Passed
Push — support-php-8.2-8.3 ( ae3b12...8c2247 )
by Akihito
01:21
created

AuraSqlQueryInsertProvider::get()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 1
Metric Value
c 1
b 0
f 1
dl 0
loc 4
rs 10
cc 1
eloc 2
nc 1
nop 0
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Ray\AuraSqlModule;
6
7
use Aura\SqlQuery\Common\InsertInterface;
8
use Aura\SqlQuery\QueryFactory;
9
use Override;
10
use Ray\AuraSqlModule\Annotation\AuraSqlQueryConfig;
11
use Ray\Di\ProviderInterface;
12
13
/** @implements ProviderInterface<InsertInterface> */
14
final class AuraSqlQueryInsertProvider implements ProviderInterface
15
{
16
    /** @param string $db The database type */
17
    public function __construct(
18
        #[AuraSqlQueryConfig]
19
        private string $db
20
    ) {
21
    }
22
23
    /**
24
     * {@inheritDoc}
25
     */
26
    #[Override]
27
    public function get(): InsertInterface
28
    {
29
        return (new QueryFactory($this->db))->newInsert();
0 ignored issues
show
Bug Best Practice introduced by
The expression return new Aura\SqlQuery...$this->db)->newInsert() returns the type Aura\SqlQuery\Common\InsertInterface 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...
30
    }
31
}
32