phpstan update

classes/Stream/PlaylistArchiveStream.php

@@ -301,7 +301,14 @@ public function read($length)
                 }
             } else {
                 // Start streaming the first video.
-                $this->startVideoStream(current($this->videos));
+                $video = current($this->videos);
+                if ($video) {
+                    $this->startVideoStream($video);
+                } else {
+                    $this->push_error('Playlist was empty');
+                    $this->finish();
+                    $this->isComplete = true;
+                }
             }
         }
 

composer.json

@@ -49,7 +49,7 @@
     "maximebf/debugbar": "^1.16",
     "php-mock/php-mock-mockery": "^1.3",
     "phpro/grumphp": "^1.1",
-    "phpstan/phpstan": "^0.12.25",
+    "phpstan/phpstan": "^0.12.72",
     "phpunit/phpunit": "^8.4",
     "smarty-gettext/smarty-gettext": "^1.6",
     "squizlabs/php_codesniffer": "^3.5",

tests/UglyRouterTest.php

@@ -37,7 +37,16 @@ protected function setUp(): void
         parent::setUp();
 
         $this->router = new UglyRouter();
-        $this->router->map(['GET'], '/foo', 'print')->setName('foo');
+        $this->router->map(['GET'], '/foo', [$this, 'fakeHandler'])->setName('foo');
+    }
+
+    /**
+     * Empty function that only exists so that our route can have a handler.
+     *
+     * @return void
+     */
+    private function fakeHandler()
+    {
     }
 
     /**
@@ -54,7 +63,7 @@ public function testDispatch()
                     Environment::mock(
                         [
                             'REQUEST_METHOD' => 'GET',
-                            'QUERY_STRING'   => 'page=foo',
+                            'QUERY_STRING' => 'page=foo',
                         ]
                     )
                 )