TrashManager::delete() - Code Metrics - Inspection of "change codecoverage in readme badge" - kaliop-uk/ezmigrationbundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5a501b...3982a6 )

by Gaetano

created 2018-06-23 17:41 UTC

TrashManager::delete() A

↳ Parent: TrashManager

Complexity

Conditions	4
Paths	3

Size

Total Lines

Duplication

Lines	3
Ratio	15.79 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

Metric	Value
dl	3
loc	19
ccs	0
cts	14
cp	0
rs	9.6333
c	0
b	0
f	0
cc	4
nc	3
nop	1
crap	20

<?php

namespace Kaliop\eZMigrationBundle\Core\Executor;

use Kaliop\eZMigrationBundle\API\Collection\LocationCollection;
use Kaliop\eZMigrationBundle\API\Collection\TrashedItemCollection;
use Kaliop\eZMigrationBundle\Core\Matcher\TrashMatcher;
use Kaliop\eZMigrationBundle\Core\Helper\SortConverter;

/**
 * Handles trash migrations.
 */
class TrashManager extends RepositoryExecutor
{
    protected $supportedActions = array('purge', 'recover', 'delete');
    protected $supportedStepTypes = array('trash');

    /** @var TrashMatcher $trashMatcher */
    protected $trashMatcher;

    protected $sortConverter;

    /**
     * @param TrashMatcher $trashMatcher
     */
    public function __construct(TrashMatcher $trashMatcher, SortConverter $sortConverter)
    {
        $this->trashMatcher = $trashMatcher;
        $this->sortConverter = $sortConverter;
    }

    /**
     * Handles emptying the trash
     */
    protected function purge($step)

    {
        $trashService = $this->repository->getTrashService();

        $trashService->emptyTrash();

        return true;
    }

    /**
     * Handles the trash-restore migration action
     *
     * @todo support handling of restoration to custom locations
     */
    protected function recover($step)
    {
        $itemsCollection = $this->matchItems('restore', $step);

        if (count($itemsCollection) > 1 && array_key_exists('references', $step->dsl)) {

            throw new \Exception("Can not execute Trash restore because multiple types match, and a references section is specified in the dsl. References can be set when only 1 section matches");
        }

        $locations = array();
        $trashService = $this->repository->getTrashService();
        foreach ($itemsCollection as $key => $item) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
            $locations[] = $trashService->recover($item);
        }

        $this->setReferences(new LocationCollection($locations), $step);

        return $itemsCollection;
    }

    /**
     * Handles the trash-delete migration action
     */
    protected function delete($step)
    {
        $itemsCollection = $this->matchItems('delete', $step);

        if (count($itemsCollection) > 1 && array_key_exists('references', $step->dsl)) {

            throw new \Exception("Can not execute Trash restore because multiple types match, and a references section is specified in the dsl. References can be set when only 1 section matches");
        }

        $this->setReferences($itemsCollection, $step);
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}

        $trashService = $this->repository->getTrashService();
        foreach ($itemsCollection as $key => $item) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
            $trashService->deleteTrashItem($item);
        }

        $this->setReferences($itemsCollection, $step);
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}

        return $itemsCollection;
    }

    /**
     * @param string $action
     * @return TrashedItemCollection
     * @throws \Exception
     */
    protected function matchItems($action, $step)

    {
        if (!isset($step->dsl['match'])) {
            throw new \Exception("A match condition is required to $action trash items");
        }

        // convert the references passed in the match
        $match = $this->resolveReferencesRecursively($step->dsl['match']);

        return $this->trashMatcher->match($match);
    }

    /**
     * @param \eZ\Publish\API\Repository\Values\Content\TrashItem|\eZ\Publish\API\Repository\Values\Content\Location $item
     * @param array $references the definitions of the references to set
     * @throws \InvalidArgumentException When trying to assign a reference to an unsupported attribute
     * @return array key: the reference names, values: the reference values
     */
    protected function getReferencesValues($item, array $references, $step)

    {
        $refs = array();

        foreach ($references as $reference) {
            switch ($reference['attribute']) {
                // a trashed item extends a location, so in theory everything 'location' here should work
                case 'location_id':
                case 'id':
                    $value = $item->id;
                    break;
                case 'remote_id':
                case 'location_remote_id':
                    $value = $item->remoteId;
                    break;
                case 'always_available':
                    $value = $item->contentInfo->alwaysAvailable;
                    break;
                case 'content_id':
                    $value = $item->contentId;
                    break;
                case 'content_type_id':
                    $value = $item->contentInfo->contentTypeId;
                    break;
                case 'content_type_identifier':
                    $contentTypeService = $this->repository->getContentTypeService();
                    $value = $contentTypeService->loadContentType($item->contentInfo->contentTypeId)->identifier;
                    break;
                case 'current_version':
                case 'current_version_no':
                    $value = $item->contentInfo->currentVersionNo;
                    break;
                case 'depth':
                    $value = $item->depth;
                    break;
                case 'is_hidden':
                    $value = $item->hidden;
                    break;
                case 'main_location_id':
                    $value = $item->contentInfo->mainLocationId;
                    break;
                case 'main_language_code':
                    $value = $item->contentInfo->mainLanguageCode;
                    break;
                case 'modification_date':
                    $value = $item->contentInfo->modificationDate->getTimestamp();
                    break;
                case 'name':
                    $value = $item->contentInfo->name;
                    break;
                case 'owner_id':
                    $value = $item->contentInfo->ownerId;
                    break;
                case 'parent_location_id':
                    $value = $item->parentLocationId;
                    break;
                case 'path':
                    $value = $item->pathString;
                    break;
                case 'priority':
                    $value = $item->priority;
                    break;
                case 'publication_date':
                    $value = $item->contentInfo->publishedDate->getTimestamp();
                    break;
                case 'section_id':
                    $value = $item->contentInfo->sectionId;
                    break;
                case 'section_identifier':
                    $sectionService = $this->repository->getSectionService();
                    $value = $sectionService->loadSection($item->contentInfo->sectionId)->identifier;
                    break;
                case 'sort_field':
                    $value = $this->sortConverter->sortField2Hash($item->sortField);
                    break;
                case 'sort_order':
                    $value = $this->sortConverter->sortOrder2Hash($item->sortOrder);
                    break;
                default:
                    throw new \InvalidArgumentException('Trash Manager does not support setting references for attribute ' . $reference['attribute']);
            }

            $refs[$reference['identifier']] = $value;
        }

        return $refs;
    }
}


