alex-kalanis / kw_rules

php-src/Rules/Factory.php

Spacing +2 added lines, -2 removed lines

@@ -52,8 +52,8 @@
 //        IRules::IS_POST_CODE           => External\IsPostCode::class,  // too many formats for simple check, use regex
 //        IRules::IS_TELEPHONE           => External\IsPhone::class,  // too many formats for simple check, use regex
 //        IRules::IS_EU_VAT              => External\IsEuVat::class,  // too many formats, needs some library for checking
-        IRules::IS_DATE                => External\IsDate::class,  // too many formats, needs some library for checking
-        IRules::IS_DATE_REGEX          => External\IsDateRegex::class,  // too many formats, needs some library for checking
+        IRules::IS_DATE                => External\IsDate::class, // too many formats, needs some library for checking
+        IRules::IS_DATE_REGEX          => External\IsDateRegex::class, // too many formats, needs some library for checking
         IRules::SAFE_EQUALS_BASIC      => Safe\HashedBasicEquals::class,
         IRules::SAFE_EQUALS_FUNC       => Safe\HashedFuncEquals::class,
         IRules::SAFE_EQUALS_PASS       => Safe\HashedPassEquals::class,

php-tests/BasicTests/CallbackRulesTest.php

Braces +6 added lines, -2 removed lines

@@ -18,11 +18,15 @@
     {
         $data = new Rules\ProcessCallback();
         $this->assertInstanceOf(Rules\ARule::class, $data);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->setAgainstValue($expectedCall);
         if ($gotResult) {
             $mock = MockEntry::init('foo', $checkValue);
-            if (!$pass) $this->expectException(RuleException::class);
+            if (!$pass) {
+                $this->expectException(RuleException::class);
+            }
             $data->validate($mock);
         }
     }

php-tests/BasicTests/LengthRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -63,9 +63,9 @@
     public function compareLenProvider(): array
     {
         return [
-            ['yxcvbnm',   8, false, false, true ],
-            ['asdfghjk',  8, true,  true,  true ],
-            ['qwertutop', 8, true,  false, false],
+            ['yxcvbnm', 8, false, false, true],
+            ['asdfghjk', 8, true, true, true],
+            ['qwertutop', 8, true, false, false],
         ];
     }
 }

Braces +9 added lines, -3 removed lines

@@ -20,7 +20,9 @@ 
         $data = new Rules\LengthMin();
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $data->setAgainstValue($length);
-        if (!$passMin) $this->expectException(RuleException::class);
+        if (!$passMin) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -38,7 +40,9 @@ 
         $data = new Rules\LengthEquals();
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $data->setAgainstValue($length);
-        if (!$passEq) $this->expectException(RuleException::class);
+        if (!$passEq) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -56,7 +60,9 @@ 
         $data = new Rules\LengthMax();
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $data->setAgainstValue($length);
-        if (!$passMax) $this->expectException(RuleException::class);
+        if (!$passMax) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 

php-tests/BasicTests/PatternRulesTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,11 +45,11 @@
     public function stringsProvider(): array
     {
         return [
-            ['foo@bar.example',     true,  false],
-            ['foo@bar.baz.example', true,  false],
+            ['foo@bar.example', true, false],
+            ['foo@bar.baz.example', true, false],
             ['foo@bar@baz.example', false, false],
-            ['6',                   false, false],
-            ['bar.example',         false, true ],
+            ['6', false, false],
+            ['bar.example', false, true],
             ['foo.bar.baz.example', false, false],
         ];
     }

Braces +6 added lines, -2 removed lines

@@ -20,7 +20,9 @@ 
         $this->assertInstanceOf(Rules\ARule::class, $data);
 
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$isEmail) $this->expectException(RuleException::class);
+        if (!$isEmail) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -38,7 +40,9 @@ 
         $this->assertInstanceOf(Rules\ARule::class, $data);
 
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$isDomain) $this->expectException(RuleException::class);
+        if (!$isDomain) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/FormatRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -57,9 +57,9 @@
     public function compareFormatProvider(): array
     {
         return [
-            [false,           true,  false, false],
-            [123,             false, true,  false],
-            ['abc',           false, false, true],
+            [false, true, false, false],
+            [123, false, true, false],
+            ['abc', false, false, true],
             [new \stdClass(), false, false, false],
         ];
     }

Braces +9 added lines, -3 removed lines

