NNTmux / newznab-tmux

app/Services/Tmux/TmuxLayoutBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
     protected function buildFullLayout(): bool
     {
         // Window 0: Monitor + Binaries + Backfill + Releases
-        if (! $this->sessionManager->createSession('Monitor')) {
+        if (!$this->sessionManager->createSession('Monitor')) {
             return false;
         }
 
@@ -93,7 +93,7 @@ 
     protected function buildBasicLayout(): bool
     {
         // Window 0: Monitor + Releases
-        if (! $this->sessionManager->createSession('Monitor')) {
+        if (!$this->sessionManager->createSession('Monitor')) {
             return false;
         }
 
@@ -138,7 +138,7 @@ 
     protected function buildStrippedLayout(): bool
     {
         // Window 0: Monitor + Sequential
-        if (! $this->sessionManager->createSession('Monitor')) {
+        if (!$this->sessionManager->createSession('Monitor')) {
             return false;
         }
 
@@ -242,7 +242,7 @@ 
             $this->paneManager->createWindow($windowIndex, 'redis');
 
             // Check if custom args provided for simple redis-cli output
-            if (! empty($redisArgs) && $redisArgs !== 'NULL') {
+            if (!empty($redisArgs) && $redisArgs !== 'NULL') {
                 $this->paneManager->respawnPane("{$windowIndex}.0", "watch -n{$refreshInterval} -c 'redis-cli -h {$redisHost} -p {$redisPort} {$redisArgs}'");
             } else {
                 // Use modern visual monitoring script

app/Console/Commands/MigrateAnimeCovers.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             if (preg_match('/^(\d+)\.jpg$/', $filename, $matches)) {
                 $anidbid = $matches[1];
                 $newFilename = "{$anidbid}-cover.jpg";
-                $newPath = $animeCoverPath . $newFilename;
+                $newPath = $animeCoverPath.$newFilename;
 
                 // Skip if new format already exists
                 if (file_exists($newPath)) {

app/Extensions/helper/helpers.php

Spacing +31 added lines, -31 removed lines

@@ -15,7 +15,7 @@ 
 use Symfony\Component\Process\Process;
 use Zip as ZipStream;
 
-if (! function_exists('getRawHtml')) {
+if (!function_exists('getRawHtml')) {
     /**
      * @param  bool  $cookie
      * @return bool|mixed|string
@@ -106,7 +106,7 @@ 
     }
 }
 
-if (! function_exists('makeFieldLinks')) {
+if (!function_exists('makeFieldLinks')) {
     /**
      * @return string
      *
@@ -136,7 +136,7 @@ 
     }
 }
 
-if (! function_exists('getUserBrowseOrder')) {
+if (!function_exists('getUserBrowseOrder')) {
     /**
      * @param  string  $orderBy
      */
@@ -162,7 +162,7 @@ 
     }
 }
 
-if (! function_exists('getUserBrowseOrdering')) {
+if (!function_exists('getUserBrowseOrdering')) {
     function getUserBrowseOrdering(): array
     {
         return [
@@ -192,7 +192,7 @@ 
     }
 }
 
-if (! function_exists('getSimilarName')) {
+if (!function_exists('getSimilarName')) {
     /**
      * @param  string  $name
      */
@@ -202,7 +202,7 @@ 
     }
 }
 
-if (! function_exists('human_filesize')) {
+if (!function_exists('human_filesize')) {
     /**
      * @param  int  $decimals
      */
@@ -215,7 +215,7 @@ 
     }
 }
 
-if (! function_exists('bcdechex')) {
+if (!function_exists('bcdechex')) {
     /**
      * @return string
      */
@@ -232,7 +232,7 @@ 
     }
 }
 
-if (! function_exists('runCmd')) {
+if (!function_exists('runCmd')) {
     /**
      * Run CLI command.
      *
@@ -260,7 +260,7 @@ 
     }
 }
 
-if (! function_exists('escapeString')) {
+if (!function_exists('escapeString')) {
 
     function escapeString($string): string
     {
@@ -268,7 +268,7 @@ 
     }
 }
 
-if (! function_exists('realDuration')) {
+if (!function_exists('realDuration')) {
 
     function realDuration($milliseconds): string
     {
@@ -278,7 +278,7 @@ 
     }
 }
 
-if (! function_exists('is_it_json')) {
+if (!function_exists('is_it_json')) {
     /**
      * @throws JsonException
      */
@@ -293,7 +293,7 @@ 
     }
 }
 
-if (! function_exists('getStreamingZip')) {
+if (!function_exists('getStreamingZip')) {
     /**
      * @throws Exception
      */
@@ -320,7 +320,7 @@ 
     }
 }
 
-if (! function_exists('release_flag')) {
+if (!function_exists('release_flag')) {
     // Function inspired by c0r3@newznabforums adds country flags on the browse page.
     /**
      * @param  string  $text  Text to match against.
@@ -457,7 +457,7 @@ 
     }
 }
 
-if (! function_exists('getReleaseCover')) {
+if (!function_exists('getReleaseCover')) {
     /**
      * Get the cover image URL for a release based on its type and ID
      *
@@ -470,7 +470,7 @@ 
         $coverId = null;
 
         // Helper function to get value from object or array
-        $getValue = function ($data, $key) {
+        $getValue = function($data, $key) {
             if (is_array($data)) {
                 return $data[$key] ?? null;
             } elseif (is_object($data)) {
@@ -489,25 +489,25 @@ 
         $xxxinfo_id = $getValue($release, 'xxxinfo_id');
         $anidbid = $getValue($release, 'anidbid');
 
-        if (! empty($imdbid) && $imdbid > 0) {
+        if (!empty($imdbid) && $imdbid > 0) {
             $coverType = 'movies';
             $coverId = str_pad($imdbid, 7, '0', STR_PAD_LEFT);
-        } elseif (! empty($musicinfo_id)) {
+        } elseif (!empty($musicinfo_id)) {
             $coverType = 'music';
             $coverId = $musicinfo_id;
-        } elseif (! empty($consoleinfo_id)) {
+        } elseif (!empty($consoleinfo_id)) {
             $coverType = 'console';
             $coverId = $consoleinfo_id;
-        } elseif (! empty($bookinfo_id)) {
+        } elseif (!empty($bookinfo_id)) {
             $coverType = 'book';
             $coverId = $bookinfo_id;
-        } elseif (! empty($gamesinfo_id)) {
+        } elseif (!empty($gamesinfo_id)) {
             $coverType = 'games';
             $coverId = $gamesinfo_id;
-        } elseif (! empty($xxxinfo_id)) {
+        } elseif (!empty($xxxinfo_id)) {
             $coverType = 'xxx';
             $coverId = $xxxinfo_id;
-        } elseif (! empty($anidbid) && $anidbid > 0) {
+        } elseif (!empty($anidbid) && $anidbid > 0) {
             $coverType = 'anime';
             $coverId = $anidbid;
         }
@@ -523,10 +523,10 @@ 
     }
 }
 
-if (! function_exists('sanitize')) {
+if (!function_exists('sanitize')) {
     function sanitize(array|string $phrases, array $doNotSanitize = []): string
     {
-        if (! is_array($phrases)) {
+        if (!is_array($phrases)) {
             $wordArray = explode(' ', str_replace('.', ' ', $phrases));
         } else {
             $wordArray = $phrases;
@@ -537,9 +537,9 @@ 
         foreach ($wordArray as $words) {
             $words = preg_split('/\s+/', $words);
             foreach ($words as $st) {
-                if (Str::startsWith($st, ['!', '+', '-', '?', '*']) && Str::length($st) > 1 && ! preg_match('/([!+?\-*]){2,}/', $st)) {
+                if (Str::startsWith($st, ['!', '+', '-', '?', '*']) && Str::length($st) > 1 && !preg_match('/([!+?\-*]){2,}/', $st)) {
                     $str = $st;
-                } elseif (Str::endsWith($st, ['+', '-', '?', '*']) && Str::length($st) > 1 && ! preg_match('/([!+?\-*]){2,}/', $st)) {
+                } elseif (Str::endsWith($st, ['+', '-', '?', '*']) && Str::length($st) > 1 && !preg_match('/([!+?\-*]){2,}/', $st)) {
                     $str = $st;
                 } else {
                     $str = Sanitizer::escape($st, $doNotSanitize);
@@ -554,7 +554,7 @@ 
     }
 }
 
-if (! function_exists('formatBytes')) {
+if (!function_exists('formatBytes')) {
     /**
      * Format bytes into human-readable file size.
      *
@@ -573,7 +573,7 @@ 
     }
 }
 
-if (! function_exists('csp_nonce')) {
+if (!function_exists('csp_nonce')) {
     /**
      * Generate a CSP nonce for inline scripts
      * This should be stored in the request and reused across the request lifecycle
@@ -590,7 +590,7 @@ 
     }
 }
 
-if (! function_exists('userDate')) {
+if (!function_exists('userDate')) {
     /**
      * Format a date/time string according to the authenticated user's timezone
      *
@@ -622,7 +622,7 @@ 
     }
 }
 
-if (! function_exists('userDateDiffForHumans')) {
+if (!function_exists('userDateDiffForHumans')) {
     /**
      * Format a date/time string as a human-readable diff according to the authenticated user's timezone
      *
@@ -653,7 +653,7 @@ 
     }
 }
 
-if (! function_exists('getAvailableTimezones')) {
+if (!function_exists('getAvailableTimezones')) {
     /**
      * Get a list of available timezones grouped by region
      *

app/Console/Commands/RefreshAnimeData.php

Spacing +16 added lines, -16 removed lines

@@ -120,7 +120,7 @@ 
         foreach ($chunks as $chunk) {
             foreach ($chunk as $release) {
                 $searchname = $release->searchname ?? '';
-                $progressBar->setMessage("Processing: " . substr($searchname, 0, 50) . "...");
+                $progressBar->setMessage("Processing: ".substr($searchname, 0, 50)."...");
 
                 try {
                     // Extract clean title from searchname
@@ -146,7 +146,7 @@ 
 
                     // Check if we should skip (if not forcing and data exists)
                     // Don't skip if we're retrying failed releases (anidbid <= 0)
-                    if (! $force && ! $missingOnly && ! $retryFailed) {
+                    if (!$force && !$missingOnly && !$retryFailed) {
                         // Check if release already has complete AniList data
                         if ($release->anidbid > 0) {
                             $anidbInfo = DB::table('anidb_info')
@@ -169,7 +169,7 @@ 
                     $this->enforceRateLimit();
                     $searchResults = $populateAniList->searchAnime($cleanTitle, 1);
                     
-                    if (! $searchResults || empty($searchResults)) {
+                    if (!$searchResults || empty($searchResults)) {
                         // Try with spaces replaced for broader matching
                         $altTitle = preg_replace('/\s+/', ' ', $cleanTitle);
                         if ($altTitle !== $cleanTitle) {
@@ -178,7 +178,7 @@ 
                         }
                     }
 
-                    if (! $searchResults || empty($searchResults)) {
+                    if (!$searchResults || empty($searchResults)) {
                         $notFound++;
                         $failedSearchnames[] = [
                             'searchname' => $searchname,
@@ -199,7 +199,7 @@ 
                     $anilistData = $searchResults[0];
                     $anilistId = $anilistData['id'] ?? null;
 
-                    if (! $anilistId) {
+                    if (!$anilistId) {
                         $notFound++;
                         $failedSearchnames[] = [
                             'searchname' => $searchname,
@@ -227,7 +227,7 @@ 
                         ->where('anilist_id', $anilistId)
                         ->value('anidbid');
 
-                    if (! $anidbid) {
+                    if (!$anidbid) {
                         // Fallback: use anilist_id as anidbid
                         $anidbid = (int) $anilistId;
                     }
@@ -271,7 +271,7 @@ 
                                 $this->line("  - {$item['searchname']} -> {$cleanedTitle} ({$item['reason']})");
                             }
                             if (count($failedSearchnames) > 10) {
-                                $this->line("  ... and " . (count($failedSearchnames) - 10) . " more.");
+                                $this->line("  ... and ".(count($failedSearchnames) - 10)." more.");
                             }
                         }
                         
@@ -281,7 +281,7 @@ 
                     $failed++;
                     if ($this->getOutput()->isVerbose()) {
                         $this->newLine();
-                        $this->error("Error processing release ID {$release->id}: " . $e->getMessage());
+                        $this->error("Error processing release ID {$release->id}: ".$e->getMessage());
                     }
                 }
 
@@ -319,8 +319,8 @@ 
             foreach ($examples as $item) {
                 $cleanedTitle = $item['cleaned_title'] ?? '(extraction failed)';
                 $rows[] = [
-                    substr($item['searchname'], 0, 60) . (strlen($item['searchname']) > 60 ? '...' : ''),
-                    substr($cleanedTitle, 0, 40) . (strlen($cleanedTitle) > 40 ? '...' : ''),
+                    substr($item['searchname'], 0, 60).(strlen($item['searchname']) > 60 ? '...' : ''),
+                    substr($cleanedTitle, 0, 40).(strlen($cleanedTitle) > 40 ? '...' : ''),
                     $item['reason'],
                 ];
             }
@@ -331,7 +331,7 @@ 
             );
             
             if (count($failedSearchnames) > 20) {
-                $this->line("... and " . (count($failedSearchnames) - 20) . " more. Use --verbose to see all.");
+                $this->line("... and ".(count($failedSearchnames) - 20)." more. Use --verbose to see all.");
             }
         }
 
@@ -467,7 +467,7 @@ 
         $now = time();
         
         // Clean old timestamps (older than 1 minute)
-        $this->requestTimestamps = array_filter($this->requestTimestamps, function ($timestamp) use ($now) {
+        $this->requestTimestamps = array_filter($this->requestTimestamps, function($timestamp) use ($now) {
             return ($now - $timestamp) < 60;
         });
 
@@ -476,20 +476,20 @@ 
         // If we're at or over the limit, wait
         if ($requestCount >= self::RATE_LIMIT_PER_MINUTE) {
             // Calculate wait time based on oldest request
-            if (! empty($this->requestTimestamps)) {
+            if (!empty($this->requestTimestamps)) {
                 $oldestRequest = min($this->requestTimestamps);
                 $waitTime = 60 - ($now - $oldestRequest) + 1; // +1 for safety margin
 
                 if ($waitTime > 0 && $waitTime <= 60) {
                     if ($this->getOutput()->isVerbose()) {
                         $this->newLine();
-                        $this->warn("Rate limit reached ({$requestCount}/" . self::RATE_LIMIT_PER_MINUTE . "). Waiting {$waitTime} seconds...");
+                        $this->warn("Rate limit reached ({$requestCount}/".self::RATE_LIMIT_PER_MINUTE."). Waiting {$waitTime} seconds...");
                     }
                     sleep($waitTime);
                     
                     // Clean timestamps again after waiting
                     $now = time();
-                    $this->requestTimestamps = array_filter($this->requestTimestamps, function ($timestamp) use ($now) {
+                    $this->requestTimestamps = array_filter($this->requestTimestamps, function($timestamp) use ($now) {
                         return ($now - $timestamp) < 60;
                     });
                 }
@@ -501,7 +501,7 @@ 
         $minDelay = 60.0 / self::RATE_LIMIT_PER_MINUTE;
         
         // If we have recent requests, ensure we wait at least the minimum delay
-        if (! empty($this->requestTimestamps)) {
+        if (!empty($this->requestTimestamps)) {
             $lastRequest = max($this->requestTimestamps);
             $timeSinceLastRequest = $now - $lastRequest;
             

app/Services/TmdbClient.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function isConfigured(): bool
     {
-        return ! empty($this->apiKey);
+        return !empty($this->apiKey);
     }
 
     /**
@@ -51,7 +51,7 @@ 
      */
     protected function get(string $endpoint, array $params = []): ?array
     {
-        if (! $this->isConfigured()) {
+        if (!$this->isConfigured()) {
             Log::warning('TMDB API key is not configured');
 
             return null;
@@ -143,7 +143,7 @@ 
     {
         $params = [];
 
-        if (! empty($appendToResponse)) {
+        if (!empty($appendToResponse)) {
             $params['append_to_response'] = implode(',', $appendToResponse);
         }
 
@@ -210,7 +210,7 @@ 
     {
         $params = [];
 
-        if (! empty($appendToResponse)) {
+        if (!empty($appendToResponse)) {
             $params['append_to_response'] = implode(',', $appendToResponse);
         }
 
@@ -309,7 +309,7 @@ 
         $value = $data;
 
         foreach ($keys as $key) {
-            if (! is_array($value) || ! array_key_exists($key, $value)) {
+            if (!is_array($value) || !array_key_exists($key, $value)) {
                 return $default;
             }
             $value = $value[$key];

app/Providers/CategorizationServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function register(): void
     {
         // Register the pipeline as a singleton
-        $this->app->singleton(CategorizationPipeline::class, function ($app) {
+        $this->app->singleton(CategorizationPipeline::class, function($app) {
             return CategorizationPipeline::createDefault();
         });
 

app/Services/Categorization/CategorizationService.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
     {
         $categorizers = $this->pipeline->getCategorizers();
 
-        return $categorizers->map(function ($categorizer) {
+        return $categorizers->map(function($categorizer) {
             return [
                 'name' => $categorizer->getName(),
                 'priority' => $categorizer->getPriority(),

app/Services/Categorization/Categorizers/BookCategorizer.php

Spacing +5 added lines, -5 removed lines

@@ -38,9 +38,9 @@ 
     protected function checkTechnical(string $name): ?CategorizationResult
     {
         $publishers = 'Apress|Addison[._ -]Wesley|Manning|No[._ -]Starch|OReilly|Packt|Pragmatic|Wiley|Wrox';
-        if (preg_match('/\b(' . $publishers . ')\b/i', $name)) return $this->matched(Category::BOOKS_TECHNICAL, 0.9, 'technical_publisher');
+        if (preg_match('/\b('.$publishers.')\b/i', $name)) return $this->matched(Category::BOOKS_TECHNICAL, 0.9, 'technical_publisher');
         $subjects = 'Programming|Python|JavaScript|Java|Database|Linux|DevOps|Machine[._ -]Learning|Data[._ -]Science';
-        if (preg_match('/\b(' . $subjects . ')\b/i', $name) && preg_match('/\b(Book|Guide|Tutorial|Learn)\b/i', $name)) {
+        if (preg_match('/\b('.$subjects.')\b/i', $name) && preg_match('/\b(Book|Guide|Tutorial|Learn)\b/i', $name)) {
             return $this->matched(Category::BOOKS_TECHNICAL, 0.85, 'technical_subject');
         }
         return null;
@@ -49,14 +49,14 @@ 
     {
         if (preg_match('/[._ -](Monthly|Weekly|Annual|Quarterly|Issue)[._ -]/i', $name)) return $this->matched(Category::BOOKS_MAGAZINES, 0.9, 'magazine_frequency');
         $magazines = 'Forbes|Fortune|GQ|National[._ -]Geographic|Newsweek|Time|Vogue|Wired|PC[._ -]Gamer';
-        if (preg_match('/\b(' . $magazines . ')\b/i', $name)) return $this->matched(Category::BOOKS_MAGAZINES, 0.85, 'magazine_title');
+        if (preg_match('/\b('.$magazines.')\b/i', $name)) return $this->matched(Category::BOOKS_MAGAZINES, 0.85, 'magazine_title');
         return null;
     }
     protected function checkEbook(string $name): ?CategorizationResult
     {
         $formats = 'EPUB|MOBI|AZW\d?|PDF|FB2|DJVU|LIT';
-        if (preg_match('/\.(' . $formats . ')$/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.9, 'ebook_format');
-        if (preg_match('/\b(' . $formats . ')\b/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.85, 'ebook_indicator');
+        if (preg_match('/\.('.$formats.')$/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.9, 'ebook_format');
+        if (preg_match('/\b('.$formats.')\b/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.85, 'ebook_indicator');
         if (preg_match('/\b(E-?book|Kindle|Kobo|Nook)\b/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.8, 'ebook_platform');
         return null;
     }

Braces +54 added lines, -18 removed lines

@@ -9,36 +9,62 @@ 
     public function getName(): string { return 'Book'; }
     public function shouldSkip(ReleaseContext $context): bool
     {
-        if ($context->hasAdultMarkers()) return true;
-        if (preg_match('/\.PS4-[A-Z0-9]+$/i', $context->releaseName)) return true;
-        if (preg_match('/\b(?:PS[1-5]|PlayStation|Xbox|Switch|Nintendo|Wii|3DS|GameCube)\b/i', $context->releaseName)) return true;
+        if ($context->hasAdultMarkers()) {
+            return true;
+        }
+        if (preg_match('/\.PS4-[A-Z0-9]+$/i', $context->releaseName)) {
+            return true;
+        }
+        if (preg_match('/\b(?:PS[1-5]|PlayStation|Xbox|Switch|Nintendo|Wii|3DS|GameCube)\b/i', $context->releaseName)) {
+            return true;
+        }
         // Skip TV shows (season patterns)
-        if (preg_match('/[._ -]S\d{1,3}[._ -]?(E\d|Complete|Full|1080|720|480|2160|WEB|HDTV|BluRay)/i', $context->releaseName)) return true;
+        if (preg_match('/[._ -]S\d{1,3}[._ -]?(E\d|Complete|Full|1080|720|480|2160|WEB|HDTV|BluRay)/i', $context->releaseName)) {
+            return true;
+        }
         // Skip movies (year + quality patterns)
-        if (preg_match('/\b(19|20)\d{2}\b.*\b(1080p|720p|2160p|BluRay|WEB-DL|BDRip|DVDRip)\b/i', $context->releaseName)) return true;
+        if (preg_match('/\b(19|20)\d{2}\b.*\b(1080p|720p|2160p|BluRay|WEB-DL|BDRip|DVDRip)\b/i', $context->releaseName)) {
+            return true;
+        }
         return false;
     }
     public function categorize(ReleaseContext $context): CategorizationResult
     {
         $name = $context->releaseName;
-        if ($result = $this->checkComic($name)) return $result;
-        if ($result = $this->checkTechnical($name)) return $result;
-        if ($result = $this->checkMagazine($name)) return $result;
-        if ($result = $this->checkEbook($name)) return $result;
+        if ($result = $this->checkComic($name)) {
+            return $result;
+        }
+        if ($result = $this->checkTechnical($name)) {
+            return $result;
+        }
+        if ($result = $this->checkMagazine($name)) {
+            return $result;
+        }
+        if ($result = $this->checkEbook($name)) {
+            return $result;
+        }
         return $this->noMatch();
     }
     protected function checkComic(string $name): ?CategorizationResult
     {
-        if (preg_match('/\b(?:CBR|CBZ|C2C)\b|\.(?:cbr|cbz)$/i', $name)) return $this->matched(Category::BOOKS_COMICS, 0.9, 'comic_format');
+        if (preg_match('/\b(?:CBR|CBZ|C2C)\b|\.(?:cbr|cbz)$/i', $name)) {
+            return $this->matched(Category::BOOKS_COMICS, 0.9, 'comic_format');
+        }
         if (preg_match('/\b(?:Marvel|DC[._ -]Comics|Image[._ -]Comics|Dark[._ -]Horse|IDW)\b/i', $name) &&
-            preg_match('/\b(?:Comics?|Annual|Issue|Vol|TPB)\b/i', $name)) return $this->matched(Category::BOOKS_COMICS, 0.85, 'comic_publisher');
-        if (preg_match('/\b(?:Manga|Manhwa|Manhua|Webtoon)\b/i', $name)) return $this->matched(Category::BOOKS_COMICS, 0.85, 'manga');
+            preg_match('/\b(?:Comics?|Annual|Issue|Vol|TPB)\b/i', $name)) {
+            return $this->matched(Category::BOOKS_COMICS, 0.85, 'comic_publisher');
+        }
+        if (preg_match('/\b(?:Manga|Manhwa|Manhua|Webtoon)\b/i', $name)) {
+            return $this->matched(Category::BOOKS_COMICS, 0.85, 'manga');
+        }
         return null;
     }
     protected function checkTechnical(string $name): ?CategorizationResult
     {
         $publishers = 'Apress|Addison[._ -]Wesley|Manning|No[._ -]Starch|OReilly|Packt|Pragmatic|Wiley|Wrox';
-        if (preg_match('/\b(' . $publishers . ')\b/i', $name)) return $this->matched(Category::BOOKS_TECHNICAL, 0.9, 'technical_publisher');
+        if (preg_match('/\b(' . $publishers . ')\b/i', $name)) {
+            return $this->matched(Category::BOOKS_TECHNICAL, 0.9, 'technical_publisher');
+        }
         $subjects = 'Programming|Python|JavaScript|Java|Database|Linux|DevOps|Machine[._ -]Learning|Data[._ -]Science';
         if (preg_match('/\b(' . $subjects . ')\b/i', $name) && preg_match('/\b(Book|Guide|Tutorial|Learn)\b/i', $name)) {
             return $this->matched(Category::BOOKS_TECHNICAL, 0.85, 'technical_subject');
@@ -47,17 +73,27 @@ 
     }
     protected function checkMagazine(string $name): ?CategorizationResult
     {
-        if (preg_match('/[._ -](Monthly|Weekly|Annual|Quarterly|Issue)[._ -]/i', $name)) return $this->matched(Category::BOOKS_MAGAZINES, 0.9, 'magazine_frequency');
+        if (preg_match('/[._ -](Monthly|Weekly|Annual|Quarterly|Issue)[._ -]/i', $name)) {
+            return $this->matched(Category::BOOKS_MAGAZINES, 0.9, 'magazine_frequency');
+        }
         $magazines = 'Forbes|Fortune|GQ|National[._ -]Geographic|Newsweek|Time|Vogue|Wired|PC[._ -]Gamer';
-        if (preg_match('/\b(' . $magazines . ')\b/i', $name)) return $this->matched(Category::BOOKS_MAGAZINES, 0.85, 'magazine_title');
+        if (preg_match('/\b(' . $magazines . ')\b/i', $name)) {
+            return $this->matched(Category::BOOKS_MAGAZINES, 0.85, 'magazine_title');
+        }
         return null;
     }
     protected function checkEbook(string $name): ?CategorizationResult
     {
         $formats = 'EPUB|MOBI|AZW\d?|PDF|FB2|DJVU|LIT';
-        if (preg_match('/\.(' . $formats . ')$/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.9, 'ebook_format');
-        if (preg_match('/\b(' . $formats . ')\b/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.85, 'ebook_indicator');
-        if (preg_match('/\b(E-?book|Kindle|Kobo|Nook)\b/i', $name)) return $this->matched(Category::BOOKS_EBOOK, 0.8, 'ebook_platform');
+        if (preg_match('/\.(' . $formats . ')$/i', $name)) {
+            return $this->matched(Category::BOOKS_EBOOK, 0.9, 'ebook_format');
+        }
+        if (preg_match('/\b(' . $formats . ')\b/i', $name)) {
+            return $this->matched(Category::BOOKS_EBOOK, 0.85, 'ebook_indicator');
+        }
+        if (preg_match('/\b(E-?book|Kindle|Kobo|Nook)\b/i', $name)) {
+            return $this->matched(Category::BOOKS_EBOOK, 0.8, 'ebook_platform');
+        }
         return null;
     }
 }

app/Services/Categorization/Categorizers/GroupNameCategorizer.php

Braces +24 added lines, -8 removed lines

@@ -10,14 +10,30 @@
     public function categorize(ReleaseContext $context): CategorizationResult
     {
         $groupName = $context->groupName;
-        if (empty($groupName)) return $this->noMatch();
-        if (preg_match('/alt\.binaries\..*?(tv|hdtv|tvseries)/i', $groupName)) return $this->matched(Category::TV_OTHER, 0.6, 'group_tv');
-        if (preg_match('/alt\.binaries\..*?(movies?|dvd|bluray|x264)/i', $groupName)) return $this->matched(Category::MOVIE_OTHER, 0.6, 'group_movie');
-        if (preg_match('/alt\.binaries\..*?(erotica|pictures\.erotica|xxx)/i', $groupName)) return $this->matched(Category::XXX_OTHER, 0.7, 'group_xxx');
-        if (preg_match('/alt\.binaries\..*?(sounds?|mp3|music|lossless)/i', $groupName)) return $this->matched(Category::MUSIC_OTHER, 0.6, 'group_music');
-        if (preg_match('/alt\.binaries\..*?(games?|console|psx|nintendo)/i', $groupName)) return $this->matched(Category::GAME_OTHER, 0.6, 'group_game');
-        if (preg_match('/alt\.binaries\..*?(warez|0day|apps?|software)/i', $groupName)) return $this->matched(Category::PC_0DAY, 0.6, 'group_pc');
-        if (preg_match('/alt\.binaries\..*?(e-?book|ebook|comics?)/i', $groupName)) return $this->matched(Category::BOOKS_EBOOK, 0.6, 'group_book');
+        if (empty($groupName)) {
+            return $this->noMatch();
+        }
+        if (preg_match('/alt\.binaries\..*?(tv|hdtv|tvseries)/i', $groupName)) {
+            return $this->matched(Category::TV_OTHER, 0.6, 'group_tv');
+        }
+        if (preg_match('/alt\.binaries\..*?(movies?|dvd|bluray|x264)/i', $groupName)) {
+            return $this->matched(Category::MOVIE_OTHER, 0.6, 'group_movie');
+        }
+        if (preg_match('/alt\.binaries\..*?(erotica|pictures\.erotica|xxx)/i', $groupName)) {
+            return $this->matched(Category::XXX_OTHER, 0.7, 'group_xxx');
+        }
+        if (preg_match('/alt\.binaries\..*?(sounds?|mp3|music|lossless)/i', $groupName)) {
+            return $this->matched(Category::MUSIC_OTHER, 0.6, 'group_music');
+        }
+        if (preg_match('/alt\.binaries\..*?(games?|console|psx|nintendo)/i', $groupName)) {
+            return $this->matched(Category::GAME_OTHER, 0.6, 'group_game');
+        }
+        if (preg_match('/alt\.binaries\..*?(warez|0day|apps?|software)/i', $groupName)) {
+            return $this->matched(Category::PC_0DAY, 0.6, 'group_pc');
+        }
+        if (preg_match('/alt\.binaries\..*?(e-?book|ebook|comics?)/i', $groupName)) {
+            return $this->matched(Category::BOOKS_EBOOK, 0.6, 'group_book');
+        }
         return $this->noMatch();
     }
 }