EntitySubscriber::onFlush() - Code Metrics - Inspection of "Ajout de la commande pour créer un utilisateur." - starker-xp/lapero.net - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fac3e1...87ae16 )

by Guillaume

created 2017-05-15 23:53 UTC

EntitySubscriber::onFlush() B

↳ Parent: EntitySubscriber

Complexity

Conditions	7
Paths	13

Size

Total Lines	21
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	21
rs	7.551
c	0
b	0
f	0
cc	7
eloc	13
nc	13
nop	1

<?php

namespace Starkerxp\StructureBundle\Listener;

use DateTime;
use Doctrine\Common\EventSubscriber;
use Doctrine\ORM\Event\LifecycleEventArgs;
use Doctrine\ORM\Event\OnFlushEventArgs;
use Ramsey\Uuid\Uuid;
use Starkerxp\StructureBundle\Entity\Entity;
use Starkerxp\StructureBundle\Entity\TimestampEntity;
use Starkerxp\StructureBundle\Entity\UtilisateurEntity;
use Starkerxp\StructureBundle\Entity\UtilisateurInterface;

//
class EntitySubscriber implements EventSubscriber
{

    /**
     * @var UtilisateurInterface
     */
    protected $utilisateur;

    public function __construct($utilisateur)
    {
        $this->utilisateur = $utilisateur;
    }

    public function getSubscribedEvents()
    {
        return [
            'prePersist',
            'onFlush',
        ];
    }

    public function prePersist(LifecycleEventArgs $args)
    {
        $entity = $args->getEntity();

        if ($entity instanceof TimestampEntity && empty($entity->getCreatedAt())) {
            $entity->setCreatedAt(new DateTime());
        }
        if ($entity instanceof Entity && empty($entity->getUuid())) {
            $uuid = Uuid::uuid4();
            $entity->setUuid($uuid);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }
        if (!$entity instanceof UtilisateurEntity) {
            return false;
        }
        if (empty($entity->getUtilisateur())) {
            $entity->setUtilisateur($this->utilisateur);
        }
    }


    public function onFlush(OnFlushEventArgs $event)
    {
        $entityManager = $event->getEntityManager();
        $uow = $entityManager->getUnitOfWork();
        foreach ($uow->getScheduledEntityUpdates() as $entity) {
            if ($entity instanceof TimestampEntity) {
                $entity->setUpdatedAt(new DateTime());
            }
            if ($entity instanceof Entity && empty($entity->getUuid())) {
                $uuid = Uuid::uuid4();
                $entity->setUuid($uuid);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }
            if (!$entity instanceof UtilisateurEntity) {
                continue;
            }
            if (empty($entity->getUtilisateur())) {
                $entity->setUtilisateur($this->utilisateur);
            }

        }
    }
}



1			<?php
2
3			namespace Starkerxp\StructureBundle\Listener;
4
5			use DateTime;
6			use Doctrine\Common\EventSubscriber;
7			use Doctrine\ORM\Event\LifecycleEventArgs;
8			use Doctrine\ORM\Event\OnFlushEventArgs;
9			use Ramsey\Uuid\Uuid;
10			use Starkerxp\StructureBundle\Entity\Entity;
11			use Starkerxp\StructureBundle\Entity\TimestampEntity;
12			use Starkerxp\StructureBundle\Entity\UtilisateurEntity;
13			use Starkerxp\StructureBundle\Entity\UtilisateurInterface;
14
15			//
16			class EntitySubscriber implements EventSubscriber
17			{
18
19			/**
20			* @var UtilisateurInterface
21			*/
22			protected $utilisateur;
23
24			public function __construct($utilisateur)
25			{
26			$this->utilisateur = $utilisateur;
27			}
28
29			public function getSubscribedEvents()
30			{
31			return [
32			'prePersist',
33			'onFlush',
34			];
35			}
36
37			public function prePersist(LifecycleEventArgs $args)
38			{
39			$entity = $args->getEntity();
40
41			if ($entity instanceof TimestampEntity && empty($entity->getCreatedAt())) {
42			$entity->setCreatedAt(new DateTime());
43			}
44			if ($entity instanceof Entity && empty($entity->getUuid())) {
45			$uuid = Uuid::uuid4();
46			$entity->setUuid($uuid);
			0 ignored issues – show Documentation introduced 2017-05-15 23:56 UTC by Report Bug Copy Issue Report `$uuid` is of type `object<Ramsey\Uuid\UuidInterface>`, but the function expects a `object<Starkerxp\StructureBundle\Entity\guid>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
47			}
48			if (!$entity instanceof UtilisateurEntity) {
49			return false;
50			}
51			if (empty($entity->getUtilisateur())) {
52			$entity->setUtilisateur($this->utilisateur);
53			}
54			}
55
56
57			public function onFlush(OnFlushEventArgs $event)
58			{
59			$entityManager = $event->getEntityManager();
60			$uow = $entityManager->getUnitOfWork();
61			foreach ($uow->getScheduledEntityUpdates() as $entity) {
62			if ($entity instanceof TimestampEntity) {
63			$entity->setUpdatedAt(new DateTime());
64			}
65			if ($entity instanceof Entity && empty($entity->getUuid())) {
66			$uuid = Uuid::uuid4();
67			$entity->setUuid($uuid);
			0 ignored issues – show Documentation introduced 2017-05-15 23:56 UTC by Report Bug Copy Issue Report `$uuid` is of type `object<Ramsey\Uuid\UuidInterface>`, but the function expects a `object<Starkerxp\StructureBundle\Entity\guid>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
68			}
69			if (!$entity instanceof UtilisateurEntity) {
70			continue;
71			}
72			if (empty($entity->getUtilisateur())) {
73			$entity->setUtilisateur($this->utilisateur);
74			}
75
76			}
77			}
78			}
			0 ignored issues – show Coding Style introduced 2017-05-15 23:56 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
79

starker-xp / lapero.net

Push — master ( fac3e1...87ae16 )

EntitySubscriber::onFlush() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like