for SQL too

atk4 · Nov 23, 2024 · f1ca317 · f1ca317
1 parent 776205f
commit f1ca317
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 7 deletions.
diff --git a/src/Persistence/Sql/Query.php b/src/Persistence/Sql/Query.php
@@ -600,11 +600,7 @@ protected function _subrenderCondition(array $row): string
                 $value = $value->getDsqlExpression($this);
             }
 
-            if ($value instanceof self && $value->mode === 'select') {
-                $operator = 'in';
-            } else {
-                $operator = '=';
-            }
+            $operator = '=';
         } else {
             $operator = strtolower($operator);
         }

diff --git a/src/Reference/HasMany.php b/src/Reference/HasMany.php
@@ -87,6 +87,7 @@ public function ref(Model $ourModelOrEntity, array $defaults = []): Model
 
         return $this->createTheirModel($defaults)->addCondition(
             $this->getTheirFieldName(),
+            $ourModelOrEntity->isEntity() ? '=' : 'in',
             $this->getOurFieldValueForRefCondition($ourModelOrEntity)
         );
     }

diff --git a/src/Reference/HasOneSql.php b/src/Reference/HasOneSql.php
@@ -135,7 +135,7 @@ public function ref(Model $ourModelOrEntity, array $defaults = []): Model
             $ourFieldExpression = $ourModelOrEntity->action('field', [$this->getOurField()]);
 
             $theirModel->getModel(true)
-                ->addCondition($this->getTheirFieldName($theirModel), $ourFieldExpression);
+                ->addCondition($this->getTheirFieldName($theirModel), 'in', $ourFieldExpression);
         }
 
         return $theirModel;

diff --git a/tests/Persistence/Sql/QueryTest.php b/tests/Persistence/Sql/QueryTest.php
@@ -650,7 +650,7 @@ public function testWhereBasic(): void
             $this->q('[where]')->where('id', 'in', [1, 2])->render()[0]
         );
         self::assertSame(
-            'where "id" in (select * from "user")',
+            'where "id" = (select * from "user")',
             $this->q('[where]')->where('id', $this->q()->table('user'))->render()[0]
         );