Browse Source

Rename MiddlewareStack to MiddlewareQueue.

ADmad 9 years ago
parent
commit
f492ddbce3

+ 2 - 2
src/Http/BaseApplication.php

@@ -43,8 +43,8 @@ abstract class BaseApplication
     }
     }
 
 
     /**
     /**
-     * @param \Cake\Http\MiddlewareStack $middleware The middleware queue to set in your App Class
-     * @return \Cake\Http\MiddlewareStack
+     * @param \Cake\Http\MiddlewareQueue $middleware The middleware queue to set in your App Class
+     * @return \Cake\Http\MiddlewareQueue
      */
      */
     abstract public function middleware($middleware);
     abstract public function middleware($middleware);
 
 

+ 1 - 1
src/Http/MiddlewareStack.php

@@ -21,7 +21,7 @@ use LogicException;
  * Provides methods for creating and manipulating a "queue" of middleware callables.
  * Provides methods for creating and manipulating a "queue" of middleware callables.
  * This queue is used to process a request and response via \Cake\Http\Runner.
  * This queue is used to process a request and response via \Cake\Http\Runner.
  */
  */
-class MiddlewareStack implements Countable
+class MiddlewareQueue implements Countable
 {
 {
     /**
     /**
      * The queue of middleware callables.
      * The queue of middleware callables.

+ 2 - 2
src/Http/Runner.php

@@ -33,12 +33,12 @@ class Runner
     /**
     /**
      * The middleware queue being run.
      * The middleware queue being run.
      *
      *
-     * @var MiddlewareStack
+     * @var MiddlewareQueue
      */
      */
     protected $middleware;
     protected $middleware;
 
 
     /**
     /**
-     * @param \Cake\Http\MiddlewareStack $middleware The middleware queue
+     * @param \Cake\Http\MiddlewareQueue $middleware The middleware queue
      * @param \Psr\Http\Message\ServerRequestInterface $request The Server Request
      * @param \Psr\Http\Message\ServerRequestInterface $request The Server Request
      * @param \Psr\Http\Message\ResponseInterface $response The response
      * @param \Psr\Http\Message\ResponseInterface $response The response
      * @return \Psr\Http\Message\ResponseInterface A response object
      * @return \Psr\Http\Message\ResponseInterface A response object

+ 2 - 2
src/Http/Server.php

@@ -73,8 +73,8 @@ class Server
         $request = $request ?: ServerRequestFactory::fromGlobals();
         $request = $request ?: ServerRequestFactory::fromGlobals();
         $response = $response ?: new Response();
         $response = $response ?: new Response();
 
 
-        $middleware = $this->app->middleware(new MiddlewareStack());
-        if (!($middleware instanceof MiddlewareStack)) {
+        $middleware = $this->app->middleware(new MiddlewareQueue());
+        if (!($middleware instanceof MiddlewareQueue)) {
             throw new RuntimeException('The application `middleware` method did not return a middleware queue.');
             throw new RuntimeException('The application `middleware` method did not return a middleware queue.');
         }
         }
         $this->dispatchEvent('Server.buildMiddleware', ['middleware' => $middleware]);
         $this->dispatchEvent('Server.buildMiddleware', ['middleware' => $middleware]);

+ 16 - 16
tests/TestCase/Http/MiddlewareStackTest.php

@@ -14,14 +14,14 @@
  */
  */
 namespace Cake\Test\TestCase\Http;
 namespace Cake\Test\TestCase\Http;
 
 
-use Cake\Http\MiddlewareStack;
+use Cake\Http\MiddlewareQueue;
 use Cake\TestSuite\TestCase;
 use Cake\TestSuite\TestCase;
 use TestApp\Middleware\SampleMiddleware;
 use TestApp\Middleware\SampleMiddleware;
 
 
 /**
 /**
- * Test case for the MiddlewareStack
+ * Test case for the MiddlewareQueue
  */
  */
-class MiddlewareStackTest extends TestCase
+class MiddlewareQueueTest extends TestCase
 {
 {
     /**
     /**
      * Test get()
      * Test get()
@@ -30,7 +30,7 @@ class MiddlewareStackTest extends TestCase
      */
      */
     public function testGet()
     public function testGet()
     {
     {
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $cb = function () {
         $cb = function () {
         };
         };
         $stack->add($cb);
         $stack->add($cb);
@@ -46,7 +46,7 @@ class MiddlewareStackTest extends TestCase
      */
      */
     public function testPushReturn()
     public function testPushReturn()
     {
     {
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $cb = function () {
         $cb = function () {
         };
         };
         $this->assertSame($stack, $stack->add($cb));
         $this->assertSame($stack, $stack->add($cb));
@@ -64,7 +64,7 @@ class MiddlewareStackTest extends TestCase
         $two = function () {
         $two = function () {
         };
         };
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $this->assertCount(0, $stack);
         $this->assertCount(0, $stack);
 
 
         $stack->add($one);
         $stack->add($one);
@@ -86,7 +86,7 @@ class MiddlewareStackTest extends TestCase
     {
     {
         $cb = function () {
         $cb = function () {
         };
         };
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $this->assertSame($stack, $stack->prepend($cb));
         $this->assertSame($stack, $stack->prepend($cb));
     }
     }
 
 
@@ -102,7 +102,7 @@ class MiddlewareStackTest extends TestCase
         $two = function () {
         $two = function () {
         };
         };
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $this->assertCount(0, $stack);
         $this->assertCount(0, $stack);
 
 
         $stack->add($one);
         $stack->add($one);
@@ -129,13 +129,13 @@ class MiddlewareStackTest extends TestCase
         $three = function () {
         $three = function () {
         };
         };
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertAt(0, $three);
         $stack->add($one)->add($two)->insertAt(0, $three);
         $this->assertSame($three, $stack->get(0));
         $this->assertSame($three, $stack->get(0));
         $this->assertSame($one, $stack->get(1));
         $this->assertSame($one, $stack->get(1));
         $this->assertSame($two, $stack->get(2));
         $this->assertSame($two, $stack->get(2));
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertAt(1, $three);
         $stack->add($one)->add($two)->insertAt(1, $three);
         $this->assertSame($one, $stack->get(0));
         $this->assertSame($one, $stack->get(0));
         $this->assertSame($three, $stack->get(1));
         $this->assertSame($three, $stack->get(1));
@@ -154,7 +154,7 @@ class MiddlewareStackTest extends TestCase
         $two = function () {
         $two = function () {
         };
         };
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->insertAt(99, $two);
         $stack->add($one)->insertAt(99, $two);
 
 
         $this->assertCount(2, $stack);
         $this->assertCount(2, $stack);
@@ -174,7 +174,7 @@ class MiddlewareStackTest extends TestCase
         $two = function () {
         $two = function () {
         };
         };
 
 
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->insertAt(-1, $two);
         $stack->add($one)->insertAt(-1, $two);
 
 
         $this->assertCount(2, $stack);
         $this->assertCount(2, $stack);
@@ -194,7 +194,7 @@ class MiddlewareStackTest extends TestCase
         $two = new SampleMiddleware();
         $two = new SampleMiddleware();
         $three = function () {
         $three = function () {
         };
         };
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertBefore(SampleMiddleware::class, $three);
         $stack->add($one)->add($two)->insertBefore(SampleMiddleware::class, $three);
 
 
         $this->assertCount(3, $stack);
         $this->assertCount(3, $stack);
@@ -217,7 +217,7 @@ class MiddlewareStackTest extends TestCase
         $two = new SampleMiddleware();
         $two = new SampleMiddleware();
         $three = function () {
         $three = function () {
         };
         };
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertBefore('InvalidClassName', $three);
         $stack->add($one)->add($two)->insertBefore('InvalidClassName', $three);
     }
     }
 
 
@@ -233,7 +233,7 @@ class MiddlewareStackTest extends TestCase
         };
         };
         $three = function () {
         $three = function () {
         };
         };
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertAfter(SampleMiddleware::class, $three);
         $stack->add($one)->add($two)->insertAfter(SampleMiddleware::class, $three);
 
 
         $this->assertCount(3, $stack);
         $this->assertCount(3, $stack);
@@ -254,7 +254,7 @@ class MiddlewareStackTest extends TestCase
         };
         };
         $three = function () {
         $three = function () {
         };
         };
-        $stack = new MiddlewareStack();
+        $stack = new MiddlewareQueue();
         $stack->add($one)->add($two)->insertAfter('InvalidClass', $three);
         $stack->add($one)->add($two)->insertAfter('InvalidClass', $three);
 
 
         $this->assertCount(3, $stack);
         $this->assertCount(3, $stack);

+ 2 - 2
tests/TestCase/Http/RunnerTest.php

@@ -14,7 +14,7 @@
  */
  */
 namespace Cake\Test\TestCase;
 namespace Cake\Test\TestCase;
 
 
-use Cake\Http\MiddlewareStack;
+use Cake\Http\MiddlewareQueue;
 use Cake\Http\Runner;
 use Cake\Http\Runner;
 use Cake\TestSuite\TestCase;
 use Cake\TestSuite\TestCase;
 use RuntimeException;
 use RuntimeException;
@@ -32,7 +32,7 @@ class RunnerTest extends TestCase
     public function setUp()
     public function setUp()
     {
     {
         parent::setUp();
         parent::setUp();
-        $this->stack = new MiddlewareStack();
+        $this->stack = new MiddlewareQueue();
 
 
         $this->ok = function ($req, $res, $next) {
         $this->ok = function ($req, $res, $next) {
             return $next($req, $res);
             return $next($req, $res);

+ 1 - 1
tests/TestCase/Http/ServerTest.php

@@ -184,7 +184,7 @@ class ServerTest extends TestCase
         $this->called = false;
         $this->called = false;
 
 
         $server->eventManager()->on('Server.buildMiddleware', function ($event, $middleware) {
         $server->eventManager()->on('Server.buildMiddleware', function ($event, $middleware) {
-            $this->assertInstanceOf('Cake\Http\MiddlewareStack', $middleware);
+            $this->assertInstanceOf('Cake\Http\MiddlewareQueue', $middleware);
             $middleware->add(function ($req, $res, $next) {
             $middleware->add(function ($req, $res, $next) {
                 $this->called = true;
                 $this->called = true;
 
 

+ 2 - 2
tests/test_app/TestApp/Http/BadResponseApplication.php

@@ -8,8 +8,8 @@ use Psr\Http\Message\ServerRequestInterface;
 class BadResponseApplication extends BaseApplication
 class BadResponseApplication extends BaseApplication
 {
 {
     /**
     /**
-     * @param \Cake\Http\MiddlewareStack $middleware The middleware stack to set in your App Class
-     * @return \Cake\Http\MiddlewareStack
+     * @param \Cake\Http\MiddlewareQueue $middleware The middleware stack to set in your App Class
+     * @return \Cake\Http\MiddlewareQueue
      */
      */
     public function middleware($middleware)
     public function middleware($middleware)
     {
     {

+ 1 - 1
tests/test_app/TestApp/Http/InvalidMiddlewareApplication.php

@@ -6,7 +6,7 @@ use Cake\Http\BaseApplication;
 class InvalidMiddlewareApplication extends BaseApplication
 class InvalidMiddlewareApplication extends BaseApplication
 {
 {
     /**
     /**
-     * @param \Cake\Http\MiddlewareStack $middleware The middleware stack to set in your App Class
+     * @param \Cake\Http\MiddlewareQueue $middleware The middleware stack to set in your App Class
      * @return null
      * @return null
      */
      */
     public function middleware($middleware)
     public function middleware($middleware)

+ 2 - 2
tests/test_app/TestApp/Http/MiddlewareApplication.php

@@ -8,8 +8,8 @@ use Psr\Http\Message\ServerRequestInterface;
 class MiddlewareApplication extends BaseApplication
 class MiddlewareApplication extends BaseApplication
 {
 {
     /**
     /**
-     * @param \Cake\Http\MiddlewareStack $middleware The middleware stack to set in your App Class
-     * @return \Cake\Http\MiddlewareStack
+     * @param \Cake\Http\MiddlewareQueue $middleware The middleware stack to set in your App Class
+     * @return \Cake\Http\MiddlewareQueue
      */
      */
     public function middleware($middleware)
     public function middleware($middleware)
     {
     {