adjust attribute interpolation deprecation warnings

Author: yyx990803

src/compiler/compile.js

@@ -709,11 +709,6 @@ function collectAttrDirective (name, value, options) {
   var tokens = textParser.parse(value)
   var isClass = name === 'class'
   if (tokens) {
-
-    if (process.env.NODE_ENV !== 'production') {
-      _.deprecation.ATTR_INTERPOLATION(name, value)
-    }
-
     var dirName = isClass ? 'class' : 'attr'
     var def = options.directives[dirName]
     var i = tokens.length
@@ -722,6 +717,9 @@ function collectAttrDirective (name, value, options) {
       var token = tokens[i]
       if (token.tag && !token.oneTime) {
         allOneTime = false
+      } else if (token.tag) {
+        process.env.NODE_ENV !== 'production' &&
+        _.deprecation.ATTR_ONETIME(name, value)
       }
     }
     var linker
@@ -732,9 +730,15 @@ function collectAttrDirective (name, value, options) {
     } else {
       linker = function (vm, el, scope) {
         var exp = textParser.tokensToExp(tokens, (scope || vm))
+
+        // silence the argument deprecation warning
+        var silent = config.silent
+        config.silent = true
         var desc = isClass
           ? dirParser.parse(exp)[0]
           : dirParser.parse(name + ':' + exp)[0]
+        config.silent = silent
+
         if (isClass) {
           desc._rawClass = value
         }

src/deprecations.js

@@ -117,11 +117,18 @@ if (process.env.NODE_ENV !== 'production') {
       )
     },
 
-    ATTR_INTERPOLATION: function (name, value) {
+    ATTR_ONETIME: function (name, value) {
       warn(
-        'Mustache interpolations inside attributes: ' + name + '="' + value + '". ' +
-        'This will be deprecated in 1.0.0. ' +
-        'Use v-bind:attr="expression" instead.' + newBindingSyntaxLink
+        name + '="' + value + '": One-time interpolations inside attributes will ' +
+        'no longer be supported in 1.0.0.'
+      )
+    },
+
+    ATTR_INVALID: function (name) {
+      warn(
+        'Mustache interpolation found in non-native attribute "' + name + '": ' +
+        'attribute interpolation will be limited to native attributes only ' +
+        'in 1.0.0. Use v-bind for custom attributes and props.'
       )
     },
 
@@ -159,8 +166,8 @@ if (process.env.NODE_ENV !== 'production') {
     PARTIAL_NAME: function (id) {
       warn(
         '<partial name="' + id + '">: mustache interpolations inside attributes ' +
-        'will be deprecated in 1.0.0. Use v-bind:name="expression" or just ' +
-        ':name="expression" instead.'
+        'will only be allowed in native attributes in 1.0.0. ' +
+        'Use v-bind:name="expression" or just :name="expression" instead.'
       )
     },
 

src/directives/attr.js

@@ -1,3 +1,5 @@
+var _ = require('../util')
+
 // xlink
 var xlinkNS = 'http://www.w3.org/1999/xlink'
 var xlinkRE = /^xlink:/
@@ -24,6 +26,19 @@ module.exports = {
 
   update: function (value) {
     if (this.arg) {
+
+      if (process.env.NODE_ENV !== 'production') {
+        var attr = this.arg
+        if (!(
+          attr === 'class' ||
+          /^data-/.test(attr) ||
+          (attr === 'for' && 'htmlFor' in this.el) ||
+          _.camelize(attr) in this.el
+        )) {
+          _.deprecation.ATTR_INVALID(attr)
+        }
+      }
+
       this.setAttr(this.arg, value)
     } else if (typeof value === 'object') {
       // TODO no longer need to support object in 1.0.0