Merge branch 'master' into fix-94

Author: rustamwin

CHANGELOG.md

@@ -8,7 +8,7 @@
   for type casting performance. Related with yiisoft/db#752 (@Tigrov)
 - Chg #348: Replace call of `SchemaInterface::getRawTableName()` to `QuoterInterface::getRawTableName()` (@Tigrov)
 - Enh #349: Add method chaining for column classes (@Tigrov)
-- New #350: Add array overlaps and JSON overlaps condition builders (@Tigrov)
+- New #350, #449: Add array overlaps and JSON overlaps condition builders (@Tigrov)
 - Enh #353: Update `bit` type according to main PR yiisoft/db#860 (@Tigrov) 
 - Enh #354: Refactor PHP type of `ColumnSchemaInterface` instances (@Tigrov)
 - Enh #356: Raise minimum PHP version to `^8.1` with minor refactoring (@Tigrov)
@@ -31,7 +31,7 @@
 - Enh #382: Replace `DbArrayHelper::getColumn()` with `array_column()` (@Tigrov)
 - New #384: Add `IndexMethod` class (@Tigrov)
 - Bug #387: Explicitly mark nullable parameters (@vjik)
-- Enh #386: Refactor array, structured and JSON expression builders (@Tigrov)
+- Enh #386, #449: Refactor array, structured and JSON expression builders (@Tigrov)
 - Chg #388: Change supported PHP versions to `8.1 - 8.4` (@Tigrov)
 - Enh #388: Minor refactoring (@Tigrov)
 - Chg #390: Remove `yiisoft/json` dependency (@Tigrov)
@@ -45,7 +45,7 @@
 - New #408, #410: Implement `DMLQueryBuilder::upsertReturning()` method (@Tigrov)
 - Enh #412: Reduce binding parameters (@Tigrov)
 - Chg #414: Rename `DMLQueryBuilder::insertWithReturningPks()` to `DMLQueryBuilder::insertReturningPks()` (@Tigrov)
-- Enh #415: Implement `CaseExpressionBuilder` class (@Tigrov)
+- Enh #415, #448: Implement `CaseExpressionBuilder` class (@Tigrov)
 - Enh #420: Provide `yiisoft/db-implementation` virtual package (@vjik)
 - Enh #424, #425, #428: Adapt to conditions refactoring in `yiisoft/db` package (@vjik)
 - Enh #431: Remove `TableSchema` class and refactor `Schema` class (@Tigrov)
@@ -54,6 +54,7 @@
 - New #439, #446: Implement `ArrayMergeBuilder` class (@Tigrov)
 - Enh #442: Refactor `DMLQueryBuilder::upsert()` method (@Tigrov)
 - Enh #444: Improve `ArrayExpressionBuilder` and `JsonExpressionBuilder` classes (@Tigrov)
+- Chg #447: Update expression namespaces according to changes in `yiisoft/db` package (@Tigrov)
 
 ## 1.3.0 March 21, 2024
 

src/Builder/ArrayOverlapsBuilder.php

@@ -4,10 +4,10 @@
 
 namespace Yiisoft\Db\Pgsql\Builder;
 
-use Yiisoft\Db\Expression\ArrayExpression;
-use Yiisoft\Db\Expression\Builder\ExpressionBuilderInterface;
+use Yiisoft\Db\Expression\Value\ArrayValue;
+use Yiisoft\Db\Expression\ExpressionBuilderInterface;
 use Yiisoft\Db\Expression\ExpressionInterface;
-use Yiisoft\Db\Expression\JsonExpression;
+use Yiisoft\Db\Expression\Value\JsonValue;
 use Yiisoft\Db\QueryBuilder\Condition\ArrayOverlaps;
 use Yiisoft\Db\QueryBuilder\QueryBuilderInterface;
 
@@ -38,10 +38,10 @@ public function build(ExpressionInterface $expression, array &$params = []): str
         $values = $expression->values;
 
         if (!$values instanceof ExpressionInterface) {
-            $values = new ArrayExpression($values);
-        } elseif ($values instanceof JsonExpression) {
+            $values = new ArrayValue($values);
+        } elseif ($values instanceof JsonValue) {
             /** @psalm-suppress MixedArgument */
-            $values = new ArrayExpression($values->getValue());
+            $values = new ArrayValue($values->value);
         }
 
         $values = $this->queryBuilder->buildExpression($values, $params);

src/Builder/ArrayExpressionBuilder.php renamed to src/Builder/ArrayValueBuilder.php

