App\Services\Streamers\S3Streamer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Test Coverage

Coverage

66.67%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	stream()	0	4	1
A	__construct()	0	4	1

<?php

namespace App\Services\Streamers;

use App\Services\S3Service;

class S3Streamer extends Streamer implements ObjectStorageStreamerInterface
{
    private $s3Service;

    public function __construct(S3Service $s3Service)
    {
        parent::__construct();
        $this->s3Service = $s3Service;
    }

    /**
     * Stream the current song through S3.
     * Actually, we just redirect the request to the S3 object's location.
     */
    public function stream()
    {
        // Get and redirect to the actual presigned-url
        return redirect($this->s3Service->getSongPublicUrl($this->song));

It seems like $this->song can also be of type string; however, parameter $song of App\Services\S3Service::getSongPublicUrl() does only seem to accept App\Models\Song, maybe add an additional type check?

    }
}