MonologProvider::get()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 8
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
eloc 5
dl 0
loc 8
rs 10
c 0
b 0
f 0
cc 1
nc 1
nop 0
1
<?php
2
3
declare(strict_types=1);
4
5
namespace BEAR\Package\Provide\Logger;
6
7
use BEAR\AppMeta\AbstractAppMeta;
8
use Monolog\Formatter\LineFormatter;
9
use Monolog\Handler\StreamHandler;
10
use Monolog\Logger;
11
use Override;
12
use Ray\Di\ProviderInterface;
13
14
/** @implements ProviderInterface<Logger> */
15
final class MonologProvider implements ProviderInterface
16
{
17
    public function __construct(
18
        private AbstractAppMeta $appMeta,
19
    ) {
20
    }
21
22
    /**
23
     * {@inheritDoc}
24
     */
25
    #[Override]
26
    public function get(): Logger
27
    {
28
        $format = "[%datetime%] %level_name%: %message% %context%\n";
29
        $stream = new StreamHandler($this->appMeta->logDir . '/app.log');
30
        $stream->setFormatter(new LineFormatter($format));
31
32
        return new Logger($this->appMeta->name, [$stream]);
0 ignored issues
show
Bug Best Practice introduced by
The expression return new Monolog\Logge...->name, array($stream)) returns the type Monolog\Logger which is incompatible with the return type mandated by Ray\Di\ProviderInterface::get() of Ray\Di\T.

In the issue above, the returned value is violating the contract defined by the mentioned interface.

Let's take a look at an example:

interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
Loading history...
33
    }
34
}
35