Uuid::initializeUuid() - Code Metrics - Inspection of "new model primary key trait" - zemit-cms/core - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 894c40...e5d2d2 )

by Julien

created 2023-04-06 17:36 UTC

Uuid::initializeUuid() A

↳ Parent: Uuid

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

/**
 * This file is part of the Zemit Framework.
 *
 * (c) Zemit Team <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE.txt
 * file that was distributed with this source code.
 */

namespace Zemit\Mvc\Model;

use Phalcon\Security;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use Zemit\Mvc\Model\AbstractTrait\AbstractBehavior;
use Zemit\Mvc\Model\AbstractTrait\AbstractInjectable;
use Zemit\Mvc\Model\Behavior\Transformable;

trait Uuid
{
    use Options;
    use AbstractBehavior;
    use AbstractInjectable;
    
    public Transformable $uuidBehavior;
    
    /**
     * Initializing Uuid
     */
    public function initializeUuid(?array $options = null): void
    {
        $options ??= $this->getOptionsManager()->get('uuid') ?? [];
        $field = $options['field'] ?? 'uuid';
        
        $security = $this->getDI()->get('security');
        assert($security instanceof Security);
        
        $this->setUuidBehavior(new Transformable([
            'beforeValidationOnCreate' => [
                $field => function ($model, $field) use ($security) {
                    return $model->getAttribute($field) ?? $security->getRandom()->uuid();
                },
            ],
        ]));
    }
    
    /**
     * Set Uuid Behavior
     */
    public function setUuidBehavior(Transformable $uuidBehavior): void
    {
        $this->uuidBehavior = $uuidBehavior;
        $this->addBehavior($this->uuidBehavior);
    }
    
    /**
     * Get Uuid Behavior
     */
    public function getUuidBehavior(): Transformable
    {
        return $this->uuidBehavior;
    }
}


1			<?php
2
3			/**
4			* This file is part of the Zemit Framework.
5			*
6			* (c) Zemit Team <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE.txt
9			* file that was distributed with this source code.
10			*/
11
12			namespace Zemit\Mvc\Model;
13
14			use Phalcon\Security;
			0 ignored issues – show Bug introduced 2023-04-06 17:47 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Zemit\Mvc\Model\Security`. 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...
15			use Zemit\Mvc\Model\AbstractTrait\AbstractBehavior;
16			use Zemit\Mvc\Model\AbstractTrait\AbstractInjectable;
17			use Zemit\Mvc\Model\Behavior\Transformable;
18
19			trait Uuid
20			{
21			use Options;
22			use AbstractBehavior;
23			use AbstractInjectable;
24
25			public Transformable $uuidBehavior;
26
27			/**
28			* Initializing Uuid
29			*/
30			public function initializeUuid(?array $options = null): void
31			{
32			$options ??= $this->getOptionsManager()->get('uuid') ?? [];
33			$field = $options['field'] ?? 'uuid';
34
35			$security = $this->getDI()->get('security');
36			assert($security instanceof Security);
37
38			$this->setUuidBehavior(new Transformable([
39			'beforeValidationOnCreate' => [
40			$field => function ($model, $field) use ($security) {
41			return $model->getAttribute($field) ?? $security->getRandom()->uuid();
42			},
43			],
44			]));
45			}
46
47			/**
48			* Set Uuid Behavior
49			*/
50			public function setUuidBehavior(Transformable $uuidBehavior): void
51			{
52			$this->uuidBehavior = $uuidBehavior;
53			$this->addBehavior($this->uuidBehavior);
54			}
55
56			/**
57			* Get Uuid Behavior
58			*/
59			public function getUuidBehavior(): Transformable
60			{
61			return $this->uuidBehavior;
62			}
63			}
64

zemit-cms / core

Push — master ( 894c40...e5d2d2 )

Uuid::initializeUuid() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like