Fix wrong variable assignment

cakephp · Jul 10, 2017 · 3240a28 · 3240a28
1 parent 597e12a
commit 3240a28
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/Routing/RouteBuilder.php b/src/Routing/RouteBuilder.php
@@ -508,12 +508,13 @@ protected function _methodRoute($method, $template, $target, $name)
             $name = $this->_namePrefix . $name;
         }
         $options = [
-            '_method' => $method,
             '_ext' => $this->_extensions,
             'routeClass' => $this->_routeClass,
             '_name' => $name,
         ];
 
+        $target['_method'] = $method;
+
         $route = $this->_makeRoute($template, $target, $options);
         $this->_collection->add($route, $options);
 

diff --git a/tests/TestCase/Routing/RouteBuilderTest.php b/tests/TestCase/Routing/RouteBuilderTest.php
@@ -907,11 +907,11 @@ public function testHttpMethods($method)
             'route-name'
         );
         $this->assertInstanceOf(Route::class, $route, 'Should return a route');
-        $this->assertSame($method, $route->options['_method']);
+        $this->assertSame($method, $route->defaults['_method']);
         $this->assertSame('app:route-name', $route->options['_name']);
         $this->assertSame('/bookmarks/:id', $route->template);
         $this->assertEquals(
-            ['plugin' => null, 'controller' => 'Bookmarks', 'action' => 'view'],
+            ['plugin' => null, 'controller' => 'Bookmarks', 'action' => 'view', '_method' => $method],
             $route->defaults
         );
     }