PHPORM-64 Remove Query\Builder::whereAll

Author: GromNaN

CHANGELOG.md

@@ -11,6 +11,7 @@ All notable changes to this project will be documented in this file.
 - Throw an exception when `Query\Builder::orderBy()` is used with invalid direction [#7](https://github.com/GromNaN/laravel-mongodb-private/pull/7) by [@GromNaN](https://github.com/GromNaN).
 - Throw an exception when `Query\Builder::push()` is used incorrectly [#5](https://github.com/GromNaN/laravel-mongodb-private/pull/5) by [@GromNaN](https://github.com/GromNaN).
 - Remove public property `Query\Builder::$paginating` [#15](https://github.com/GromNaN/laravel-mongodb-private/pull/15) by [@GromNaN](https://github.com/GromNaN).
+- Remove `Query\Builder::whereAll($column, $values)`. Use `Query\Builder::where($column, 'all', $values)` instead. [#16](https://github.com/GromNaN/laravel-mongodb-private/pull/16) by [@GromNaN](https://github.com/GromNaN).
 
 ## [3.9.2] - 2022-09-01
 

src/Query/Builder.php

@@ -528,24 +528,6 @@ public function orderBy($column, $direction = 'asc')
         return $this;
     }
 
-    /**
-     * Add a "where all" clause to the query.
-     *
-     * @param  string  $column
-     * @param  array  $values
-     * @param  string  $boolean
-     * @param  bool  $not
-     * @return $this
-     */
-    public function whereAll($column, array $values, $boolean = 'and', $not = false)
-    {
-        $type = 'all';
-
-        $this->wheres[] = compact('column', 'type', 'boolean', 'values', 'not');
-
-        return $this;
-    }
-
     /**
      * @inheritdoc
      * @param list{mixed, mixed}|CarbonPeriod $values
@@ -1038,17 +1020,6 @@ protected function compileWheres(): array
         return $compiled;
     }
 
-    /**
-     * @param  array  $where
-     * @return array
-     */
-    protected function compileWhereAll(array $where): array
-    {
-        extract($where);
-
-        return [$column => ['$all' => array_values($values)]];
-    }
-
     /**
      * @param  array  $where
      * @return array

tests/Query/BuilderTest.php

@@ -247,6 +247,11 @@ public static function provideQueryBuilderToMql(): iterable
                 ]),
         ];
 
+        yield 'where all' => [
+            ['find' => [['tags' => ['$all' => ['ssl', 'security']]], []]],
+            fn (Builder $builder) => $builder->where('tags', 'all', ['ssl', 'security']),
+        ];
+
         /** @see DatabaseQueryBuilderTest::testForPage() */
         yield 'forPage' => [
             ['find' => [[], ['limit' => 20, 'skip' => 40]]],