Add psr-15 support, drop 5.5 support

Author: snapshotpl

.travis.yml

@@ -1,7 +1,6 @@
 language: php
 
 php:
-  - 5.5
   - 5.6
   - 7.0
   - 7.1
@@ -13,8 +12,8 @@ env:
 
 before_script:
   - composer self-update
-  - if [[ $DEPS == 'lowest' ]]; then travis_retry composer update --prefer-lowest --prefer-stable --no-interaction ; fi
-  - travis_retry composer install --no-interaction
+  - if [[ $DEPS == 'lowest' ]]; then composer update --prefer-stable --no-interaction --prefer-lowest ; fi
+  - if [[ $DEPS == 'latest' ]]; then composer update --prefer-stable --no-interaction ; fi
 
 script:
   - ./vendor/bin/phpunit

composer.json

@@ -6,17 +6,20 @@
         "debug",
         "middleware",
         "psr",
-        "psr-7"
+        "psr-7",
+        "psr-15"
     ],
     "require": {
-        "php": "^5.5 || ^7.0",
+        "php": ">=5.6",
         "maximebf/debugbar": "^1.10",
         "psr/http-message": "^1.0",
         "container-interop/container-interop": "^1.1",
-        "zendframework/zend-diactoros": "^1.1.3"
+        "zendframework/zend-diactoros": "^1.1.3",
+        "http-interop/http-middleware": "^0.4.1",
+        "php-middleware/double-pass-compatibility": "^1.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^4.8.6",
+        "phpunit/phpunit": "^5.7.19",
         "mikey179/vfsStream": "^1.6",
         "slim/slim": "^3.0",
         "zendframework/zend-expressive": "^1.0",

phpunit.xml

@@ -2,7 +2,7 @@
 
 <phpunit bootstrap="./vendor/autoload.php" colors="true">
     <testsuites>
-        <testsuite name="PhpMiddleware\\PhpDebugBar Tests">
+        <testsuite>
             <directory>./test</directory>
         </testsuite>
     </testsuites>

src/PhpDebugBarMiddleware.php

@@ -3,6 +3,9 @@
 namespace PhpMiddleware\PhpDebugBar;
 
 use DebugBar\JavascriptRenderer as DebugBarRenderer;
+use Interop\Http\ServerMiddleware\DelegateInterface;
+use Interop\Http\ServerMiddleware\MiddlewareInterface;
+use PhpMiddleware\DoublePassCompatibilityTrait;
 use Psr\Http\Message\MessageInterface;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
@@ -18,64 +21,71 @@
  *
  * @author Witold Wasiczko <witold@wasiczko.pl>
  */
-class PhpDebugBarMiddleware
+class PhpDebugBarMiddleware implements MiddlewareInterface
 {
-    /**
-     * @var DebugBarRenderer
-     */
+    use DoublePassCompatibilityTrait;
+
     protected $debugBarRenderer;
 
-    /**
-     * @param DebugBarRenderer $debugbarRenderer
-     */
     public function __construct(DebugBarRenderer $debugbarRenderer)
     {
         $this->debugBarRenderer = $debugbarRenderer;
     }
 
     /**
-     * @param ServerRequestInterface $request
-     * @param ResponseInterface $response
-     * @param callable $next
-     *
-     * @return ResponseInterface
+     * @inheritDoc
      */
-    public function __invoke(ServerRequestInterface $request, ResponseInterface $response, callable $next)
+    public function process(ServerRequestInterface $request, DelegateInterface $delegate)
     {
         if ($staticFile = $this->getStaticFile($request->getUri())) {
             return $staticFile;
         }
 
-        $outResponse = $next($request, $response);
+        $response = $delegate->process($request);
 
         if (!$this->isHtmlAccepted($request)) {
-            return $outResponse;
+            return $response;
         }
 
-        $debugBarHead = $this->debugBarRenderer->renderHead();
-        $debugBarBody = $this->debugBarRenderer->render();
-
-        if ($this->isHtmlResponse($outResponse)) {
-            $body = $outResponse->getBody();
-            if (! $body->eof() && $body->isSeekable()) {
-                $body->seek(0, SEEK_END);
-            }
-            $body->write($debugBarHead . $debugBarBody);
-
-            return $outResponse;
+        if ($this->isHtmlResponse($response)) {
+            return $this->attachDebugBarToResponse($response);
         }
+        return $this->prepareHtmlResponseWithDebugBar($response);
+    }
 
-        $outResponseBody = Serializer::toString($outResponse);
+    /**
+     * @return HtmlResponse
+     */
+    private function prepareHtmlResponseWithDebugBar(ResponseInterface $response)
+    {
+        $head = $this->debugBarRenderer->renderHead();
+        $body = $this->debugBarRenderer->render();
+        $outResponseBody = Serializer::toString($response);
         $template = '<html><head>%s</head><body><h1>DebugBar</h1><p>Response:</p><pre>%s</pre>%s</body></html>';
         $escapedOutResponseBody = htmlspecialchars($outResponseBody);
-        $result = sprintf($template, $debugBarHead, $escapedOutResponseBody, $debugBarBody);
+        $result = sprintf($template, $head, $escapedOutResponseBody, $body);
 
         return new HtmlResponse($result);
     }
 
     /**
-     * @param UriInterface $uri
-     *
+     * @return ResponseInterface
+     */
+    private function attachDebugBarToResponse(ResponseInterface $response)
+    {
+        $head = $this->debugBarRenderer->renderHead();
+        $body = $this->debugBarRenderer->render();
+        $responseBody = $response->getBody();
+
+        if (! $responseBody->eof() && $responseBody->isSeekable()) {
+            $responseBody->seek(0, SEEK_END);
+        }
+        $responseBody->write($head . $body);
+
+        return $response;
+    }
+
+    /**
      * @return ResponseInterface|null
      */
     private function getStaticFile(UriInterface $uri)
@@ -139,11 +149,7 @@ private function getContentTypeByFileName($filename)
             'woff2' => 'application/font-woff2',
         ];
 
-        if (isset($map[$ext])) {
-            return $map[$ext];
-        }
-
-        return 'text/plain';
+        return isset($map[$ext]) ? $map[$ext] : 'text/plain';
     }
 
     /**