Fixed heredoc inside function calls inside other function calls (#1064)

Author: jodysimpson

src/util.js

@@ -258,7 +258,9 @@ function docShouldHaveTrailingNewline(path) {
   }
 
   if (
-    (parentParent && ["call", "new", "echo"].includes(parentParent.kind)) ||
+    (parentParent &&
+      ["call", "new", "echo"].includes(parentParent.kind) &&
+      parent.kind !== "call") ||
     parent.kind === "parameter"
   ) {
     const lastIndex = parentParent.arguments.length - 1;

tests/nowdoc/__snapshots__/jsfmt.spec.js.snap

@@ -52,6 +52,11 @@ foo
 EOD
 , true);
 
+$str = sprintf(sprintf(<<<END
+foo
+END
+, true));
+
 function foo($a = 1, $b = <<<'EOD'
 Example of string
 spanning multiple lines
@@ -982,6 +987,16 @@ EOD
     true
 );
 
+$str = sprintf(
+    sprintf(
+        <<<END
+foo
+END
+        ,
+        true
+    )
+);
+
 function foo(
     $a = 1,
     $b = <<<'EOD'

tests/nowdoc/nowdoc.php

@@ -44,6 +44,11 @@
 EOD
 , true);
 
+$str = sprintf(sprintf(<<<END
+foo
+END
+, true));
+
 function foo($a = 1, $b = <<<'EOD'
 Example of string
 spanning multiple lines