List support only works with Connection::execute() (#21)

DamienHarper · Feb 6, 2021 · feae12d · feae12d
1 parent fe26ddc
commit feae12d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/Provider/Doctrine/Persistence/Reader/Query.php b/src/Provider/Doctrine/Persistence/Reader/Query.php
@@ -235,7 +235,7 @@ private function buildWhere(QueryBuilder $queryBuilder): QueryBuilder
             } else {
                 $queryBuilder
                     ->andWhere(sprintf('%s IN (:%s)', $filter, $filter))
-                    ->setParameter($filter, $values)
+                    ->setParameter($filter, $values, Connection::PARAM_STR_ARRAY)
                 ;
             }
         }

diff --git a/tests/Provider/Doctrine/Persistence/Reader/ReaderTest.php b/tests/Provider/Doctrine/Persistence/Reader/ReaderTest.php
@@ -181,6 +181,21 @@ public function testGetAuditsPager(): void
         self::assertSame(3, $pager['numPages'], 'Pager has 3 pages.');
     }
 
+    public function testMultipleFilters(): void
+    {
+        $reader = $this->createReader();
+
+        $query = $reader->createQuery(Author::class);
+        $audits = $query->execute();
+        self::assertCount(5, $audits);
+
+        $query = $reader->createQuery(Author::class);
+        $query->addFilter('object_id', 1);
+        $query->addFilter('object_id', 2);
+        $audits = $query->execute();
+        self::assertCount(3, $audits);
+    }
+
     /**
      * @depends testGetAudits
      */