fix(query-builder): use AND for multiple where statements

AlexandreGerault · Jul 19, 2021 · 68b5c6b · 68b5c6b
1 parent 7134ea1
commit 68b5c6b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/AGerault/Database/QueryBuilder.php b/src/AGerault/Database/QueryBuilder.php
@@ -263,7 +263,7 @@ private function appendConditions(string $query): string
     {
         if ($this->conditions) {
             $conditions = implode(
-                ', ',
+                ' AND ',
                 array_map(
                     function ($name, $payload) {
                         $value = $payload['value'] ?? ":{$name}";

diff --git a/tests/Unit/Database/QueryBuilderTest.php b/tests/Unit/Database/QueryBuilderTest.php
@@ -145,7 +145,7 @@ function () {
             ->delete();
 
         expect($query->from('posts')->toSQL())->toBeString()->toBe(
-            'DELETE FROM posts WHERE name = :name, slug = :slug'
+            'DELETE FROM posts WHERE name = :name AND slug = :slug'
         );
     }
 );
@@ -201,7 +201,7 @@ function () {
 
         expect($query->toSQL())
             ->toBeString()
-            ->toBe('UPDATE posts SET title = :title, slug = :slug WHERE title = :title, slug = :slug');
+            ->toBe('UPDATE posts SET title = :title, slug = :slug WHERE title = :title AND slug = :slug');
     }
 );