13766674441

Committed 10 Mar 2025 02:07PM UTC coverage: 87.322% (-0.09%) from 87.414%

Build # 13766674441

Build Type

push

github

Committed by

BenMorel

Commit Message

Support for boolean parameters in DatabaseEngine

Run Details

1 of 2 new or added lines in 1 file covered. (50.0%)

1 existing line in 1 file now uncovered.

1653 of 1893 relevant lines covered (87.32%)

1944.87 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

90.63

/src/Engine/PDOEngine.php

<?php

declare(strict_types=1);

namespace Brick\Geo\Engine;

use Brick\Geo\Exception\GeometryEngineException;
use Brick\Geo\Geometry;
use Override;
use PDO;
use PDOException;
use PDOStatement;

/**
 * Database engine based on a PDO driver.
 */
final class PDOEngine extends DatabaseEngine
{
    /**
     * The database connection.
     */
    private readonly PDO $pdo;

    /**
     * A cache of the prepared statements, indexed by query.
     *
     * @var array<string, PDOStatement>
     */
    private array $statements = [];

    public function __construct(PDO $pdo, bool $useProxy = true)
    {
        parent::__construct($useProxy);

        $this->pdo = $pdo;
    }

    public function getPDO() : PDO
    {
        return $this->pdo;
    }

    #[Override]
    protected function executeQuery(string $query, array $parameters) : array
    {
        /** @var int $errMode */
        $errMode = $this->pdo->getAttribute(PDO::ATTR_ERRMODE);
        $this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

        try {
            if (! isset($this->statements[$query])) {
                $this->statements[$query] = $this->pdo->prepare($query);
            }

            $statement = $this->statements[$query];

            $index = 1;

            foreach ($parameters as $parameter) {
                if ($parameter instanceof GeometryParameter) {
                    $statement->bindValue($index++, $parameter->data, $parameter->isBinary ? PDO::PARAM_LOB : PDO::PARAM_STR);
                    $statement->bindValue($index++, $parameter->srid, PDO::PARAM_INT);
                } else {
                    if (is_int($parameter)) {
                        $type = PDO::PARAM_INT;
                    } elseif (is_bool($parameter)) {
                        $type = PDO::PARAM_BOOL;
                    } else {
                        $type = PDO::PARAM_STR;
                    }

                    $statement->bindValue($index++, $parameter, $type);
                }
            }

            $statement->execute();

            /** @var list<mixed>|false $result */
            $result = $statement->fetch(PDO::FETCH_NUM);
        } catch (PDOException $e) {
            throw GeometryEngineException::wrap($e);
        } finally {
            $this->pdo->setAttribute(PDO::ATTR_ERRMODE, $errMode);
        }

        assert($result !== false);

        return $result;
    }

    #[Override]
    protected function getGeomFromWKBSyntax(): string
    {
        if ($this->pdo->getAttribute(PDO::ATTR_DRIVER_NAME) === 'mysql') {
            return 'ST_GeomFromWKB(BINARY ?, ?)';
        }

        return parent::getGeomFromWKBSyntax();
    }

    #[Override]
    protected function getParameterPlaceholder(string|float|int|bool $parameter): string
    {
        if ($this->pdo->getAttribute(PDO::ATTR_DRIVER_NAME) === 'pgsql') {
            if (is_int($parameter)) {
                // https://stackoverflow.com/q/66625661/759866
                // https://externals.io/message/113521
                return 'CAST (? AS INTEGER)';
            }
        }

        return parent::getParameterPlaceholder($parameter);
    }
}

1	<?php
2
3	declare(strict_types=1);
4
5	namespace Brick\Geo\Engine;
6
7	use Brick\Geo\Exception\GeometryEngineException;
8	use Brick\Geo\Geometry;
9	use Override;
10	use PDO;
11	use PDOException;
12	use PDOStatement;
13
14	/**
15	* Database engine based on a PDO driver.
16	*/
17	final class PDOEngine extends DatabaseEngine
18	{
19	/**
20	* The database connection.
21	*/
22	private readonly PDO $pdo;
23
24	/**
25	* A cache of the prepared statements, indexed by query.
26	*
27	* @var array<string, PDOStatement>
28	*/
29	private array $statements = [];
30
31	public function __construct(PDO $pdo, bool $useProxy = true)
32	{
33	parent::__construct($useProxy);	×
34
35	$this->pdo = $pdo;	×
36	}
37
38	public function getPDO() : PDO
39	{
40	return $this->pdo;	1,032✔
41	}
42
43	#[Override]
44	protected function executeQuery(string $query, array $parameters) : array
45	{
46	/** @var int $errMode */
47	$errMode = $this->pdo->getAttribute(PDO::ATTR_ERRMODE);	1,412✔
48	$this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);	1,412✔
49
50	try {
51	if (! isset($this->statements[$query])) {	1,412✔
52	$this->statements[$query] = $this->pdo->prepare($query);	333✔
53	}
54
55	$statement = $this->statements[$query];	1,275✔
56
57	$index = 1;	1,275✔
58
59	foreach ($parameters as $parameter) {	1,275✔
60	if ($parameter instanceof GeometryParameter) {	1,275✔
61	$statement->bindValue($index++, $parameter->data, $parameter->isBinary ? PDO::PARAM_LOB : PDO::PARAM_STR);	1,275✔
62	$statement->bindValue($index++, $parameter->srid, PDO::PARAM_INT);	1,275✔
63	} else {
64	if (is_int($parameter)) {	295✔
65	$type = PDO::PARAM_INT;	28✔
66	} elseif (is_bool($parameter)) {	267✔
NEW 67	$type = PDO::PARAM_BOOL;	×
68	} else {
69	$type = PDO::PARAM_STR;	267✔
70	}
71
72	$statement->bindValue($index++, $parameter, $type);	295✔
73	}
74	}
75
76	$statement->execute();	1,275✔
77
78	/** @var list<mixed>\|false $result */
79	$result = $statement->fetch(PDO::FETCH_NUM);	1,018✔
80	} catch (PDOException $e) {	404✔
81	throw GeometryEngineException::wrap($e);	404✔
82	} finally {
83	$this->pdo->setAttribute(PDO::ATTR_ERRMODE, $errMode);	1,412✔
84	}
85
86	assert($result !== false);
87
88	return $result;	1,018✔
89	}
90
91	#[Override]
92	protected function getGeomFromWKBSyntax(): string
93	{
94	if ($this->pdo->getAttribute(PDO::ATTR_DRIVER_NAME) === 'mysql') {	1,297✔
95	return 'ST_GeomFromWKB(BINARY ?, ?)';	1,036✔
96	}
97
98	return parent::getGeomFromWKBSyntax();	261✔
99	}
100
101	#[Override]
102	protected function getParameterPlaceholder(string\|float\|int\|bool $parameter): string
103	{
104	if ($this->pdo->getAttribute(PDO::ATTR_DRIVER_NAME) === 'pgsql') {	375✔
105	if (is_int($parameter)) {	75✔
106	// https://stackoverflow.com/q/66625661/759866
107	// https://externals.io/message/113521
108	return 'CAST (? AS INTEGER)';	7✔
109	}
110	}
111
112	return parent::getParameterPlaceholder($parameter);	368✔
113	}
114	}

brick / geo / 13766674441

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous