Thinktomorrow\AssetLibrary\Application\ReorderAssets

Complexity

Total Complexity

5

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	batchUpdateColumn()	0	22	4
A	handle()	0	16	1

<?php

namespace Thinktomorrow\AssetLibrary\Application;

use Illuminate\Support\Facades\DB;
use Thinktomorrow\AssetLibrary\Asset;
use Thinktomorrow\AssetLibrary\HasAsset;

class ReorderAssets
{
    public function handle(HasAsset $model, string $type, string $locale, array $orderedAssetIds): void
    {
        $model->assetRelation()
            ->where('assets_pivot.type', $type)
            ->where('assets_pivot.locale', $locale)
            ->get()
            ->each(function(Asset $asset) use($model, $orderedAssetIds, $type, $locale){

                DB::table('assets_pivot')
                    ->where('asset_id', $asset->id)
                    ->where('entity_type', $model->getMorphClass())

The method getMorphClass() does not exist on Thinktomorrow\AssetLibrary\HasAsset. Since it exists in all sub-types, consider adding an abstract or default implementation to Thinktomorrow\AssetLibrary\HasAsset.

                    ->where('entity_id', $model->getKey())

The method getKey() does not exist on Thinktomorrow\AssetLibrary\HasAsset. Since it exists in all sub-types, consider adding an abstract or default implementation to Thinktomorrow\AssetLibrary\HasAsset.

                    ->where('type', $type)
                    ->where('locale', $locale)
                    ->update([
                        'order' => array_search($asset->id, $orderedAssetIds),
                    ]);
            });

    }

    /**
     * Taken from: https://github.com/laravel/ideas/issues/575
     *
     * @return void
     */
    private static function batchUpdateColumn(string $table, string $column, array $indices, string $indexColumn = 'id', bool $castIdToIntegers = true, string $extraWhere = null): void

The method batchUpdateColumn() is not used, and could be removed.

    {
        $cases = [];
        $ids = [];
        $params = [];

        foreach ($indices as $index => $modelId) {
            $id = $castIdToIntegers ? (int) $modelId : $modelId;
            $ids[] = "'{$id}'";

            $cases[] = "WHEN '{$id}' then ?";
            $params[] = $index;
        }

        $ids = implode(',', $ids);
        $cases = implode(' ', $cases);

        if ($extraWhere) {
            $extraWhere = ' AND ' . DB::raw($extraWhere);
        }

        DB::update("UPDATE `{$table}` SET `{$column}` = CASE `".$indexColumn."` {$cases} END WHERE `".$indexColumn."` in ({$ids})".$extraWhere, $params);
    }
}