Issues in PageBuilder.php - All Issues - Inspection of "Merge branch '5.3'" - Kunstmaan/KunstmaanBundlesCMS - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 770316...74fc07 )

by Jeroen

created 2019-10-18 13:07 UTC

Kunstmaan/FixturesBundle/Builder/PageBuilder.php (1 issue)

Labels

Bug 1

Severity

Minor 1

Joeri Verdeyen 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace Kunstmaan\FixturesBundle\Builder;

use Doctrine\ORM\EntityManager;
use Kunstmaan\FixturesBundle\Loader\Fixture;
use Kunstmaan\FixturesBundle\Populator\Populator;
use Kunstmaan\NodeBundle\Entity\HasNodeInterface;
use Kunstmaan\NodeBundle\Entity\Node;
use Kunstmaan\NodeBundle\Entity\NodeTranslation;
use Kunstmaan\NodeBundle\Entity\NodeVersion;
use Kunstmaan\NodeBundle\Entity\StructureNode;
use Kunstmaan\NodeBundle\Helper\PagesConfiguration;
use Kunstmaan\NodeBundle\Helper\Services\ACLPermissionCreatorService;
use Kunstmaan\PagePartBundle\Entity\PageTemplateConfiguration;
use Kunstmaan\UtilitiesBundle\Helper\ClassLookup;
use Kunstmaan\UtilitiesBundle\Helper\Slugifier;

class PageBuilder implements BuilderInterface
{
    private $manager;

    private $userRepo;

    private $nodeRepo;

    private $nodeTranslationRepo;

    private $aclPermissionCreatorService;

    private $populator;

    private $slugifier;

    /**
     * @var PagesConfiguration
     */
    private $pagesConfiguration;

    public function __construct(
        EntityManager $em,
function someFunction(ManagerRegistry $registry) {
    $em = $registry->getManager();
    $em->getConnection()->beginTransaction();
    try {
        // Do something.
        $em->getConnection()->commit();
    } catch (\Exception $ex) {
        $em->getConnection()->rollback();
        $em->close();

        throw $ex;
    }
}
        ACLPermissionCreatorService $aclPermissionCreatorService,
        Populator $populator,
        Slugifier $slugifier,
        PagesConfiguration $pagesConfiguration,
        string $userClass
    ) {
        $this->manager = $em;
        $this->nodeRepo = $em->getRepository('KunstmaanNodeBundle:Node');
        $this->nodeTranslationRepo = $em->getRepository('KunstmaanNodeBundle:NodeTranslation');
        $this->userRepo = $em->getRepository($userClass);
        $this->aclPermissionCreatorService = $aclPermissionCreatorService;
        $this->populator = $populator;
        $this->slugifier = $slugifier;
        $this->pagesConfiguration = $pagesConfiguration;
    }

    public function canBuild(Fixture $fixture)
    {
        if ($fixture->getEntity() instanceof HasNodeInterface) {
            return true;
        }

        return false;
    }

    public function preBuild(Fixture $fixture)
    {
        return;
    }

    public function postBuild(Fixture $fixture)
    {
        $entity = $fixture->getEntity();
        $fixtureParams = $fixture->getParameters();
        $translations = $fixture->getTranslations();
        if (empty($translations)) {
            throw new \Exception('No translations detected for page fixture ' . $fixture->getName() . ' (' . $fixture->getClass() . ')');
        }

        $internalName = array_key_exists('page_internal_name', $fixtureParams) ?
            $fixtureParams['page_internal_name'] : null;

        $rootNode = null;
        foreach ($fixture->getTranslations() as $language => $data) {
            if ($rootNode === null) {
                $page = $entity;
                $rootNode = $this->createRootNode($page, $language, $internalName, $fixtureParams);
                $this->manager->persist($rootNode);
            } else {
                $cloned = clone $entity;
                $page = $cloned;
                $this->manager->persist($page);
            }

            // Create the translationNode.
            $translationNode = $this->createTranslationNode($rootNode, $language, $page);
            if (!$page instanceof StructureNode) {
                $translationNode->setOnline(isset($fixtureParams['set_online']) ? $fixtureParams['set_online'] : true);
            }

            $fixture->addAdditional($fixture->getName() . '_' . $language, $page);
            $fixture->addAdditional('translationNode_' . $language, $translationNode);
            $fixture->addAdditional('nodeVersion_' . $language, $translationNode->getPublicNodeVersion());
            $fixture->addAdditional('rootNode', $rootNode);

            $this->populator->populate($translationNode, $data);
            $this->populator->populate($page, $data);
            if ($translationNode->getSlug() === null && $rootNode->getParent() !== null) {
                $translationNode->setSlug($this->slugifier->slugify($translationNode->getTitle()));
            }
            $this->ensureUniqueUrl($translationNode, $page);

            $this->manager->persist($translationNode);
            $rootNode->addNodeTranslation($translationNode);
        }

        $this->manager->flush();
        $this->aclPermissionCreatorService->createPermission($rootNode);
    }

    public function postFlushBuild(Fixture $fixture)
    {
        $entities = $fixture->getAdditionalEntities();
        $fixtureParams = $fixture->getParameters();

        foreach ($fixture->getTranslations() as $language => $data) {
            /** @var HasNodeInterface $page */
            $page = $entities[$fixture->getName() . '_' . $language];
            /** @var NodeTranslation $translationNode */
            $translationNode = $entities['translationNode_' . $language];

            $pagecreator = array_key_exists('creator', $fixtureParams) ? $fixtureParams['creator'] : 'pagecreator';
            $creator = $this->userRepo->findOneBy(array('username' => $pagecreator));

            $nodeVersion = new NodeVersion();
            $nodeVersion->setNodeTranslation($translationNode);
            $nodeVersion->setType('public');
            $nodeVersion->setOwner($creator);
            $nodeVersion->setRef($page);

            $translationNode->setPublicNodeVersion($nodeVersion);

            if (isset($fixtureParams['template'])) {
                $pageTemplateConfiguration = new PageTemplateConfiguration();
                $pageTemplateConfiguration->setPageId($page->getId());
                $pageTemplateConfiguration->setPageEntityName(ClassLookup::getClass($page));
                $pageTemplateConfiguration->setPageTemplate($fixtureParams['template']);
                $this->manager->persist($pageTemplateConfiguration);
            }

            $this->manager->persist($nodeVersion);
            $this->manager->persist($translationNode);
        }
        $this->manager->flush();
    }

    private function getParentNode($params, $language)
    {
        if (!isset($params['parent'])) {
            return;
        }

        $parent = $params['parent'];
        if ($parent instanceof Fixture) {
            $additionals = $parent->getAdditionalEntities();
            $parent = $additionals['rootNode'];
        } elseif (is_string($parent)) {
            $nodes = $this->nodeRepo->getNodesByInternalName($parent, $language, false, true);
            if (count($nodes) > 0) {
                $parent = $nodes[0];
            }
        }

        return $parent;
    }

    private function createRootNode($page, $language, $internalName, $fixtureParams)
    {
        $rootNode = new Node();
        $rootNode->setRef($page);
        $rootNode->setDeleted(false);
        $rootNode->setInternalName($internalName);
        $rootNode->setHiddenFromNav(
            isset($fixtureParams['hidden_from_nav']) ? $fixtureParams['hidden_from_nav'] : false
        );
        $parent = $this->getParentNode($fixtureParams, $language);

        if ($parent instanceof Node) {
            $rootNode->setParent($parent);

            if (!$this->canHaveChild($parent->getRefEntityName(), get_class($page))) {
                throw new \Exception(
                    sprintf('A %s can\'t have a %s as child. Forgot to add in allowed_children or getPossibleChildTypes?', $parent->getRefEntityName(), get_class($page))
                );
            }
        }

        return $rootNode;
    }

    private function createTranslationNode(Node $rootNode, $language, HasNodeInterface $page)
    {
        $translationNode = new NodeTranslation();
        $translationNode
            ->setNode($rootNode)
            ->setLang($language)
            ->setTitle($page->getTitle())
            ->setOnline(false)
            ->setWeight(0);

        return $translationNode;
    }

