Merge pull request #712 from Demivan/fix-empty-text-nodes

Fix component interpolation with empty text nodes

src/components/interpolation.js

@@ -54,8 +54,14 @@ function onlyHasDefaultPlace (params) {
 
 function useLegacyPlaces (children, places) {
   const params = places ? createParamsFromPlaces(places) : {}
+
   if (!children) { return params }
 
+  // Filter empty text nodes
+  children = children.filter(child => {
+    return child.tag || child.text.trim() !== ''
+  })
+
   const everyPlace = children.every(vnodeHasPlaceAttribute)
   if (process.env.NODE_ENV !== 'production' && everyPlace) {
     warn('`place` attribute is deprecated in next major version. Please switch to Vue slots.')

test/unit/interpolation.test.js

@@ -80,6 +80,25 @@ describe('component interpolation', () => {
       })
     })
 
+    describe('empty text node between components', () => {
+      it('should NOT be interpolated', done => {
+        const el = document.createElement('div')
+        const vm = new Vue({
+          i18n,
+          render (h) {
+            return h('i18n', { props: { path: 'primitive' } }, [
+              h('p', ['1']),
+              this._v(''),
+              h('p', ['2'])
+            ])
+          }
+        }).$mount(el)
+        nextTick(() => {
+          assert.strictEqual(vm.$el.innerHTML, 'one: <p>1</p>, two: <p>2</p>')
+        }).then(done)
+      })
+    })
+
     describe('components', () => {
       it('should be interpolated', done => {
         const el = document.createElement('div')