Fix tests, update implementation to use php 7.1 and update docs

Author: snapshotpl

README.md

@@ -1,5 +1,5 @@
 # phpdebugbar middleware [![Build Status](https://travis-ci.org/php-middleware/phpdebugbar.svg?branch=master)](https://travis-ci.org/php-middleware/phpdebugbar)
-PHP Debug bar PSR-15 middleware with PSR-7
+PHP Debug bar [PSR-15](https://www.php-fig.org/psr/psr-15/) middleware with [PSR-7](https://www.php-fig.org/psr/psr-7/). Also supports [PSR-11]([PSR-7](https://www.php-fig.org/psr/psr-11/))
 
 This middleware provide framework-agnostic possibility to attach [PHP Debug Bar](http://phpdebugbar.com/) to your response (html on non-html!).
 
@@ -30,17 +30,7 @@ You don't need to copy any static assets from phpdebugbar vendor!
 
 ### How to install on Zend Expressive?
 
-Use [mtymek/expressive-config-manager](https://github.com/mtymek/expressive-config-manager) and add
-`PhpMiddleware\PhpDebugBar\ConfigProvider` class name:
-
-```php
-$configManager = new \Zend\Expressive\ConfigManager\ConfigManager([
-    \PhpMiddleware\PhpDebugBar\ConfigProvider::class,
-    new \Zend\Expressive\ConfigManager\PhpFileProvider('config/autoload/{{,*.}global,{,*.}local}.php'),
-]);
-```
-
-more [about config manager](https://zendframework.github.io/zend-expressive/cookbook/modular-layout/).
+[Follow Zend Expressive documentation](https://docs.zendframework.com/zend-expressive/v3/features/modular-applications/).
 
 ### How to install on Slim 3?
 

composer.json

@@ -12,15 +12,15 @@
     ],
     "require": {
         "php": "^7.1",
+        "maximebf/debugbar": "^1.4",
         "psr/http-server-handler": "^1.0",
         "psr/http-server-middleware": "^1.0",
-        "maximebf/debugbar": "^1.15",
         "psr/container": "^1.0",
         "psr/http-message": "^1.0.1",
-        "zendframework/zend-diactoros": "^1.7.1"
+        "zendframework/zend-diactoros": "^1.1.3"
     },
     "require-dev": {
-        "phpunit/phpunit": "^6.5",
+        "phpunit/phpunit": "^7.1.2",
         "mikey179/vfsStream": "^1.6.4",
         "slim/slim": "^3.0",
         "zendframework/zend-expressive": "^3.0",

src/ConfigCollectorFactory.php

@@ -1,4 +1,5 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
@@ -7,7 +8,7 @@
 
 final class ConfigCollectorFactory
 {
-    public function __invoke(ContainerInterface $container)
+    public function __invoke(ContainerInterface $container): ConfigCollector
     {
         $data = $container->get('config');
 

src/ConfigProvider.php

@@ -1,16 +1,17 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
 final class ConfigProvider
 {
-    public static function getConfig()
+    public static function getConfig(): array
     {
         $self = new self();
         return $self();
     }
 
-    public function __invoke()
+    public function __invoke(): array
     {
         $config = include __DIR__ . '/../config/phpdebugbar.config.php';
         $config['dependencies'] = include __DIR__ . '/../config/dependency.config.php';

src/JavascriptRendererFactory.php

@@ -1,4 +1,5 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
@@ -8,7 +9,7 @@
 
 final class JavascriptRendererFactory
 {
-    public function __invoke(ContainerInterface $container = null)
+    public function __invoke(ContainerInterface $container = null): JavascriptRenderer
     {
         if ($container === null || !$container->has(DebugBar::class)) {
             $standardDebugBarFactory = new StandardDebugBarFactory();

src/PhpDebugBarMiddleware.php

@@ -1,9 +1,9 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
 use DebugBar\JavascriptRenderer as DebugBarRenderer;
-use PhpMiddleware\DoublePassCompatibilityTrait;
 use Psr\Http\Message\MessageInterface;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
@@ -21,7 +21,7 @@
  *
  * @author Witold Wasiczko <[email protected]>
  */
-class PhpDebugBarMiddleware implements MiddlewareInterface
+final class PhpDebugBarMiddleware implements MiddlewareInterface
 {
     protected $debugBarRenderer;
 
@@ -51,6 +51,26 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
         return $this->prepareHtmlResponseWithDebugBar($response);
     }
 
+    public function __invoke(ServerRequestInterface $request, ResponseInterface $response, callable $next): ResponseInterface
+    {
+        $handler = new class($next, $response) implements RequestHandlerInterface {
+            private $next;
+            private $response;
+
+            public function __construct(callable $next, ResponseInterface $response)
+            {
+                $this->next = $next;
+                $this->response = $response;
+            }
+
+            public function handle(ServerRequestInterface $request): ResponseInterface
+            {
+                return ($this->next)($request, $this->response);
+            }
+        };
+        return $this->process($request, $handler);
+    }
+
     /**
      * @return HtmlResponse
      */

src/PhpDebugBarMiddlewareFactory.php

@@ -1,4 +1,5 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
@@ -7,7 +8,7 @@
 
 final class PhpDebugBarMiddlewareFactory
 {
-    public function __invoke(ContainerInterface $container = null)
+    public function __invoke(ContainerInterface $container = null): PhpDebugBarMiddleware
     {
         if ($container === null || !$container->has(JavascriptRenderer::class)) {
             $rendererFactory = new JavascriptRendererFactory();

src/ResponseInjector/AlwaysInjector.php

@@ -0,0 +1,40 @@
+<?php
+declare (strict_types=1);
+
+namespace PhpMiddleware\PhpDebugBar\ResponseInjector;
+
+use DebugBar\JavascriptRenderer;
+use Psr\Http\Message\ResponseInterface;
+use Zend\Diactoros\Response\HtmlResponse;
+use Zend\Diactoros\Response\Serializer;
+
+final class AlwaysInjector implements ResponseInjectorInterface
+{
+    public function injectPhpDebugBar(ResponseInterface $response, JavascriptRenderer $debugBarRenderer): ResponseInterface
+    {
+        $debugBarHead = $debugBarRenderer->renderHead();
+        $debugBarBody = $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;
+        }
+
+        $outResponseBody = Serializer::toString($outResponse);
+        $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);
+
+        return new HtmlResponse($result);
+    }
+
+    private function isHtmlResponse(ResponseInterface $response): bool
+    {
+        return $this->hasHeaderContains($response, 'Content-Type', 'text/html');
+    }
+}

src/ResponseInjector/ResponseInjectorInterface.php

@@ -0,0 +1,15 @@
+<?php
+declare (strict_types=1);
+
+namespace PhpMiddleware\PhpDebugBar\ResponseInjector;
+
+use DebugBar\JavascriptRenderer;
+use Psr\Http\Message\ResponseInterface;
+
+/**
+ * @author Witold Wasiczko <[email protected]>
+ */
+interface ResponseInjectorInterface
+{
+    public function injectPhpDebugBar(ResponseInterface $response, JavascriptRenderer $debugBar): ResponseInterface;
+}

src/StandardDebugBarFactory.php

@@ -1,4 +1,5 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddleware\PhpDebugBar;
 
@@ -7,7 +8,7 @@
 
 final class StandardDebugBarFactory
 {
-    public function __invoke(ContainerInterface $container = null)
+    public function __invoke(ContainerInterface $container = null): StandardDebugBar
     {
         $debugBar = new StandardDebugBar();
 

test/AbstractMiddlewareRunnerTest.php

@@ -1,22 +1,25 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddlewareTest\PhpDebugBar;
 
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
+use Zend\Diactoros\Response;
 
 abstract class AbstractMiddlewareRunnerTest extends TestCase
 {
 
-    final public function testAppendJsIntoHtmlContent()
+    final public function testAppendJsIntoHtmlContent(): void
     {
         $response = $this->dispatchApplication([
             'REQUEST_URI' => '/hello',
             'REQUEST_METHOD' => 'GET',
             'HTTP_ACCEPT' => 'text/html',
         ], [
-            '/hello' => function (ServerRequestInterface $request, ResponseInterface $response, $next) {
+            '/hello' => function (ServerRequestInterface $request) {
+                $response = new Response();
                 $response->getBody()->write('Hello!');
                 return $response;
             },
@@ -29,7 +32,7 @@ final public function testAppendJsIntoHtmlContent()
         $this->assertContains('"/phpdebugbar/debugbar.js"', $responseBody);
     }
 
-    final public function testGetStatics()
+    final public function testGetStatics(): void
     {
         $response = $this->dispatchApplication([
             'DOCUMENT_ROOT' => __DIR__,
@@ -53,8 +56,5 @@ final public function testGetStatics()
         $this->assertContains('text/javascript', $contentType);
     }
 
-    /**
-     * @return ResponseInterface
-     */
-    abstract protected function dispatchApplication(array $server, array $pipe = []);
+    abstract protected function dispatchApplication(array $server, array $pipe = []): ResponseInterface;
 }

test/PhpDebugBarMiddlewareFactoryTest.php

@@ -1,4 +1,5 @@
 <?php
+declare (strict_types=1);
 
 namespace PhpMiddlewareTest\PhpDebugBar;
 
@@ -11,7 +12,7 @@
  */
 class PhpDebugBarMiddlewareFactoryTest extends TestCase
 {
-    public function testFactory()
+    public function testFactory(): void
     {
         $factory = new PhpDebugBarMiddlewareFactory();