@@ -7,10 +7,10 @@
 use Yiisoft\Db\Constant\ColumnType;
 use Yiisoft\Db\Constant\DataType;
 use Yiisoft\Db\Constant\GettypeResult;
-use Yiisoft\Db\Expression\ArrayExpression;
-use Yiisoft\Db\Expression\Builder\AbstractArrayExpressionBuilder;
+use Yiisoft\Db\Expression\Value\ArrayValue;
+use Yiisoft\Db\Expression\Value\Builder\AbstractArrayValueBuilder;
 use Yiisoft\Db\Expression\ExpressionInterface;
-use Yiisoft\Db\Expression\Param;
+use Yiisoft\Db\Expression\Value\Param;
 use Yiisoft\Db\Pgsql\Data\LazyArray;
 use Yiisoft\Db\Query\QueryInterface;
 use Yiisoft\Db\Schema\Column\AbstractArrayColumn;
@@ -25,11 +25,11 @@
 use function str_repeat;
 
 /**
- * Builds expressions for {@see ArrayExpression} for PostgreSQL Server.
+ * Builds expressions for {@see ArrayValue} for PostgreSQL Server.
  */
-final class ArrayExpressionBuilder extends AbstractArrayExpressionBuilder
+final class ArrayValueBuilder extends AbstractArrayValueBuilder
 {
-    protected function buildStringValue(string $value, ArrayExpression $expression, array &$params): string
+    protected function buildStringValue(string $value, ArrayValue $expression, array &$params): string
     {
         $param = new Param($value, DataType::STRING);
 
@@ -41,15 +41,15 @@ protected function buildStringValue(string $value, ArrayExpression $expression,
         return $this->queryBuilder->bindParam($param, $params) . $typeHint;
     }
 
-    protected function buildSubquery(QueryInterface $query, ArrayExpression $expression, array &$params): string
+    protected function buildSubquery(QueryInterface $query, ArrayValue $expression, array &$params): string
     {
         $column = $this->getColumn($expression);
         $dbType = $this->getColumnDbType($column);
 
         return $this->buildNestedSubquery($query, $dbType, $column?->getDimension() ?? 1, $params);
     }
 
-    protected function buildValue(iterable $value, ArrayExpression $expression, array &$params): string
+    protected function buildValue(iterable $value, ArrayValue $expression, array &$params): string
     {
         $column = $this->getColumn($expression);
         $dbType = $this->getColumnDbType($column);
@@ -125,9 +125,9 @@ private function buildNestedValue(iterable $value, string|null &$dbType, ColumnI
         return $this->buildNestedArray($placeholders, $dbType, $dimension);
     }
 
-    private function getColumn(ArrayExpression $expression): AbstractArrayColumn|null
+    private function getColumn(ArrayValue $expression): AbstractArrayColumn|null
     {
-        $type = $expression->getType();
+        $type = $expression->type;
 
         if ($type === null || $type instanceof AbstractArrayColumn) {
             return $type;

src/Builder/CaseExpressionBuilder.php renamed to src/Builder/CaseXBuilder.php

@@ -4,44 +4,35 @@
 
 namespace Yiisoft\Db\Pgsql\Builder;
 
-use InvalidArgumentException;
-use Yiisoft\Db\Expression\CaseExpression;
+use Yiisoft\Db\Expression\Statement\CaseX;
 use Yiisoft\Db\Expression\ExpressionInterface;
 use Yiisoft\Db\Schema\Column\ColumnInterface;
 
 use function is_string;
 
 /**
- * Builds expressions for {@see CaseExpression}.
+ * Builds expressions for {@see CaseX}.
  */
-final class CaseExpressionBuilder extends \Yiisoft\Db\Expression\Builder\CaseExpressionBuilder
+final class CaseXBuilder extends \Yiisoft\Db\Expression\Statement\Builder\CaseXBuilder
 {
     public function build(ExpressionInterface $expression, array &$params = []): string
     {
-        $whenClauses = $expression->getWhen();
-
-        if (empty($whenClauses)) {
-            throw new InvalidArgumentException('The CASE expression must have at least one WHEN clause.');
-        }
-
         $sql = 'CASE';
 
-        $case = $expression->getCase();
-
-        if ($case !== null) {
-            $caseTypeHint = $this->buildTypeHint($expression->getCaseType());
-            $sql .= ' ' . $this->buildConditionWithTypeHint($case, $caseTypeHint, $params);
+        if ($expression->value !== null) {
+            $caseTypeHint = $this->buildTypeHint($expression->valueType);
+            $sql .= ' ' . $this->buildConditionWithTypeHint($expression->value, $caseTypeHint, $params);
         } else {
             $caseTypeHint = '';
         }
 
-        foreach ($whenClauses as $when) {
+        foreach ($expression->when as $when) {
             $sql .= ' WHEN ' . $this->buildConditionWithTypeHint($when->condition, $caseTypeHint, $params);
             $sql .= ' THEN ' . $this->buildResult($when->result, $params);
         }
 
         if ($expression->hasElse()) {
-            $sql .= ' ELSE ' . $this->buildResult($expression->getElse(), $params);
+            $sql .= ' ELSE ' . $this->buildResult($expression->else, $params);
         }
 
         return $sql . ' END';

src/Builder/JsonExpressionBuilder.php

@@ -4,10 +4,10 @@
 
 namespace Yiisoft\Db\Pgsql\Builder;
 
-use Yiisoft\Db\Expression\ArrayExpression;
-use Yiisoft\Db\Expression\Builder\ExpressionBuilderInterface;
+use Yiisoft\Db\Expression\Value\ArrayValue;
+use Yiisoft\Db\Expression\ExpressionBuilderInterface;
 use Yiisoft\Db\Expression\ExpressionInterface;
-use Yiisoft\Db\Expression\JsonExpression;
+use Yiisoft\Db\Expression\Value\JsonValue;
 use Yiisoft\Db\QueryBuilder\Condition\JsonOverlaps;
 use Yiisoft\Db\QueryBuilder\QueryBuilderInterface;
 
@@ -37,11 +37,11 @@ public function build(ExpressionInterface $expression, array &$params = []): str
             : $this->queryBuilder->getQuoter()->quoteColumnName($expression->column);
         $values = $expression->values;
 
-        if ($values instanceof JsonExpression) {
+        if ($values instanceof JsonValue) {
             /** @psalm-suppress MixedArgument */
-            $values = new ArrayExpression($values->getValue());
+            $values = new ArrayValue($values->value);
         } elseif (!$values instanceof ExpressionInterface) {
-            $values = new ArrayExpression($values);
+            $values = new ArrayValue($values);
         }
 
         $values = $this->queryBuilder->buildExpression($values, $params);

src/Builder/JsonOverlapsBuilder.php

@@ -0,0 +1,60 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Db\Pgsql\Builder;
+
+use Yiisoft\Db\Expression\Value\ArrayValue;
+use Yiisoft\Db\Expression\ExpressionBuilderInterface;
+use Yiisoft\Db\Expression\ExpressionInterface;
+use Yiisoft\Db\Expression\Value\JsonValue;
+use Yiisoft\Db\Expression\Value\Builder\JsonValueBuilder as BaseJsonValueBuilder;
+use Yiisoft\Db\QueryBuilder\QueryBuilderInterface;
+
+/**
+ * Builds expressions for {@see `Yiisoft\Db\Expression\Value\JsonValue`} for PostgreSQL Server.
+ *
+ * @implements ExpressionBuilderInterface<JsonValue>
+ */
+final class JsonValueBuilder implements ExpressionBuilderInterface
+{
+    private BaseJsonValueBuilder $baseValueBuilder;
+
+    public function __construct(QueryBuilderInterface $queryBuilder)
+    {
+        $this->baseValueBuilder = new BaseJsonValueBuilder($queryBuilder);
+    }
+
+    /**
+     * The Method builds the raw SQL from the $expression that won't be additionally escaped or quoted.
+     *
+     * @param JsonValue $expression The expression to build.
+     * @param array $params The binding parameters.
+     *
+     * @return string The raw SQL that won't be additionally escaped or quoted.
+     */
+    public function build(ExpressionInterface $expression, array &$params = []): string
+    {
+        $statement = $this->baseValueBuilder->build($expression, $params);
+
+        if ($expression->value instanceof ArrayValue) {
+            $statement = 'array_to_json(' . $statement . ')';
+        }
+
+        return $statement . $this->getTypeHint($expression);
+    }
+
+    /**
+     * @return string The typecast expression based on {@see JsonValue::type}.
+     */
+    private function getTypeHint(JsonValue $expression): string
+    {
+        $type = $expression->type;
+
+        if ($type === null) {
+            return '';
+        }
+
+        return '::' . $type;
+    }
+}