1		<?php
2
3		namespace Kaliop\eZMigrationBundle\Core\Executor;
4
5		use Kaliop\eZMigrationBundle\API\Collection\LocationCollection;
6		use Kaliop\eZMigrationBundle\API\Collection\TrashedItemCollection;
7		use Kaliop\eZMigrationBundle\Core\Matcher\TrashMatcher;
8		use Kaliop\eZMigrationBundle\Core\Helper\SortConverter;
9
10		/**
11		* Handles trash migrations.
12		*/
13		class TrashManager extends RepositoryExecutor
14		{
15		protected $supportedActions = array('purge', 'recover', 'delete');
16		protected $supportedStepTypes = array('trash');
17
18		/** @var TrashMatcher $trashMatcher */
19		protected $trashMatcher;
20
21		protected $sortConverter;
22
23		/**
24		* @param TrashMatcher $trashMatcher
25		*/
26		public function __construct(TrashMatcher $trashMatcher, SortConverter $sortConverter)
27		{
28		$this->trashMatcher = $trashMatcher;
29		$this->sortConverter = $sortConverter;
30		}
31
32		/**
33		* Handles emptying the trash
34		*/
35		protected function purge($step)
		0 ignored issues – show Unused Code introduced 2017-10-28 23:27 UTC by Report Bug Copy Issue Report The parameter `$step` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
36		{
37		$trashService = $this->repository->getTrashService();
38
39		$trashService->emptyTrash();
40
41		return true;
42		}
43
44		/**
45		* Handles the trash-restore migration action
46		*
47		* @todo support handling of restoration to custom locations
48		*/
49		protected function recover($step)
50		{
51		$itemsCollection = $this->matchItems('restore', $step);
52
53	View Code Duplication	if (count($itemsCollection) > 1 && array_key_exists('references', $step->dsl)) {
		0 ignored issues – show Duplication introduced 2017-11-09 09:37 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
54		throw new \Exception("Can not execute Trash restore because multiple types match, and a references section is specified in the dsl. References can be set when only 1 section matches");
55		}
56
57		$locations = array();
58		$trashService = $this->repository->getTrashService();
59		foreach ($itemsCollection as $key => $item) {
		0 ignored issues – show Bug introduced 2017-10-28 23:27 UTC by Report Bug Copy Issue Report The expression `$itemsCollection` of type `object<Kaliop\eZMigratio...hedItemCollection>\|null` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
60		$locations[] = $trashService->recover($item);
61		}
62
63		$this->setReferences(new LocationCollection($locations), $step);
64
65		return $itemsCollection;
66		}
67
68		/**
69		* Handles the trash-delete migration action
70		*/
71		protected function delete($step)
72		{
73		$itemsCollection = $this->matchItems('delete', $step);
74
75	View Code Duplication	if (count($itemsCollection) > 1 && array_key_exists('references', $step->dsl)) {
		0 ignored issues – show Duplication introduced 2017-11-09 09:37 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
76		throw new \Exception("Can not execute Trash restore because multiple types match, and a references section is specified in the dsl. References can be set when only 1 section matches");
77		}
78
79		$this->setReferences($itemsCollection, $step);
		0 ignored issues – show Bug introduced 2018-05-13 13:57 UTC by Report Bug Copy Issue Report It seems like `$itemsCollection` defined by `$this->matchItems('delete', $step)` on line `73` can be `null`; however, `Kaliop\eZMigrationBundle...ecutor::setReferences()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
80
81		$trashService = $this->repository->getTrashService();
82		foreach ($itemsCollection as $key => $item) {
		0 ignored issues – show Bug introduced 2017-10-28 23:27 UTC by Report Bug Copy Issue Report The expression `$itemsCollection` of type `object<Kaliop\eZMigratio...hedItemCollection>\|null` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
83		$trashService->deleteTrashItem($item);
84		}
85
86		$this->setReferences($itemsCollection, $step);
		0 ignored issues – show Bug introduced 2018-05-13 13:57 UTC by Report Bug Copy Issue Report It seems like `$itemsCollection` defined by `$this->matchItems('delete', $step)` on line `73` can be `null`; however, `Kaliop\eZMigrationBundle...ecutor::setReferences()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
87
88		return $itemsCollection;
89		}
90
91		/**
92		* @param string $action
93		* @return TrashedItemCollection
94		* @throws \Exception
95		*/
96	View Code Duplication	protected function matchItems($action, $step)
		0 ignored issues – show Duplication introduced 2017-08-29 17:16 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
97		{
98		if (!isset($step->dsl['match'])) {
99		throw new \Exception("A match condition is required to $action trash items");
100		}
101
102		// convert the references passed in the match
103		$match = $this->resolveReferencesRecursively($step->dsl['match']);
104
105		return $this->trashMatcher->match($match);
106		}
107
108		/**
109		* @param \eZ\Publish\API\Repository\Values\Content\TrashItem\|\eZ\Publish\API\Repository\Values\Content\Location $item
110		* @param array $references the definitions of the references to set
111		* @throws \InvalidArgumentException When trying to assign a reference to an unsupported attribute
112		* @return array key: the reference names, values: the reference values
113		*/
114	View Code Duplication	protected function getReferencesValues($item, array $references, $step)
		0 ignored issues – show Duplication introduced 2018-06-22 22:02 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
115		{
116		$refs = array();
117
118		foreach ($references as $reference) {
119		switch ($reference['attribute']) {
120		// a trashed item extends a location, so in theory everything 'location' here should work
121		case 'location_id':
122		case 'id':
123		$value = $item->id;
124		break;
125		case 'remote_id':
126		case 'location_remote_id':
127		$value = $item->remoteId;
128		break;
129		case 'always_available':
130		$value = $item->contentInfo->alwaysAvailable;
131		break;
132		case 'content_id':
133		$value = $item->contentId;
134		break;
135		case 'content_type_id':
136		$value = $item->contentInfo->contentTypeId;
137		break;
138		case 'content_type_identifier':
139		$contentTypeService = $this->repository->getContentTypeService();
140		$value = $contentTypeService->loadContentType($item->contentInfo->contentTypeId)->identifier;
141		break;
142		case 'current_version':
143		case 'current_version_no':
144		$value = $item->contentInfo->currentVersionNo;
145		break;
146		case 'depth':
147		$value = $item->depth;
148		break;
149		case 'is_hidden':
150		$value = $item->hidden;
151		break;
152		case 'main_location_id':
153		$value = $item->contentInfo->mainLocationId;
154		break;
155		case 'main_language_code':
156		$value = $item->contentInfo->mainLanguageCode;
157		break;
158		case 'modification_date':
159		$value = $item->contentInfo->modificationDate->getTimestamp();
160		break;
161		case 'name':
162		$value = $item->contentInfo->name;
163		break;
164		case 'owner_id':
165		$value = $item->contentInfo->ownerId;
166		break;
167		case 'parent_location_id':
168		$value = $item->parentLocationId;
169		break;
170		case 'path':
171		$value = $item->pathString;
172		break;
173		case 'priority':
174		$value = $item->priority;
175		break;
176		case 'publication_date':
177		$value = $item->contentInfo->publishedDate->getTimestamp();
178		break;
179		case 'section_id':
180		$value = $item->contentInfo->sectionId;
181		break;
182		case 'section_identifier':
183		$sectionService = $this->repository->getSectionService();
184		$value = $sectionService->loadSection($item->contentInfo->sectionId)->identifier;
185		break;
186		case 'sort_field':
187		$value = $this->sortConverter->sortField2Hash($item->sortField);
188		break;
189		case 'sort_order':
190		$value = $this->sortConverter->sortOrder2Hash($item->sortOrder);
191		break;
192		default:
193		throw new \InvalidArgumentException('Trash Manager does not support setting references for attribute ' . $reference['attribute']);
194		}
195
196		$refs[$reference['identifier']] = $value;
197		}
198
199		return $refs;
200		}
201		}
202

kaliop-uk / ezmigrationbundle

Push — master ( 5a501b...3982a6 )

TrashManager::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like