jclaveau / php-logical-filter

src/Filterer/FiltererInterface.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      * @param LogicalFilter $filter
      * @param Iterable      $data_to_filter
      */
-    public function apply( LogicalFilter $filter, $tree_to_filter, $options=[] );
+    public function apply(LogicalFilter $filter, $tree_to_filter, $options = []);
 
     public function validateRule($field, $operator, $value, $row, array $path, $all_operands, $options);
 

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 {
     /**
      * @param LogicalFilter $filter
-     * @param Iterable      $data_to_filter
+     * @param Iterable      $tree_to_filter
      */
     public function apply( LogicalFilter $filter, $tree_to_filter, $options=[] );
 

Unused Use Statements +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @author  Jean Claveau
  */
 namespace JClaveau\LogicalFilter\Filterer;
-use       JClaveau\LogicalFilter\LogicalFilter;
+use JClaveau\LogicalFilter\LogicalFilter;
 
 /**
  */

src/Filterer/CustomizableFilterer.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 
     /**
      */
-    public function __construct( callable $rule_validator )
+    public function __construct(callable $rule_validator)
     {
         $this->rule_validator = $rule_validator;
     }
@@ -25,7 +25,7 @@ 
      */
     public function validateRule($field, $operator, $value, $row, array $path, $all_operands, $options)
     {
-        return call_user_func_array( $this->rule_validator, get_defined_vars() );
+        return call_user_func_array($this->rule_validator, get_defined_vars());
     }
 
     /**/

src/Converter/ConverterInterface.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     /**
      * @param LogicalFilter $filter
      */
-    public function convert( LogicalFilter $filter );
+    public function convert(LogicalFilter $filter);
 
     /**/
 }

src/Converter/CustomizableMinimalConverter.php

Spacing +3 added lines, -3 removed lines

@@ -30,14 +30,14 @@ 
      */
     public function onOpenOr()
     {
-        call_user_func( $this->callbacks[ __FUNCTION__ ] );
+        call_user_func($this->callbacks[__FUNCTION__]);
     }
 
     /**
      */
     public function onCloseOr()
     {
-        call_user_func( $this->callbacks[ __FUNCTION__ ] );
+        call_user_func($this->callbacks[__FUNCTION__]);
     }
 
     /**
@@ -47,7 +47,7 @@ 
     public function onAndPossibility($field, $operator, $operand, array $allOperandsByField)
     {
         call_user_func(
-            $this->callbacks[ __FUNCTION__ ],
+            $this->callbacks[__FUNCTION__],
             $field,
             $operator,
             $operand,

src/Rule/BetweenOrEqualBothRule.php

Spacing +3 added lines, -3 removed lines

@@ -14,10 +14,10 @@
 
     /**
      */
-    public function __construct( $field, array $limits )
+    public function __construct($field, array $limits)
     {
-        $this->addOperand( new AboveOrEqualRule($field, $limits[0]) );
-        $this->addOperand( new BelowOrEqualRule($field, $limits[1]) );
+        $this->addOperand(new AboveOrEqualRule($field, $limits[0]));
+        $this->addOperand(new BelowOrEqualRule($field, $limits[1]));
     }
 
     /**

src/Rule/Trait_RuleWithOptions.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
     /**
      */
