spaark / composite-utils

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

src/Factory/Reflection/ReflectionCompositeFactory.php

Spacing +4 added lines, -8 removed lines

@@ -21,8 +21,7 @@ 
             (new ReflectionFileFactory($this->reflector->getFileName()))
                 ->build();
         $this->accessor->setRawValue('file', $file);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'namespace',
             $file->namespaces[$this->reflector->getNamespaceName()]
         );
@@ -48,15 +47,13 @@ 
 
     protected function buildProperty($reflect)
     {
-        $properties = $this->accessor->getRawValue
-        (
+        $properties = $this->accessor->getRawValue(
             'properties'
         );
 
         $properties[$reflect->getName()] = 
             (new ReflectionPropertyFactory($reflect))
-                ->build
-                (
+                ->build(
                     $this->object,
                     $this->reflector
                         ->getDefaultProperties()[$reflect->getName()]
@@ -65,8 +62,7 @@ 
 
     protected function buildMethod($reflect)
     {
-        $this->accessor->rawAddToValue
-        (
+        $this->accessor->rawAddToValue(
             'methods',
             (new ReflectionMethodFactory($reflect))
                 ->build($this->object)