PostgresHandler::assertValid() - Code Metrics - Inspection of "Merge pull request #134: Change JSON mapping from..." - cycle/database - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 2.x ( 67d9b2...73d5f4 )

by Aleksei

created 2023-10-10 16:05 UTC

PostgresHandler::assertValid() A

↳ Parent: PostgresHandler

Complexity

Conditions	4
Paths	3

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

Metric	Value
cc	4
eloc	6
nc	3
nop	1
dl	0
loc	8
ccs	0
cts	0
cp	0
crap	20
rs	10
c	0
b	0
f	0

<?php

/**
 * This file is part of Cycle ORM package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Cycle\Database\Driver\Postgres;

use Cycle\Database\Driver\Handler;
use Cycle\Database\Driver\Postgres\Exception\PostgresException;
use Cycle\Database\Driver\Postgres\Schema\PostgresColumn;
use Cycle\Database\Driver\Postgres\Schema\PostgresTable;
use Cycle\Database\Exception\SchemaException;
use Cycle\Database\Schema\AbstractColumn;
filter:
    dependency_paths: ["lib/*"]
use Cycle\Database\Schema\AbstractTable;

/**
 * @property PostgresDriver $driver
 */
class PostgresHandler extends Handler
{
    /**
     * @psalm-param non-empty-string $table
     */
    public function getSchema(string $table, string $prefix = null): AbstractTable
    {
        return new PostgresTable($this->driver, $table, $prefix ?? '');
    }

    public function getTableNames(string $prefix = ''): array
    {
        $query = "SELECT table_schema, table_name
            FROM information_schema.tables
            WHERE table_type = 'BASE TABLE'";

        if ($this->driver->shouldUseDefinedSchemas()) {
            $query .= " AND table_schema in ('" . implode("','", $this->driver->getSearchSchemas()) . "')";
        } else {
            $query .= " AND table_schema !~ '^pg_.*' AND table_schema != 'information_schema'";
        }

        $tables = [];
        foreach ($this->driver->query($query) as $row) {
            if ($prefix !== '' && !str_starts_with($row['table_name'], $prefix)) {
                continue;
            }

            $tables[] = $row['table_schema'] . '.' . $row['table_name'];
        }

        return $tables;
    }

    /**
     * @psalm-param non-empty-string $table
     */
    public function hasTable(string $table): bool
    {
        [$schema, $name] = $this->driver->parseSchemaAndTable($table);

        $query = "SELECT COUNT(table_name)
            FROM information_schema.tables
            WHERE table_schema = ?
            AND table_type = 'BASE TABLE'
            AND table_name = ?";

        return (bool)$this->driver->query($query, [$schema, $name])->fetchColumn();
    }

    public function eraseTable(AbstractTable $table, bool $restartIdentity = false): void
    {
        $query = "TRUNCATE TABLE {$this->driver->identifier($table->getFullName())}";

        if ($restartIdentity) {
            $query .= ' RESTART IDENTITY CASCADE';
        }

        $this->driver->execute($query);
    }

    /**
     * @psalm-param non-empty-string $table
     * @psalm-param non-empty-string $name
     */
    public function renameTable(string $table, string $name): void
    {
        // New table name should not contain a schema
        [, $name] = $this->driver->parseSchemaAndTable($name);

        parent::renameTable($table, $name);
    }

    /**
     * @throws SchemaException
     */
    public function alterColumn(
        AbstractTable $table,
        AbstractColumn $initial,
        AbstractColumn $column
    ): void {
        if (!$initial instanceof PostgresColumn || !$column instanceof PostgresColumn) {
            throw new SchemaException('Postgres handler can work only with Postgres columns');
        }

        //Rename is separate operation
        if ($column->getName() !== $initial->getName()) {
            $this->renameColumn($table, $initial, $column);

            //This call is required to correctly built set of alter operations
            $initial->setName($column->getName());
        }

        //Postgres columns should be altered using set of operations
        $operations = $column->alterOperations($this->driver, $initial);
        if (empty($operations)) {
            return;
        }

        //Postgres columns should be altered using set of operations
        $query = sprintf(
            'ALTER TABLE %s %s',
            $this->identify($table),
            trim(implode(', ', $operations), ', ')
        );

        $this->run($query);
    }

