fix: monitors check for episodes in folders w/ & w/o the year appended (backwards compatible)

.gitea/workflows/build_base_images.yml

@@ -0,0 +1,59 @@
+name: CI
+
+on:
+  workflow_dispatch:
+
+jobs:
+  build-base-app:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5
+
+      - name: Login to registry
+        uses: docker/login-action@v2
+        with:
+          registry: "${{ vars.REGISTRY_URL }}"
+          username: "${{ vars.REGISTRY_USER }}"
+          password: "${{ vars.REGISTRY_PASS }}"
+
+      - name: Build and push
+        uses: docker/build-push-action@v5
+        env:
+          APP_FRANKENPHP_TAG: php8.4
+        with:
+          context: .
+          push: true
+          file: docker/Dockerfile.base.worker
+          platforms: linux/amd64
+          build-args: |
+            APP_FRANKENPHP_TAG=${{ env.APP_FRANKENPHP_TAG }}
+          tags: |
+            code.caldwell.digital/home/torsearch-base:${APP_FRANKENPHP_TAG}
+            code.caldwell.digital/home/torsearch-base:latest
+
+  build-base-worker-supervisord:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5
+
+      - name: Login to registry
+        uses: docker/login-action@v2
+        with:
+          registry: "${{ vars.REGISTRY_URL }}"
+          username: "${{ vars.REGISTRY_USER }}"
+          password: "${{ vars.REGISTRY_PASS }}"
+
+      - name: Build and push
+        uses: docker/build-push-action@v5
+        with:
+          context: .
+          push: true
+          file: docker/Dockerfile.base.worker
+          platforms: linux/amd64
+          build-args: |
+            ALPINE_VERSION=3.22
+          tags: |
+            code.caldwell.digital/home/torsearch-base-worker-supervisord:latest

src/Base/Service/MediaFiles.php

@@ -113,9 +113,14 @@ class MediaFiles
         return Map::from($results);
     }
 
-    public function createMovieDirectory(string $path): string
+    public function createMovieDirectory(string $title, string|int $year): string
     {
-        $path = $this->moviesPath . DIRECTORY_SEPARATOR . $path;
+        $path = sprintf(
+            '%s' . DIRECTORY_SEPARATOR . '%s (%s)',
+            $this->moviesPath,
+            $title,
+            $year
+        );
 
         if (false === $this->filesystem->exists($path)) {
             $this->filesystem->mkdir($path);
@@ -124,9 +129,14 @@ class MediaFiles
         return $path;
     }
 
-    public function createTvShowDirectory(string $path): string
+    public function createTvShowDirectory(string $title, string|int $year): string
     {
-        $path = $this->tvShowsPath . DIRECTORY_SEPARATOR . $path;
+        $path = sprintf(
+            '%s' . DIRECTORY_SEPARATOR . '%s (%s)',
+            $this->tvShowsPath,
+            $title,
+            $year
+        );
 
         if (false === $this->filesystem->exists($path)) {
             $this->filesystem->mkdir($path);

src/Download/Downloader/ProcessDownloader.php

@@ -7,6 +7,8 @@ use App\Base\Service\MediaFiles;
 use App\Download\DownloadEvents;
 use App\Download\Framework\Entity\Download;
 use App\EventLog\Action\Command\AddEventLogCommand;
+use App\Search\Action\Command\GetMediaInfoCommand;
+use App\Search\Action\Handler\GetMediaInfoHandler;
 use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\Cache\Adapter\RedisAdapter;
 use Symfony\Component\Messenger\MessageBusInterface;
@@ -25,6 +27,7 @@ class ProcessDownloader implements DownloaderInterface
         private MediaFiles $mediaFiles,
         private CacheInterface $cache,
         private readonly Broadcaster $broadcaster,
+        private readonly GetMediaInfoHandler $getMediaInfoHandler,
     ) {}
 
     /**
@@ -37,7 +40,7 @@ class ProcessDownloader implements DownloaderInterface
         $this->entityManager->flush();
 
         $downloadPreferences = $downloadEntity->getUser()->getDownloadPreferences();
-        $path = $this->getDownloadPath($mediaType, $title, $downloadPreferences);
+        $path = $this->getDownloadPath($mediaType, $title, $downloadEntity->getImdbId(), $downloadPreferences);
 
         $processArgs = ['wget', '-O', $downloadEntity->getFilename(), $url];
 
@@ -103,17 +106,22 @@ class ProcessDownloader implements DownloaderInterface
         $this->entityManager->flush();
     }
 
-    public function getDownloadPath(string $mediaType, string $title, array $downloadPreferences): string
+    public function getDownloadPath(string $mediaType, string $title, string $imdbId, array $downloadPreferences): string
     {
+        $mediaInfo = $this->getMediaInfoHandler->handle(new GetMediaInfoCommand(
+            $imdbId,
+            $mediaType,
+        ));
+
         if ($mediaType === 'movies') {
             if ((bool) $downloadPreferences['movie_folder']->getPreferenceValue() === true) {
-                return $this->mediaFiles->createMovieDirectory($title);
+                return $this->mediaFiles->createMovieDirectory($title, $mediaInfo->media->year);
             }
             return $this->mediaFiles->getMoviesPath();
         }
 
         if ($mediaType === 'tvshows') {
-            return $this->mediaFiles->createTvShowDirectory($title);
+            return $this->mediaFiles->createTvShowDirectory($title, $mediaInfo->media->year);
         }
 
         throw new \Exception("There is no download path for media type: $mediaType");

src/Monitor/Action/Handler/MonitorTvShowHandler.php

@@ -11,6 +11,7 @@ use App\Monitor\Framework\Entity\Monitor;
 use App\Monitor\Framework\Repository\MonitorRepository;
 use App\Tmdb\Dto\TmdbEpisodeDto;
 use App\Tmdb\TmdbClient;
+use App\Tmdb\TmdbResult;
 use Carbon\Carbon;
 use DateTimeImmutable;
 use Doctrine\ORM\EntityManagerInterface;
@@ -36,18 +37,14 @@ readonly class MonitorTvShowHandler implements HandlerInterface
     public function handle(CommandInterface $command): ResultInterface
     {
         $this->logger->info('> [MonitorTvShowHandler] Executing MonitorTvShowHandler');
+
         $monitor = $this->monitorRepository->find($command->monitorId);
-        $this->refreshData($monitor);
+        $showTmdbData = $this->tmdb->tvshowDetails($monitor->getImdbId());
+
+        $this->refreshData($monitor, $showTmdbData);
 
         // Check current episodes
-        $downloadedEpisodes = $this->mediaFiles
-            ->getEpisodes($monitor->getTitle())
-            ->map(fn($episode) => (object)(new PTN())->parse($episode))
-            ->filter(fn($episode) => property_exists($episode, 'episode')
-                && property_exists($episode, 'season')
-                && null !== $episode->episode
-                && null !== $episode->season
-            );
+        $downloadedEpisodes = $this->getDownloadedEpisodes($monitor->getTitle(), $showTmdbData);
 
         $this->logger->info('> [MonitorTvShowHandler] Found ' . count($downloadedEpisodes) . ' downloaded episodes for title: ' . $monitor->getTitle());
 
@@ -130,6 +127,33 @@ readonly class MonitorTvShowHandler implements HandlerInterface
         );
     }
 
+    private function getDownloadedEpisodes(string $title, TmdbResult $showTmdbData): Map
+    {
+        // Episodes in folder w/o the year
+        $downloadedEpisodes = $this->mediaFiles
+            ->getEpisodes($title)
+            ->map(fn($episode) => (object) new PTN()->parse($episode))
+            ->filter(fn($episode) =>
+                property_exists($episode, 'episode')
+                && property_exists($episode, 'season')
+                && null !== $episode->episode
+                && null !== $episode->season
+            );
+
+        return $downloadedEpisodes->concat(
+            // Episodes in folder w/ the year
+            $this->mediaFiles
+                ->getEpisodes(sprintf("%s (%s)", $title, $showTmdbData->year))
+                ->map(fn($episode) => (object) new PTN()->parse($episode))
+                ->filter(fn($episode) =>
+                    property_exists($episode, 'episode')
+                    && property_exists($episode, 'season')
+                    && null !== $episode->episode
+                    && null !== $episode->season
+                )
+        );
+    }
+
     private function episodeReleasedAfterMonitorCreated(
         string|DateTimeImmutable $monitorStartDate,
         TmdbEpisodeDto $episodeInShow
@@ -159,11 +183,11 @@ readonly class MonitorTvShowHandler implements HandlerInterface
             ]) !== null;
     }
 
-    private function refreshData(Monitor $monitor)
+    private function refreshData(Monitor $monitor, TmdbResult $showTmdbData)
     {
         if (null === $monitor->getPoster()) {
             $this->logger->info('> [MonitorTvShowHandler] Refreshing poster for "' . $monitor->getTitle() . '"');
-            $poster = $this->tmdb->tvshowDetails($monitor->getImdbId())->poster;
+            $poster = $showTmdbData->poster;
             if (null !== $poster && "" !== $poster) {
                 $monitor->setPoster($poster);
             }

src/Monitor/MonitorTypes.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace App\Monitor;
+
+enum MonitorTypes: string
+{
+    case TV_EPISODE = "tvepisode";
+    case TV_SEASON = "tvseason";
+    case TV_SERIES = "tvseries";
+}