newLine at end of @this(...) and @(...) are not part of content (#169)

playframework · Feb 21, 2018 · b4b8e96 · b4b8e96
1 parent fc57c5c
commit b4b8e96
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/parser/src/main/scala/play/twirl/parser/TwirlParser.scala b/parser/src/main/scala/play/twirl/parser/TwirlParser.scala
@@ -902,8 +902,13 @@ class TwirlParser(val shouldParseInclusiveDot: Boolean) {
       input.regress(1)
       val p = input.offset()
       val args = templateArgs()
-      if (args != null) Some(position(PosString(args), p))
-      else None
+      if (args != null) {
+        val result = position(PosString(args), p)
+        check("\n")
+        Some(result)
+      } else {
+        None
+      }
     } else None
   }
 

diff --git a/sbt-twirl/src/sbt-test/twirl/compile/src/test/scala/test/Test.scala b/sbt-twirl/src/sbt-test/twirl/compile/src/test/scala/test/Test.scala
@@ -7,7 +7,7 @@ object Test extends App {
     assert(template.body == expected, s"Found '$template' but expected '$expected'")
   }
 
-  test(a.b.html.c.render("world"), "\nHello, world.\n")
+  test(a.b.html.c.render("world"), "Hello, world.\n")
 
-  test(html.template.render("42"), "\nAnswer: 42\n")
+  test(html.template.render("42"), "Answer: 42\n")
 }
diff --git a/sbt-twirl/src/sbt-test/twirl/scalajs-compile/src/main/scala/Test.scala b/sbt-twirl/src/sbt-test/twirl/scalajs-compile/src/main/scala/Test.scala
@@ -6,7 +6,7 @@ object Test extends App {
     assert(template.body == expected, s"Found '$template' but expected '$expected'")
   }
 
-  test(a.b.html.c.render("world"), "\nHello, world.\n")
+  test(a.b.html.c.render("world"), "Hello, world.\n")
 
-  test(html.template.render("42"), "\nAnswer: 42\n")
+  test(html.template.render("42"), "Answer: 42\n")
 }