App\Console\Commands\NntmuxResetTvShowPostProcessing::handle()

Complexity

Conditions	5
Paths	4

Size

Total Lines	57
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(): int
    {
        $videosId = (int) $this->argument('videos_id');
        $dryRun = (bool) $this->option('dry-run');

        // Validate target show exists and is TV type (0)
        $video = Video::query()->where('id', $videosId)->where('type', 0)->first(['id', 'title']);
        if ($video === null) {
            $this->error("videos_id {$videosId} not found or is not a TV show (type=0)");

            return self::FAILURE;
        }

        // Gather releases to reset
        $baseQuery = Release::query()
            ->select(['id'])
            ->where('videos_id', $videosId)
            ->whereBetween('categories_id', [Category::TV_ROOT, Category::TV_OTHER]);

        $total = (clone $baseQuery)->count();
        if ($total === 0) {
            $this->info('No releases found for this videos_id in TV categories');

            return self::SUCCESS;
        }

        if ($dryRun) {
            $this->line("[Dry Run] Would reset {$total} release(s) for TV show: {$video->title} (videos_id={$videosId})");

            return self::SUCCESS;
        }

        $this->info("Resetting postprocessing for TV show: {$video->title} (videos_id={$videosId})");

        // Progress bar
        $bar = $this->output->createProgressBar($total);

It seems like $total can also be of type Illuminate\Database\Eloquent\Builder and Illuminate\Database\Query\Builder; however, parameter $max of Symfony\Component\Console\Style\SymfonyStyle::createProgressBar() does only seem to accept integer, maybe add an additional type check?

        $bar->setOverwrite(true);
        $bar->start();

        // Chunk through releases to avoid loading all models simultaneously
        $baseQuery->chunkById(1000, function ($chunk) use ($bar) {
            foreach ($chunk as $release) {
                Release::query()->where('id', $release->id)->update([
                    // Reset association so TV pipeline picks them up again
                    'videos_id' => 0,
                    'tv_episodes_id' => 0,
                ]);
                $bar->advance();
            }
        });

        $bar->finish();
        $this->newLine();
        $this->info(number_format($total).' release(s) reset. They will be considered for TV postprocessing again.');

It seems like $total can also be of type Illuminate\Database\Eloquent\Builder and Illuminate\Database\Query\Builder; however, parameter $num of number_format() does only seem to accept double, maybe add an additional type check?

        $this->line('You can now run: php artisan postprocess:tv-pipeline <guid-bucket> (or your existing TV processing routines)');

        return self::SUCCESS;
    }