Fix styling

src/Concerns/LogFile/HasMetadata.php

@@ -11,7 +11,7 @@ public function setMetadata(string $attribute, $value): void
         $this->metadata[$attribute] = $value;
     }
 
-    public function getMetadata(string $attribute = null, $default = null): mixed
+    public function getMetadata(?string $attribute = null, $default = null): mixed
     {
         if (! isset($this->metadata)) {
             $this->loadMetadata();

src/Concerns/LogIndex/CanFilterIndex.php

@@ -13,7 +13,7 @@ trait CanFilterIndex
     protected ?int $limit = null;
     protected ?int $skip = null;
 
-    public function setQuery(string $query = null): self
+    public function setQuery(?string $query = null): self
     {
         if ($this->query !== $query) {
             $this->query = $query;
@@ -29,7 +29,7 @@ public function getQuery(): ?string
         return $this->query;
     }
 
-    public function forDateRange(int|CarbonInterface $from = null, int|CarbonInterface $to = null): self
+    public function forDateRange(int|CarbonInterface|null $from = null, int|CarbonInterface|null $to = null): self
     {
         if ($from instanceof CarbonInterface) {
             $from = $from->timestamp;
@@ -45,7 +45,7 @@ public function forDateRange(int|CarbonInterface $from = null, int|CarbonInterfa
         return $this;
     }
 
-    public function forLevels(string|array $levels = null): self
+    public function forLevels(string|array|null $levels = null): self
     {
         if (is_string($levels)) {
             $levels = [$levels];
@@ -60,7 +60,7 @@ public function forLevels(string|array $levels = null): self
         return $this;
     }
 
-    public function exceptLevels(string|array $levels = null): self
+    public function exceptLevels(string|array|null $levels = null): self
     {
         if (is_null($levels)) {
             $this->excludeLevels = null;
@@ -73,7 +73,7 @@ public function exceptLevels(string|array $levels = null): self
         return $this;
     }
 
-    public function forLevel(string $level = null): self
+    public function forLevel(?string $level = null): self
     {
         return $this->forLevels($level);
     }
@@ -84,7 +84,7 @@ public function isLevelSelected(string $level): bool
             && (is_null($this->excludeLevels) || ! in_array($level, $this->excludeLevels));
     }
 
-    public function skip(int $skip = null): self
+    public function skip(?int $skip = null): self
     {
         $this->skip = $skip;
 
@@ -96,7 +96,7 @@ public function getSkip(): ?int
         return $this->skip;
     }
 
-    public function limit(int $limit = null): self
+    public function limit(?int $limit = null): self
     {
         $this->limit = $limit;
 

src/Concerns/LogReader/CanFilterUsingIndex.php

@@ -76,12 +76,12 @@ public function limit(int $number): static
         return $this;
     }
 
-    public function search(string $query = null): static
+    public function search(?string $query = null): static
     {
         return $this->setQuery($query);
     }
 
-    protected function setQuery(string $query = null): static
+    protected function setQuery(?string $query = null): static
     {
         $this->closeFile();
 

src/Concerns/LogReader/CanSetDirectionUsingIndex.php

@@ -16,7 +16,7 @@ public function forward(): static
         return $this->setDirection(Direction::Forward);
     }
 
-    public function setDirection(string $direction = null): static
+    public function setDirection(?string $direction = null): static
     {
         $direction = $direction === Direction::Backward
             ? Direction::Backward

src/LogFile.php

@@ -24,7 +24,7 @@ class LogFile
     private ?string $type = null;
     private array $_logIndexCache;
 
-    public function __construct(string $path, string $type = null)
+    public function __construct(string $path, ?string $type = null)
     {
         $this->path = $path;
         $this->name = basename($path);
@@ -59,7 +59,7 @@ public function type(): LogType
         return new LogType($this->type ?? LogType::DEFAULT);
     }
 
-    public function index(string $query = null): LogIndex
+    public function index(?string $query = null): LogIndex
     {
         if (! isset($this->_logIndexCache[$query])) {
             $this->_logIndexCache[$query] = new LogIndex($this, $query);
@@ -171,7 +171,7 @@ public function latestTimestamp(): int
         return $this->getMetadata('latest_timestamp') ?? $this->mtime();
     }
 
-    public function scan(int $maxBytesToScan = null, bool $force = false): void
+    public function scan(?int $maxBytesToScan = null, bool $force = false): void
     {
         $this->logs()->scan($maxBytesToScan, $force);
     }
@@ -184,7 +184,7 @@ public function requiresScan(): bool
     /**
      * @throws InvalidRegularExpression
      */
-    public function search(string $query = null): LogReaderInterface
+    public function search(?string $query = null): LogReaderInterface
     {
         return $this->logs()->search($query);
     }

src/LogIndex.php

@@ -30,7 +30,7 @@ public function __construct(
         $this->loadMetadata();
     }
 
-    public function addToIndex(int $filePosition, int|CarbonInterface $timestamp, string $severity, int $index = null): int
+    public function addToIndex(int $filePosition, int|CarbonInterface $timestamp, string $severity, ?int $index = null): int
     {
         $logIndex = $index ?? $this->nextLogIndexToCreate ?? 0;
 
@@ -75,7 +75,7 @@ public function getPositionForIndex(int $indexToFind): ?int
         return null;
     }
 
-    public function get(int $limit = null): array
+    public function get(?int $limit = null): array
     {
         $results = [];
         $itemsAdded = 0;
@@ -157,7 +157,7 @@ public function get(int $limit = null): array
         return $results;
     }
 
-    public function getFlatIndex(int $limit = null): array
+    public function getFlatIndex(?int $limit = null): array
     {
         $results = [];
         $itemsAdded = 0;

src/LogLevels/HorizonStatusLevel.php

@@ -11,7 +11,7 @@ public function __construct(string $value)
         $this->value = $value;
     }
 
-    public static function from(string $value = null): LevelInterface
+    public static function from(?string $value = null): LevelInterface
     {
         return new static($value);
     }

src/LogLevels/HttpStatusCodeLevel.php

@@ -9,7 +9,7 @@ public function __construct(
     ) {
     }
 
-    public static function from(string $value = null): LevelInterface
+    public static function from(?string $value = null): LevelInterface
     {
         return new static($value);
     }

src/LogLevels/LaravelLogLevel.php

@@ -16,7 +16,7 @@ class LaravelLogLevel implements LevelInterface
 
     public string $value;
 
-    public function __construct(string $value = null)
+    public function __construct(?string $value = null)
     {
         $this->value = $value ?? self::None;
     }
@@ -36,7 +36,7 @@ public static function cases(): array
         ];
     }
 
-    public static function from(string $value = null): self
+    public static function from(?string $value = null): self
     {
         return new self($value);
     }

src/LogLevels/LevelClass.php

@@ -15,7 +15,7 @@ public function __construct(
     ) {
     }
 
-    public static function from(string $value = null): LevelClass
+    public static function from(?string $value = null): LevelClass
     {
         return new static($value);
     }

src/LogLevels/LevelInterface.php

@@ -6,7 +6,7 @@ interface LevelInterface
 {
     public function __construct(string $value);
 
-    public static function from(string $value = null): self;
+    public static function from(?string $value = null): self;
 
     public static function caseValues(): array;
 

src/LogLevels/NginxStatusLevel.php

@@ -15,7 +15,7 @@ class NginxStatusLevel implements LevelInterface
 
     public string $value;
 
-    public function __construct(string $value = null)
+    public function __construct(?string $value = null)
     {
         $this->value = $value ?? self::Error;
     }
@@ -34,7 +34,7 @@ public static function cases(): array
         ];
     }
 
-    public static function from(string $value = null): self
+    public static function from(?string $value = null): self
     {
         return new self($value);
     }

src/LogLevels/PostgresLevel.php

@@ -11,7 +11,7 @@ public function __construct(string $value)
         $this->value = $value;
     }
 
-    public static function from(string $value = null): LevelInterface
+    public static function from(?string $value = null): LevelInterface
     {
         return new static($value);
     }

src/LogLevels/RedisLogLevel.php

@@ -11,7 +11,7 @@ public function __construct(string $value)
         $this->value = $value;
     }
 
-    public static function from(string $value = null): LevelInterface
+    public static function from(?string $value = null): LevelInterface
     {
         return new static($value);
     }

src/LogLevels/SupervisorLogLevel.php

@@ -25,7 +25,7 @@ public function __construct(string $value)
         $this->value = $value;
     }
 
-    public static function from(string $value = null): LevelInterface
+    public static function from(?string $value = null): LevelInterface
     {
         return new static($value);
     }

src/Logs/HorizonLog.php

@@ -31,7 +31,7 @@ protected function fillMatches(array $matches = []): void
         ]);
     }
 
