props: only warn twoWay when prop is present (fix vuejs#1400)

yyx990803 · yyx990803 · commit d8862a177493 · 2015-10-09T22:15:51.000-04:00
diff --git a/src/compiler/compile-props.js b/src/compiler/compile-props.js
@@ -85,6 +85,18 @@ module.exports = function compileProps (el, propOptions) {
           }
         }
         prop.parentPath = value
+
+        // warn required two-way
+        if (
+          process.env.NODE_ENV !== 'production' &&
+          options.twoWay &&
+          prop.mode !== propBindingModes.TWO_WAY
+        ) {
+          _.warn(
+            'Prop "' + name + '" expects a two-way binding type.'
+          )
+        }
+
       } else if (options.required) {
         // warn missing required
         process.env.NODE_ENV !== 'production' && _.warn(
@@ -93,17 +105,6 @@ module.exports = function compileProps (el, propOptions) {
       }
     }
 
-    // warn required two-way
-    if (
-      process.env.NODE_ENV !== 'production' &&
-      options.twoWay &&
-      prop.mode !== propBindingModes.TWO_WAY
-    ) {
-      _.warn(
-        'Prop "' + name + '" expects a two-way binding type.'
-      )
-    }
-
     // push prop
     props.push(prop)
   }
