add enum for conjunction of WhereGroup

Author: pavog

README.md

@@ -225,7 +225,7 @@ User::query(new \Aternos\Model\Query\SelectQuery(
         new \Aternos\Model\Query\WhereGroup([
             new \Aternos\Model\Query\WhereCondition("hello", "world"),
             new \Aternos\Model\Query\WhereCondition("foo", "bar")
-        ], \Aternos\Model\Query\WhereGroup:: OR)
+        ], \Aternos\Model\Query\Conjunction::OR)
     ]),
     [
         new \Aternos\Model\Query\OrderField("field", \Aternos\Model\Query\Direction::DESCENDING),

src/Driver/Test/TestTableEntry.php

@@ -3,7 +3,7 @@
 namespace Aternos\Model\Driver\Test;
 
 use Aternos\Model\ModelInterface;
-use Aternos\Model\Query\Field;
+use Aternos\Model\Query\Conjunction;
 use Aternos\Model\Query\SelectField;
 use Aternos\Model\Query\UpdateField;
 use Aternos\Model\Query\WhereCondition;
@@ -42,14 +42,14 @@ public function matchesWhereGroup(?WhereGroup $where): bool
             } else if ($condition instanceof WhereCondition) {
                 $matches = $this->matchesWhereCondition($condition);
             }
-            if ($where->conjunction === WhereGroup::AND && !$matches) {
+            if ($where->conjunction === Conjunction::AND && !$matches) {
                 return false;
             }
-            if ($where->conjunction === WhereGroup::OR && $matches) {
+            if ($where->conjunction === Conjunction::OR && $matches) {
                 return true;
             }
         }
-        return $where->conjunction === WhereGroup::AND;
+        return $where->conjunction === Conjunction::AND;
     }
 
     /**

src/GenericModel.php

@@ -18,7 +18,8 @@
 };
 use Aternos\Model\Driver\Mysqli\Mysqli;
 use Aternos\Model\Driver\Redis\Redis;
-use Aternos\Model\Query\{CountField,
+use Aternos\Model\Query\{Conjunction,
+    CountField,
     DeleteQuery,
     GroupField,
     Limit,
@@ -28,8 +29,7 @@
     SelectQuery,
     UpdateQuery,
     WhereCondition,
-    WhereGroup
-};
+    WhereGroup};
 use Aternos\Model\Search\Search;
 use Aternos\Model\Search\SearchResult;
 use BadMethodCallException;
@@ -441,7 +441,7 @@ public static function query(Query $query): QueryResult
         $query->modelClassName = static::class;
 
         if (static::$filters !== null && count(static::$filters) > 0) {
-            $wrappedWhereGroup = new WhereGroup(conjunction: WhereGroup::AND);
+            $wrappedWhereGroup = new WhereGroup(conjunction: Conjunction::AND);
             foreach (static::$filters as $key => $value) {
                 $wrappedWhereGroup->add(new WhereCondition($key, $value));
             }

src/Query/Conjunction.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Aternos\Model\Query;
+
+enum Conjunction
+{
+    case AND;
+    case OR;
+}

src/Query/Generator/SQL.php

@@ -2,7 +2,8 @@
 
 namespace Aternos\Model\Query\Generator;
 
-use Aternos\Model\Query\{DeleteQuery,
+use Aternos\Model\Query\{Conjunction,
+    DeleteQuery,
     Direction,
     OrderField,
     Query,
@@ -12,7 +13,6 @@
     UpdateQuery,
     WhereCondition,
     WhereGroup};
-use UnexpectedValueException;
 
 /**
  * Class SQL
@@ -140,9 +140,8 @@ private function generateWhere(Query $query, WhereCondition|WhereGroup|null $whe
             return $field . " " . $where->operator . " " . $value;
         } elseif ($where instanceof WhereGroup) {
             $conjunction = match ($where->conjunction) {
-                WhereGroup:: AND => " AND ",
-                WhereGroup:: OR => " OR ",
-                default => throw new UnexpectedValueException("Invalid conjunction: " . $where->conjunction),
+                Conjunction::AND => " AND ",
+                Conjunction::OR => " OR ",
             };
 
             $whereStrings = [];

src/Query/WhereGroup.php

@@ -12,11 +12,6 @@
  */
 class WhereGroup implements Iterator, Countable
 {
-    /**
-     * Conjunction values
-     */
-    const int AND = 0;
-    const int OR = 1;
 
     /**
      * Multiple WhereGroup or WhereCondition objects
@@ -26,9 +21,9 @@ class WhereGroup implements Iterator, Countable
     protected array $group = [];
 
     /**
-     * @var int
+     * @var Conjunction
      */
-    public int $conjunction = self:: AND;
+    public Conjunction $conjunction = Conjunction::AND;
 
     /**
      * Group iterator
@@ -41,9 +36,9 @@ class WhereGroup implements Iterator, Countable
      * WhereGroup constructor.
      *
      * @param array $conditions
-     * @param int $conjunction
+     * @param Conjunction $conjunction
      */
-    public function __construct(array $conditions = [], int $conjunction = self:: AND)
+    public function __construct(array $conditions = [], Conjunction $conjunction = Conjunction::AND)
     {
         $this->group = $conditions;
         $this->conjunction = $conjunction;

test/tests/SQLTest.php

@@ -2,6 +2,7 @@
 
 namespace Aternos\Model\Test\Tests;
 
+use Aternos\Model\Query\Conjunction;
 use Aternos\Model\Query\DeleteQuery;
 use Aternos\Model\Query\Generator\SQL;
 use Aternos\Model\Query\Limit;
@@ -142,7 +143,7 @@ public function testSelectWhereGroupOR()
             new WhereGroup([
                 new WhereCondition('number', 1, '>'),
                 new WhereCondition('number', 0, '<'),
-            ], WhereGroup::OR),
+            ], Conjunction::OR),
         );
         $query->modelClassName = TestModel::class;
 

test/tests/TestDriverTest.php

@@ -4,12 +4,12 @@
 
 use Aternos\Model\Driver\DriverRegistry;
 use Aternos\Model\Driver\Test\TestDriver;
+use Aternos\Model\Query\Conjunction;
 use Aternos\Model\Query\CountField;
 use Aternos\Model\Query\DeleteQuery;
 use Aternos\Model\Query\Direction;
 use Aternos\Model\Query\MaxField;
 use Aternos\Model\Query\MinField;
-use Aternos\Model\Query\OrderField;
 use Aternos\Model\Query\SelectField;
 use Aternos\Model\Query\SumField;
 use Aternos\Model\Query\WhereCondition;
@@ -145,7 +145,7 @@ public function testSelectWhereOr(): void
         $models = TestModel::select(new WhereGroup([
             new WhereCondition("number", 1),
             new WhereCondition("number", 2)
-        ], WhereGroup::OR));
+        ], Conjunction::OR));
         $this->assertCount(2, $models);
         $this->assertEquals("1B", $models[0]->id);
         $this->assertEquals("B", $models[0]->text);