[5.6] Fix failed returning Responsable from Middleware

src/Illuminate/Pipeline/Pipeline.php

@@ -5,6 +5,7 @@
 use Closure;
 use RuntimeException;
 use Illuminate\Contracts\Container\Container;
+use Illuminate\Contracts\Support\Responsable;
 use Illuminate\Contracts\Pipeline\Pipeline as PipelineContract;
 
 class Pipeline implements PipelineContract
@@ -96,7 +97,9 @@ public function via($method)
     public function then(Closure $destination)
     {
         $pipeline = array_reduce(
-            array_reverse($this->pipes), $this->carry(), $this->prepareDestination($destination)
+            array_reverse($this->pipes),
+            $this->carry(),
+            $this->prepareDestination($destination)
         );
 
         return $pipeline($this->passable);
@@ -145,9 +148,13 @@ protected function carry()
                     $parameters = [$passable, $stack];
                 }
 
-                return method_exists($pipe, $this->method)
-                                ? $pipe->{$this->method}(...$parameters)
-                                : $pipe(...$parameters);
+                $response = method_exists($pipe, $this->method)
+                    ? $pipe->{$this->method}(...$parameters)
+                    : $pipe(...$parameters);
+
+                return $response instanceof Responsable
+                    ? $response->toResponse($this->container->make('Illuminate\Http\Request'))
+                    : $response;
             };
         };
     }

tests/Pipeline/PipelineTest.php

@@ -4,6 +4,7 @@
 
 use PHPUnit\Framework\TestCase;
 use Illuminate\Pipeline\Pipeline;
+use Illuminate\Contracts\Support\Responsable;
 
 class PipelineTest extends TestCase
 {
@@ -26,8 +27,7 @@ public function testPipelineBasicUsage()
         $this->assertEquals('foo', $_SERVER['__test.pipe.one']);
         $this->assertEquals('foo', $_SERVER['__test.pipe.two']);
 
-        unset($_SERVER['__test.pipe.one']);
-        unset($_SERVER['__test.pipe.two']);
+        unset($_SERVER['__test.pipe.one'], $_SERVER['__test.pipe.two']);
     }
 
     public function testPipelineUsageWithObjects()
@@ -62,6 +62,23 @@ function ($piped) {
         unset($_SERVER['__test.pipe.one']);
     }
 
+    public function testPipelineUsageWithResponsableObjects()
+    {
+        $result = (new Pipeline(new \Illuminate\Container\Container))
+            ->send('foo')
+            ->through([new PipelineTestPipeResponsable])
+            ->then(
+                function ($piped) {
+                    return $piped;
+                }
+            );
+
+        $this->assertEquals('bar', $result);
+        $this->assertEquals('foo', $_SERVER['__test.pipe.responsable']);
+
+        unset($_SERVER['__test.pipe.responsable']);
+    }
+
     public function testPipelineUsageWithCallable()
     {
         $function = function ($piped, $next) {
@@ -160,6 +177,14 @@ public function differentMethod($piped, $next)
     }
 }
 
+class PipeResponsable implements Responsable
+{
+    public function toResponse($request)
+    {
+        return 'bar';
+    }
+}
+
 class PipelineTestPipeTwo
 {
     public function __invoke($piped, $next)
@@ -170,6 +195,16 @@ public function __invoke($piped, $next)
     }
 }
 
+class PipelineTestPipeResponsable
+{
+    public function handle($piped, $next)
+    {
+        $_SERVER['__test.pipe.responsable'] = $piped;
+
+        return new PipeResponsable;
+    }
+}
+
 class PipelineTestParameterPipe
 {
     public function handle($piped, $next, $parameter1 = null, $parameter2 = null)