fix(graphql-printer): Object value will respect `Settings::isAlwaysMu…

…ltilineArguments()`.
LastDragon-ru · Aug 29, 2023 · 4a8b916 · 4a8b916
1 parent 212b61c
commit 4a8b916
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 7 deletions.
diff --git a/packages/graphql-printer/src/Blocks/Document/ValueTest.php b/packages/graphql-printer/src/Blocks/Document/ValueTest.php
@@ -327,13 +327,13 @@ public static function dataProviderSerialize(): array {
                 0,
                 Parser::valueLiteral(
                     <<<'STRING'
-                {
-                    object: {
-                        a: "a"
-                        b: "b"
+                    {
+                        object: {
+                            a: "a"
+                            b: "b"
+                        }
                     }
-                }
-                STRING,
+                    STRING,
                 ),
                 null,
                 null,
@@ -371,6 +371,27 @@ public static function dataProviderSerialize(): array {
                 null,
                 null,
             ],
+            ObjectValueNode::class.' (one line)'                  => [
+                <<<'STRING'
+                {object: {a: "a", b: "b"}}
+                STRING,
+                $settings
+                    ->setAlwaysMultilineArguments(false),
+                0,
+                0,
+                Parser::valueLiteral(
+                    <<<'STRING'
+                    {
+                        object: {
+                            a: "a"
+                            b: "b"
+                        }
+                    }
+                    STRING,
+                ),
+                null,
+                null,
+            ],
             'all'                                                 => [
                 <<<'STRING'
                 {

diff --git a/packages/graphql-printer/src/Blocks/Values/ObjectValue.php b/packages/graphql-printer/src/Blocks/Values/ObjectValue.php
@@ -40,7 +40,8 @@ protected function getEmptyValue(): string {
     }
 
     protected function isAlwaysMultiline(): bool {
-        return true;
+        return parent::isAlwaysMultiline()
+            || $this->getSettings()->isAlwaysMultilineArguments();
     }
 
     protected function isNormalized(): bool {