phpstan

Author: samsonasik

.travis.yml

@@ -9,7 +9,6 @@ before_script:
   - mkdir -p build/logs
   - composer self-update
   - travis_retry composer install --prefer-source --no-interaction
-  - composer require --dev phpstan/phpstan
   - composer dump-autoload -o
 
 script:

composer.json

@@ -26,6 +26,7 @@
     ],
     "require": {
         "php": "^7.1",
+        "webmozart/assert": "^1.4",
         "zendframework/zend-console": "^2.5"
     },
     "conflict": {
@@ -35,11 +36,14 @@
     "require-dev": {
         "kahlan/kahlan": "^4.0",
         "php-coveralls/php-coveralls": "^2.1",
+        "phpstan/phpstan": "^0.10.8",
+        "phpstan/phpstan-webmozart-assert": "^0.10.0",
         "zendframework/zend-expressive": "^3.0",
         "zendframework/zend-mvc": "^3.0"
     },
     "config": {
-        "bin-dir": "bin"
+        "bin-dir": "bin",
+        "sort-packages": true
     },
     "extra": {
         "zf": {

phpstan.neon

@@ -0,0 +1,3 @@
+includes:
+	- vendor/phpstan/phpstan-webmozart-assert/extension.neon
+

src/HttpsTrait.php

@@ -5,6 +5,7 @@
 namespace ForceHttpsModule;
 
 use Psr\Http\Message\ResponseInterface;
+use Webmozart\Assert\Assert;
 use Zend\Expressive\Router\RouteResult;
 use Zend\Http\PhpEnvironment\Response;
 use Zend\Router\RouteMatch;
@@ -41,6 +42,7 @@ private function isGoingToBeForcedToHttps($match = null) : bool
             return false;
         }
 
+        Assert::notNull($match);
         if (! $this->config['force_all_routes'] &&
             ! \in_array(
                 $match->getMatchedRouteName(),