Completed
Push — master ( 6f1b7e...a981b6 )
by Rafal
02:18
created

Factory::createConfig()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 6
Code Lines 4

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
dl 0
loc 6
rs 9.4285
c 0
b 0
f 0
cc 1
eloc 4
nc 1
nop 0
1
<?php
2
3
4
namespace SpotifyApiConnect;
5
6
7
use Dotenv\Dotenv;
8
use SpotifyApiConnect\Application\SpotifyApiAuth;
9
use SpotifyApiConnect\Application\SpotifyApiAuthInterface;
10
use SpotifyApiConnect\Application\SpotifyWebApiPhp\Session;
11
use SpotifyApiConnect\Application\SpotifyWebApiPhp\SessionInterface;
12
use SpotifyApiConnect\Domain\Model\Config;
13
14
final class Factory
15
{
16
    public function __construct()
17
    {
18
        $envFile = __DIR__ . '/../.env';
19
        if (!file_exists($envFile)) {
20
            throw new \RuntimeException('Pleas create ".env"-File and fill this file with info');
21
        }
22
        if (null === getenv('CLIENT_ID')) {
23
            (new Dotenv())->load($envFile);
0 ignored issues
show
Unused Code introduced by
The call to Dotenv\Dotenv::load() has too many arguments starting with $envFile. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

23
            (new Dotenv())->/** @scrutinizer ignore-call */ load($envFile);

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
Bug introduced by
The call to Dotenv\Dotenv::__construct() has too few arguments starting with path. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

23
            (/** @scrutinizer ignore-call */ new Dotenv())->load($envFile);

This check compares calls to functions or methods with their respective definitions. If the call has less arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
24
        }
25
    }
26
27
    /**
28
     * @return SpotifyApiAuthInterface
29
     */
30
    public function createSpotifyApiAuth() : SpotifyApiAuthInterface
31
    {
32
        return new SpotifyApiAuth(
33
            $this->createSpotifyWebApiPhpSession()
34
        );
35
    }
36
37
    /**
38
     * @return SessionInterface
39
     */
40
    private function createSpotifyWebApiPhpSession(): SessionInterface
41
    {
42
        return new Session(
43
            $this->createConfig()
44
        );
45
    }
46
47
    /**
48
     * @return Config
49
     */
50
    private function createConfig(): Config
51
    {
52
        return new Config(
53
            getenv('CLIENT_ID'),
54
            getenv('CLIENT_SECRET'),
55
            getenv('REDIRECT_URI')
56
        );
57
    }
58
59
}