AbstractResource::postUpdate() - Code Metrics - Inspection of "Add "Illustration" entity to set the image of a re..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 323135...9f183a )

by Julito

created 2019-10-30 14:05 UTC

AbstractResource::postUpdate() A

↳ Parent: AbstractResource

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	8
nc	2
nop	1
dl	0
loc	15
rs	10
c	0
b	0
f	0

<?php
/* For licensing terms, see /license.txt */

namespace Chamilo\CoreBundle\Entity\Resource;

use Doctrine\ORM\Event\LifecycleEventArgs;
use Doctrine\ORM\Mapping as ORM;
use Sylius\Component\Resource\Model\ResourceInterface;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

/**
 * @ORM\MappedSuperclass
 * @ORM\HasLifecycleCallbacks
 */
abstract class AbstractResource implements ResourceInterface
{
    /**
     * @ORM\OneToOne(targetEntity="Chamilo\CoreBundle\Entity\Resource\ResourceNode", cascade={"remove"}, orphanRemoval=true)
     * @ORM\JoinColumn(name="resource_node_id", referencedColumnName="id", onDelete="CASCADE")
     */
    public $resourceNode;

    /**
     * @return string
     */
    abstract public function getResourceName(): string;

    /**
     * @ORM\PostUpdate()
     *
     * @param LifecycleEventArgs $args
     */
    public function postUpdate(LifecycleEventArgs $args)
    {
        $em = $args->getEntityManager();
        // Updates resource node name with the resource name.
        $node = $this->getResourceNode();
        $name = $this->getResourceName();
        $node->setName($name);

        if ($node->hasResourceFile()) {
            // Update file name if exists too.
            $node->getResourceFile()->setOriginalName($name);
        }

        $em->persist($node);
        $em->flush();
    }

    /**
     * @param ResourceNode $resourceNode
     *
     * @return $this
     */
    public function setResourceNode(ResourceNode $resourceNode): self
    {
        $this->resourceNode = $resourceNode;

        return $this;
    }

    /**
     * @return ResourceNode
     */
    public function getResourceNode(): ResourceNode
    {
        return $this->resourceNode;
    }
}


1			<?php
2			/* For licensing terms, see /license.txt */
3
4			namespace Chamilo\CoreBundle\Entity\Resource;
5
6			use Doctrine\ORM\Event\LifecycleEventArgs;
7			use Doctrine\ORM\Mapping as ORM;
8			use Sylius\Component\Resource\Model\ResourceInterface;
			0 ignored issues – show Bug introduced 2018-08-24 07:58 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Chamilo\CoreBundle\Entit...ource\ResourceInterface`. Consider defining an alias. Let?s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let?s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
9
10			/**
11			* @ORM\MappedSuperclass
12			* @ORM\HasLifecycleCallbacks
13			*/
14			abstract class AbstractResource implements ResourceInterface
15			{
16			/**
17			* @ORM\OneToOne(targetEntity="Chamilo\CoreBundle\Entity\Resource\ResourceNode", cascade={"remove"}, orphanRemoval=true)
18			* @ORM\JoinColumn(name="resource_node_id", referencedColumnName="id", onDelete="CASCADE")
19			*/
20			public $resourceNode;
21
22			/**
23			* @return string
24			*/
25			abstract public function getResourceName(): string;
26
27			/**
28			* @ORM\PostUpdate()
29			*
30			* @param LifecycleEventArgs $args
31			*/
32			public function postUpdate(LifecycleEventArgs $args)
33			{
34			$em = $args->getEntityManager();
35			// Updates resource node name with the resource name.
36			$node = $this->getResourceNode();
37			$name = $this->getResourceName();
38			$node->setName($name);
39
40			if ($node->hasResourceFile()) {
41			// Update file name if exists too.
42			$node->getResourceFile()->setOriginalName($name);
43			}
44
45			$em->persist($node);
46			$em->flush();
47			}
48
49			/**
50			* @param ResourceNode $resourceNode
51			*
52			* @return $this
53			*/
54			public function setResourceNode(ResourceNode $resourceNode): self
55			{
56			$this->resourceNode = $resourceNode;
57
58			return $this;
59			}
60
61			/**
62			* @return ResourceNode
63			*/
64			public function getResourceNode(): ResourceNode
65			{
66			return $this->resourceNode;
67			}
68			}
69

chamilo / chamilo-lms

Push — master ( 323135...9f183a )

AbstractResource::postUpdate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like