BaseEntity::postUpdate() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/raivie..." - raivieira/base-api-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c1f454...99d563 )

by Rai

created 2017-02-01 18:47 UTC

BaseEntity::postUpdate() A

↳ Parent: BaseEntity

Complexity

Conditions	3
Paths	4

Size

Total Lines	12
Code Lines	6

Duplication

Lines	12
Ratio	100 %

Importance

Changes

Metric	Value
cc	3
eloc	6
nc	4
nop	0
dl	12
loc	12
rs	9.4285
c	0
b	0
f	0

<?php

namespace Bludata\Doctrine\ODM\MongoDB\Entities;

use Bludata\Doctrine\Common\Interfaces\BaseEntityInterface;
use Bludata\Doctrine\Common\Interfaces\EntityManagerInterface;
use Bludata\Doctrine\Common\Interfaces\EntityTimestampInterface;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;
use Doctrine\ODM\MongoDB\PersistentCollection;

/**
 * @ODM\MappedSuperclass
 * @ODM\HasLifecycleCallbacks
 */
abstract class BaseEntity implements BaseEntityInterface, EntityTimestampInterface

{
    /**
     * @ODM\Id
     */
    protected $id;

    /**
     * @var \DateTime
     *
     * @ODM\Field(type="timestamp", name="createdAt")
     */
    protected $createdAt;

    /**
     * @var \DateTime
     *
     * @ODM\Field(type="timestamp", name="updatedAt")
     */
    protected $updatedAt;

    /**
     * @ODM\Field(type="timestamp", nullable=true, name="deletedAt")
     */
    protected $deletedAt;

    public function getCreatedAt()
    {
        return $this->createdAt;
    }

    public function getUpdatedAt()
    {
        return $this->updatedAt;
    }

    public function getDeletedAt()

    {
        return $this->deletedAt;
    }

    public function setDeletedAt($deletedAt)
    {
        $this->deletedAt = $deletedAt;
    }

    /**
     * Altera o campo updatedAt para forçar o persist da entity.
     */
    public function forcePersist()
    {
        $this->updatedAt = new \DateTime();

        return $this;
    }

    /**
     * @ODM\PrePersist
     */
    public function prePersist()

    {
        $repository = $this->getRepository();

        if (method_exists($repository, 'preSave')) {
            $repository->preSave($this);
        }

        if (method_exists($repository, 'preSave')) {
            $repository->validate($this);
        }
    }

    /**
     * @ODM\PostPersist
     */
    public function postPersist()

    {
        $repository = $this->getRepository();

        if (method_exists($repository, 'postSave')) {
            $repository->postSave($this);
        }

        if (method_exists($repository, 'postSave')) {
            $repository->validate($this);
        }
    }

    /**
     * @ODM\PreUpdate
     */
    public function preUpdate()

    {
        $repository = $this->getRepository();

        if (method_exists($repository, 'preSave')) {
            $repository->preSave($this);
        }

        if (method_exists($repository, 'preSave')) {
            $repository->validate($this);
        }
    }

    /**
     * @ODM\PostUpdate
     */
    public function postUpdate()

    {
        $repository = $this->getRepository();

        if (method_exists($repository, 'postSave')) {
            $repository->postSave($this);
        }

        if (method_exists($repository, 'postSave')) {
            $repository->validate($this);
        }
    }

    public function getRepository()

    {
        return app(EntityManagerInterface::class)->getRepository(get_class($this));

    }

    public function save($flush = false)
    {
        $this->getRepository()->save($this);

        return $this;
    }

    public function remove()
    {
        $this->getRepository()->remove($this);

        return $this;
    }

    public function flush($all = true)
    {
        $this->getRepository()->flush($all ? null : $this);

        return $this;
    }

    public function getId()

    {
        return $this->id;
    }

    protected function getFillable()
    {
        return ['id', 'createdAt', 'updatedAt', 'deletedAt'];
    }

