AssetLibrary::removeByIds() - Code Metrics - Inspection of "0.6" - thinktomorrow/assetlibrary - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

by Philippe

created 2019-10-05 12:54 UTC

AssetLibrary::removeByIds() A

↳ Parent: AssetLibrary

Complexity

Conditions	4
Paths	4

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace Thinktomorrow\AssetLibrary\Models;

use Illuminate\Support\Collection;

class AssetLibrary
{
    /**
     * Removes one or more assets by their ids.
     */
    public static function removeByIds($imageIds)
    {
        if (is_array($imageIds)) {
            foreach ($imageIds as $id) {
                self::remove($id);
            }
        } else {
            if (! $imageIds) {
                return;
            }

            self::remove($imageIds);
        }
    }

    /**
     * Removes one assets by id.
     * It also checks if you have the permissions to remove the file.
     *
     * @param $imageIds
     */
    public static function remove($id)
    {
        if (! $id) {
            return false;
        }

        $asset = Asset::find($id)->first();
        $media = $asset->media;

        foreach ($media as $file) {
            if (! is_file(public_path($file->getUrl())) || ! is_writable(public_path($file->getUrl()))) {
                return;
            }
        }

        $asset->delete();
    }

    /**
     * Returns a collection of all the assets in the library.
     * @return \Illuminate\Support\Collection
     */
    public static function getAllAssets(): Collection
    {
        return Asset::all()->sortByDesc('created_at');
    }
}


1		<?php
2
3		namespace Thinktomorrow\AssetLibrary\Models;
4
5		use Illuminate\Support\Collection;
6
7		class AssetLibrary
8		{
9		/**
10		* Removes one or more assets by their ids.
11		*/
12	5	public static function removeByIds($imageIds)
13		{
14	5	if (is_array($imageIds)) {
15	2	foreach ($imageIds as $id) {
16	2	self::remove($id);
17		}
18		} else {
19	4	if (! $imageIds) {
20	1	return;
21		}
22
23	3	self::remove($imageIds);
24		}
25	5	}
26
27		/**
28		* Removes one assets by id.
29		* It also checks if you have the permissions to remove the file.
30		*
31		* @param $imageIds
32		*/
33	5	public static function remove($id)
34		{
35	5	if (! $id) {
36	1	return false;
37		}
38
39	4	$asset = Asset::find($id)->first();
40	4	$media = $asset->media;
41
42	4	foreach ($media as $file) {
43	4	if (! is_file(public_path($file->getUrl())) \|\| ! is_writable(public_path($file->getUrl()))) {
44	4	return;
45		}
46		}
47
48	4	$asset->delete();
49	4	}
50
51		/**
52		* Returns a collection of all the assets in the library.
53		* @return \Illuminate\Support\Collection
54		*/
55	7	public static function getAllAssets(): Collection
56		{
57	7	return Asset::all()->sortByDesc('created_at');
58		}
59		}
60

thinktomorrow / assetlibrary

Pull Request — master (#38)

AssetLibrary::removeByIds() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like