limoncello-php-dist / validation

src/Rules/Converters/StringToDateTime.php

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
      * @param string $input
      * @param string $format
      *
-     * @return DateTimeInterface|null
+     * @return DateTimeImmutable|null
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */

src/Rules/Comparisons/ScalarInValues.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     public function __construct(array $scalars)
     {
-        assert(call_user_func(function () use ($scalars) {
+        assert(call_user_func(function() use ($scalars) {
             foreach ($scalars as $scalar) {
                 assert(static::isValidType($scalar) === true);
             }

src/Contracts/Execution/BlockStateInterface.php

Doc Comments +3 added lines, -3 removed lines

@@ -23,15 +23,15 @@
 {
     /**
      * @param int   $key
-     * @param mixed $default
+     * @param boolean $default
      *
-     * @return mixed
+     * @return ContextInterface
      */
     public function getState(int $key, $default = null);
 
     /**
      * @param int   $key
-     * @param mixed $value
+     * @param boolean $value
      *
      * @return self
      */

src/Validator/Comparisons.php

Spacing +9 added lines, -18 removed lines

@@ -105,8 +105,7 @@ 
     protected static function lessThan($value, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            $value instanceof DateTimeInterface ? new DateTimeLessThan($value) : new NumericLessThan($value) :
-            new AndOperator(static::lessThan($value), $next);
+            $value instanceof DateTimeInterface ? new DateTimeLessThan($value) : new NumericLessThan($value) : new AndOperator(static::lessThan($value), $next);
     }
 
     /**
@@ -118,8 +117,7 @@ 
     protected static function lessOrEquals($value, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            $value instanceof DateTimeInterface ? new DateTimeLessOrEquals($value) : new NumericLessOrEquals($value) :
-            new AndOperator(static::lessOrEquals($value), $next);
+            $value instanceof DateTimeInterface ? new DateTimeLessOrEquals($value) : new NumericLessOrEquals($value) : new AndOperator(static::lessOrEquals($value), $next);
     }
 
     /**
@@ -131,8 +129,7 @@ 
     protected static function moreThan($value, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            $value instanceof DateTimeInterface ? new DateTimeMoreThan($value) : new NumericMoreThan($value) :
-            new AndOperator(static::moreThan($value), $next);
+            $value instanceof DateTimeInterface ? new DateTimeMoreThan($value) : new NumericMoreThan($value) : new AndOperator(static::moreThan($value), $next);
     }
 
     /**
@@ -145,8 +142,7 @@ 
     {
         return $next === null ?
             ($value instanceof DateTimeInterface ?
-                new DateTimeMoreOrEquals($value) : new NumericMoreOrEqualsThan($value)) :
-            new AndOperator(static::moreOrEquals($value), $next);
+                new DateTimeMoreOrEquals($value) : new NumericMoreOrEqualsThan($value)) : new AndOperator(static::moreOrEquals($value), $next);
     }
 
     /**
@@ -162,8 +158,7 @@ 
 
         return $next === null ?
             ($areLimitsDates ?
-                new DateTimeBetween($lowerLimit, $upperLimit) : new NumericBetween($lowerLimit, $upperLimit)) :
-            new AndOperator(static::between($lowerLimit, $upperLimit), $next);
+                new DateTimeBetween($lowerLimit, $upperLimit) : new NumericBetween($lowerLimit, $upperLimit)) : new AndOperator(static::between($lowerLimit, $upperLimit), $next);
     }
 
     /**
@@ -176,8 +171,7 @@ 
     protected static function stringLengthBetween(int $min, int $max, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            new StringLengthBetween($min, $max) :
-            new AndOperator(static::stringLengthBetween($min, $max), $next);
+            new StringLengthBetween($min, $max) : new AndOperator(static::stringLengthBetween($min, $max), $next);
     }
 
     /**
@@ -189,8 +183,7 @@ 
     protected static function stringLengthMin(int $min, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            new StringLengthMin($min) :
-            new AndOperator(static::stringLengthMin($min), $next);
+            new StringLengthMin($min) : new AndOperator(static::stringLengthMin($min), $next);
     }
 
     /**
@@ -202,8 +195,7 @@ 
     protected static function stringLengthMax(int $max, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            new StringLengthMax($max) :
-            new AndOperator(static::stringLengthMax($max), $next);
+            new StringLengthMax($max) : new AndOperator(static::stringLengthMax($max), $next);
     }
 
     /**
@@ -215,8 +207,7 @@ 
     protected static function regexp(string $pattern, RuleInterface $next = null): RuleInterface
     {
         return $next === null ?
-            new StringRegExp($pattern) :
-            new AndOperator(static::regexp($pattern), $next);
+            new StringRegExp($pattern) : new AndOperator(static::regexp($pattern), $next);
     }
 
     /**

sample/Application.php

Doc Comments +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
     }
 
     /**
-     * @param iterable $errors
+     * @param ErrorInterface[] $errors
      *
      * @return void
      */

Spacing +8 added lines, -8 removed lines

@@ -67,10 +67,10 @@ 
             // 'address1'   => 'Dow 1', // missed required parameter
             'accepted'      => 'false',
         ];
-        $this->console('Invalid data (errors)' . PHP_EOL);
+        $this->console('Invalid data (errors)'.PHP_EOL);
         $validator->validate($invalidInput);
         $this->printErrors($validator->getErrors());
-        $this->console('Invalid data (captures)' . PHP_EOL);
+        $this->console('Invalid data (captures)'.PHP_EOL);
         $this->printCaptures($validator->getCaptures());
 
         // Check with valid data
@@ -83,10 +83,10 @@ 
             'address2'      => null,
             'accepted'      => 'true',
         ];
-        $this->console(PHP_EOL . 'Valid data (errors)' . PHP_EOL);
+        $this->console(PHP_EOL.'Valid data (errors)'.PHP_EOL);
         $validator->validate($validInput);
         $this->printErrors($validator->getErrors());
-        $this->console('Valid data (captures)' . PHP_EOL);
+        $this->console('Valid data (captures)'.PHP_EOL);
         $this->printCaptures($validator->getCaptures());
 
         // The output would be
@@ -134,11 +134,11 @@ 
             $context    = $error->getMessageParameters();
             $errorMsg   = MessageFormatter::formatMessage('en', $errorMsg, $context !== null ? $context : []);
 
-            $this->console("$entry failed for `$paramValue` with: $errorMsg" . PHP_EOL);
+            $this->console("$entry failed for `$paramValue` with: $errorMsg".PHP_EOL);
         }
 
         if ($hasErrors === false) {
-            $this->console('No errors' . PHP_EOL);
+            $this->console('No errors'.PHP_EOL);
         }
     }
 
@@ -155,11 +155,11 @@ 
             $hasCaptures = true;
             $type        = gettype($value);
             $value       = $value instanceof DateTimeInterface ? $value->format(DateTime::ISO8601) : $value;
-            $this->console("`$name` = `$value` ($type)" . PHP_EOL);
+            $this->console("`$name` = `$value` ($type)".PHP_EOL);
         }
 
         if ($hasCaptures === false) {
-            $this->console('No captures' . PHP_EOL);
+            $this->console('No captures'.PHP_EOL);
         }
     }
 

src/Rules/Generic/Required.php

Spacing +1 added lines, -2 removed lines

@@ -94,8 +94,7 @@
     {
         $isOk = $context->getStates()->getState(static::STATE_HAS_BEEN_CALLED, false);
 
-        return $isOk === true ? BlockReplies::createEndSuccessReply() :
-            BlockReplies::createEndErrorReply($context, ErrorCodes::REQUIRED, Messages::REQUIRED, []);
+        return $isOk === true ? BlockReplies::createEndSuccessReply() : BlockReplies::createEndErrorReply($context, ErrorCodes::REQUIRED, Messages::REQUIRED, []);
     }
 
     /**

src/Rules/Generic/Filter.php

Spacing +1 added lines, -2 removed lines

@@ -90,7 +90,6 @@
         $output = filter_var($value, $filterId, $filterOptions);
 
         return $output !== false ?
-            static::createSuccessReply($output) :
-            static::createErrorReply($context, $value, $errorCode, $messageTemplate, [$filterId, $filterOptions]);
+            static::createSuccessReply($output) : static::createErrorReply($context, $value, $errorCode, $messageTemplate, [$filterId, $filterOptions]);
     }
 }

src/Rules/Generic/Enum.php

Spacing +1 added lines, -2 removed lines

@@ -61,7 +61,6 @@
         $isOk   = in_array($value, $values);
 
         return $isOk === true ?
-            static::createSuccessReply($value) :
-            static::createErrorReply($context, $value, ErrorCodes::INVALID_VALUE, Messages::INVALID_VALUE, []);
+            static::createSuccessReply($value) : static::createErrorReply($context, $value, ErrorCodes::INVALID_VALUE, Messages::INVALID_VALUE, []);
     }
 }

src/Rules/Converters/StringToInt.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public static function execute($value, ContextInterface $context): array
     {
         if (is_string($value) === true || is_numeric($value) === true) {
-            return static::createSuccessReply((int)$value);
+            return static::createSuccessReply((int) $value);
         }
 
         return static::createErrorReply($context, $value, ErrorCodes::IS_INT, Messages::IS_INT, []);