Merge pull request #419 from jingu/fix_head_request_query_issue

Fixed issue with GET parameters not being passed correctly in HEAD requests

Author: koriym

src/Provide/Router/HttpMethodParams.php

@@ -44,9 +44,9 @@ public function get(array $server, array $get, array $post)
         // set the original value
         $method = strtolower($server['REQUEST_METHOD']);
 
-        // early return on GET
-        if ($method === 'get') {
-            return ['get', $get];
+        // early return on GET or HEAD
+        if ($method === 'get' || $method === 'head') {
+            return [$method, $get];
         }
 
         return $this->unsafeMethod($method, $server, $post);

tests/Provide/Router/HttpMethodParamsTest.php

@@ -19,6 +19,16 @@ public function testGet(): void
         $this->assertSame(['id' => '1'], $params);
     }
 
+    public function testHead(): void
+    {
+        $server = ['REQUEST_METHOD' => 'HEAD'];
+        $get = ['id' => '1'];
+        $post = [];
+        [$method, $params] = (new HttpMethodParams())->get($server, $get, $post);
+        $this->assertSame('head', $method);
+        $this->assertSame(['id' => '1'], $params);
+    }
+
     public function testPost(): void
     {
         $server = ['REQUEST_METHOD' => 'POST'];