fix: namespace followed by inline node (#1265)

Fixes #1052
prettier · Jan 2, 2020 · cee7f95 · cee7f95
1 parent 4f58488
commit cee7f95
Show file tree

Hide file tree

Showing 5 changed files with 83 additions and 10 deletions.
diff --git a/src/printer.js b/src/printer.js
@@ -1037,12 +1037,12 @@ function printLines(path, options, print, childrenAttribute = "children") {
       const beforeInline =
         childNode.leadingComments && childNode.leadingComments.length
           ? concat([
-              isFirstNode ? openTag : "",
+              isFirstNode && node.kind !== "namespace" ? openTag : "",
               hardline,
               comments.printComments(childNode.leadingComments, options),
               "?>"
             ])
-          : isProgramLikeNode(node) && isFirstNode
+          : isProgramLikeNode(node) && isFirstNode && node.kind !== "namespace"
           ? ""
           : concat([beforeCloseTagInlineNode, "?>"]);
       const afterInline =
@@ -1623,14 +1623,18 @@ function printNode(path, options, print) {
           ? node.withBrackets
             ? indent(concat([hardline, printLines(path, options, print)]))
             : concat([
-                hardline,
-                isNextLineEmptyAfterNamespace(
-                  options.originalText,
-                  node,
-                  options.locStart
-                )
-                  ? hardline
-                  : "",
+                node.children[0].kind === "inline"
+                  ? ""
+                  : concat([
+                      hardline,
+                      isNextLineEmptyAfterNamespace(
+                        options.originalText,
+                        node,
+                        options.locStart
+                      )
+                        ? hardline
+                        : ""
+                    ]),
                 printLines(path, options, print)
               ])
           : "",

diff --git a/tests/inline/__snapshots__/jsfmt.spec.js.snap b/tests/inline/__snapshots__/jsfmt.spec.js.snap
@@ -2305,6 +2305,64 @@ $var = 2;
 ================================================================================
 `;
 
+exports[`namespace-1.php 1`] = `
+====================================options=====================================
+parsers: ["php"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+<?php namespace foo; ?>
+<html></html>
+
+=====================================output=====================================
+<?php namespace foo; ?>
+<html></html>
+
+================================================================================
+`;
+
+exports[`namespace-2.php 1`] = `
+====================================options=====================================
+parsers: ["php"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+<?php namespace foo; ?>
+
+<html></html>
+
+=====================================output=====================================
+<?php namespace foo; ?>
+
+<html></html>
+
+================================================================================
+`;
+
+exports[`namespace-3.php 1`] = `
+====================================options=====================================
+parsers: ["php"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+<?php
+namespace foo;
+// comment
+?>
+
+<html></html>
+
+=====================================output=====================================
+<?php
+namespace foo;
+// comment
+?>
+
+<html></html>
+
+================================================================================
+`;
+
 exports[`only-html.php 1`] = `
 ====================================options=====================================
 parsers: ["php"]

diff --git a/tests/inline/namespace-1.php b/tests/inline/namespace-1.php
@@ -0,0 +1,2 @@
+<?php namespace foo; ?>
+<html></html>
diff --git a/tests/inline/namespace-2.php b/tests/inline/namespace-2.php
@@ -0,0 +1,3 @@
+<?php namespace foo; ?>
+
+<html></html>
diff --git a/tests/inline/namespace-3.php b/tests/inline/namespace-3.php
@@ -0,0 +1,6 @@
+<?php
+namespace foo;
+// comment
+?>
+
+<html></html>