for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace App\Services\Streamers;
use DaveRandom\Resume\FileResource;
use DaveRandom\Resume\InvalidRangeHeaderException;
use DaveRandom\Resume\NonExistentFileException;
use DaveRandom\Resume\RangeSet;
use DaveRandom\Resume\Resource;
use DaveRandom\Resume\ResourceServlet;
use DaveRandom\Resume\SendFileFailureException;
use DaveRandom\Resume\UnreadableFileException;
use DaveRandom\Resume\UnsatisfiableRangeException;
use function DaveRandom\Resume\get_request_header;
class PHPStreamer extends Streamer implements DirectStreamerInterface
{
public function stream()
try {
$rangeSet = RangeSet::createFromHeader(get_request_header('Range'));
/** @var resource $resource */
$resource = new FileResource($this->song->path, 'application/octet-stream');
(new ResourceServlet($resource))->sendResource($rangeSet);
$resource
resource
object<DaveRandom\Resume\Resource>
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example:
function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x);
} catch (InvalidRangeHeaderException $e) {
abort(400);
} catch (UnsatisfiableRangeException $e) {
abort(416);
} catch (NonExistentFileException $e) {
abort(404);
} catch (UnreadableFileException $e) {
abort(500);
} catch (SendFileFailureException $e) {
abort_unless(headers_sent(), 500);
echo "An error occurred while attempting to send the requested resource: {$e->getMessage()}";
}
exit;
phanan /
koel
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example: