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

@@ -7,7 +7,7 @@
  */
 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);
 
     /**/
 }

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

@@ -8,7 +8,7 @@
  */
 namespace JClaveau\LogicalFilter\Converter;
 
-use       JClaveau\LogicalFilter\LogicalFilter;
+use JClaveau\LogicalFilter\LogicalFilter;
 
 /**
  */

src/Converter/MinimalConverter.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     /**
      * @param LogicalFilter $filter
      */
-    public function convert( LogicalFilter $filter )
+    public function convert(LogicalFilter $filter)
     {
         $rootOr = $filter->simplify(['force_logical_core' => true])->getRules();
 
@@ -44,7 +44,7 @@ 
                         );
                     }
 
-                    $operandsByFields[ $field ][ $operator ] = $operandsOfOperator[0];
+                    $operandsByFields[$field][$operator] = $operandsOfOperator[0];
                 }
             }
 

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
                 foreach ($operandsByOperator as $operator => $operandsOfOperator) {
                     if (1 != count($operandsOfOperator)) {
                         throw new \RuntimeException(
-                             "Once a logical filter is simplified, there MUST be "
+                                "Once a logical filter is simplified, there MUST be "
                             ."no more than one operand by operator instead of for '$field' / '$operator': "
                             .var_export($operandsOfOperator, true)
                         );

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

@@ -7,8 +7,8 @@
  */
 namespace JClaveau\LogicalFilter\Converter;
 
-use       JClaveau\LogicalFilter\Converter\ConverterInterface;
-use       JClaveau\LogicalFilter\LogicalFilter;
+use JClaveau\LogicalFilter\Converter\ConverterInterface;
+use JClaveau\LogicalFilter\LogicalFilter;
 
 /**
  * Basic tools to implements minimal converters: Converters that can

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

Doc Comments +1 added lines

@@ -30,6 +30,7 @@
     }
 
     /**
+     * @param string $name
      */
     public function getOption($name, array $contextual_options=[])
     {

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

@@ -7,7 +7,7 @@
  */
 namespace JClaveau\LogicalFilter\Rule;
 
-use       JClaveau\LogicalFilter\LogicalFilter;
+use JClaveau\LogicalFilter\LogicalFilter;
 
 /**
  * Options are defined in three levels:

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/BetweenOrEqualUpperRule.php

Spacing +5 added lines, -5 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 AboveRule($field, $limits[0]) );
-            $this->addOperand( new BelowOrEqualRule($field, $limits[1]) );
+            $this->addOperand(new AboveRule($field, $limits[0]));
+            $this->addOperand(new BelowOrEqualRule($field, $limits[1]));
         }
     }