spaark / composite-utils

src/Factory/Reflection/ReflectionFileFactory.php

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 +6 added lines, -12 removed lines

@@ -47,8 +47,7 @@ 
         {
             if (!isset($args[$i]))
             {
-                throw new MissingRequiredParameterException
-                (
+                throw new MissingRequiredParameterException(
                     get_class($this->object),
                     $property->name
                 );
@@ -109,14 +108,12 @@ 
     {
         if (!$this->reflect->properties->contains($property))
         {
-            throw new CannotWritePropertyException
-            (
+            throw new CannotWritePropertyException(
                 get_class($this->object), $property
             );
         }
 
-        $this->setAnyValue
-        (
+        $this->setAnyValue(
             $this->reflect->properties[$property],
             $value
         );
@@ -130,8 +127,7 @@ 
         }
         else
         {
-            $this->setNonNullValue
-            (
+            $this->setNonNullValue(
                 $property->name,
                 $value,
                 $property->type
@@ -198,8 +194,7 @@ 
         }
         elseif (is_object($value) && method_exists([$value, $method]))
         {
-            $this->setScalarValue
-            (
+            $this->setScalarValue(
                 $property,
                 $value->$method(),
                 $method,
@@ -238,8 +233,7 @@ 
 
     private function throwError($property, $expected, $value)
     {
-        throw new IllegalPropertyTypeException
-        (
+        throw new IllegalPropertyTypeException(
             get_class($this->object),
             $property,
             $expected,

Braces +9 added lines, -18 removed lines

@@ -65,15 +65,13 @@ 
             if ($building)
             {
                 $this->buildProperty($property);
-            }
-            else
+            } else
             {
                 if (isset($args[$i]))
                 {
                     $this->setAnyValue($property, $args[$i]);
                     $i++;
-                }
-                else
+                } else
                 {
                     $building = true;
                     $this->buildProperty($property);
@@ -93,8 +91,7 @@ 
         {
             $class = $property->type->classname;
             $this->setRawValue($property->name, new $class());
-        }
-        else
+        } else
         {
             $this->setAnyValue($property, 0);
         }
@@ -127,8 +124,7 @@ 
         if (is_null($value))
         {
             $this->setNullValue($property->name, $property->type);
-        }
-        else
+        } else
         {
             $this->setNonNullValue
             (
@@ -144,8 +140,7 @@ 
         if ($type->nullable)
         {
             $this->setRawValue($property, null);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'NonNull', null);
         }
@@ -195,8 +190,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
             (
@@ -205,8 +199,7 @@ 
                 $method,
                 $cast
             );
-        }
-        else
+        } else
         {
             $this->throwError($property, $name, $value);
         }
@@ -217,8 +210,7 @@ 
         if (is_a($value, $type->classname))
         {
             $this->setRawValue($property, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, $type->classname, $value);
         }
@@ -229,8 +221,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

src/Exception/PropertyNotReadableException.php

Indentation +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     {
         parent::__construct
         (
-              'Tried to read unreadable property '
+                'Tried to read unreadable property '
             . $class . '::$' . $property,
             0,
             $previous

Spacing +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
 {
     public function __construct($class, $property, $previous = null)
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Tried to read unreadable property '
             . $class . '::$' . $property,
             0,

src/Exception/PropertyNotWritableException.php

Indentation +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     {
         parent::__construct
         (
-              'Tried to write to unwritable property '
+                'Tried to write to unwritable property '
             . $class . '::$' . $property,
             0,
             $previous

Spacing +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
 {
     public function __construct($class, $property, $previous = null)
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Tried to write to unwritable property '
             . $class . '::$' . $property,
             0,

src/Service/ConditionalPropertyAccessor.php

Spacing +3 added lines, -6 removed lines

@@ -15,8 +15,7 @@ 
         }
         else
         {
-            throw new PropertyNotWritableException
-            (
+            throw new PropertyNotWritableException(
                 get_class($this->object),
                 $property->name
             );
@@ -27,16 +26,14 @@ 
     {
         if (!$this->reflect->properties->contains($property))
         {
-            throw new CannotReadPropertyException
-            (
+            throw new CannotReadPropertyException(
                 get_class($this->object),
                 $property
             );
         }
         elseif (!$this->reflect->properties[$property]->readable)
         {
-            throw new PropertyNotReadableException
-            (
+            throw new PropertyNotReadableException(
                 get_class($this->object),
                 $property
             );

Braces +2 added lines, -4 removed lines

@@ -12,8 +12,7 @@ 
         if ($property->writable)
         {
             parent::setAnyValue($property, $value);
-        }
-        else
+        } else
         {
             throw new PropertyNotWritableException
             (
@@ -32,8 +31,7 @@ 
                 get_class($this->object),
                 $property
             );
-        }
-        elseif (!$this->reflect->properties[$property]->readable)
+        } elseif (!$this->reflect->properties[$property]->readable)
         {
             throw new PropertyNotReadableException
             (

src/Traits/HasReflectorTrait.php

Spacing +1 added lines, -2 removed lines

@@ -25,8 +25,7 @@
         if (!static::$reflectionComposite)
         {
             static::$reflectionComposite =
-                ReflectionCompositeFactory::fromClassName
-                (
+                ReflectionCompositeFactory::fromClassName(
                     get_called_class()
                 )
                 ->build();

src/Traits/PropertyAccessTrait.php

Spacing +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
     {
         if (!$this->accessor)
         {
-            $this->accessor = new ConditionalPropertyAccessor
-            (
+            $this->accessor = new ConditionalPropertyAccessor(
                 $this,
                 self::getReflectionComposite()
             );

src/Exception/MissingRequiredParameterException.php

Indentation +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     {
         parent::__construct
         (
-              'Missing required parameter in constructor. '
+                'Missing required parameter in constructor. '
             . $class . ' requires a value for ' . $property,
             0,
             $previous

Spacing +1 added lines, -2 removed lines

@@ -18,8 +18,7 @@
 {
     public function __construct($class, $property, $previous = null)
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Missing required parameter in constructor. '
             . $class . ' requires a value for ' . $property,
             0,