Inspection of "Merge pull request #9 from Micro-PHP/1.6.1" - Micro-PHP/dto - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( fc14ff...049ae2 )

unknown

created 2023-02-16 08:05 UTC

Status

Indentation +30 added lines, -30 removed lines patch added patch discarded remove patch

@@ -116,66 +116,66 @@
 block discarded – undo
     protected static function attributesMetadata(): array
     {
         return array (
-          'username' =>
-          array (
+            'username' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'int',
-              2 => 'null',
+                0 => 'string',
+                1 => 'int',
+                2 => 'null',
             ),
             'required' => false,
             'actionName' => 'username',
-          ),
-          'books' =>
-          array (
+            ),
+            'books' =>
+            array (
             'type' =>
             array (
-              0 => 'iterable',
-              1 => 'null',
+                0 => 'iterable',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'books',
-          ),
-          'first_name' =>
-          array (
+            ),
+            'first_name' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'int',
+                0 => 'string',
+                1 => 'int',
             ),
             'required' => true,
             'actionName' => 'firstName',
-          ),
-          'updatedAt' =>
-          array (
+            ),
+            'updatedAt' =>
+            array (
             'type' =>
             array (
-              0 => 'DateTimeInterface',
-              1 => 'null',
+                0 => 'DateTimeInterface',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'updatedAt',
-          ),
-          'someclass' =>
-          array (
+            ),
+            'someclass' =>
+            array (
             'type' =>
             array (
-              0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
-              1 => 'null',
+                0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'someclass',
-          ),
-          'testMixed' =>
-          array (
+            ),
+            'testMixed' =>
+            array (
             'type' =>
             array (
-              0 => 'mixed',
+                0 => 'mixed',
             ),
             'required' => false,
             'actionName' => 'testMixed',
-          ),
+            ),
         );
     }
 }

Please login to merge, or discard this patch.

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -68,17 +68,17 @@  discard block
 block discarded – undo
 
     public function setBooks(iterable|null $books): self
     {
-        if(!$books) {
+        if (!$books) {
                         $this->books = null;
 
                         return $this;
                     }
 
-                    if(!$this->books) {
+                    if (!$this->books) {
                         $this->books = new Collection();
                     }
 
-                    foreach($books as $item) {
+                    foreach ($books as $item) {
                         $this->books->add($item);
                     }
 
@@ -115,11 +115,11 @@  discard block
 block discarded – undo
 
     protected static function attributesMetadata(): array
     {
-        return array (
+        return array(
           'username' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'int',
               2 => 'null',
@@ -128,9 +128,9 @@  discard block
 block discarded – undo
             'actionName' => 'username',
           ),
           'books' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'iterable',
               1 => 'null',
             ),
@@ -138,9 +138,9 @@  discard block
 block discarded – undo
             'actionName' => 'books',
           ),
           'first_name' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'int',
             ),
@@ -148,9 +148,9 @@  discard block
 block discarded – undo
             'actionName' => 'firstName',
           ),
           'updatedAt' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'DateTimeInterface',
               1 => 'null',
             ),
@@ -158,9 +158,9 @@  discard block
 block discarded – undo
             'actionName' => 'updatedAt',
           ),
           'someclass' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
               1 => 'null',
             ),
@@ -168,9 +168,9 @@  discard block
 block discarded – undo
             'actionName' => 'someclass',
           ),
           'testMixed' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'mixed',
             ),
             'required' => false,

Please login to merge, or discard this patch.

example/out/Simple/SimpleObjectTransfer.php 2 patches

Indentation +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -59,36 +59,36 @@
 block discarded – undo
     protected static function attributesMetadata(): array
     {
         return array (
-          'weight' =>
-          array (
+            'weight' =>
+            array (
             'type' =>
             array (
-              0 => 'int',
-              1 => 'null',
+                0 => 'int',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'weight',
-          ),
-          'height' =>
-          array (
+            ),
+            'height' =>
+            array (
             'type' =>
             array (
-              0 => 'int',
-              1 => 'null',
+                0 => 'int',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'height',
-          ),
-          'parent' =>
-          array (
+            ),
+            'parent' =>
+            array (
             'type' =>
             array (
-              0 => 'Transfer\\Simple\\SimpleObjectTransfer',
-              1 => 'null',
+                0 => 'Transfer\\Simple\\SimpleObjectTransfer',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'parent',
-          ),
+            ),
         );
     }
 }

Please login to merge, or discard this patch.

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -58,11 +58,11 @@  discard block
 block discarded – undo
 
     protected static function attributesMetadata(): array
     {
-        return array (
+        return array(
           'weight' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'int',
               1 => 'null',
             ),
@@ -70,9 +70,9 @@  discard block
 block discarded – undo
             'actionName' => 'weight',
           ),
           'height' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'int',
               1 => 'null',
             ),
@@ -80,9 +80,9 @@  discard block
 block discarded – undo
             'actionName' => 'height',
           ),
           'parent' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'Transfer\\Simple\\SimpleObjectTransfer',
               1 => 'null',
             ),

Please login to merge, or discard this patch.

example/out/Simple/SimpleUserTransfer.php 2 patches

