Thinktomorrow\Chief\MediaGallery\Application\RemovalAction

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	26	4

<?php
declare(strict_types=1);

namespace Thinktomorrow\Chief\MediaGallery\Application;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use Thinktomorrow\Chief\Audit\Audit;
use Thinktomorrow\AssetLibrary\Application\DeleteAsset;
use Thinktomorrow\AssetLibrary\Exceptions\FileNotAccessibleException;

final class RemovalAction
{
    /** @var DeleteAsset */
    private $deleteAsset;

    public function __construct(DeleteAsset $deleteAsset)
    {
        $this->deleteAsset = $deleteAsset;
    }

    public function handle(Request $request)
    {
        // Strict protection enabled: we won't remove assets who are still being used...
        $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId){
            return DB::table('asset_pivots')
                ->where('asset_id', $assetId)
                ->where('unused', 0)
                ->exists();
        })->toArray();

        if($assetIds)

The expression $assetIds of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

        {
                foreach($assetIds as $k => $assetId) {
                    try{
                        $this->deleteAsset->remove($assetId);
                    } catch(FileNotAccessibleException $e) {
                        unset($assetIds[$k]); // So our count of removed assets is correct in the log.
                    }
                }

                Audit::activity()->log('removed ' . count($assetIds) . ' assets from the mediagallery.');

                return true;
        }

        return false;
    }
}