-    public static function matches(string $text, int &$timestamp = null, string &$level = null): bool
+    public static function matches(string $text, ?int &$timestamp = null, ?string &$level = null): bool
     {
         return parent::matches($text, $timestamp, $level)
             || (str_contains($text, 'Horizon started successfully') && throw new SkipLineException);

src/Logs/Log.php

@@ -46,7 +46,7 @@ class Log
     public ?int $filePosition;
     public ?int $index;
 
-    public function __construct(string $text, string $fileIdentifier = null, int $filePosition = null, int $index = null)
+    public function __construct(string $text, ?string $fileIdentifier = null, ?int $filePosition = null, ?int $index = null)
     {
         $this->text = rtrim($text);
         $this->fileIdentifier = $fileIdentifier;
@@ -60,7 +60,7 @@ public function __construct(string $text, string $fileIdentifier = null, int $fi
         unset($matches);
     }
 
-    public static function matches(string $text, int &$timestamp = null, string &$level = null): bool
+    public static function matches(string $text, ?int &$timestamp = null, ?string &$level = null): bool
     {
         $matches = [];
         $result = preg_match(static::$regex, $text, $matches) === 1;

src/Readers/IndexedLogReader.php

@@ -48,7 +48,7 @@ public function lazyScanning($lazy = true): static
      *
      * @throws CannotOpenFileException
      */
-    public function scan(int $maxBytesToScan = null, bool $force = false): static
+    public function scan(?int $maxBytesToScan = null, bool $force = false): static
     {
         if (is_null($maxBytesToScan)) {
             $maxBytesToScan = LogViewer::lazyScanChunkSize();
@@ -188,7 +188,7 @@ public function getLevelCounts(): array
      *
      * @throws CannotOpenFileException
      */
-    public function get(int $limit = null): array
+    public function get(?int $limit = null): array
     {
         if (! is_null($limit) && method_exists($this, 'limit')) {
             $this->limit($limit);
@@ -230,7 +230,7 @@ public function total(): int
         return $this->index()->count();
     }
 
-    public function paginate(int $perPage = 25, int $page = null)
+    public function paginate(int $perPage = 25, ?int $page = null)
     {
         $page = $page ?: Paginator::resolveCurrentPage('page');
 

src/Readers/LogReaderInterface.php

@@ -15,7 +15,7 @@ public static function clearInstance(LogFile $file): void;
     public static function clearInstances(): void;
 
     // Search/querying
-    public function search(string $query = null): static;
+    public function search(?string $query = null): static;
 
     public function skip(int $number): static;
 
@@ -26,7 +26,7 @@ public function reverse(): static;
 
     public function forward(): static;
 
-    public function setDirection(string $direction = null): static;
+    public function setDirection(?string $direction = null): static;
 
     public function getLevelCounts(): array;
 
@@ -41,20 +41,20 @@ public function except($levels = null): static;
     public function exceptLevels($levels = null): static;
 
     // Retrieving actual logs
-    public function get(int $limit = null): array;
+    public function get(?int $limit = null): array;
 
     public function next(): ?Log;
 
     /** @return LengthAwarePaginator<Log> */
-    public function paginate(int $perPage = 25, int $page = null);
+    public function paginate(int $perPage = 25, ?int $page = null);
 
     public function total(): int;
 
     // Functional
     public function reset(): static;
 
     // We should decouple scanning from the LogReader
-    public function scan(int $maxBytesToScan = null, bool $force = false): static;
+    public function scan(?int $maxBytesToScan = null, bool $force = false): static;
 
     public function numberOfNewBytes(): int;
 

src/Readers/MultipleLogReader.php

@@ -48,7 +48,7 @@ public function allLevels(): self
         return $this;
     }
 
-    public function setDirection(string $direction = null): self
+    public function setDirection(?string $direction = null): self
     {
         $this->direction = $direction === Direction::Backward
             ? Direction::Backward
@@ -87,7 +87,7 @@ public function limit(int $number): self
         return $this;
     }
 
-    public function search(string $query = null): self
+    public function search(?string $query = null): self
     {
         $this->query = $query;
 
@@ -121,7 +121,7 @@ public function total(): int
         });
     }
 
-    public function paginate($perPage = 25, int $page = null): LengthAwarePaginator
+    public function paginate($perPage = 25, ?int $page = null): LengthAwarePaginator
     {
         $page = $page ?: Paginator::resolveCurrentPage('page');
 
@@ -140,7 +140,7 @@ public function paginate($perPage = 25, int $page = null): LengthAwarePaginator
      *
      * @return array|Log[]
      */
-    public function get(int $limit = null): array
+    public function get(?int $limit = null): array
     {
         $skip = $this->skip ?? null;
         $limit = $limit ?? $this->limit ?? null;
@@ -204,7 +204,7 @@ public function percentScanned(): int
         return 100 - intval($missingScansBytes / $totalFileBytes * 100);
     }
 
-    public function scan(int $maxBytesToScan = null, bool $force = false): void
+    public function scan(?int $maxBytesToScan = null, bool $force = false): void
     {
         $fileSizeScanned = 0;
         $stopScanningAfter = microtime(true) + LogViewer::lazyScanTimeout();