Sort schema printer methods

Author: a-melnikov

src/Utils/FederatedSchemaPrinter.php

@@ -100,28 +100,6 @@ public static function printType(Type $type, array $options = []): string
         return parent::printType($type, $options);
     }
 
-    /**
-     * @param array<string, bool> $options
-     */
-    protected static function printObject(ObjectType $type, array $options): string
-    {
-        if (empty($type->getFields())) {
-            return '';
-        }
-
-        $implementedInterfaces = static::printImplementedInterfaces($type);
-        $extends = FederatedSchema::isReservedRootType($type->name) ? 'extend ' : '';
-
-        return static::printDescription($options, $type) .
-            sprintf(
-                "%stype %s%s {\n%s\n}",
-                $extends,
-                $type->name,
-                $implementedInterfaces,
-                static::printFields($options, $type)
-            );
-    }
-
     /**
      * @param array<string, bool> $options
      */
@@ -142,6 +120,27 @@ protected static function printEntityObject(EntityObjectType $type, array $optio
             );
     }
 
+    protected static function printFieldFederatedDirectives(FieldDefinition $field): string
+    {
+        $directives = [];
+
+        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_IS_EXTERNAL])
+            && true === $field->config[EntityObjectType::FIELD_DIRECTIVE_IS_EXTERNAL]
+        ) {
+            $directives[] = '@external';
+        }
+
+        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_PROVIDES])) {
+            $directives[] = sprintf('@provides(fields: "%s")', static::printKeyFields($field->config[EntityObjectType::FIELD_DIRECTIVE_PROVIDES]));
+        }
+
+        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_REQUIRES])) {
+            $directives[] = sprintf('@requires(fields: "%s")', static::printKeyFields($field->config[EntityObjectType::FIELD_DIRECTIVE_REQUIRES]));
+        }
+
+        return implode(' ', $directives);
+    }
+
     /**
      * @param array<string, bool> $options
      * @param EntityObjectType|InterfaceType|ObjectType|TypeWithFields $type
@@ -178,27 +177,6 @@ static function (FieldDefinition $f, $i) use ($options) {
         );
     }
 
-    protected static function printFieldFederatedDirectives(FieldDefinition $field): string
-    {
-        $directives = [];
-
-        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_IS_EXTERNAL])
-            && true === $field->config[EntityObjectType::FIELD_DIRECTIVE_IS_EXTERNAL]
-        ) {
-            $directives[] = '@external';
-        }
-
-        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_PROVIDES])) {
-            $directives[] = sprintf('@provides(fields: "%s")', static::printKeyFields($field->config[EntityObjectType::FIELD_DIRECTIVE_PROVIDES]));
-        }
-
-        if (isset($field->config[EntityObjectType::FIELD_DIRECTIVE_REQUIRES])) {
-            $directives[] = sprintf('@requires(fields: "%s")', static::printKeyFields($field->config[EntityObjectType::FIELD_DIRECTIVE_REQUIRES]));
-        }
-
-        return implode(' ', $directives);
-    }
-
     protected static function printImplementedInterfaces(ObjectType $type): string
     {
         $interfaces = $type->getInterfaces();
@@ -240,4 +218,26 @@ protected static function printKeyFields($keyFields): string
 
         return implode(' ', $parts);
     }
+
+    /**
+     * @param array<string, bool> $options
+     */
+    protected static function printObject(ObjectType $type, array $options): string
+    {
+        if (empty($type->getFields())) {
+            return '';
+        }
+
+        $implementedInterfaces = static::printImplementedInterfaces($type);
+        $extends = FederatedSchema::isReservedRootType($type->name) ? 'extend ' : '';
+
+        return static::printDescription($options, $type) .
+            sprintf(
+                "%stype %s%s {\n%s\n}",
+                $extends,
+                $type->name,
+                $implementedInterfaces,
+                static::printFields($options, $type)
+            );
+    }
 }