    public function enableForeignKeyConstraints(): void
    {
        $this->run('SET CONSTRAINTS ALL IMMEDIATE;');
    }

    public function disableForeignKeyConstraints(): void
    {
        $this->run('SET CONSTRAINTS ALL DEFERRED;');
    }

    /**
     * @psalm-param non-empty-string $statement
     */
    protected function run(string $statement, array $parameters = []): int
    {
        if ($this->driver instanceof PostgresDriver) {

            // invaliding primary key cache
            $this->driver->resetPrimaryKeys();
        }

        return parent::run($statement, $parameters);
    }

    /**
     * @throws PostgresException
     */
    protected function assertValid(AbstractColumn $column): void
    {
        if ($column->getDefaultValue() !== null && \in_array($column->getAbstractType(), ['json', 'jsonb'])) {
            try {
                \json_decode($column->getDefaultValue(), true, 512, JSON_THROW_ON_ERROR);
            } catch (\Throwable) {
                throw new PostgresException(
                    \sprintf('Column `%s` of type json/jsonb has an invalid default json value.', $column),
                );
            }
        }
    }

    private function renameColumn(
        AbstractTable $table,
        AbstractColumn $initial,
        AbstractColumn $column
    ): void {
        $statement = sprintf(
            'ALTER TABLE %s RENAME COLUMN %s TO %s',
            $this->identify($table),
            $this->identify($initial),
            $this->identify($column)
        );

        $this->run($statement);
    }
}


1		<?php
2
3		/**
4		* This file is part of Cycle ORM package.
5		*
6		* For the full copyright and license information, please view the LICENSE
7		* file that was distributed with this source code.
8		*/
9
10		declare(strict_types=1);
11
12		namespace Cycle\Database\Driver\Postgres;
13
14		use Cycle\Database\Driver\Handler;
15		use Cycle\Database\Driver\Postgres\Exception\PostgresException;
16		use Cycle\Database\Driver\Postgres\Schema\PostgresColumn;
17		use Cycle\Database\Driver\Postgres\Schema\PostgresTable;
18		use Cycle\Database\Exception\SchemaException;
19		use Cycle\Database\Schema\AbstractColumn;
		0 ignored issues – show Bug introduced 2021-12-22 14:01 UTC by Report Bug Copy Issue Report The type `Cycle\Database\Schema\AbstractColumn` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
