revert: revert prop object validation

close #7279
vuejs · Dec 19, 2017 · 01c0750 · 01c0750
1 parent 2e0f6d5
commit 01c0750
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 40 deletions.
diff --git a/src/core/util/options.js b/src/core/util/options.js
@@ -292,9 +292,6 @@ function normalizeProps (options: Object, vm: ?Component) {
     for (const key in props) {
       val = props[key]
       name = camelize(key)
-      if (process.env.NODE_ENV !== 'production' && isPlainObject(val)) {
-        validatePropObject(name, val, vm)
-      }
       res[name] = isPlainObject(val)
         ? val
         : { type: val }
@@ -309,26 +306,6 @@ function normalizeProps (options: Object, vm: ?Component) {
   options.props = res
 }
 
-/**
- * Validate whether a prop object keys are valid.
- */
-const propOptionsRE = /^(type|default|required|validator)$/
-
-function validatePropObject (
-  propName: string,
-  prop: Object,
-  vm: ?Component
-) {
-  for (const key in prop) {
-    if (!propOptionsRE.test(key)) {
-      warn(
-        `Invalid key "${key}" in validation rules object for prop "${propName}".`,
-        vm
-      )
-    }
-  }
-}
-
 /**
  * Normalize all injections into Object-based format
  */

diff --git a/test/unit/features/options/props.spec.js b/test/unit/features/options/props.spec.js
@@ -513,23 +513,6 @@ describe('Options props', () => {
     })
   })
 
-  it('should warn about misspelled keys in prop validation object', () => {
-    new Vue({
-      template: '<test></test>',
-      components: {
-        test: {
-          template: '<div></div>',
-          props: {
-            value: { reqquired: true },
-            count: { deafult: 1 }
-          }
-        }
-      }
-    }).$mount()
-    expect(`Invalid key "reqquired" in validation rules object for prop "value".`).toHaveBeenWarned()
-    expect(`Invalid key "deafult" in validation rules object for prop "count".`).toHaveBeenWarned()
-  })
-
   it('should not trigger re-render on non-changed inline literals', done => {
     const updated = jasmine.createSpy('updated')
     const vm = new Vue({