Merge pull request #19 from ArondeParon/bugfix/sanitizer-chaining

Bugfix/sanitizer chaining

src/Traits/SanitizesInputs.php

@@ -2,9 +2,9 @@
 
 namespace ArondeParon\RequestSanitizer\Traits;
 
+use ArondeParon\RequestSanitizer\Contracts\Sanitizer;
 use Illuminate\Support\Arr;
 use InvalidArgumentException;
-use ArondeParon\RequestSanitizer\Contracts\Sanitizer;
 
 trait SanitizesInputs
 {
@@ -42,7 +42,10 @@ public function sanitize()
                 } else {
                     throw new InvalidArgumentException('Could not resolve sanitizer from given properties');
                 }
+
                 Arr::set($input, $formKey, $sanitizer->sanitize($this->input($formKey, null)));
+
+                $this->replace($input);
             }
         }
 

tests/SanitizesInputsTest.php

@@ -3,6 +3,8 @@
 namespace ArondeParon\RequestSanitizer\Tests;
 
 use ArondeParon\RequestSanitizer\Contracts\Sanitizer;
+use ArondeParon\RequestSanitizer\Sanitizers\Capitalize;
+use ArondeParon\RequestSanitizer\Sanitizers\Lowercase;
 use ArondeParon\RequestSanitizer\Sanitizers\TrimDuplicateSpaces;
 use ArondeParon\RequestSanitizer\Tests\Objects\Request;
 use Illuminate\Validation\ValidationException;
@@ -56,6 +58,23 @@ public function test_it_will_call_each_sanitizer_if_the_key_exists()
         $request->validateResolved();
     }
 
+    public function test_it_will_chain_the_result_of_each_sanitizer()
+    {
+        $sanitizers = [
+            Lowercase::class,
+            Capitalize::class,
+        ];
+
+        $request = $this->createRequest([
+            'foo' => 'this is a lower case string with a CAPITAL word'
+        ]);
+        $request->addSanitizers('foo', $sanitizers);
+
+        $request->validateResolved();
+
+        $this->assertEquals('This is a lower case string with a capital word', $request->input('foo'));
+    }
+
     public function test_it_will_handle_dot_notation()
     {
         $request = $this->createRequest([