spaark / composite-utils

src/Factory/Reflection/ReflectionFileFactory.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Spaark\CompositeUtils\Factory\Reflection;
 
-use Spaark\CompositeUtils\Factory\BaseFactory;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionFile;
 use Spaark\CompositeUtils\Model\Reflection\NamespaceBlock;
 use Spaark\CompositeUtils\Model\Reflection\UseStatement;

Braces +2 added lines, -4 removed lines

@@ -93,8 +93,7 @@ 
                 {
                     $as .= $token[1];
                 }
-            }
-            elseif ($matching)
+            } elseif ($matching)
             {
                 switch ($token[0])
                 {
@@ -105,8 +104,7 @@ 
                     case T_AS:
                         $matching = T_AS;
                 }
-            }
-            else
+            } else
             {
                 switch ($token[0])
                 {

Spacing +2 added lines, -4 removed lines

@@ -60,8 +60,7 @@ 
                     case T_NAMESPACE:
                         $ns = new NamespaceBlock($classname);
                         $currentNS = new RawPropertyAccessor($ns);
-                        $this->accessor->getRawValue
-                        (
+                        $this->accessor->getRawValue(
                             'namespaces'
                         )
                         ->add($classname, $ns);
@@ -76,8 +75,7 @@ 
                             $as = end($as);
                         }
 
-                        $currentNS->getRawValue
-                        (
+                        $currentNS->getRawValue(
                             'useStatements'
                         )
                         ->add($as, new UseStatement($classname, $as));

src/Service/PropertyAccessor.php

Doc Comments +4 added lines

@@ -103,6 +103,10 @@
         }
     }
 
+    /**
+     * @param string $name
+     * @param \Closure $cast
+     */
     private function setScalarValue($property, $value, $name, $cast)
     {
         $method = '__to' . $name;

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Spaark\CompositeUtils\Model\Reflection\Type\StringType;
 use Spaark\CompositeUtils\Model\Reflection\Type\IntegerType;
 use Spaark\CompositeUtils\Model\Reflection\Type\BooleanType;
-use Spaark\CompositeUtils\Model\Reflection\Type\FloatType;
 use Spaark\CompositeUtils\Model\Reflection\Type\CollectionType;
 use Spaark\CompositeUtils\Exception\CannotWritePropertyException;
 use Spaark\CompositeUtils\Exception\IllegalPropertyTypeException;

Spacing +3 added lines, -6 removed lines

@@ -36,8 +36,7 @@ 
     {
         if (!$this->reflect->properties->contains($property))
         {
-            throw new CannotWritePropertyException
-            (
+            throw new CannotWritePropertyException(
                 get_class($this->object), $property
             );
         }
@@ -113,8 +112,7 @@ 
         }
         elseif (is_object($value) && method_exists([$value, $method]))
         {
-            $this->setScalarValue
-            (
+            $this->setScalarValue(
                 $property,
                 $value->$method(),
                 $method,
@@ -153,8 +151,7 @@ 
 
     private function throwError($property, $expected, $value)
     {
-        throw new IllegalPropertyTypeException
-        (
+        throw new IllegalPropertyTypeException(
             get_class($this->object),
             $property,
             $expected,

Braces +6 added lines, -12 removed lines

@@ -47,8 +47,7 @@ 
         if (is_null($value))
         {
             $this->setNullValue($property, $type);
-        }
-        else
+        } else
         {
             $this->setNonNullValue($property, $value, $type);
         }
@@ -59,8 +58,7 @@ 
         if ($type->nullable)
         {
             $this->setRawValue($property, null);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'NonNull', null);
         }
@@ -110,8 +108,7 @@ 
         if (is_scalar($value))
         {
             $this->setRawValue($property, $cast($value));
-        }
-        elseif (is_object($value) && method_exists([$value, $method]))
+        } elseif (is_object($value) && method_exists([$value, $method]))
         {
             $this->setScalarValue
             (
@@ -120,8 +117,7 @@ 
                 $method,
                 $cast
             );
-        }
-        else
+        } else
         {
             $this->throwError($property, $name, $value);
         }
@@ -132,8 +128,7 @@ 
         if (is_a($value, $type->classname))
         {
             $this->setRawValue($property, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, $type->classname, $value);
         }
@@ -144,8 +139,7 @@ 
         if (is_a($value, \ArrayAccess::class) || is_array($value))
         {
             $this->setRawValue($property, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'Collection', $value);
         }

src/Exception/IllegalPropertyTypeException.php

Indentation +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         parent::__construct
         (
-              'Tried to set an illegal property type for '
+                'Tried to set an illegal property type for '
             . $class .'::$' . $property . '. Excpected ' . $expected
             . ', got ' . $got,
             0,

Spacing +3 added lines, -5 removed lines

@@ -6,8 +6,7 @@ 
 
 class IllegalPropertyTypeException extends Exception
 {
-    public function __construct
-    (
+    public function __construct(
         string $class,
         string $property,
         string $expected,
@@ -15,10 +14,9 @@ 
         Exception $previous = null
     )
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Tried to set an illegal property type for '
-            . $class .'::$' . $property . '. Excpected ' . $expected
+            . $class . '::$' . $property . '. Excpected ' . $expected
             . ', got ' . $got,
             0,
             $previous