fix(graphql/@sortby): Random order will be available only for Query…

… and Eloquent builders.
LastDragon-ru · Jan 5, 2024 · 7ffc919 · 7ffc919
1 parent 58719df
commit 7ffc919
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 12 deletions.
diff --git a/packages/graphql/src/SortBy/Directives/DirectiveTest~full-expected.graphql b/packages/graphql/src/SortBy/Directives/DirectiveTest~full-expected.graphql
@@ -399,12 +399,6 @@ input SortByScoutClauseNested {
     nested: SortByScoutClauseNested
     @sortByOperatorProperty
 
-    """
-    By random
-    """
-    random: SortByTypeFlag
-    @sortByOperatorRandom
-
     """
     Property clause.
     """
@@ -478,12 +472,6 @@ input SortByScoutClauseProperties {
     """
     nestedNotNull: SortByScoutClauseNested
     @sortByOperatorProperty
-
-    """
-    By random
-    """
-    random: SortByTypeFlag
-    @sortByOperatorRandom
 }
 
 interface Eloquent

diff --git a/packages/graphql/src/SortBy/Operators/Extra/Random.php b/packages/graphql/src/SortBy/Operators/Extra/Random.php
@@ -16,6 +16,7 @@
 use Override;
 
 use function array_merge;
+use function is_a;
 
 class Random extends BaseOperator {
     // <editor-fold desc="Directive">
@@ -48,6 +49,12 @@ public function getFieldType(TypeProvider $provider, TypeSource $source): string
         return $provider->getType(Flag::class, $source);
     }
 
+    #[Override]
+    public function isBuilderSupported(string $builder): bool {
+        return is_a($builder, EloquentBuilder::class, true)
+            || is_a($builder, QueryBuilder::class, true);
+    }
+
     #[Override]
     public function call(Handler $handler, object $builder, Property $property, Argument $argument): object {
         if (!($builder instanceof EloquentBuilder || $builder instanceof QueryBuilder)) {