Check data before merging (#91)

* try to map parameters to their types

* Update RequestValidator.php

* check data before casting

* Refactoring

* style ci

Co-authored-by: Adam Campbell <adam@hotmeteor.com>
Co-authored-by: Bastien Philippe <bastien.philippe@soyhuce.fr>

src/Validation/RequestValidator.php

@@ -149,6 +149,30 @@ protected function validateParameters()
         }
     }
 
+    /**
+     * @param  mixed  $parameter
+     * @param  string|null  $type
+     * @return mixed
+     */
+    private function castParameter($parameter, ?string $type)
+    {
+        if ($type === null) {
+            return $parameter;
+        }
+
+        if ($type === 'integer' && filter_var($parameter, FILTER_VALIDATE_INT, FILTER_NULL_ON_FAILURE) !== null) {
+            return (int) $parameter;
+        } elseif ($type === 'number' && filter_var($parameter, FILTER_VALIDATE_FLOAT, FILTER_NULL_ON_FAILURE) !== null) {
+            return (float) $parameter;
+        } elseif ($type === 'boolean' && filter_var($parameter, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE) !== null) {
+            return filter_var($parameter, FILTER_VALIDATE_BOOLEAN);
+        } elseif ($type === 'string') {
+            return (string) $parameter;
+        }
+
+        return $parameter;
+    }
+
     /**
      * @throws RequestValidationException|SchemaValidationException
      */

tests/RequestValidatorTest.php

@@ -530,6 +530,9 @@ public function test_handles_query_parameters_int(): void
             return [];
         })->middleware(Middleware::class);
 
+        $this->getJson('/users-by-id/foo')
+            ->assertInvalidRequest();
+
         $this->getJson('/users-by-id/1')
             ->assertValidRequest();
     }