legalthings / data-enricher

src/DataEnricher.php

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)

Spacing +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
         }
         
         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));
         }
         
         if (is_string($target)) {
@@ -144,7 +144,7 @@ 
      */
     protected function hasProcessorProperty($value)
     {
-        $processorProps = array_map(function ($processor) {
+        $processorProps = array_map(function($processor) {
             return $processor->getProperty();
         }, $this->processors);
         

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);

src/DataEnricher/Processor.php

Doc Comments +2 added lines

@@ -13,6 +13,7 @@ 
      * Class constructor
      * 
      * @param string $property  Property key with the processing instruction
+     * @return void
      */
     public function __construct($property);
 
@@ -27,6 +28,7 @@ 
      * Apply processing to a node
      * 
      * @param Node $node
+     * @return null|boolean
      */
     public function applyToNode(Node $node);
 }

src/DataEnricher/Processor/Mustache.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         $template = $node->getInstruction($this);
         
         if (!is_string($template) && !is_array($template) && !is_object($template)) {
-            return trigger_error("Unable to parse given template of type: " . gettype($template), E_WARNING);
+            return trigger_error("Unable to parse given template of type: ".gettype($template), E_WARNING);
         }
         
         $result = $this->getParsedResult($template);