spaark / composite-utils

src/Factory/Reflection/ReflectionPropertyFactory.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Spaark\CompositeUtils\Model\Reflection\ReflectionComposite;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionProperty;
-use Spaark\CompositeUtils\Model\Reflection\ReflectionParameter;
 use Spaark\CompositeUtils\Model\Reflection\Type\BooleanType;
 use Spaark\CompositeUtils\Model\Reflection\Type\CollectionType;
 use Spaark\CompositeUtils\Model\Reflection\Type\IntegerType;

Braces +2 added lines, -4 removed lines

@@ -52,8 +52,7 @@ 
         if ($value{0} !== '?')
         {
             $nullable = false;
-        }
-        else
+        } else
         {
             $nullable = true;
             $value = substr($value, 1);
@@ -62,8 +61,7 @@ 
         if (substr($value, -2) !== '[]')
         {
             $collection = false;
-        }
-        else
+        } else
         {
             $collection = true;
             $value = substr($value, 0, -2);

Spacing +10 added lines, -20 removed lines

@@ -40,8 +40,7 @@ 
 
     public static function fromName($class, $property)
     {
-        return new static(new PHPNativeReflectionProperty
-        (
+        return new static(new PHPNativeReflectionProperty(
             $class, $property
         ));
     }
@@ -50,8 +49,7 @@ 
     {
         $this->accessor->setRawValue('owner', $parent);
         $this->accessor->setRawValue('defaultValue', $default);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );
@@ -138,50 +136,42 @@ 
         switch ($value[0])
         {
             case 'required':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'passedToConstructor',
                     true
                 );
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'requiredInConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'requiredProperties',
                     $this->object
                 );
                 break;
             case 'new':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'builtInConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'builtProperties',
                     $this->object
                 );
                 break;
             case 'optional':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'passedToConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'optionalProperties',
                     $this->object
                 );
 
                 if (isset($value[1]) && $value[1] === 'new')
                 {
-                    $this->accessor->setRawValue
-                    (
+                    $this->accessor->setRawValue(
                         'builtInConstructor',
                         true
                     );

src/Factory/Reflection/ReflectorFactory.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     {
         preg_match_all
         (
-              '/^'
+                '/^'
             .     '[ \t]*\*[ \t]*'
             .     '@([a-zA-Z]+)'
             .     '(.*)'

Spacing +3 added lines, -5 removed lines

@@ -37,8 +37,7 @@ 
 
     protected function parseDocComment()
     {
-        preg_match_all
-        (
+        preg_match_all(
               '/^'
             .     '[ \t]*\*[ \t]*'
             .     '@([a-zA-Z]+)'
@@ -55,8 +54,7 @@ 
 
             if (isset($this->acceptedParams[$name]))
             {
-                call_user_func
-                (
+                call_user_func(
                     array($this, $this->acceptedParams[$name]),
                     $name, $value
                 );
@@ -66,7 +64,7 @@ 
 
     protected function setBool($name, $value)
     {
-        switch(strtolower($value))
+        switch (strtolower($value))
         {
             case '':
             case 'true':

src/Factory/Reflection/ReflectionMethodFactory.php

Spacing +3 added lines, -5 removed lines

@@ -13,17 +13,15 @@
 
     public static function fromName($class, $method)
     {
-        return new static(new PHPNativeReflectionMethod
-        (
+        return new static(new PHPNativeReflectionMethod(
             $class, $method
         ));
     }
 
-    public function build(?ReflectionComposite $parent = null)
+    public function build(? ReflectionComposite $parent = null)
     {
         $this->accessor->setRawValue('owner', $parent);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );

src/Model/Reflection/Reflector.php

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,7 @@
         if ($this->locked)
         {
             return parent::__set($var, $val);
-        }
-        else
+        } else
         {
             $this->$var = $val;
         }

src/Model/Collection/Set.php

Indentation +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     private function getScalar($value)
     {
         return
-              (is_object($value) ? spl_object_hash($value)
+                (is_object($value) ? spl_object_hash($value)
             : (is_array($value)  ? implode($value)
             : (                    (string)$value)));
     }

Spacing +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
     {
         return
               (is_object($value) ? spl_object_hash($value)
-            : (is_array($value)  ? implode($value)
-            : (                    (string)$value)));
+            : (is_array($value) ? implode($value)
+            : ((string)$value)));
     }
 
     /**

src/Model/Collection/Stack.php

Braces +3 added lines, -1 removed lines

@@ -43,7 +43,9 @@
      */
     public function popMultiple($i = 1)
     {
-        if ($i < 1) return NULL;
+        if ($i < 1) {
+            return NULL;
+        }
 
         $ret = array_slice($this->data, -1 * $i);
         $this->data = array_slice($this->data, 0, -1 * $i);

src/Exception/NonExistentPropertyException.php

Indentation +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     {
         parent::__construct
         (
-              'Cannot ' . static::ACCESS_TYPE . ' non existent '
+                'Cannot ' . static::ACCESS_TYPE . ' non existent '
             . '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(
               'Cannot ' . static::ACCESS_TYPE . ' non existent '
             . 'property: ' . $class . '::$' . $property,
             0,

src/Model/Collection/HashMap.php

Indentation +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     private function getScalar($value)
     {
         return
-              (is_object($value) ? spl_object_hash($value)
+                (is_object($value) ? spl_object_hash($value)
             : (is_array($value)  ? implode($value)
             : (                    (string)$value)));
     }

Spacing +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
     {
         return
               (is_object($value) ? spl_object_hash($value)
-            : (is_array($value)  ? implode($value)
-            : (                    (string)$value)));
+            : (is_array($value) ? implode($value)
+            : ((string)$value)));
     }
 
     public function key()

src/Factory/Reflection/ReflectionCompositeFactory.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\ReflectionComposite;
 use \ReflectionClass as PHPNativeReflectionClass;
 

Spacing +5 added lines, -10 removed lines

@@ -21,13 +21,11 @@ 
             (new ReflectionFileFactory($this->reflector->getFileName()))
                 ->build();
         $this->accessor->setRawValue('file', $file);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'classname',
             $this->reflector->name
         );
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'namespace',
             $file->namespaces[$this->reflector->getNamespaceName()]
         );
@@ -53,15 +51,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()]
@@ -70,8 +66,7 @@ 
 
     protected function buildMethod($reflect)
     {
-        $this->accessor->rawAddToValue
-        (
+        $this->accessor->rawAddToValue(
             'methods',
             (new ReflectionMethodFactory($reflect))
                 ->build($this->object)