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

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 +5 added lines, -10 removed lines

@@ -36,14 +36,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
         );
@@ -57,8 +55,7 @@ 
         }
         else
         {
-            $this->setNonNullValue
-            (
+            $this->setNonNullValue(
                 $property->name,
                 $value,
                 $property->type
@@ -125,8 +122,7 @@ 
         }
         elseif (is_object($value) && method_exists([$value, $method]))
         {
-            $this->setScalarValue
-            (
+            $this->setScalarValue(
                 $property,
                 $value->$method(),
                 $method,
@@ -165,8 +161,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

@@ -54,8 +54,7 @@ 
         if (is_null($value))
         {
             $this->setNullValue($property->name, $property->type);
-        }
-        else
+        } else
         {
             $this->setNonNullValue
             (
@@ -71,8 +70,7 @@ 
         if ($type->nullable)
         {
             $this->setRawValue($property, null);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'NonNull', null);
         }
@@ -122,8 +120,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
             (
@@ -132,8 +129,7 @@ 
                 $method,
                 $cast
             );
-        }
-        else
+        } else
         {
             $this->throwError($property, $name, $value);
         }
@@ -144,8 +140,7 @@ 
         if (is_a($value, $type->classname))
         {
             $this->setRawValue($property, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, $type->classname, $value);
         }
@@ -156,8 +151,7 @@ 
         if (is_a($value, \ArrayAccess::class) || is_array($value))
         {
             $this->setRawValue($property, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'Collection', $value);
         }

Doc Comments +7 added lines

@@ -20,6 +20,9 @@ 
      */
     protected $reflect;
 
+    /**
+     * @param \Spaark\Composite\Traits\PropertyAccessTrait $object
+     */
     public function __construct($object, ReflectionComposite $reflect)
     {
         parent::__construct($object);
@@ -115,6 +118,10 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     * @param \Closure $cast
+     */
     private function setScalarValue($property, $value, $name, $cast)
     {
         $method = '__to' . $name;

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/PropertyAccessTrait.php

Spacing +2 added lines, -4 removed lines

@@ -14,8 +14,7 @@ 
         if (!static::$reflectionComposite)
         {
             static::$reflectComposite =
-                ReflectionCompositeFactory::fromClassName
-                (
+                ReflectionCompositeFactory::fromClassName(
                     get_called_class()
                 )
                 ->build();
@@ -36,8 +35,7 @@ 
 
     protected function initPropertyAccessTrait()
     {
-        $this->accessor = new ConditionalPropertyAccessor
-        (
+        $this->accessor = new ConditionalPropertyAccessor(
             $this,
             self::getReflectionComposite()
         );