add v-model read only filter warning

yyx990803 · yyx990803 · commit 99ebd511d7aa · 2014-10-27T15:25:44.000-04:00
diff --git a/src/directives/model/index.js b/src/directives/model/index.js
@@ -26,6 +26,15 @@ module.exports = {
    */
 
   bind: function () {
+    // friendly warning...
+    var filters = this.filters
+    if (filters && filters.read && !filters.write) {
+      _.warn(
+        'It seems you are using a read-only filter with ' +
+        'v-model. You might want to use a two-way filter ' +
+        'to ensure correct behavior.'
+      )
+    }
     var el = this.el
     var tag = el.tagName
     var handler
diff --git a/test/unit/specs/directives/model_spec.js b/test/unit/specs/directives/model_spec.js
@@ -473,5 +473,18 @@ if (_.inBrowser) {
       expect(_.warn).toHaveBeenCalled()
     })
 
+    it('warn read-only filters', function () {
+      var vm = new Vue({
+        el: el,
+        template: '<input v-model="abc | test">',
+        filters: {
+          test: function (v) {
+            return v
+          }
+        }
+      })
+      expect(_.warn).toHaveBeenCalled()
+    })
+
   })
 }
