DetachAsset::detachAll() - Code Metrics - Inspection of "Fix: detaching asset with entity id being uuid fai..." - thinktomorrow/assetlibrary - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 8a9f0f...befa7d )

by Ben

created 2023-03-06 12:23 UTC

DetachAsset::detachAll() A

↳ Parent: DetachAsset

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
cc	2
eloc	4
c	2
b	0
f	1
nc	2
nop	2
dl	0
loc	7
ccs	5
cts	5
cp	1
crap	2
rs	10

<?php

namespace Thinktomorrow\AssetLibrary\Application;

use Thinktomorrow\AssetLibrary\HasAsset;

class DetachAsset
{
    /**
     * Detaches an asset from a model.
     *
     * @param $ids
     */
    public function detach(HasAsset $model, $ids, $type, $locale): void
    {
        if (! is_array($ids)) {
            $ids = (array) $ids;
        }

        $ids = $this->ensureParameterIsString($ids);

        foreach ($ids as $id) {
            $model->assetRelation()->where('asset_pivots.type', $type)->where('asset_pivots.locale', $locale)->detach($id);
        }
    }

    /**
     * Detaches all assets or for a specific type from a model.
     *
     * @param $ids
     */
    public function detachAll(HasAsset $model, ?string $type = null): void
    {
        $assetIds = $type
            ? $model->assetRelation()->where('asset_pivots.type', $type)->get()->pluck('id')
            : $model->assetRelation()->get()->pluck('id');

        $model->assetRelation()->detach($assetIds);
    }

    /**
     * @param mixed $ids
     * @return string[]
     */
    public function ensureParameterIsString(mixed $ids): array
    {
        return array_map(fn($id) => (string)$id, $ids);
    }
}


1		<?php
2
3		namespace Thinktomorrow\AssetLibrary\Application;
4
5		use Thinktomorrow\AssetLibrary\HasAsset;
6
7		class DetachAsset
8		{
9		/**
10		* Detaches an asset from a model.
11		*
12		* @param $ids
13		*/
14	5	public function detach(HasAsset $model, $ids, $type, $locale): void
15		{
16	5	if (! is_array($ids)) {
17	4	$ids = (array) $ids;
18		}
19
20	5	$ids = $this->ensureParameterIsString($ids);
21	5
22		foreach ($ids as $id) {
23	5	$model->assetRelation()->where('asset_pivots.type', $type)->where('asset_pivots.locale', $locale)->detach($id);
24		}
25		}
26
27		/**
28		* Detaches all assets or for a specific type from a model.
29		*
30	2	* @param $ids
31		*/
32	2	public function detachAll(HasAsset $model, ?string $type = null): void
33		{
34	2	$assetIds = $type
35	1	? $model->assetRelation()->where('asset_pivots.type', $type)->get()->pluck('id')
36	1	: $model->assetRelation()->get()->pluck('id');
37
38	1	$model->assetRelation()->detach($assetIds);
39		}
40	2
41		/**
42		* @param mixed $ids
43		* @return string[]
44		*/
45		public function ensureParameterIsString(mixed $ids): array
46		{
47		return array_map(fn($id) => (string)$id, $ids);
48		}
49		}
50

thinktomorrow / assetlibrary

Push — master ( 8a9f0f...befa7d )

DetachAsset::detachAll() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like