-    public function getOption($name, array $contextual_options=[])
+    public function getOption($name, array $contextual_options = [])
     {
         $options = $this->getOptions($contextual_options);
 
@@ -39,7 +39,7 @@ 
     /**
      * @param $contextual_options
      */
-    public function getOptions(array $contextual_options=[])
+    public function getOptions(array $contextual_options = [])
     {
         $default_options = LogicalFilter::getDefaultOptions();
 

src/Rule/EqualRule.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
      * @param string $field The field to apply the rule on.
      * @param array  $value The value the field can equal to.
      */
-    public function __construct( $field, $value )
+    public function __construct($field, $value)
     {
         $this->field = $field;
         $this->value = $value;
@@ -42,7 +42,7 @@ 
      *
      * @return bool
      */
-    public function hasSolution(array $contextual_options=[])
+    public function hasSolution(array $contextual_options = [])
     {
         return true;
     }

src/Rule/Trait_RuleWithField.php

Braces +2 added lines, -4 removed lines

@@ -50,13 +50,11 @@
     {
         if (is_callable($renamings)) {
             $this->setField( call_user_func($renamings, $this->field) );
-        }
-        elseif (is_array($renamings)) {
+        } elseif (is_array($renamings)) {
             if (isset($renamings[$this->field])) {
                 $this->setField( $renamings[$this->field] );
             }
-        }
-        else {
+        } else {
             throw new \InvalidArgumentException(
                 "\$renamings MUST be a callable or an associative array "
                 ."instead of: " . var_export($renamings, true)

Spacing +5 added lines, -5 removed lines

@@ -23,9 +23,9 @@ 
     /**
      * @return string $field
      */
-    final public function setField( $field )
+    final public function setField($field)
     {
-        if ( ! is_scalar($field)) {
+        if (!is_scalar($field)) {
             throw new \InvalidArgumentEXception(
                 "\$field property of a logical rule must be a scalar contrary to: "
                 .var_export($field, true)
@@ -49,17 +49,17 @@ 
     final public function renameField($renamings)
     {
         if (is_callable($renamings)) {
-            $this->setField( call_user_func($renamings, $this->field) );
+            $this->setField(call_user_func($renamings, $this->field));
         }
         elseif (is_array($renamings)) {
             if (isset($renamings[$this->field])) {
-                $this->setField( $renamings[$this->field] );
+                $this->setField($renamings[$this->field]);
             }
         }
         else {
             throw new \InvalidArgumentException(
                 "\$renamings MUST be a callable or an associative array "
-                ."instead of: " . var_export($renamings, true)
+                ."instead of: ".var_export($renamings, true)
             );
         }
 

Doc Comments +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
     }
 
     /**
-     * @return string $field
+     * @return Trait_RuleWithField $field
      */
     final public function setField( $field )
     {
@@ -44,7 +44,7 @@ 
      * @param  array|callable Associative array of renamings or callable
      *                        that would rename the fields.
      *
-     * @return AbstractAtomicRule $this
+     * @return Trait_RuleWithField $this
      */
     final public function renameField($renamings)
     {

src/Rule/BetweenOrEqualLowerRule.php

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
             // So if the two limits are equal we only consider the equality
             $this->addOperand( new EqualRule($field, $limits[0]) );
             $this->addOperand( new EqualRule($field, $limits[0]) );
-        }
-        else {
+        } else {
             $this->addOperand( new AboveOrEqualRule($field, $limits[0]) );
             $this->addOperand( new BelowRule($field, $limits[1]) );
         }

Spacing +6 added lines, -6 removed lines

@@ -14,17 +14,17 @@ 
 
     /**
      */
-    public function __construct( $field, array $limits )
+    public function __construct($field, array $limits)
     {
         if ($limits[0] == $limits[1]) {
             // (A = 1 || A > 1) && A < 1 <=> no sens
             // So if the two limits are equal we only consider the equality
-            $this->addOperand( new EqualRule($field, $limits[0]) );
-            $this->addOperand( new EqualRule($field, $limits[0]) );
+            $this->addOperand(new EqualRule($field, $limits[0]));
+            $this->addOperand(new EqualRule($field, $limits[0]));
         }
         else {
-            $this->addOperand( new AboveOrEqualRule($field, $limits[0]) );
-            $this->addOperand( new BelowRule($field, $limits[1]) );
+            $this->addOperand(new AboveOrEqualRule($field, $limits[0]));
+            $this->addOperand(new BelowRule($field, $limits[1]));
         }
     }
 
@@ -45,7 +45,7 @@ 
      */
     public function getMaximum()
     {
-        if ( ! $this->getOperandAt(1)) {
+        if (!$this->getOperandAt(1)) {
             return $this->getOperandAt(0)->getValue();
         }