SimpleFileStreamable::stream() - Code Metrics - Inspection of "Add some test coverage" - uxmp/core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 75e993...93dd37 )

by Daniel

created 2022-09-02 16:57 UTC

SimpleFileStreamable::stream() A

↳ Parent: SimpleFileStreamable

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	9
nc	1
nop	2
dl	0
loc	15
ccs	9
cts	9
cp	1
crap	1
rs	9.9666
c	1
b	0
f	0

<?php

declare(strict_types=1);

namespace Uxmp\Core\Component\Io;

use Nyholm\Psr7\Factory\Psr17Factory;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\StreamInterface;
use Uxmp\Core\Orm\Model\SongInterface;

final class SimpleFileStreamable implements StreamableInterface
{
    public function __construct(
        private readonly Psr17Factory $psr17Factory
    ) {
    }

    public function stream(
        ResponseInterface $response,
        SongInterface $song,
    ): ResponseInterface {
        $size = $song->getFileSize();

        return $response
            ->withHeader('Content-Type', (string) $song->getMimeType())
            ->withHeader('Content-Disposition', sprintf('attachment; filename=song%d.mp3', $song->getId()))
            ->withHeader('Content-Length', (string) $size)
            ->withHeader('Cache-Control', 'no-cache')
            ->withHeader('Content-Range', 'bytes 0-'.$size)
            //->withHeader('Accept-Ranges', 'bytes') @todo add seek
            ->withBody(
                $this->createStream($song)
            );
    }

    public function createStream(SongInterface $song): StreamInterface
    {
        return $this->psr17Factory->createStreamFromFile($song->getFilename());
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Uxmp\Core\Component\Io;
6
7		use Nyholm\Psr7\Factory\Psr17Factory;
8		use Psr\Http\Message\ResponseInterface;
9		use Psr\Http\Message\StreamInterface;
10		use Uxmp\Core\Orm\Model\SongInterface;
11
12		final class SimpleFileStreamable implements StreamableInterface
13		{
14	2	public function __construct(
15		private readonly Psr17Factory $psr17Factory
16		) {
17		}
18
19	1	public function stream(
20		ResponseInterface $response,
21		SongInterface $song,
22		): ResponseInterface {
23	1	$size = $song->getFileSize();
24
25		return $response
26	1	->withHeader('Content-Type', (string) $song->getMimeType())
27	1	->withHeader('Content-Disposition', sprintf('attachment; filename=song%d.mp3', $song->getId()))
28	1	->withHeader('Content-Length', (string) $size)
29	1	->withHeader('Cache-Control', 'no-cache')
30	1	->withHeader('Content-Range', 'bytes 0-'.$size)
31		//->withHeader('Accept-Ranges', 'bytes') @todo add seek
32	1	->withBody(
33	1	$this->createStream($song)
34		);
35		}
36
37	1	public function createStream(SongInterface $song): StreamInterface
38		{
39	1	return $this->psr17Factory->createStreamFromFile($song->getFilename());
40		}
41		}
42

uxmp / core

Push — main ( 75e993...93dd37 )

SimpleFileStreamable::stream() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like