Fix _method HTTP param ignoring (#9)

Co-authored-by: Paramtamtam <[email protected]>

Author: assurrussa

CHANGELOG.md

@@ -4,6 +4,22 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog][keepachangelog] and this project adheres to [Semantic Versioning][semver].
 
+## v3.5.0
+
+### Added
+
+- Listener `EnableHttpMethodParameterOverride` for forced support of `_method` request parameter (for determining the intended HTTP method) [#9]
+
+### Changed
+
+- Listener `EnableHttpMethodParameterOverride` is enabled by default in the configuration file [#9]
+
+### Fixed
+
+- Sending any form data with a `DELETE` or `PUT` method (the application ignored the hidden field `_method` and as a result the action necessary for the form did not occur) [#9]
+
+[#9]:https://github.com/spiral/roadrunner-laravel/pull/9
+
 ## v3.4.0
 
 ### Added

config/roadrunner.php

@@ -64,6 +64,7 @@
         ],
 
         Events\BeforeLoopIterationEvent::class => [
+            Listeners\EnableHttpMethodParameterOverride::class,
             Listeners\RebindHttpKernelListener::class, // Laravel 7 issue: <https://git.io/JvPpf>
             Listeners\RebindViewListener::class,
             Listeners\CloneConfigListener::class,

src/Listeners/EnableHttpMethodParameterOverride.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spiral\RoadRunnerLaravel\Listeners;
+
+use Spiral\RoadRunnerLaravel\Events\Contracts\WithHttpRequest;
+
+/**
+ * This listener must be registered BEFORE `RebindRouterListener` if you need to process
+ * the hidden `_hidden` field when submitted using the HTML form.
+ *
+ * @link https://github.com/spiral/roadrunner-laravel/pull/9
+ * @see RebindRouterListener
+ */
+class EnableHttpMethodParameterOverride implements ListenerInterface
+{
+    /**
+     * {@inheritdoc}
+     */
+    public function handle($event): void
+    {
+        if ($event instanceof WithHttpRequest) {
+            $event->httpRequest()::enableHttpMethodParameterOverride();
+        }
+    }
+}

tests/Listeners/EnableHttpMethodParameterOverrideTest.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spiral\RoadRunnerLaravel\Tests\Listeners;
+
+use Mockery as m;
+use Symfony\Component\HttpFoundation\Request;
+use Spiral\RoadRunnerLaravel\Events\Contracts\WithHttpRequest;
+use Spiral\RoadRunnerLaravel\Listeners\EnableHttpMethodParameterOverride;
+
+/**
+ * @covers \Spiral\RoadRunnerLaravel\Listeners\EnableHttpMethodParameterOverride<extended>
+ */
+class EnableHttpMethodParameterOverrideTest extends AbstractListenerTestCase
+{
+    /**
+     * {@inheritdoc}
+     */
+    public function testHandle(): void
+    {
+        $request = $this->app->make('request');
+        
+        /** @var m\MockInterface|WithHttpRequest $event_mock */
+        $event_mock = m::mock(WithHttpRequest::class)
+            ->makePartial()
+            ->expects('httpRequest')
+            ->andReturn($request)
+            ->getMock();
+
+        $this->assertFalse($request::getHttpMethodParameterOverride());
+
+        $this->listenerFactory()->handle($event_mock);
+
+        $this->assertTrue($request::getHttpMethodParameterOverride());
+    }
+
+    /**
+     * @return EnableHttpMethodParameterOverride
+     */
+    protected function listenerFactory(): EnableHttpMethodParameterOverride
+    {
+        return new EnableHttpMethodParameterOverride();
+    }
+}