    /**
     * Retona um array com o nome das propriedade que o cliente pode setar para realizar o store

     * É usado principalmente em $this->setPropertiesEntity e nos Controllers.
     * Este método não evita que uma propriedade seja alterada caso tenha seu método set().

     *
     * @return array
     */
    abstract public function getOnlyStore();

    /**
     * Retona um array com o nome das propriedade que o cliente pode setar para realizar o update.

     * Por padrão retorna os mesmos valores de $this->getOnlyStore().
     * Este método pode ser sobrescrito nas classes filhas.
     * É usado principalmente em $this->setPropertiesEntity e nos Controllers.
     * Este método não evita que uma propriedade seja alterada caso tenha seu método set().

     *
     * @return array
     */
    public function getOnlyUpdate()
    {
        return $this->getOnlyStore();
    }

    public function setPropertiesEntity(array $data)
    {
        foreach ($data as $key => $value) {
            $set = true;

            if (

                ((!isset($data['id']) || !is_numeric($data['id'])) && !in_array($key, $this->getOnlyStore()))

                ||
                (isset($data['id']) && is_numeric($data['id']) && !in_array($key, $this->getOnlyUpdate()))

            ) {
                $set = false;
            }

            $method = 'set'.ucfirst($key);

            if (method_exists($this, $method) && $set) {
                $this->$method($value);
            }
        }

        return $this;
    }

    final protected function checkOnyExceptInArray($key, array $options = null)

    {
        if (
            $options
            &&
            (
                (isset($options['only']) && is_array($options['only']) && !in_array($key, $options['only']))

                ||
                (isset($options['except']) && is_array($options['except']) && in_array($key, $options['except']))

            )
        ) {
            return false;
        }

        return true;
    }

    public function toArray(array $options = null)
    {
        $classMetadata = $this->getRepository()->getClassMetadata();
        $array = [];
$a = "a";
$ab = "ab";
$abc = "abc";

        foreach ($this->getFillable() as $key) {
            if ($this->checkOnyExceptInArray($key, $options)) {
                if (is_object($this->$key) && $this->$key instanceof \DateTime || $this->$key instanceof \MongoTimestamp) {

                    $metaDataKey = $classMetadata->hasField($key) ? $classMetadata->getFieldMapping($key) : null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

                    if ($this->$key) {
                        $dateFormat = 'Y-m-d H:i:s';

                        if ($this->$key instanceof \MongoTimestamp) {
                            $timestamp = new \DateTime();
                            $timestamp->setTimestamp($this->$key->sec);
                            $this->$key = $timestamp;
                        }

                        $array[$key] = $this->$key->format($dateFormat);
                    }
                } elseif (is_object($this->$key) && $this->$key instanceof ArrayCollection || $this->$key instanceof PersistentCollection) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
                    $ids = [];
                    foreach ($this->$key->getValues() as $item) {
                        $ids[] = $item->getId();
                    }
                    $array[$key] = $ids;
                } elseif (method_exists($this->$key, 'getId')) {
                    $array[$key] = $this->$key->getId();
                } else {
                    $array[$key] = $this->$key;
                }
            }
        }

        return $array;
    }
}


Push — master ( c1f454...99d563 )

BaseEntity::postUpdate() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

