do not set up watchers if v-with is bound to literal value (fix vuejs#776 & vuejs#813)

yyx990803 · yyx990803 · commit 2d3cf2d589b4 · 2015-04-30T14:52:13.000-04:00
diff --git a/src/directives/with.js b/src/directives/with.js
@@ -1,5 +1,7 @@
 var _ = require('../util')
 var Watcher = require('../watcher')
+var expParser = require('../parsers/expression')
+var literalRE = /^(true|false|\s?('[^']*'|"[^"]")\s?)$/
 
 module.exports = {
 
@@ -20,6 +22,17 @@ module.exports = {
       _.warn(
         'v-with must be used on an instance with a parent.'
       )
+    } else if (literalRE.test(parentKey)) {
+      // no need to setup watchers for literal bindings
+      if (!this.arg) {
+        _.warn(
+          'v-with cannot bind literal value as $data: ' +
+          parentKey
+        )
+      } else {
+        var value = expParser.parse(parentKey).get()
+        child.$set(childKey, value)
+      }
     } else {
 
       // simple lock to avoid circular updates.
diff --git a/test/unit/specs/directives/with_spec.js b/test/unit/specs/directives/with_spec.js
@@ -152,5 +152,35 @@ if (_.inBrowser) {
       expect(el.innerHTML).toBe('<!--v-start--><p>AAA</p><p>DDD</p><!--v-end--><!--v-component-->')
     })
 
+    it('bind literal values should not trigger setter warning', function (done) {
+      var vm = new Vue({
+        el: el,
+        template: '<div v-component="test" v-with="a:\'test\'"></div>',
+        components: {
+          test: {
+            template: '{{a}}'
+          }
+        }
+      })
+      expect(el.firstChild.innerHTML).toBe('test')
+      vm._children[0].a = 'changed'
+      _.nextTick(function () {
+        expect(el.firstChild.innerHTML).toBe('changed')
+        expect(_.warn).not.toHaveBeenCalled()
+        done()
+      })
+    })
+
+    it('should warn when binding literal value without childKey', function () {
+      var vm = new Vue({
+        el: el,
+        template: '<div v-component="test" v-with="\'test\'"></div>',
+        components: {
+          test: {}
+        }
+      })
+      expect(_.warn).toHaveBeenCalled()
+    })
+
   })
 }
