BenTools\SimpleDBAL\Model\Adapter\Mysqli\EmulatedStatement

Complexity

Total Complexity

7

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getWrappedStatement()	0	4	1
A	bind()	0	4	1
A	getRunnableQuery()	0	7	2
A	createResult()	0	6	2

<?php

namespace BenTools\SimpleDBAL\Model\Adapter\Mysqli;

use BenTools\SimpleDBAL\Contract\ResultInterface;

class EmulatedStatement extends Statement
{
    /**
     * @inheritDoc
     */
    public function __construct(MysqliAdapter $connection, array $values = null, $queryString)
    {
        $this->connection          = $connection;
        $this->values              = $values;

It seems like $values can be null. However, the property $values is declared as array. Maybe change the type of the property to array|null or add a type check?

        $this->queryString         = $queryString;
    }

    /**
     * @inheritDoc
     */
    public function getWrappedStatement()
    {
        throw new \RuntimeException("Wrapped statement is unavailable on emulated statements.");
    }

    /**
     * @inheritDoc
     */
    public function bind(): void
    {
        $this->runnableQueryString = $this->preview();
    }

    /**
     * @return string
     */
    public function getRunnableQuery()
    {
        if (null === $this->runnableQueryString) {
            throw new \RuntimeException("Unbound query, run bind() before.");
        }
        return $this->runnableQueryString;
    }

    /**
     * @inheritDoc
     */
    public function createResult(): ResultInterface
    {
        $this->bind();
        $mysqliResult = $this->getConnection()->getWrappedConnection()->query($this->getRunnableQuery());
        return new Result($this->getConnection()->getWrappedConnection(), $mysqliResult instanceof \mysqli_result ? $mysqliResult : null);

It seems like $this->getConnection()->getWrappedConnection() targeting BenTools\SimpleDBAL\Model\Adapter\Mysqli\MysqliAdapter::getWrappedConnection() can also be of type object<PDO>; however, BenTools\SimpleDBAL\Model\Adapter\Mysqli\Result::__construct() does only seem to accept object<mysqli>, maybe add an additional type check?

    }
}