eZ\Publish\Core\Repository\ContentService::removeTranslation()

Complexity

Conditions	8
Paths	21

Size

Total Lines	66
Code Lines	38

Duplication

Lines	10
Ratio	15.15 %

Importance

Changes

0

<?php
    public function removeTranslation(ContentInfo $contentInfo, $languageCode)
    {
        if ($contentInfo->mainLanguageCode === $languageCode) {
            throw new BadStateException(
                '$languageCode',
                'Specified translation is the main translation of the Content Object'
            );
        }
        // before actual removal, collect information on Versions
        $versions = [];
        foreach ($this->loadVersions($contentInfo) as $versionInfo) {
            // check if user is authorized to delete Version
            if (!$this->repository->canUser('content', 'delete', $versionInfo)) {

The method eZ\Publish\Core\Repository\Repository::canUser() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::canUser() instead. Check if user has access to a given action on a given value object. Indicates if the current user is allowed to perform an action given by the function on the given
objects.

This code seems to be duplicated across your project.

                throw new UnauthorizedException(
                    'content',
                    'delete',
                    [
                        'contentId' => $contentInfo->id,
                        'versionNo' => $versionInfo->versionNo,
                    ]
                );
            }
            // check if the specified translation exists for the Version
            if (!in_array($languageCode, $versionInfo->languageCodes)) {
                // if translation does not exist, simply ignore Version (see InvalidArgumentException later on)
                continue;
            }
            // check if the specified translation is not the only one
            if (count($versionInfo->languageCodes) < 2) {
                throw new BadStateException(
                    '$languageCode',
                    'Specified translation is the only one Content Object Version has'
                );
            }
            // add version to the list
            $versions[] = $versionInfo->versionNo;
        }

        // if there are no Versions with the given translation, $languageCode arg is invalid
        if (empty($versions)) {
            throw new InvalidArgumentException(
                '$languageCode',
                sprintf(
                    'Specified translation does not exist in the Content Object(id=%d)',
                     $contentInfo->id
                )
            );
        }

        $this->repository->beginTransaction();
        try {
            $this->persistenceHandler->contentHandler()->removeTranslationFromContent($contentInfo->id, $languageCode);
            $locationIds = array_map(
                function (Location $location) {
                    return $location->id;

The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

                },
                $this->repository->getLocationService()->loadLocations($contentInfo)
            );
            $this->persistenceHandler->urlAliasHandler()->translationRemoved($locationIds, $languageCode);
            $this->repository->commit();
        } catch (Exception $e) {
            $this->repository->rollback();
            // cover generic unexpected exception to fulfill API promise on @throws
            throw new BadStateException('$contentInfo', 'Translation removal failed', $e);
        }
    }