legalthings / data-enricher

src/DataEnricher/Processor.php

Doc Comments +2 added lines

@@ -14,6 +14,7 @@ 
      * 
      * @param DataEnricher $invoker
      * @param string       $property  Property key which should trigger the processor
+     * @return void
      */
     public function __construct(DataEnricher $invoker, $property);
 
@@ -28,6 +29,7 @@ 
      * Apply processing to nodes
      * 
      * @param Node[] $nodes
+     * @return void
      */
     public function applyTo(array $nodes);
 }

src/DataEnricher.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     public function __construct($source)
     {
         if (!is_object($source)) {
-            throw new \Exception("Data enricher on works on an object, not on a " . gettype($source));
+            throw new \Exception("Data enricher on works on an object, not on a ".gettype($source));
         }
         
         $this->source = $source;
@@ -123,7 +123,7 @@ 
      */
     protected function hasProcessorProperty($value)
     {
-        $processorProps = array_map(function ($processor) {
+        $processorProps = array_map(function($processor) {
             return $processor->getProperty();
         }, $this->processors);
         

Doc Comments -1 removed lines

@@ -137,7 +137,6 @@
      * Check if object has at leas one process property
      * 
      * @param \stdClass    $value
-     * @param Processor[]  $processors
      * @return boolean
      */
     protected function hasProcessorProperty($value)

src/DataEnricher/Processor/Http.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     {
         $url = $node->getInstruction($this);
         
-        $type = (is_object($url) ? get_class($url) . ' ' : '') . gettype($url);
+        $type = (is_object($url) ? get_class($url).' ' : '').gettype($url);
         if (!assert(is_string($url), "Expected '{$this->property}' to be a string, but got a $type")) {
             return null;
         }
@@ -45,7 +45,7 @@ 
         }
 
         $client = new Client(['http_errors' => false]);
-        $promise = $client->getAsync($url)->then(function (Response $response) use ($node) {
+        $promise = $client->getAsync($url)->then(function(Response $response) use ($node) {
             $this->applyResult($node, $response);
         });
         

src/DataEnricher/Node.php

Braces +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             $target = $target->getResult();
             $this->applyNodeResults($target);
         } elseif (is_array($target) || is_object($target)) {
-            foreach ($target as &$value) {         
+            foreach ($target as &$value) {
                 $this->applyNodeResults($value);
             }
         }

src/DataEnricher/Processor/Merge.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         
         if (isset($list) && !is_array($list)) {
             throw new \Exception("Unable to apply {$this->property} processing instruction:"
-                . " Expected an array, got a " . (is_object($list) ? get_class($list) . ' ' : '') . gettype($list));
+                . " Expected an array, got a ".(is_object($list) ? get_class($list).' ' : '').gettype($list));
         }
         
         $result = $this->merge((array)$list);