    private function ensureUniqueUrl(NodeTranslation $translation, HasNodeInterface $page)
    {
        if ($page instanceof StructureNode) {
            $translation->setSlug('');
            $translation->setUrl($translation->getFullSlug());

            return $translation;
        }

        $translation->setUrl($translation->getFullSlug());

        // Find all translations with this new URL, whose nodes are not deleted.
        $translationWithSameUrl = $this->nodeTranslationRepo->getNodeTranslationForUrl($translation->getUrl(), $translation->getLang(), false, $translation);

        if ($translationWithSameUrl instanceof NodeTranslation) {
            $translation->setSlug($this->slugifier->slugify($this->incrementString($translation->getSlug())));
            $this->ensureUniqueUrl($translation, $page);
        }

        return $translation;
    }

    private function incrementString($string, $append = '-v')
    {
        $finalDigitGrabberRegex = '/\d+$/';
        $matches = array();

        preg_match($finalDigitGrabberRegex, $string, $matches);

        if (count($matches) > 0) {
            $digit = (int) $matches[0];
            ++$digit;

            // Replace the integer with the new digit.
            return preg_replace($finalDigitGrabberRegex, $digit, $string);
        } else {
            return $string . $append . '1';
        }
    }

    /**
     * @param string $parentPageClass
     * @param string $childPageClass
     *
     * @return bool
     */
    private function canHaveChild($parentPageClass, $childPageClass)
    {
        $childTypes = $this->pagesConfiguration->getPossibleChildTypes($parentPageClass);

        foreach ($childTypes as $childType) {
            if ($childType['class'] == $childPageClass) {
                return true;
            }
        }

        return false;
    }
}


