limoncello-php-dist / validation

src/Execution/BlockInterpreter.php

Doc Comments +2 added lines, -2 removed lines

@@ -337,7 +337,7 @@ 
     /**
      * @param array $serializedBlocks
      *
-     * @return array
+     * @return integer[]
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */
@@ -357,7 +357,7 @@ 
     /**
      * @param array $serializedBlocks
      *
-     * @return array
+     * @return integer[]
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */

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/Blocks/ProcedureBlock.php

Doc Comments +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
     }
 
     /**
-     * @param callable|null $endCallable
+     * @param callable $endCallable
      *
      * @return self
      */

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
      */

sample/main.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php namespace Sample;
 
-require_once __DIR__ . '/vendor/autoload.php';
+require_once __DIR__.'/vendor/autoload.php';
 
 /**
  * Copyright 2015-2017 info@neomerx.com

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

@@ -66,10 +66,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
@@ -82,10 +82,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
@@ -133,11 +133,11 @@ 
             $context    = $error->getMessageContext();
             $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);
         }
     }
 
@@ -154,11 +154,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/Types/AsArray.php

Spacing +1 added lines, -2 removed lines

@@ -36,7 +36,6 @@
     public static function execute($value, ContextInterface $context): array
     {
         return is_array($value) === true ?
-            static::createSuccessReply($value) :
-            static::createErrorReply($context, $value, ErrorCodes::IS_ARRAY);
+            static::createSuccessReply($value) : static::createErrorReply($context, $value, ErrorCodes::IS_ARRAY);
     }
 }