Fix bugs where Query parser ignored 0 strings

utopia-php · Sep 23, 2022 · 7b8d5ff · 7b8d5ff
1 parent 2883de8
commit 7b8d5ff
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 5 deletions.
diff --git a/src/Database/Query.php b/src/Database/Query.php
@@ -234,7 +234,7 @@ public static function parse(string $filter): self
                     \array_pop($stack);
                     $stackCount--;
 
-                    if (!empty($currentParam)) {
+                    if (strlen($currentParam)) {
                         $currentArrayParam[] = $currentParam;
                     }
 
@@ -251,7 +251,9 @@ public static function parse(string $filter): self
                     } else {
                         // Append from parap builder. Either value, or array
                         if (empty($currentArrayParam)) {
-                            if (!empty($currentParam)) {
+                            // if (!empty($currentParam)) {
+                            if (strlen($currentParam)) {
+                                var_dump($currentParam);
                                 $params[] = $currentParam;
                             }
 
@@ -650,7 +652,7 @@ public static function groupByType(array $queries): array
      * 
      * @return Query[]
      */
-    public static function parseQueries(array $queries) : array
+    public static function parseQueries(array $queries): array
     {
         $parsed = [];
         foreach ($queries as $query) {

diff --git a/tests/Database/QueryTest.php b/tests/Database/QueryTest.php
@@ -148,6 +148,21 @@ public function testParseV2()
         $this->assertEquals("attr", $query->getAttribute());
         $this->assertEquals([1], $query->getValues());
 
+        $query = Query::parse('equal("attr", [0])');
+        $this->assertCount(1, $query->getValues());
+        $this->assertEquals("attr", $query->getAttribute());
+        $this->assertEquals([0], $query->getValues());
+
+        $query = Query::parse('equal("attr", 0,)');
+        $this->assertCount(1, $query->getValues());
+        $this->assertEquals("attr", $query->getAttribute());
+        $this->assertEquals([0], $query->getValues());
+
+        $query = Query::parse('equal("attr", ["0"])');
+        $this->assertCount(1, $query->getValues());
+        $this->assertEquals("attr", $query->getAttribute());
+        $this->assertEquals(["0"], $query->getValues());
+
         $query = Query::parse('equal(1, ["[Hello] World"])');
         $this->assertCount(1, $query->getValues());
         $this->assertEquals(1, $query->getAttribute());
@@ -247,7 +262,7 @@ public function testParseV2()
         $this->assertCount(1, $query->getValues());
         $this->assertEquals("value", $query->getAttribute());
         $this->assertEquals('{"type":"json","somekey":"someval"}', $query->getValue());
-        
+
         $query = Query::parse('equal("value", "{ NormalStringInBraces }")');
         $this->assertCount(1, $query->getValues());
         $this->assertEquals("value", $query->getAttribute());
@@ -282,7 +297,7 @@ public function testParseV2()
         $this->assertCount(1, $query->getValues());
         $this->assertEquals("value", $query->getAttribute());
         $this->assertEquals('DoubleQuote"InMiddle', $query->getValue());
-        
+
         $query = Query::parse('equal("value", "Slash/InMiddle")');
         $this->assertCount(1, $query->getValues());
         $this->assertEquals("value", $query->getAttribute());