1		<?php
2
3		namespace Kunstmaan\FixturesBundle\Builder;
4
5		use Doctrine\ORM\EntityManager;
6		use Kunstmaan\FixturesBundle\Loader\Fixture;
7		use Kunstmaan\FixturesBundle\Populator\Populator;
8		use Kunstmaan\NodeBundle\Entity\HasNodeInterface;
9		use Kunstmaan\NodeBundle\Entity\Node;
10		use Kunstmaan\NodeBundle\Entity\NodeTranslation;
11		use Kunstmaan\NodeBundle\Entity\NodeVersion;
12		use Kunstmaan\NodeBundle\Entity\StructureNode;
13		use Kunstmaan\NodeBundle\Helper\PagesConfiguration;
14		use Kunstmaan\NodeBundle\Helper\Services\ACLPermissionCreatorService;
15		use Kunstmaan\PagePartBundle\Entity\PageTemplateConfiguration;
16		use Kunstmaan\UtilitiesBundle\Helper\ClassLookup;
17		use Kunstmaan\UtilitiesBundle\Helper\Slugifier;
18
19		class PageBuilder implements BuilderInterface
20		{
21		private $manager;
22
23		private $userRepo;
24
25		private $nodeRepo;
26
27		private $nodeTranslationRepo;
28
29		private $aclPermissionCreatorService;
30
31		private $populator;
32
33		private $slugifier;
34
35		/**
36		* @var PagesConfiguration
37		*/
38		private $pagesConfiguration;
39
40		public function __construct(
41		EntityManager $em,
		0 ignored issues – show Bug introduced 2017-11-21 06:15 UTC by Report Bug Copy Issue Report Show Similar Issues like this You have injected the EntityManager via parameter `$em`. This is generally not recommended as it might get closed and become unusable. Instead, it is recommended to inject the ManagerRegistry and retrieve the EntityManager via `getManager()` each time you need it. The `EntityManager` might become unusable for example if a transaction is rolled back and it gets closed. Let’s assume that somewhere in your application, or in a third-party library, there is code such as the following: function someFunction(ManagerRegistry $registry) { $em = $registry->getManager(); $em->getConnection()->beginTransaction(); try { // Do something. $em->getConnection()->commit(); } catch (\Exception $ex) { $em->getConnection()->rollback(); $em->close(); throw $ex; } } If that code throws an exception and the `EntityManager` is closed. Any other code which depends on the same instance of the `EntityManager` during this request will fail. On the other hand, if you instead inject the `ManagerRegistry`, the `getManager()` method guarantees that you will always get a usable manager instance. Loading history...
42		ACLPermissionCreatorService $aclPermissionCreatorService,
43		Populator $populator,
44		Slugifier $slugifier,
45		PagesConfiguration $pagesConfiguration,
46		string $userClass
47		) {
48		$this->manager = $em;
49		$this->nodeRepo = $em->getRepository('KunstmaanNodeBundle:Node');
50		$this->nodeTranslationRepo = $em->getRepository('KunstmaanNodeBundle:NodeTranslation');
51		$this->userRepo = $em->getRepository($userClass);
52		$this->aclPermissionCreatorService = $aclPermissionCreatorService;
53		$this->populator = $populator;
54		$this->slugifier = $slugifier;
55		$this->pagesConfiguration = $pagesConfiguration;
56		}
57
58		public function canBuild(Fixture $fixture)
59		{
60		if ($fixture->getEntity() instanceof HasNodeInterface) {
61		return true;
62		}
63
64		return false;
65		}
66
67		public function preBuild(Fixture $fixture)
68		{
69		return;
70		}
71
72		public function postBuild(Fixture $fixture)
73		{
74		$entity = $fixture->getEntity();
75		$fixtureParams = $fixture->getParameters();
76		$translations = $fixture->getTranslations();
77	View Code Duplication	if (empty($translations)) {
78		throw new \Exception('No translations detected for page fixture ' . $fixture->getName() . ' (' . $fixture->getClass() . ')');
79		}
80
81		$internalName = array_key_exists('page_internal_name', $fixtureParams) ?
82		$fixtureParams['page_internal_name'] : null;
83
84		$rootNode = null;
85		foreach ($fixture->getTranslations() as $language => $data) {
86		if ($rootNode === null) {
87		$page = $entity;
88		$rootNode = $this->createRootNode($page, $language, $internalName, $fixtureParams);
89		$this->manager->persist($rootNode);
90		} else {
91		$cloned = clone $entity;
92		$page = $cloned;
93		$this->manager->persist($page);
94		}
95
96		// Create the translationNode.
97		$translationNode = $this->createTranslationNode($rootNode, $language, $page);
98		if (!$page instanceof StructureNode) {
99		$translationNode->setOnline(isset($fixtureParams['set_online']) ? $fixtureParams['set_online'] : true);
100		}
101
102		$fixture->addAdditional($fixture->getName() . '_' . $language, $page);
103		$fixture->addAdditional('translationNode_' . $language, $translationNode);
104		$fixture->addAdditional('nodeVersion_' . $language, $translationNode->getPublicNodeVersion());
105		$fixture->addAdditional('rootNode', $rootNode);
106
107		$this->populator->populate($translationNode, $data);
108		$this->populator->populate($page, $data);
109	View Code Duplication	if ($translationNode->getSlug() === null && $rootNode->getParent() !== null) {
110		$translationNode->setSlug($this->slugifier->slugify($translationNode->getTitle()));
111		}
112		$this->ensureUniqueUrl($translationNode, $page);
113
114		$this->manager->persist($translationNode);
115		$rootNode->addNodeTranslation($translationNode);
116		}
117
118		$this->manager->flush();
119		$this->aclPermissionCreatorService->createPermission($rootNode);
120		}
121
122		public function postFlushBuild(Fixture $fixture)
123		{
124		$entities = $fixture->getAdditionalEntities();
125		$fixtureParams = $fixture->getParameters();
126
127		foreach ($fixture->getTranslations() as $language => $data) {
128		/** @var HasNodeInterface $page */
129		$page = $entities[$fixture->getName() . '_' . $language];
130		/** @var NodeTranslation $translationNode */
131		$translationNode = $entities['translationNode_' . $language];
132
133		$pagecreator = array_key_exists('creator', $fixtureParams) ? $fixtureParams['creator'] : 'pagecreator';
134		$creator = $this->userRepo->findOneBy(array('username' => $pagecreator));
135
136		$nodeVersion = new NodeVersion();
137		$nodeVersion->setNodeTranslation($translationNode);
138		$nodeVersion->setType('public');
139		$nodeVersion->setOwner($creator);
140		$nodeVersion->setRef($page);
141
142		$translationNode->setPublicNodeVersion($nodeVersion);
143
144		if (isset($fixtureParams['template'])) {
145		$pageTemplateConfiguration = new PageTemplateConfiguration();
146		$pageTemplateConfiguration->setPageId($page->getId());
147		$pageTemplateConfiguration->setPageEntityName(ClassLookup::getClass($page));
148		$pageTemplateConfiguration->setPageTemplate($fixtureParams['template']);
149		$this->manager->persist($pageTemplateConfiguration);
150		}
151
152		$this->manager->persist($nodeVersion);
153		$this->manager->persist($translationNode);
154		}
155		$this->manager->flush();
156		}
157
158		private function getParentNode($params, $language)
159		{
160		if (!isset($params['parent'])) {
161		return;
162		}
163
164		$parent = $params['parent'];
165		if ($parent instanceof Fixture) {
166		$additionals = $parent->getAdditionalEntities();
167		$parent = $additionals['rootNode'];
168		} elseif (is_string($parent)) {
169		$nodes = $this->nodeRepo->getNodesByInternalName($parent, $language, false, true);
170		if (count($nodes) > 0) {
171		$parent = $nodes[0];
172		}
173		}
174
175		return $parent;
176		}
177
178		private function createRootNode($page, $language, $internalName, $fixtureParams)
179		{
180		$rootNode = new Node();
181		$rootNode->setRef($page);
182		$rootNode->setDeleted(false);
183		$rootNode->setInternalName($internalName);
184		$rootNode->setHiddenFromNav(
185		isset($fixtureParams['hidden_from_nav']) ? $fixtureParams['hidden_from_nav'] : false
186		);
187		$parent = $this->getParentNode($fixtureParams, $language);
188
189		if ($parent instanceof Node) {
190		$rootNode->setParent($parent);
191
192		if (!$this->canHaveChild($parent->getRefEntityName(), get_class($page))) {
193		throw new \Exception(
194		sprintf('A %s can\'t have a %s as child. Forgot to add in allowed_children or getPossibleChildTypes?', $parent->getRefEntityName(), get_class($page))
195		);
196		}
197		}
198
199		return $rootNode;
200		}
201
202		private function createTranslationNode(Node $rootNode, $language, HasNodeInterface $page)
203		{
204		$translationNode = new NodeTranslation();
205		$translationNode
206		->setNode($rootNode)
207		->setLang($language)
208		->setTitle($page->getTitle())
209		->setOnline(false)
210		->setWeight(0);
211
212		return $translationNode;
213		}
214
215		private function ensureUniqueUrl(NodeTranslation $translation, HasNodeInterface $page)
216		{
217		if ($page instanceof StructureNode) {
218		$translation->setSlug('');
219		$translation->setUrl($translation->getFullSlug());
220
221		return $translation;
222		}
223
224		$translation->setUrl($translation->getFullSlug());
225
226		// Find all translations with this new URL, whose nodes are not deleted.
227		$translationWithSameUrl = $this->nodeTranslationRepo->getNodeTranslationForUrl($translation->getUrl(), $translation->getLang(), false, $translation);
228
229		if ($translationWithSameUrl instanceof NodeTranslation) {
230		$translation->setSlug($this->slugifier->slugify($this->incrementString($translation->getSlug())));
231		$this->ensureUniqueUrl($translation, $page);
232		}
233
234		return $translation;
235		}
236
237	View Code Duplication	private function incrementString($string, $append = '-v')
238		{
239		$finalDigitGrabberRegex = '/\d+$/';
240		$matches = array();
241
242		preg_match($finalDigitGrabberRegex, $string, $matches);
243
244		if (count($matches) > 0) {
245		$digit = (int) $matches[0];
246		++$digit;
247
248		// Replace the integer with the new digit.
249		return preg_replace($finalDigitGrabberRegex, $digit, $string);
250		} else {
251		return $string . $append . '1';
252		}
253		}
254
255		/**
256		* @param string $parentPageClass
257		* @param string $childPageClass
258		*
259		* @return bool
260		*/
261		private function canHaveChild($parentPageClass, $childPageClass)
262		{
263		$childTypes = $this->pagesConfiguration->getPossibleChildTypes($parentPageClass);
264
265		foreach ($childTypes as $childType) {
266		if ($childType['class'] == $childPageClass) {
267		return true;
268		}
269		}
270
271		return false;
272		}
273		}
274

Kunstmaan / KunstmaanBundlesCMS

Push — master ( 770316...74fc07 )

Kunstmaan/FixturesBundle/Builder/PageBuilder.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like