1		<?php
2
3		namespace Bludata\Doctrine\ODM\MongoDB\Entities;
4
5		use Bludata\Doctrine\Common\Interfaces\BaseEntityInterface;
6		use Bludata\Doctrine\Common\Interfaces\EntityManagerInterface;
7		use Bludata\Doctrine\Common\Interfaces\EntityTimestampInterface;
8		use Doctrine\Common\Collections\ArrayCollection;
9		use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;
10		use Doctrine\ODM\MongoDB\PersistentCollection;
11
12		/**
13		* @ODM\MappedSuperclass
14		* @ODM\HasLifecycleCallbacks
15		*/
16		abstract class BaseEntity implements BaseEntityInterface, EntityTimestampInterface
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 82 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
17		{
18		/**
19		* @ODM\Id
20		*/
21		protected $id;
22
23		/**
24		* @var \DateTime
25		*
26		* @ODM\Field(type="timestamp", name="createdAt")
27		*/
28		protected $createdAt;
29
30		/**
31		* @var \DateTime
32		*
33		* @ODM\Field(type="timestamp", name="updatedAt")
34		*/
35		protected $updatedAt;
36
37		/**
38		* @ODM\Field(type="timestamp", nullable=true, name="deletedAt")
39		*/
40		protected $deletedAt;
41
42		public function getCreatedAt()
43		{
44		return $this->createdAt;
45		}
46
47		public function getUpdatedAt()
48		{
49		return $this->updatedAt;
50		}
51
52		public function getDeletedAt()
		0 ignored issues – show Documentation introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
53		{
54		return $this->deletedAt;
55		}
56
57		public function setDeletedAt($deletedAt)
58		{
59		$this->deletedAt = $deletedAt;
60		}
61
62		/**
63		* Altera o campo updatedAt para forçar o persist da entity.
64		*/
65		public function forcePersist()
66		{
67		$this->updatedAt = new \DateTime();
68
69		return $this;
70		}
71
72		/**
73		* @ODM\PrePersist
74		*/
75	View Code Duplication	public function prePersist()
		0 ignored issues – show Duplication introduced 2016-10-27 16: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...
76		{
77		$repository = $this->getRepository();
78
79		if (method_exists($repository, 'preSave')) {
80		$repository->preSave($this);
81		}
82
83		if (method_exists($repository, 'preSave')) {
84		$repository->validate($this);
85		}
86		}
87
88		/**
89		* @ODM\PostPersist
90		*/
91	View Code Duplication	public function postPersist()
		0 ignored issues – show Duplication introduced 2017-02-01 18:49 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...
92		{
93		$repository = $this->getRepository();
94
95		if (method_exists($repository, 'postSave')) {
96		$repository->postSave($this);
97		}
98
99		if (method_exists($repository, 'postSave')) {
100		$repository->validate($this);
101		}
102		}
103
104		/**
105		* @ODM\PreUpdate
106		*/
107	View Code Duplication	public function preUpdate()
		0 ignored issues – show Duplication introduced 2016-10-27 16: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...
108		{
109		$repository = $this->getRepository();
110
111		if (method_exists($repository, 'preSave')) {
112		$repository->preSave($this);
113		}
114
115		if (method_exists($repository, 'preSave')) {
116		$repository->validate($this);
117		}
118		}
119
120		/**
121		* @ODM\PostUpdate
122		*/
123	View Code Duplication	public function postUpdate()
		0 ignored issues – show Duplication introduced 2017-02-01 18:49 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...
124		{
125		$repository = $this->getRepository();
126
127		if (method_exists($repository, 'postSave')) {
128		$repository->postSave($this);
129		}
130
131		if (method_exists($repository, 'postSave')) {
132		$repository->validate($this);
133		}
134		}
135
136		public function getRepository()
		0 ignored issues – show Documentation introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
137		{
138		return app(EntityManagerInterface::class)->getRepository(get_class($this));
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 83 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
139		}
140
141		public function save($flush = false)
142		{
143		$this->getRepository()->save($this);
144
145		return $this;
146		}
147
148		public function remove()
149		{
150		$this->getRepository()->remove($this);
151
152		return $this;
153		}
154
155		public function flush($all = true)
156		{
157		$this->getRepository()->flush($all ? null : $this);
158
159		return $this;
160		}
161
162		public function getId()
		0 ignored issues – show Documentation introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
163		{
164		return $this->id;
165		}
166
167		protected function getFillable()
168		{
169		return ['id', 'createdAt', 'updatedAt', 'deletedAt'];
170		}
171
172		/**
173		* Retona um array com o nome das propriedade que o cliente pode setar para realizar o store
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 96 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
174		* É usado principalmente em $this->setPropertiesEntity e nos Controllers.
175		* Este método não evita que uma propriedade seja alterada caso tenha seu método set().
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 91 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
176		*
177		* @return array
178		*/
179		abstract public function getOnlyStore();
180
181		/**
182		* Retona um array com o nome das propriedade que o cliente pode setar para realizar o update.
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 98 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
183		* Por padrão retorna os mesmos valores de $this->getOnlyStore().
184		* Este método pode ser sobrescrito nas classes filhas.
185		* É usado principalmente em $this->setPropertiesEntity e nos Controllers.
186		* Este método não evita que uma propriedade seja alterada caso tenha seu método set().
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 91 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
187		*
188		* @return array
189		*/
190		public function getOnlyUpdate()
191		{
192		return $this->getOnlyStore();
193		}
194
195		public function setPropertiesEntity(array $data)
196		{
197		foreach ($data as $key => $value) {
198		$set = true;
199
200	View Code Duplication	if (
		0 ignored issues – show Duplication introduced 2016-10-05 16:49 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...
201		((!isset($data['id']) \|\| !is_numeric($data['id'])) && !in_array($key, $this->getOnlyStore()))
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 109 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
202		\|\|
203		(isset($data['id']) && is_numeric($data['id']) && !in_array($key, $this->getOnlyUpdate()))
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 106 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
204		) {
205		$set = false;
206		}
207
208		$method = 'set'.ucfirst($key);
209
210		if (method_exists($this, $method) && $set) {
211		$this->$method($value);
212		}
213		}
214
215		return $this;
216		}
217
218	View Code Duplication	final protected function checkOnyExceptInArray($key, array $options = null)
		0 ignored issues – show Duplication introduced 2016-10-05 16:49 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...
219		{
220		if (
221		$options
222		&&
223		(
224		(isset($options['only']) && is_array($options['only']) && !in_array($key, $options['only']))
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 108 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
225		\|\|
226		(isset($options['except']) && is_array($options['except']) && in_array($key, $options['except']))
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 113 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
227		)
228		) {
229		return false;
230		}
231
232		return true;
233		}
234
235		public function toArray(array $options = null)
236		{
237		$classMetadata = $this->getRepository()->getClassMetadata();
238		$array = [];
		0 ignored issues – show Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
239
240		foreach ($this->getFillable() as $key) {
241		if ($this->checkOnyExceptInArray($key, $options)) {
242		if (is_object($this->$key) && $this->$key instanceof \DateTime \|\| $this->$key instanceof \MongoTimestamp) {
		0 ignored issues – show Coding Style introduced 2016-12-14 12:26 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 123 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
243		$metaDataKey = $classMetadata->hasField($key) ? $classMetadata->getFieldMapping($key) : null;
		0 ignored issues – show Unused Code introduced 2016-11-07 11:50 UTC by Report Bug Copy Issue Report `$metaDataKey` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history... Coding Style introduced 2016-09-06 13:58 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 113 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
244
245		if ($this->$key) {
246		$dateFormat = 'Y-m-d H:i:s';
247
248		if ($this->$key instanceof \MongoTimestamp) {
249		$timestamp = new \DateTime();
250		$timestamp->setTimestamp($this->$key->sec);
251		$this->$key = $timestamp;
252		}
253
254		$array[$key] = $this->$key->format($dateFormat);
255		}
256	View Code Duplication	} elseif (is_object($this->$key) && $this->$key instanceof ArrayCollection \|\| $this->$key instanceof PersistentCollection) {
		0 ignored issues – show Bug introduced 2016-12-14 12:26 UTC by Report Bug Copy Issue Report The class `Doctrine\ODM\MongoDB\PersistentCollection` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history... Duplication introduced 2016-12-14 12:26 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... Coding Style introduced 2016-12-14 12:26 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 80 characters; contains 140 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
257		$ids = [];
258		foreach ($this->$key->getValues() as $item) {
259		$ids[] = $item->getId();
260		}
261		$array[$key] = $ids;
262		} elseif (method_exists($this->$key, 'getId')) {
263		$array[$key] = $this->$key->getId();
264		} else {
265		$array[$key] = $this->$key;
266		}
267		}
268		}
269
270		return $array;
271		}
272		}
273

raivieira / base-api-php

Push — master ( c1f454...99d563 )

BaseEntity::postUpdate() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like