20		use Cycle\Database\Schema\AbstractTable;
21
22		/**
23		* @property PostgresDriver $driver
24		*/
25		class PostgresHandler extends Handler
26		{
27		/**
28		* @psalm-param non-empty-string $table
29	522	*/
30		public function getSchema(string $table, string $prefix = null): AbstractTable
31	522	{
32		return new PostgresTable($this->driver, $table, $prefix ?? '');
33		}
34	920
35		public function getTableNames(string $prefix = ''): array
36	920	{
37		$query = "SELECT table_schema, table_name
38		FROM information_schema.tables
39		WHERE table_type = 'BASE TABLE'";
40	920
41	920	if ($this->driver->shouldUseDefinedSchemas()) {
42		$query .= " AND table_schema in ('" . implode("','", $this->driver->getSearchSchemas()) . "')";
43		} else {
44		$query .= " AND table_schema !~ '^pg_.*' AND table_schema != 'information_schema'";
45		}
46	920
47	920	$tables = [];
48	478	foreach ($this->driver->query($query) as $row) {
49	2	if ($prefix !== '' && !str_starts_with($row['table_name'], $prefix)) {
50		continue;
51		}
52	478
53		$tables[] = $row['table_schema'] . '.' . $row['table_name'];
54		}
55	920
56		return $tables;
57		}
58
59		/**
60		* @psalm-param non-empty-string $table
61	534	*/
62		public function hasTable(string $table): bool
63	534	{
64		[$schema, $name] = $this->driver->parseSchemaAndTable($table);
65	534
66		$query = "SELECT COUNT(table_name)
67		FROM information_schema.tables
68		WHERE table_schema = ?
69		AND table_type = 'BASE TABLE'
70		AND table_name = ?";
71	534
72		return (bool)$this->driver->query($query, [$schema, $name])->fetchColumn();
73		}
74	12
75		public function eraseTable(AbstractTable $table, bool $restartIdentity = false): void
76	12	{
77	12	$query = "TRUNCATE TABLE {$this->driver->identifier($table->getFullName())}";
78
79	10	if ($restartIdentity) {
80		$query .= ' RESTART IDENTITY CASCADE';
81		}
82
83		$this->driver->execute($query);
84		}
85	2
86		/**
87		* @psalm-param non-empty-string $table
88	2	* @psalm-param non-empty-string $name
89		*/
90	2	public function renameTable(string $table, string $name): void
91	2	{
92		// New table name should not contain a schema
93		[, $name] = $this->driver->parseSchemaAndTable($name);
94
95		parent::renameTable($table, $name);
96	40	}
97
98		/**
99		* @throws SchemaException
100		*/
101	40	public function alterColumn(
102		AbstractTable $table,
103		AbstractColumn $initial,
104		AbstractColumn $column
105		): void {
106	40	if (!$initial instanceof PostgresColumn \|\| !$column instanceof PostgresColumn) {
107	18	throw new SchemaException('Postgres handler can work only with Postgres columns');
108		}
109
110	18	//Rename is separate operation
111		if ($column->getName() !== $initial->getName()) {
112		$this->renameColumn($table, $initial, $column);
113
114	40	//This call is required to correctly built set of alter operations
115	40	$initial->setName($column->getName());
116	12	}
117
118		//Postgres columns should be altered using set of operations
119		$operations = $column->alterOperations($this->driver, $initial);
120	28	if (empty($operations)) {
121	28	return;
122	28	}
123	28
124		//Postgres columns should be altered using set of operations
125		$query = sprintf(
126	28	'ALTER TABLE %s %s',
127	28	$this->identify($table),
128		trim(implode(', ', $operations), ', ')
129		);
130
131		$this->run($query);
132	516	}
133
134	516	public function enableForeignKeyConstraints(): void
135		{
136	516	$this->run('SET CONSTRAINTS ALL IMMEDIATE;');
137		}
138
139	516	public function disableForeignKeyConstraints(): void
140		{
141		$this->run('SET CONSTRAINTS ALL DEFERRED;');
142	18	}
143
144		/**
145		* @psalm-param non-empty-string $statement
146		*/
147	18	protected function run(string $statement, array $parameters = []): int
148	18	{
149	18	if ($this->driver instanceof PostgresDriver) {
		0 ignored issues – show introduced 2021-12-21 11:12 UTC by Report Bug Copy Issue Report `$this->driver` is always a sub-type of `Cycle\Database\Driver\Postgres\PostgresDriver`. Loading history...
150	18	// invaliding primary key cache
151	18	$this->driver->resetPrimaryKeys();
152		}
153
154	18	return parent::run($statement, $parameters);
155	18	}
156
157		/**
158		* @throws PostgresException
159		*/
160		protected function assertValid(AbstractColumn $column): void
161		{
162		if ($column->getDefaultValue() !== null && \in_array($column->getAbstractType(), ['json', 'jsonb'])) {
163		try {
164		\json_decode($column->getDefaultValue(), true, 512, JSON_THROW_ON_ERROR);
165		} catch (\Throwable) {
166		throw new PostgresException(
167		\sprintf('Column `%s` of type json/jsonb has an invalid default json value.', $column),
168		);
169		}
170		}
171		}
172
173		private function renameColumn(
174		AbstractTable $table,
175		AbstractColumn $initial,
176		AbstractColumn $column
177		): void {
178		$statement = sprintf(
179		'ALTER TABLE %s RENAME COLUMN %s TO %s',
180		$this->identify($table),
181		$this->identify($initial),
182		$this->identify($column)
183		);
184
185		$this->run($statement);
186		}
187		}
188

cycle / database

Push — 2.x ( 67d9b2...73d5f4 )

PostgresHandler::assertValid() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like