Indentation +100 added lines, -100 removed lines patch added patch discarded remove patch

@@ -327,206 +327,206 @@
 block discarded – undo
     protected static function attributesMetadata(): array
     {
         return array (
-          'parent' =>
-          array (
+            'parent' =>
+            array (
             'type' =>
             array (
-              0 => 'Transfer\\Simple\\SimpleObjectTransfer',
-              1 => 'null',
+                0 => 'Transfer\\Simple\\SimpleObjectTransfer',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'parent',
-          ),
-          'username' =>
-          array (
+            ),
+            'username' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'username',
-          ),
-          'age' =>
-          array (
+            ),
+            'age' =>
+            array (
             'type' =>
             array (
-              0 => 'int',
-              1 => 'null',
+                0 => 'int',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'age',
-          ),
-          'email' =>
-          array (
+            ),
+            'email' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'email',
-          ),
-          'ip' =>
-          array (
+            ),
+            'ip' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'ip',
-          ),
-          'hostname' =>
-          array (
+            ),
+            'hostname' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'hostname',
-          ),
-          'sometext' =>
-          array (
+            ),
+            'sometext' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'sometext',
-          ),
-          'url' =>
-          array (
+            ),
+            'url' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'url',
-          ),
-          'json' =>
-          array (
+            ),
+            'json' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'json',
-          ),
-          'uuid' =>
-          array (
+            ),
+            'uuid' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'uuid',
-          ),
-          'created_at' =>
-          array (
+            ),
+            'created_at' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'createdAt',
-          ),
-          'updated_at' =>
-          array (
+            ),
+            'updated_at' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'updatedAt',
-          ),
-          'timezone' =>
-          array (
+            ),
+            'timezone' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'timezone',
-          ),
-          'card_scheme' =>
-          array (
+            ),
+            'card_scheme' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'cardScheme',
-          ),
-          'bic' =>
-          array (
+            ),
+            'bic' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'bic',
-          ),
-          'currency' =>
-          array (
+            ),
+            'currency' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'currency',
-          ),
-          'iban' =>
-          array (
+            ),
+            'iban' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'iban',
-          ),
-          'isbn' =>
-          array (
+            ),
+            'isbn' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'isbn',
-          ),
-          'issn' =>
-          array (
+            ),
+            'issn' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'issn',
-          ),
-          'isin' =>
-          array (
+            ),
+            'isin' =>
+            array (
             'type' =>
             array (
-              0 => 'string',
-              1 => 'null',
+                0 => 'string',
+                1 => 'null',
             ),
             'required' => false,
             'actionName' => 'isin',
-          ),
+            ),
         );
     }
 }

Please login to merge, or discard this patch.

Spacing +41 added lines, -41 removed lines patch added patch discarded remove patch

@@ -326,11 +326,11 @@  discard block
 block discarded – undo
 
     protected static function attributesMetadata(): array
     {
-        return array (
+        return array(
           'parent' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'Transfer\\Simple\\SimpleObjectTransfer',
               1 => 'null',
             ),
@@ -338,9 +338,9 @@  discard block
 block discarded – undo
             'actionName' => 'parent',
           ),
           'username' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -348,9 +348,9 @@  discard block
 block discarded – undo
             'actionName' => 'username',
           ),
           'age' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'int',
               1 => 'null',
             ),
@@ -358,9 +358,9 @@  discard block
 block discarded – undo
             'actionName' => 'age',
           ),
           'email' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -368,9 +368,9 @@  discard block
 block discarded – undo
             'actionName' => 'email',
           ),
           'ip' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -378,9 +378,9 @@  discard block
 block discarded – undo
             'actionName' => 'ip',
           ),
           'hostname' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -388,9 +388,9 @@  discard block
 block discarded – undo
             'actionName' => 'hostname',
           ),
           'sometext' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -398,9 +398,9 @@  discard block
 block discarded – undo
             'actionName' => 'sometext',
           ),
           'url' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -408,9 +408,9 @@  discard block
 block discarded – undo
             'actionName' => 'url',
           ),
           'json' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -418,9 +418,9 @@  discard block
 block discarded – undo
             'actionName' => 'json',
           ),
           'uuid' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -428,9 +428,9 @@  discard block
 block discarded – undo
             'actionName' => 'uuid',
           ),
           'created_at' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -438,9 +438,9 @@  discard block
 block discarded – undo
             'actionName' => 'createdAt',
           ),
           'updated_at' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -448,9 +448,9 @@  discard block
 block discarded – undo
             'actionName' => 'updatedAt',
           ),
           'timezone' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -458,9 +458,9 @@  discard block
 block discarded – undo
             'actionName' => 'timezone',
           ),
           'card_scheme' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -468,9 +468,9 @@  discard block
 block discarded – undo
             'actionName' => 'cardScheme',
           ),
           'bic' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -478,9 +478,9 @@  discard block
 block discarded – undo
             'actionName' => 'bic',
           ),
           'currency' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -488,9 +488,9 @@  discard block
 block discarded – undo
             'actionName' => 'currency',
           ),
           'iban' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -498,9 +498,9 @@  discard block
 block discarded – undo
             'actionName' => 'iban',
           ),
           'isbn' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -508,9 +508,9 @@  discard block
 block discarded – undo
             'actionName' => 'isbn',
           ),
           'issn' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),
@@ -518,9 +518,9 @@  discard block
 block discarded – undo
             'actionName' => 'issn',
           ),
           'isin' =>
-          array (
+          array(
             'type' =>
-            array (
+            array(
               0 => 'string',
               1 => 'null',
             ),

Please login to merge, or discard this patch.

src/Generator/Generator.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -39,7 +39,7 @@
 block discarded – undo
         /** @var ClassDefinition $classDef */
         foreach ($this->classCollectionPreparation->process($this->reader) as $classDef) {
             $classRendered = $this->renderer->render($classDef);
-            $classname = $classDef->getNamespace().'\\'.$classDef->getName();
+            $classname = $classDef->getNamespace() . '\\' . $classDef->getName();
 
             $this->writer->write($classname, $classRendered);
             $this->logger->debug(sprintf('Generated class "%s"', $classname));

Please login to merge, or discard this patch.

src/Helper/ClassMetadataHelper.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -60,7 +60,7 @@  discard block
 block discarded – undo
             return $className;
         }
 
-        return $this->generateNamespace($className).'\\'.$this->generateClassnameShort($className);
+        return $this->generateNamespace($className) . '\\' . $this->generateClassnameShort($className);
     }
 
     /**
@@ -76,10 +76,10 @@  discard block
 block discarded – undo
         }
 
         if (1 === \count($exploded)) {
-            return $className.ucfirst($this->classSuffix);
+            return $className . ucfirst($this->classSuffix);
         }
 
-        return array_pop($exploded).ucfirst($this->classSuffix);
+        return array_pop($exploded) . ucfirst($this->classSuffix);
     }
 
     /**

Please login to merge, or discard this patch.

src/Preparation/Processor/MethodSetProcessor.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -41,7 +41,7 @@
 block discarded – undo
         $methodSuffix = $this->nameNormalizer->normalize($propertyName);
 
         $methodDef = new MethodDefinition();
-        $methodDef->setName('set'.ucfirst($methodSuffix));
+        $methodDef->setName('set' . ucfirst($methodSuffix));
         // $methodDef->setBody(sprintf("\$this->%s = $%s;\r\n\r\nreturn \$this;", $propertyName, $propertyName));
 
         if ($propertyDefinition->isCollection()) {

Please login to merge, or discard this patch.

src/Preparation/Processor/MethodAttributesMetadataProcessor.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -43,7 +43,7 @@
 block discarded – undo
         $attributesMetaMethod->setTypesReturn(['array']);
         $attributesMetaMethod->setVisibility('protected');
         $attributesMetaMethod->setBody(
-            'return '.var_export($metadataArray, true).';'
+            'return ' . var_export($metadataArray, true) . ';'
         );
 
         $classDefinition->addMethod($attributesMetaMethod);

Please login to merge, or discard this patch.

src/Preparation/Processor/MethodGetProcessor.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -38,7 +38,7 @@
 block discarded – undo
         $propertyName = $propertyDefinition->getName();
         $methodSuffix = $this->nameNormalizer->normalize($propertyName);
         $methodDefinition = new MethodDefinition();
-        $methodDefinition->setName('get'.ucfirst($methodSuffix));
+        $methodDefinition->setName('get' . ucfirst($methodSuffix));
         $methodDefinition->setTypesReturn($propertyDefinition->getTypes());
         $methodDefinition->setVisibility('public');
         $methodDefinition->setBody(sprintf('return $this->%s;', $propertyName));

Please login to merge, or discard this patch.

src/Preparation/Processor/ClassCommentProcessor.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -44,7 +44,7 @@
 block discarded – undo
      */
     protected function addComment(ClassDefinition $classDefinition, string $comment, ?string $commentPrefix): void
     {
-        $text = $commentPrefix ? $commentPrefix.' '.$comment : $comment;
+        $text = $commentPrefix ? $commentPrefix . ' ' . $comment : $comment;
 
         $classDefinition->addComment($text);
     }

Please login to merge, or discard this patch.

		@@ -116,66 +116,66 @@
		block discarded – undo
116	116	protected static function attributesMetadata(): array
117	117	{
118	118	return array (
119		- 'username' =>
120		- array (
	119	+ 'username' =>
	120	+ array (
121	121	'type' =>
122	122	array (
123		- 0 => 'string',
124		- 1 => 'int',
125		- 2 => 'null',
	123	+ 0 => 'string',
	124	+ 1 => 'int',
	125	+ 2 => 'null',
126	126	),
127	127	'required' => false,
128	128	'actionName' => 'username',
129		- ),
130		- 'books' =>
131		- array (
	129	+ ),
	130	+ 'books' =>
	131	+ array (
132	132	'type' =>
133	133	array (
134		- 0 => 'iterable',
135		- 1 => 'null',
	134	+ 0 => 'iterable',
	135	+ 1 => 'null',
136	136	),
137	137	'required' => false,
138	138	'actionName' => 'books',
139		- ),
140		- 'first_name' =>
141		- array (
	139	+ ),
	140	+ 'first_name' =>
	141	+ array (
142	142	'type' =>
143	143	array (
144		- 0 => 'string',
145		- 1 => 'int',
	144	+ 0 => 'string',
	145	+ 1 => 'int',
146	146	),
147	147	'required' => true,
148	148	'actionName' => 'firstName',
149		- ),
150		- 'updatedAt' =>
151		- array (
	149	+ ),
	150	+ 'updatedAt' =>
	151	+ array (
152	152	'type' =>
153	153	array (
154		- 0 => 'DateTimeInterface',
155		- 1 => 'null',
	154	+ 0 => 'DateTimeInterface',
	155	+ 1 => 'null',
156	156	),
157	157	'required' => false,
158	158	'actionName' => 'updatedAt',
159		- ),
160		- 'someclass' =>
161		- array (
	159	+ ),
	160	+ 'someclass' =>
	161	+ array (
162	162	'type' =>
163	163	array (
164		- 0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
165		- 1 => 'null',
	164	+ 0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
	165	+ 1 => 'null',
166	166	),
167	167	'required' => false,
168	168	'actionName' => 'someclass',
169		- ),
170		- 'testMixed' =>
171		- array (
	169	+ ),
	170	+ 'testMixed' =>
	171	+ array (
172	172	'type' =>
173	173	array (
174		- 0 => 'mixed',
	174	+ 0 => 'mixed',
175	175	),
176	176	'required' => false,
177	177	'actionName' => 'testMixed',
178		- ),
	178	+ ),
179	179	);
180	180	}
181	181	}

		@@ -68,17 +68,17 @@ discard block
		block discarded – undo
68	68
69	69	public function setBooks(iterable\|null $books): self
70	70	{
71		- if(!$books) {
	71	+ if (!$books) {
72	72	$this->books = null;
73	73
74	74	return $this;
75	75	}
76	76
77		- if(!$this->books) {
	77	+ if (!$this->books) {
78	78	$this->books = new Collection();
79	79	}
80	80
81		- foreach($books as $item) {
	81	+ foreach ($books as $item) {
82	82	$this->books->add($item);
83	83	}
84	84
		@@ -115,11 +115,11 @@ discard block
		block discarded – undo
115	115
116	116	protected static function attributesMetadata(): array
117	117	{
118		- return array (
	118	+ return array(
119	119	'username' =>
120		- array (
	120	+ array(
121	121	'type' =>
122		- array (
	122	+ array(
123	123	0 => 'string',
124	124	1 => 'int',
125	125	2 => 'null',
		@@ -128,9 +128,9 @@ discard block
		block discarded – undo
128	128	'actionName' => 'username',
129	129	),
130	130	'books' =>
131		- array (
	131	+ array(
132	132	'type' =>
133		- array (
	133	+ array(
134	134	0 => 'iterable',
135	135	1 => 'null',
136	136	),
		@@ -138,9 +138,9 @@ discard block
		block discarded – undo
138	138	'actionName' => 'books',
139	139	),
140	140	'first_name' =>
141		- array (
	141	+ array(
142	142	'type' =>
143		- array (
	143	+ array(
144	144	0 => 'string',
145	145	1 => 'int',
146	146	),
		@@ -148,9 +148,9 @@ discard block
		block discarded – undo
148	148	'actionName' => 'firstName',
149	149	),
150	150	'updatedAt' =>
151		- array (
	151	+ array(
152	152	'type' =>
153		- array (
	153	+ array(
154	154	0 => 'DateTimeInterface',
155	155	1 => 'null',
156	156	),
		@@ -158,9 +158,9 @@ discard block
		block discarded – undo
158	158	'actionName' => 'updatedAt',
159	159	),
160	160	'someclass' =>
161		- array (
	161	+ array(
162	162	'type' =>
163		- array (
	163	+ array(
164	164	0 => 'Micro\\Library\\DTO\\Object\\AbstractDto',
165	165	1 => 'null',
166	166	),
		@@ -168,9 +168,9 @@ discard block
		block discarded – undo
168	168	'actionName' => 'someclass',
169	169	),
170	170	'testMixed' =>
171		- array (
	171	+ array(
172	172	'type' =>
173		- array (
	173	+ array(
174	174	0 => 'mixed',
175	175	),
176	176	'required' => false,

		@@ -59,36 +59,36 @@
		block discarded – undo
59	59	protected static function attributesMetadata(): array
60	60	{
61	61	return array (
62		- 'weight' =>
63		- array (
	62	+ 'weight' =>
	63	+ array (
64	64	'type' =>
65	65	array (
66		- 0 => 'int',
67		- 1 => 'null',
	66	+ 0 => 'int',
	67	+ 1 => 'null',
68	68	),
69	69	'required' => false,
70	70	'actionName' => 'weight',
71		- ),
72		- 'height' =>
73		- array (
	71	+ ),
	72	+ 'height' =>
	73	+ array (
74	74	'type' =>
75	75	array (
76		- 0 => 'int',
77		- 1 => 'null',
	76	+ 0 => 'int',
	77	+ 1 => 'null',
78	78	),
79	79	'required' => false,
80	80	'actionName' => 'height',
81		- ),
82		- 'parent' =>
83		- array (
	81	+ ),
	82	+ 'parent' =>
	83	+ array (
84	84	'type' =>
85	85	array (
86		- 0 => 'Transfer\\Simple\\SimpleObjectTransfer',
87		- 1 => 'null',
	86	+ 0 => 'Transfer\\Simple\\SimpleObjectTransfer',
	87	+ 1 => 'null',
88	88	),
89	89	'required' => false,
90	90	'actionName' => 'parent',
91		- ),
	91	+ ),
92	92	);
93	93	}
94	94	}

		@@ -58,11 +58,11 @@ discard block
		block discarded – undo
58	58
59	59	protected static function attributesMetadata(): array
60	60	{
61		- return array (
	61	+ return array(
62	62	'weight' =>
63		- array (
	63	+ array(
64	64	'type' =>
65		- array (
	65	+ array(
66	66	0 => 'int',
67	67	1 => 'null',
68	68	),
		@@ -70,9 +70,9 @@ discard block
		block discarded – undo
70	70	'actionName' => 'weight',
71	71	),
72	72	'height' =>
73		- array (
	73	+ array(
74	74	'type' =>
75		- array (
	75	+ array(
76	76	0 => 'int',
77	77	1 => 'null',
78	78	),
		@@ -80,9 +80,9 @@ discard block
		block discarded – undo
80	80	'actionName' => 'height',
81	81	),
82	82	'parent' =>
83		- array (
	83	+ array(
84	84	'type' =>
85		- array (
	85	+ array(
86	86	0 => 'Transfer\\Simple\\SimpleObjectTransfer',
87	87	1 => 'null',
88	88	),

		@@ -327,206 +327,206 @@
		block discarded – undo
327	327	protected static function attributesMetadata(): array
328	328	{
329	329	return array (
330		- 'parent' =>
331		- array (
	330	+ 'parent' =>
	331	+ array (
332	332	'type' =>
333	333	array (
334		- 0 => 'Transfer\\Simple\\SimpleObjectTransfer',
335		- 1 => 'null',
	334	+ 0 => 'Transfer\\Simple\\SimpleObjectTransfer',
	335	+ 1 => 'null',
336	336	),
337	337	'required' => false,
338	338	'actionName' => 'parent',
339		- ),
340		- 'username' =>
341		- array (
	339	+ ),
	340	+ 'username' =>
	341	+ array (
342	342	'type' =>
343	343	array (
344		- 0 => 'string',
345		- 1 => 'null',
	344	+ 0 => 'string',
	345	+ 1 => 'null',
346	346	),
347	347	'required' => false,
348	348	'actionName' => 'username',
349		- ),
350		- 'age' =>
351		- array (
	349	+ ),
	350	+ 'age' =>
	351	+ array (
352	352	'type' =>
353	353	array (
354		- 0 => 'int',
355		- 1 => 'null',
	354	+ 0 => 'int',
	355	+ 1 => 'null',
356	356	),
357	357	'required' => false,
358	358	'actionName' => 'age',
359		- ),
360		- 'email' =>
361		- array (
	359	+ ),
	360	+ 'email' =>
	361	+ array (
362	362	'type' =>
363	363	array (
364		- 0 => 'string',
365		- 1 => 'null',
	364	+ 0 => 'string',
	365	+ 1 => 'null',
366	366	),
367	367	'required' => false,
368	368	'actionName' => 'email',
369		- ),
370		- 'ip' =>
371		- array (
	369	+ ),
	370	+ 'ip' =>
	371	+ array (
372	372	'type' =>
373	373	array (
374		- 0 => 'string',
375		- 1 => 'null',
	374	+ 0 => 'string',
	375	+ 1 => 'null',
376	376	),
377	377	'required' => false,
378	378	'actionName' => 'ip',
379		- ),
380		- 'hostname' =>
381		- array (
	379	+ ),
	380	+ 'hostname' =>
	381	+ array (
382	382	'type' =>
383	383	array (
384		- 0 => 'string',
385		- 1 => 'null',
	384	+ 0 => 'string',
	385	+ 1 => 'null',
386	386	),
387	387	'required' => false,
388	388	'actionName' => 'hostname',
389		- ),
390		- 'sometext' =>
391		- array (
	389	+ ),
	390	+ 'sometext' =>
	391	+ array (
392	392	'type' =>
393	393	array (
394		- 0 => 'string',
395		- 1 => 'null',
	394	+ 0 => 'string',
	395	+ 1 => 'null',
396	396	),
397	397	'required' => false,
398	398	'actionName' => 'sometext',
399		- ),
400		- 'url' =>
401		- array (
	399	+ ),
	400	+ 'url' =>
	401	+ array (
402	402	'type' =>
403	403	array (
404		- 0 => 'string',
405		- 1 => 'null',
	404	+ 0 => 'string',
	405	+ 1 => 'null',
406	406	),
407	407	'required' => false,
408	408	'actionName' => 'url',
409		- ),
410		- 'json' =>
411		- array (
	409	+ ),
	410	+ 'json' =>
	411	+ array (
412	412	'type' =>
413	413	array (
414		- 0 => 'string',
415		- 1 => 'null',
	414	+ 0 => 'string',
	415	+ 1 => 'null',
416	416	),
417	417	'required' => false,
418	418	'actionName' => 'json',
419		- ),
420		- 'uuid' =>
421		- array (
	419	+ ),
	420	+ 'uuid' =>
	421	+ array (
422	422	'type' =>
423	423	array (
424		- 0 => 'string',
425		- 1 => 'null',
	424	+ 0 => 'string',
	425	+ 1 => 'null',
426	426	),
427	427	'required' => false,
428	428	'actionName' => 'uuid',
429		- ),
430		- 'created_at' =>
431		- array (
	429	+ ),
	430	+ 'created_at' =>
	431	+ array (
432	432	'type' =>
433	433	array (
434		- 0 => 'string',
435		- 1 => 'null',
	434	+ 0 => 'string',
	435	+ 1 => 'null',
436	436	),
437	437	'required' => false,
438	438	'actionName' => 'createdAt',
439		- ),
440		- 'updated_at' =>
441		- array (
	439	+ ),
	440	+ 'updated_at' =>
	441	+ array (
442	442	'type' =>
443	443	array (
444		- 0 => 'string',
445		- 1 => 'null',
	444	+ 0 => 'string',
	445	+ 1 => 'null',
446	446	),
447	447	'required' => false,
448	448	'actionName' => 'updatedAt',
449		- ),
450		- 'timezone' =>
451		- array (
	449	+ ),
	450	+ 'timezone' =>
	451	+ array (
452	452	'type' =>
453	453	array (
454		- 0 => 'string',
455		- 1 => 'null',
	454	+ 0 => 'string',
	455	+ 1 => 'null',
456	456	),
457	457	'required' => false,
458	458	'actionName' => 'timezone',
459		- ),
460		- 'card_scheme' =>
461		- array (
	459	+ ),
	460	+ 'card_scheme' =>
	461	+ array (
462	462	'type' =>
463	463	array (
464		- 0 => 'string',
465		- 1 => 'null',
	464	+ 0 => 'string',
	465	+ 1 => 'null',
466	466	),
467	467	'required' => false,
468	468	'actionName' => 'cardScheme',
469		- ),
470		- 'bic' =>
471		- array (
	469	+ ),
	470	+ 'bic' =>
	471	+ array (
472	472	'type' =>
473	473	array (
474		- 0 => 'string',
475		- 1 => 'null',
	474	+ 0 => 'string',
	475	+ 1 => 'null',
476	476	),
477	477	'required' => false,
478	478	'actionName' => 'bic',
479		- ),
480		- 'currency' =>
481		- array (
	479	+ ),
	480	+ 'currency' =>
	481	+ array (
482	482	'type' =>
483	483	array (
484		- 0 => 'string',
485		- 1 => 'null',
	484	+ 0 => 'string',
	485	+ 1 => 'null',
486	486	),
487	487	'required' => false,
488	488	'actionName' => 'currency',
489		- ),
490		- 'iban' =>
491		- array (
	489	+ ),
	490	+ 'iban' =>
	491	+ array (
492	492	'type' =>
493	493	array (
494		- 0 => 'string',
495		- 1 => 'null',
	494	+ 0 => 'string',
	495	+ 1 => 'null',
496	496	),
497	497	'required' => false,
498	498	'actionName' => 'iban',
499		- ),
500		- 'isbn' =>
501		- array (
	499	+ ),
	500	+ 'isbn' =>
	501	+ array (
502	502	'type' =>
503	503	array (
504		- 0 => 'string',
505		- 1 => 'null',
	504	+ 0 => 'string',
	505	+ 1 => 'null',
506	506	),
507	507	'required' => false,
508	508	'actionName' => 'isbn',
509		- ),
510		- 'issn' =>
511		- array (
	509	+ ),
	510	+ 'issn' =>
	511	+ array (
512	512	'type' =>
513	513	array (
514		- 0 => 'string',
515		- 1 => 'null',
	514	+ 0 => 'string',
	515	+ 1 => 'null',
516	516	),
517	517	'required' => false,
518	518	'actionName' => 'issn',
519		- ),
520		- 'isin' =>
521		- array (
	519	+ ),
	520	+ 'isin' =>
	521	+ array (
522	522	'type' =>
523	523	array (
524		- 0 => 'string',
525		- 1 => 'null',
	524	+ 0 => 'string',
	525	+ 1 => 'null',
526	526	),
527	527	'required' => false,
528	528	'actionName' => 'isin',
529		- ),
	529	+ ),
530	530	);
531	531	}
532	532	}

		@@ -326,11 +326,11 @@ discard block
		block discarded – undo
326	326
327	327	protected static function attributesMetadata(): array
328	328	{
329		- return array (
	329	+ return array(
330	330	'parent' =>
331		- array (
	331	+ array(
332	332	'type' =>
333		- array (
	333	+ array(
334	334	0 => 'Transfer\\Simple\\SimpleObjectTransfer',
335	335	1 => 'null',
336	336	),
		@@ -338,9 +338,9 @@ discard block
		block discarded – undo
338	338	'actionName' => 'parent',
339	339	),
340	340	'username' =>
341		- array (
	341	+ array(
342	342	'type' =>
343		- array (
	343	+ array(
344	344	0 => 'string',
345	345	1 => 'null',
346	346	),
		@@ -348,9 +348,9 @@ discard block
		block discarded – undo
348	348	'actionName' => 'username',
349	349	),
350	350	'age' =>
351		- array (
	351	+ array(
352	352	'type' =>
353		- array (
	353	+ array(
354	354	0 => 'int',
355	355	1 => 'null',
356	356	),
		@@ -358,9 +358,9 @@ discard block
		block discarded – undo
358	358	'actionName' => 'age',
359	359	),
360	360	'email' =>
361		- array (
	361	+ array(
362	362	'type' =>
363		- array (
	363	+ array(
364	364	0 => 'string',
365	365	1 => 'null',
366	366	),
		@@ -368,9 +368,9 @@ discard block
		block discarded – undo
368	368	'actionName' => 'email',
369	369	),
370	370	'ip' =>
371		- array (
	371	+ array(
372	372	'type' =>
373		- array (
	373	+ array(
374	374	0 => 'string',
375	375	1 => 'null',
376	376	),
		@@ -378,9 +378,9 @@ discard block
		block discarded – undo
378	378	'actionName' => 'ip',
379	379	),
380	380	'hostname' =>
381		- array (
	381	+ array(
382	382	'type' =>
383		- array (
	383	+ array(
384	384	0 => 'string',
385	385	1 => 'null',
386	386	),
		@@ -388,9 +388,9 @@ discard block
		block discarded – undo
388	388	'actionName' => 'hostname',
389	389	),
390	390	'sometext' =>
391		- array (
	391	+ array(
392	392	'type' =>
393		- array (
	393	+ array(
394	394	0 => 'string',
395	395	1 => 'null',
396	396	),
		@@ -398,9 +398,9 @@ discard block
		block discarded – undo
398	398	'actionName' => 'sometext',
399	399	),
400	400	'url' =>
401		- array (
	401	+ array(
402	402	'type' =>
403		- array (
	403	+ array(
404	404	0 => 'string',
405	405	1 => 'null',
406	406	),
		@@ -408,9 +408,9 @@ discard block
		block discarded – undo
408	408	'actionName' => 'url',
409	409	),
410	410	'json' =>
411		- array (
	411	+ array(
412	412	'type' =>
413		- array (
	413	+ array(
414	414	0 => 'string',
415	415	1 => 'null',
416	416	),
		@@ -418,9 +418,9 @@ discard block
		block discarded – undo
418	418	'actionName' => 'json',
419	419	),
420	420	'uuid' =>
421		- array (
	421	+ array(
422	422	'type' =>
423		- array (
	423	+ array(
424	424	0 => 'string',
425	425	1 => 'null',
426	426	),
		@@ -428,9 +428,9 @@ discard block
		block discarded – undo
428	428	'actionName' => 'uuid',
429	429	),
430	430	'created_at' =>
431		- array (
	431	+ array(
432	432	'type' =>
433		- array (
	433	+ array(
434	434	0 => 'string',
435	435	1 => 'null',
436	436	),
		@@ -438,9 +438,9 @@ discard block
		block discarded – undo
438	438	'actionName' => 'createdAt',
439	439	),
440	440	'updated_at' =>
441		- array (
	441	+ array(
442	442	'type' =>
443		- array (
	443	+ array(
444	444	0 => 'string',
445	445	1 => 'null',
446	446	),
		@@ -448,9 +448,9 @@ discard block
		block discarded – undo
448	448	'actionName' => 'updatedAt',
449	449	),
450	450	'timezone' =>
451		- array (
	451	+ array(
452	452	'type' =>
453		- array (
	453	+ array(
454	454	0 => 'string',
455	455	1 => 'null',
456	456	),
		@@ -458,9 +458,9 @@ discard block
		block discarded – undo
458	458	'actionName' => 'timezone',
459	459	),
460	460	'card_scheme' =>
461		- array (
	461	+ array(
462	462	'type' =>
463		- array (
	463	+ array(
464	464	0 => 'string',
465	465	1 => 'null',
466	466	),
		@@ -468,9 +468,9 @@ discard block
		block discarded – undo
468	468	'actionName' => 'cardScheme',
469	469	),
470	470	'bic' =>
471		- array (
	471	+ array(
472	472	'type' =>
473		- array (
	473	+ array(
474	474	0 => 'string',
475	475	1 => 'null',
476	476	),
		@@ -478,9 +478,9 @@ discard block
		block discarded – undo
478	478	'actionName' => 'bic',
479	479	),
480	480	'currency' =>
481		- array (
	481	+ array(
482	482	'type' =>
483		- array (
	483	+ array(
484	484	0 => 'string',
485	485	1 => 'null',
486	486	),
		@@ -488,9 +488,9 @@ discard block
		block discarded – undo
488	488	'actionName' => 'currency',
489	489	),
490	490	'iban' =>
491		- array (
	491	+ array(
492	492	'type' =>
493		- array (
	493	+ array(
494	494	0 => 'string',
495	495	1 => 'null',
496	496	),
		@@ -498,9 +498,9 @@ discard block
		block discarded – undo
498	498	'actionName' => 'iban',
499	499	),
500	500	'isbn' =>
501		- array (
	501	+ array(
502	502	'type' =>
503		- array (
	503	+ array(
504	504	0 => 'string',
505	505	1 => 'null',
506	506	),
		@@ -508,9 +508,9 @@ discard block
		block discarded – undo
508	508	'actionName' => 'isbn',
509	509	),
510	510	'issn' =>
511		- array (
	511	+ array(
512	512	'type' =>
513		- array (
	513	+ array(
514	514	0 => 'string',
515	515	1 => 'null',
516	516	),
		@@ -518,9 +518,9 @@ discard block
		block discarded – undo
518	518	'actionName' => 'issn',
519	519	),
520	520	'isin' =>
521		- array (
	521	+ array(
522	522	'type' =>
523		- array (
	523	+ array(
524	524	0 => 'string',
525	525	1 => 'null',
526	526	),

		@@ -39,7 +39,7 @@
		block discarded – undo
39	39	/** @var ClassDefinition $classDef */
40	40	foreach ($this->classCollectionPreparation->process($this->reader) as $classDef) {
41	41	$classRendered = $this->renderer->render($classDef);
42		- $classname = $classDef->getNamespace().'\\'.$classDef->getName();
	42	+ $classname = $classDef->getNamespace() . '\\' . $classDef->getName();
43	43
44	44	$this->writer->write($classname, $classRendered);
45	45	$this->logger->debug(sprintf('Generated class "%s"', $classname));

		@@ -60,7 +60,7 @@ discard block
		block discarded – undo
60	60	return $className;
61	61	}
62	62
63		- return $this->generateNamespace($className).'\\'.$this->generateClassnameShort($className);
	63	+ return $this->generateNamespace($className) . '\\' . $this->generateClassnameShort($className);
64	64	}
65	65
66	66	/**
		@@ -76,10 +76,10 @@ discard block
		block discarded – undo
76	76	}
77	77
78	78	if (1 === \count($exploded)) {
79		- return $className.ucfirst($this->classSuffix);
	79	+ return $className . ucfirst($this->classSuffix);
80	80	}
81	81
82		- return array_pop($exploded).ucfirst($this->classSuffix);
	82	+ return array_pop($exploded) . ucfirst($this->classSuffix);
83	83	}
84	84
85	85	/**

		@@ -41,7 +41,7 @@
		block discarded – undo
41	41	$methodSuffix = $this->nameNormalizer->normalize($propertyName);
42	42
43	43	$methodDef = new MethodDefinition();
44		- $methodDef->setName('set'.ucfirst($methodSuffix));
	44	+ $methodDef->setName('set' . ucfirst($methodSuffix));
45	45	// $methodDef->setBody(sprintf("\$this->%s = $%s;\r\n\r\nreturn \$this;", $propertyName, $propertyName));
46	46
47	47	if ($propertyDefinition->isCollection()) {

		@@ -43,7 +43,7 @@
		block discarded – undo
43	43	$attributesMetaMethod->setTypesReturn(['array']);
44	44	$attributesMetaMethod->setVisibility('protected');
45	45	$attributesMetaMethod->setBody(
46		- 'return '.var_export($metadataArray, true).';'
	46	+ 'return ' . var_export($metadataArray, true) . ';'
47	47	);
48	48
49	49	$classDefinition->addMethod($attributesMetaMethod);

		@@ -38,7 +38,7 @@
		block discarded – undo
38	38	$propertyName = $propertyDefinition->getName();
39	39	$methodSuffix = $this->nameNormalizer->normalize($propertyName);
40	40	$methodDefinition = new MethodDefinition();
41		- $methodDefinition->setName('get'.ucfirst($methodSuffix));
	41	+ $methodDefinition->setName('get' . ucfirst($methodSuffix));
42	42	$methodDefinition->setTypesReturn($propertyDefinition->getTypes());
43	43	$methodDefinition->setVisibility('public');
44	44	$methodDefinition->setBody(sprintf('return $this->%s;', $propertyName));

		@@ -44,7 +44,7 @@
		block discarded – undo
44	44	*/
45	45	protected function addComment(ClassDefinition $classDefinition, string $comment, ?string $commentPrefix): void
46	46	{
47		- $text = $commentPrefix ? $commentPrefix.' '.$comment : $comment;
	47	+ $text = $commentPrefix ? $commentPrefix . ' ' . $comment : $comment;
48	48
49	49	$classDefinition->addComment($text);
50	50	}

Micro-PHP / dto

Push — master ( fc14ff...049ae2 )

Status

Category

Indentation +30 added lines, -30 removed lines patch added patch discarded remove patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

Indentation +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

Indentation +100 added lines, -100 removed lines patch added patch discarded remove patch

Spacing +41 added lines, -41 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch