fix: Don't call PlaylistArchiveStream parent constructor

Because it messes with the output buffer

Fixes #215
This commit is contained in:
Pierre Rudloff 2019-04-20 11:27:13 +02:00
parent 7ab4c55550
commit 07261db27b
5 changed files with 537 additions and 202 deletions

View file

@ -36,7 +36,8 @@ module.exports = function (grunt) {
phpstan: {
options: {
level: 'max',
bin: 'vendor/bin/phpstan'
bin: 'vendor/bin/phpstan',
config: 'phpstan.neon'
},
php: {
src: ['*.php', 'classes/*.php', 'controllers/*.php']

View file

@ -66,8 +66,6 @@ class PlaylistArchiveStream extends TarArchive
*/
public function __construct(Config $config = null)
{
parent::__construct();
$this->client = new Client();
$this->download = new VideoDownload($config);
}

View file

@ -21,7 +21,7 @@
"grunt-markdownlint": "~2.1.0",
"grunt-phpcs": "~0.4.0",
"grunt-phpdocumentor": "~0.4.1",
"grunt-phpstan": "~0.1.0",
"grunt-phpstan": "~0.2.0",
"grunt-phpunit": "~0.3.6"
},
"homepage": "https://www.alltubedownload.net/",

4
phpstan.neon Normal file
View file

@ -0,0 +1,4 @@
parameters:
ignoreErrors:
# The Archive constructor messes up the output buffering.
- '#Alltube\\PlaylistArchiveStream::__construct\(\) does not call parent constructor from Barracuda\\ArchiveStream\\TarArchive\.#'

728
yarn.lock

File diff suppressed because it is too large Load diff