Doctrine\ORM\Query\Exec\SingleSelectExecutor

Complexity

Total Complexity

2

Size/Duplication

Total Lines	13
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	3	1
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace Doctrine\ORM\Query\Exec;

use Doctrine\DBAL\Connection;
use Doctrine\ORM\Query\AST\SelectStatement;
use Doctrine\ORM\Query\SqlWalker;

/**
 * Executor that executes the SQL statement for simple DQL SELECT statements.
 */
class SingleSelectExecutor extends AbstractSqlExecutor
{
    public function __construct(SelectStatement $AST, SqlWalker $sqlWalker)
    {
        $this->sqlStatements = $sqlWalker->walkSelectStatement($AST);

It seems like $sqlWalker->walkSelectStatement($AST) of type string is incompatible with the declared type string[] of property $sqlStatements.

    }

    /**
     * {@inheritDoc}
     */
    public function execute(Connection $conn, array $params, array $types)
    {
        return $conn->executeQuery($this->sqlStatements, $params, $types, $this->queryCacheProfile);

$this->sqlStatements of type string[] is incompatible with the type string expected by parameter $query of Doctrine\DBAL\Connection::executeQuery().

The expression return $conn->executeQuery($this->sqlStatements, $params, $types, $this->queryCacheProfile) also could return the type Doctrine\DBAL\Driver\ResultStatement which includes types incompatible with the return type mandated by Doctrine\ORM\Query\Exec\AbstractSqlExecutor::execute() of Doctrine\DBAL\Driver\Statement. Consider adding a type-check to rule them out.

    }
}