Phauthentic\Authentication\Identifier\Resolver\PdoResolver

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

79.17%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	buildStatement()	0	17	3
A	__construct()	0	4	1
A	find()	0	16	3

<?php
declare(strict_types=1);
/**
 * Copyright (c) Phauthentic (https://github.com/Phauthentic)
 *
 * Licensed under The MIT License
 * For full copyright and license information, please see the LICENSE.txt
 * Redistributions of files must retain the above copyright notice.
 *
 * @copyright     Copyright (c) Phauthentic (https://github.com/Phauthentic)
 * @link          https://github.com/Phauthentic
 * @license       https://opensource.org/licenses/mit-license.php MIT License
 */
namespace Phauthentic\Authentication\Identifier\Resolver;

use ArrayAccess;
use ArrayObject;
use PDO;
use PDOException;
use PDOStatement;
use Phauthentic\Authentication\Identifier\Resolver\ResolverInterface;
use RuntimeException;

/**
 * PDO Resolver
 */
class PdoResolver implements ResolverInterface
{
    /**
     * @var \PDO
     */
    protected $pdo;

    /**
     * @var string
     */
    protected $sql;

    /**
     * Constructor.
     *
     * @param \PDO PDO Instance
     * @param string $sql SQL String
     */
    public function __construct(PDO $pdo, string $sql)
    {
        $this->pdo = $pdo;
        $this->sql = $sql;
    }

    /**
     * Builds the statement
     *
     * @return \PDOStatement
     */
    protected function buildStatement(): PDOStatement
    {
        $statement = $this->pdo->prepare($this->sql);

        $error = $this->pdo->errorInfo();
        if ($error[0] !== '00000') {
            throw new PDOException($error[2], (int)$error[0]);
        }

        if ($statement === false) {
            throw new RuntimeException(sprintf(
                'There was an error running your PDO resolver using this query: %s',
                $this->sql
            ));
        }

        return $statement;

The expression return $statement could return the type true which is incompatible with the type-hinted return PDOStatement. Consider adding an additional type-check to rule them out.

    }

    /**
     * {@inheritDoc}
     */
    public function find(array $conditions): ?ArrayAccess
    {
        foreach ($conditions as $key => $value) {
            unset($conditions[$key]);
            $conditions[':' . $key] = $value;
        }

        $statement = $this->buildStatement();
        $statement->execute($conditions);
        $result = $statement->fetchAll();

        if (empty($result)) {
            return null;
        }

        return new ArrayObject($result[0]);
    }
}