MediaLibraryEventHandler::whenMediaHasBeenAdded() - Code Metrics - spatie-custom/blender - Measure and Improve Code Quality continuously with Scrutinizer

MediaLibraryEventHandler::whenMediaHasBeenAdded() A
last analyzed 2017-01-20 14:45 UTC

↳ Parent: MediaLibraryEventHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	6
nc	1
nop	1
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0

<?php

namespace App\Services\Medialibrary;

use Spatie\Color\Rgb;
use ColorThief\ColorThief;
use Illuminate\Contracts\Events\Dispatcher;
use Spatie\MediaLibrary\Events\MediaHasBeenAdded;

class MediaLibraryEventHandler
{
    public function subscribe(Dispatcher $events)
    {
        $events->listen(
            MediaHasBeenAdded::class,
            [$this, 'whenMediaHasBeenAdded']
        );
    }

    public function whenMediaHasBeenAdded(MediaHasBeenAdded $event)
    {
        $media = $event->media;

        $dominantColor = ColorThief::getColor($media->getPath());

        $hexColor = (new Rgb(...$dominantColor))->toHex();
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $media->setCustomProperty('dominantColor', $hexColor);

        $media->save();
    }
}


1			<?php
2
3			namespace App\Services\Medialibrary;
4
5			use Spatie\Color\Rgb;
6			use ColorThief\ColorThief;
7			use Illuminate\Contracts\Events\Dispatcher;
8			use Spatie\MediaLibrary\Events\MediaHasBeenAdded;
9
10			class MediaLibraryEventHandler
11			{
12			public function subscribe(Dispatcher $events)
13			{
14			$events->listen(
15			MediaHasBeenAdded::class,
16			[$this, 'whenMediaHasBeenAdded']
17			);
18			}
19
20			public function whenMediaHasBeenAdded(MediaHasBeenAdded $event)
21			{
22			$media = $event->media;
23
24			$dominantColor = ColorThief::getColor($media->getPath());
25
26			$hexColor = (new Rgb(...$dominantColor))->toHex();
			0 ignored issues – show Bug introduced 2016-10-18 11:59 UTC by Report Bug Copy Issue Report The call to `Rgb::__construct()` misses some required arguments starting with `$green`. Loading history... Documentation introduced 2016-10-18 11:59 UTC by Report Bug Copy Issue Report `$dominantColor` is of type `array\|boolean`, but the function expects a `integer`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
27
28			$media->setCustomProperty('dominantColor', $hexColor);
29
30			$media->save();
31			}
32			}
33

spatie-custom / blender

MediaLibraryEventHandler::whenMediaHasBeenAdded() A last analyzed 2017-01-20 14:45 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

MediaLibraryEventHandler::whenMediaHasBeenAdded() A
last analyzed 2017-01-20 14:45 UTC