Fix placeables in pattern at boundries.

fluent-syntax/src/ftlstream.js

@@ -173,8 +173,7 @@ export class FTLParserStream extends ParserStream {
         this.currentPeekIs('.') ||
         this.currentPeekIs('#') ||
         this.currentPeekIs('[') ||
-        this.currentPeekIs('*') ||
-        this.currentPeekIs('{')) {
+        this.currentPeekIs('*')) {
       this.resetPeek();
       return false;
     }

fluent-syntax/src/parser.js

@@ -360,11 +360,10 @@ function getPattern(ps) {
       ps.next();
       ps.skipLineWS();
 
-      firstLine = false;
-
-      if (buffer.length !== 0) {
+      if (!firstLine) {
         buffer += ch;
       }
+      firstLine = false;
       continue;
     } else if (ch === '\\') {
       const ch2 = ps.peek();

fluent-syntax/test/fixtures_behavior/placeable_at_line_extremes.ftl

@@ -0,0 +1,8 @@
+key1 =
+    { foo }
+
+key2 =
+    Foo { foo }
+
+key3 =
+    { foo } Foo

fluent-syntax/test/fixtures_structure/placeable_at_eol.ftl

@@ -0,0 +1,4 @@
+key =
+    A multiline message with a { placeable }
+    at the end of line.  The message should
+    consist of three lines of text.

fluent-syntax/test/fixtures_structure/placeable_at_eol.json

@@ -0,0 +1,42 @@
+{
+  "type": "Resource",
+  "body": [
+    {
+      "type": "Message",
+      "span": {
+        "type": "Span",
+        "start": 0,
+        "end": 130
+      },
+      "annotations": [],
+      "id": {
+        "type": "Identifier",
+        "name": "key"
+      },
+      "value": {
+        "type": "Pattern",
+        "elements": [
+          {
+            "type": "TextElement",
+            "value": "A multiline message with a "
+          },
+          {
+            "type": "MessageReference",
+            "id": {
+              "type": "Identifier",
+              "name": "placeable"
+            }
+          },
+          {
+            "type": "TextElement",
+            "value": "\nat the end of line.  The message should\nconsist of three lines of text."
+          }
+        ]
+      },
+      "attributes": null,
+      "tags": null,
+      "comment": null
+    }
+  ],
+  "comment": null
+}

fluent-syntax/test/serializer_test.js

@@ -122,7 +122,6 @@ suite('Serializer', function() {
     assert.equal(pretty(input), input);
   });
 
-  // XXX The Parser ignores the new line after the closing brace
   test.skip('multiline with placeable', function() {
     const input = ftl`
       foo =
@@ -132,18 +131,6 @@ suite('Serializer', function() {
     assert.equal(pretty(input), input);
   });
 
-  test('multiline with placeable (current)', function() {
-    const input = ftl`
-      foo =
-          Foo { bar }
-          Baz
-    `;
-    const output = ftl`
-      foo = Foo { bar }Baz
-    `;
-    assert.equal(pretty(input), output);
-  });
-
   test('tag', function() {
     const input = ftl`
       foo = Foo

fluent/src/parser.js

@@ -321,7 +321,7 @@ class RuntimeParser {
           break;
         }
 
-        if (buffer.length) {
+        if (buffer.length || content.length) {
           buffer += '\n';
         }
         ch = this._source[this._index];

fluent/test/fixtures_structure/placeable_at_eol.json

@@ -0,0 +1,12 @@
+{
+  "key": {
+    "val": [
+      "A multiline message with a ",
+      {
+        "type": "ref",
+        "name": "placeable"
+      },
+      "\nat the end of line.  The message should\nconsist of three lines of text."
+    ]
+  }
+}