AbstractManipulator::performConversions() - Code Metrics - bytic/MediaLibrary - Measure and Improve Code Quality continuously with Scrutinizer

AbstractManipulator::performConversions() A
last analyzed 2024-07-12 11:13 UTC

↳ Parent: AbstractManipulator

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	5.1971

Importance

Changes

Metric	Value
cc	3
eloc	7
nc	3
nop	2
dl	0
loc	13
ccs	3
cts	8
cp	0.375
crap	5.1971
rs	10
c	0
b	0
f	0

<?php

namespace ByTIC\MediaLibrary\Media\Manipulators;

use ByTIC\MediaLibrary\Conversions\Conversion;
use ByTIC\MediaLibrary\Conversions\ConversionCollection;
use ByTIC\MediaLibrary\Media\Media;
use Nip\Collection;

/**
 * Class AbstractManipulator.
 */
abstract class AbstractManipulator implements ManipulatorInterface
{
    /**
     * @param Media $media
     *
     * @return bool
     */
    public function canConvert(Media $media): bool
    {
        if (!$this->requirementsAreInstalled()) {
            return false;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
        }

        if (!$this->supportedExtensions()->contains(strtolower($media->getExtension()))) {
            return false;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
        }
//        $urlGenerator = UrlGeneratorFactory::createForMedia($media);
//        if (method_exists($urlGenerator, 'getPath') && file_exists($media->getPath())
//            && $this->supportedMimetypes()->contains(strtolower(File::getMimetype($media->getPath())))) {
//            return true;
//        }
        return true;
    }

    /**
     * @param ConversionCollection $conversions
     * @param Media                $media
     *
     * @return int Number of convertions performed
     */
    public function performConversions(ConversionCollection $conversions, Media $media)
    {
        if ($conversions->isEmpty()) {
            return;
        }

        $i = 0;
        foreach ($conversions as $conversion) {
            $this->performConversion($media, $conversion);
            $i++;
        }

        return $i;
    }

    /**
     * @param Media      $media
     * @param Conversion $conversion
     */
    abstract public function performConversion(Media $media, Conversion $conversion);

    /**
     * @return bool
     */
    abstract public function requirementsAreInstalled(): bool;

    /**
     * @return Collection
     */
    abstract public function supportedExtensions(): Collection;

    /**
     * @return Collection
     */
    abstract public function supportedMimetypes(): Collection;
}


1		<?php
2
3		namespace ByTIC\MediaLibrary\Media\Manipulators;
4
5		use ByTIC\MediaLibrary\Conversions\Conversion;
6		use ByTIC\MediaLibrary\Conversions\ConversionCollection;
7		use ByTIC\MediaLibrary\Media\Media;
8		use Nip\Collection;
9
10		/**
11		* Class AbstractManipulator.
12		*/
13		abstract class AbstractManipulator implements ManipulatorInterface
14		{
15		/**
16		* @param Media $media
17		*
18		* @return bool
19		*/
20	4	public function canConvert(Media $media): bool
21		{
22	4	if (!$this->requirementsAreInstalled()) {
23		return false;
		0 ignored issues – show Bug Best Practice introduced 2019-01-29 16:27 UTC by Report Bug Copy Issue Report The expression `return false` returns the type `false` which is incompatible with the return type mandated by `ByTIC\MediaLibrary\Media...Interface::canConvert()` of `true`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
24		}
25
26	4	if (!$this->supportedExtensions()->contains(strtolower($media->getExtension()))) {
27	1	return false;
		0 ignored issues – show Bug Best Practice introduced 2019-01-29 16:27 UTC by Report Bug Copy Issue Report The expression `return false` returns the type `false` which is incompatible with the return type mandated by `ByTIC\MediaLibrary\Media...Interface::canConvert()` of `true`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
28		}
29		// $urlGenerator = UrlGeneratorFactory::createForMedia($media);
30		// if (method_exists($urlGenerator, 'getPath') && file_exists($media->getPath())
31		// && $this->supportedMimetypes()->contains(strtolower(File::getMimetype($media->getPath())))) {
32		// return true;
33		// }
34	3	return true;
35		}
36
37		/**
38		* @param ConversionCollection $conversions
39		* @param Media $media
40		*
41		* @return int Number of convertions performed
42		*/
43	3	public function performConversions(ConversionCollection $conversions, Media $media)
44		{
45	3	if ($conversions->isEmpty()) {
46	3	return;
47		}
48
49		$i = 0;
50		foreach ($conversions as $conversion) {
51		$this->performConversion($media, $conversion);
52		$i++;
53		}
54
55		return $i;
56		}
57
58		/**
59		* @param Media $media
60		* @param Conversion $conversion
61		*/
62		abstract public function performConversion(Media $media, Conversion $conversion);
63
64		/**
65		* @return bool
66		*/
67		abstract public function requirementsAreInstalled(): bool;
68
69		/**
70		* @return Collection
71		*/
72		abstract public function supportedExtensions(): Collection;
73
74		/**
75		* @return Collection
76		*/
77		abstract public function supportedMimetypes(): Collection;
78		}
79

bytic / MediaLibrary

AbstractManipulator::performConversions() A last analyzed 2024-07-12 11:13 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

AbstractManipulator::performConversions() A
last analyzed 2024-07-12 11:13 UTC