Browse Source

Revert "fix #4041, warn overriding Vue's internal methods (#4111)"

This reverts commit 4078ce9c58.
dev
Evan You 8 years ago
parent
commit
1bcc571739
  1. 10
      src/core/instance/state.js
  2. 10
      test/unit/features/options/methods.spec.js

10
src/core/instance/state.js

@ -21,8 +21,6 @@ import {
noop noop
} from '../util/index' } from '../util/index'
import BuiltinVue from '../index'
export function initState (vm: Component) { export function initState (vm: Component) {
vm._watchers = [] vm._watchers = []
initProps(vm) initProps(vm)
@ -145,16 +143,12 @@ function initMethods (vm: Component) {
if (methods) { if (methods) {
for (const key in methods) { for (const key in methods) {
vm[key] = methods[key] == null ? noop : bind(methods[key], vm) vm[key] = methods[key] == null ? noop : bind(methods[key], vm)
if (process.env.NODE_ENV !== 'production') { if (process.env.NODE_ENV !== 'production' && methods[key] == null) {
methods[key] == null && warn( warn(
`method "${key}" has an undefined value in the component definition. ` + `method "${key}" has an undefined value in the component definition. ` +
`Did you reference the function correctly?`, `Did you reference the function correctly?`,
vm vm
) )
hasOwn(BuiltinVue.prototype, key) && warn(
`Avoid overriding Vue's internal method "${key}".`,
vm
)
} }
} }
} }

10
test/unit/features/options/methods.spec.js

@ -24,14 +24,4 @@ describe('Options methods', () => {
}) })
expect(`method "hello" has an undefined value in the component definition`).toHaveBeenWarned() expect(`method "hello" has an undefined value in the component definition`).toHaveBeenWarned()
}) })
it('should warn overriding builtin methods', () => {
new Vue({
methods: {
$emit () {
}
}
})
expect(`Avoid overriding Vue's internal method "$emit".`).toHaveBeenWarned()
})
}) })

Loading…
Cancel
Save