@@ -18,7 +18,9 @@ 
     {
         $data = new Rules\IsBool();
         $this->assertInstanceOf(Rules\ARule::class, $data);
-        if (!$resultBool) $this->expectException(RuleException::class);
+        if (!$resultBool) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -34,7 +36,9 @@ 
     {
         $data = new Rules\IsNumeric();
         $this->assertInstanceOf(Rules\ARule::class, $data);
-        if (!$resultNum) $this->expectException(RuleException::class);
+        if (!$resultNum) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 
@@ -50,7 +54,9 @@ 
     {
         $data = new Rules\IsString();
         $this->assertInstanceOf(Rules\ARule::class, $data);
-        if (!$resultStr) $this->expectException(RuleException::class);
+        if (!$resultStr) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate(MockEntry::init('foo', $value));
     }
 

php-tests/BasicTests/FileRulesTest.php

Spacing +3 added lines, -3 removed lines

@@ -79,7 +79,7 @@ 
         $this->assertInstanceOf(File\AFileRule::class, $data);
         $data->setAgainstValue($maxSize);
         $mock = MockFile::init('foo', 'text0.txt', 'text/plain',
-            '', $fileSize, UPLOAD_ERR_OK );
+            '', $fileSize, UPLOAD_ERR_OK);
         if (!$match) $this->expectException(RuleException::class);
         $data->validate($mock);
     }
@@ -87,10 +87,10 @@ 
     public function sizeMatchProvider(): array
     {
         return [
-            ['32',  128,   false],
+            ['32', 128, false],
             ['10g', 46843, true],
             ['15m', 84641, true],
-            ['30k', 3534,  true],
+            ['30k', 3534, true],
             ['30k', 35534, false],
         ];
     }

php-tests/BasicTests/SafeRulesTest.php

Braces +6 added lines, -2 removed lines

@@ -20,7 +20,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init($key, $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -38,7 +40,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init($key, $checkValue);
-        if (!$gotResult) $this->expectException(RuleException::class);
+        if (!$gotResult) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 }

php-tests/BasicTests/MatchRulesTest.php

Braces +6 added lines, -2 removed lines

@@ -69,7 +69,9 @@ 
         $this->assertInstanceOf(Rules\ARule::class, $data);
 
         $mock = MockEntry::init('foo', 'bar');
-        if (!$matchAll) $this->expectException(RuleException::class);
+        if (!$matchAll) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -86,7 +88,9 @@ 
         $data->setAgainstValue($expectedValue);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init('foo', 'bar');
-        if (!$matchAny) $this->expectException(RuleException::class);
+        if (!$matchAny) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 

php-tests/BasicTests/RangeRulesTest.php

Spacing +5 added lines, -5 removed lines

@@ -111,11 +111,11 @@
     public function compareRangesProvider(): array
     {
         return [
-            [2,  '8', '4',  false, false, ],
-            [2,  '8', '2',  false, true,  ],
-            [6,  '4', '8',  true,  true,  ],
-            [10, '2', '10', false, true,  ],
-            [10, '2', '6',  false, false, ],
+            [2, '8', '4', false, false, ],
+            [2, '8', '2', false, true, ],
+            [6, '4', '8', true, true, ],
+            [10, '2', '10', false, true, ],
+            [10, '2', '6', false, false, ],
         ];
     }
 }

Braces +15 added lines, -5 removed lines

@@ -16,7 +16,9 @@ 
     {
         $data = new Rules\InRange();
         $this->assertInstanceOf(Rules\ARule::class, $data);
-        if ($gotException) $this->expectException(RuleException::class);
+        if ($gotException) {
+            $this->expectException(RuleException::class);
+        }
         $data->setAgainstValue($expectedValue);
     }
 
@@ -47,7 +49,9 @@ 
         $data->setAgainstValue([$lowerLimit, $upperLimit]);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$range) $this->expectException(RuleException::class);
+        if (!$range) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -66,7 +70,9 @@ 
         $data->setAgainstValue([$lowerLimit, $upperLimit]);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if (!$rangeEq) $this->expectException(RuleException::class);
+        if (!$rangeEq) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -85,7 +91,9 @@ 
         $data->setAgainstValue([$lowerLimit, $upperLimit]);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if ($range) $this->expectException(RuleException::class);
+        if ($range) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }
 
@@ -104,7 +112,9 @@ 
         $data->setAgainstValue([$lowerLimit, $upperLimit]);
         $this->assertInstanceOf(Rules\ARule::class, $data);
         $mock = MockEntry::init('foo', $checkValue);
-        if ($rangeEq) $this->expectException(RuleException::class);
+        if ($rangeEq) {
+            $this->expectException(RuleException::class);
+        }
         $data->validate($mock);
     }