theluyuan/vue3-yuanma
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(keep-alive): should remove wrapped version of injected keep alive hooks (#1959)

Browse Source
This commit is contained in:
zhangzhonghe 2020-08-25 22:02:39 +08:00 committed by GitHub
parent a47626a4a3
commit 1ea24000c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
packages/runtime-core/src/apiLifecycle.ts
View File

@ -18,7 +18,7 @@ export function injectHook(
hook: Function & { __weh?: Function },
target: ComponentInternalInstance | null = currentInstance,
prepend: boolean = false
) {
): Function | undefined {
if (target) {
const hooks = target[type] || (target[type] = [])
// cache the error handling wrapper for injected hooks so the same hook
@ -47,6 +47,7 @@ export function injectHook(
} else {
hooks.push(wrappedHook)
}
return wrappedHook
} else if (__DEV__) {
const apiName = `on${capitalize(
ErrorTypeStrings[type].replace(/ hook$/, '')

8
packages/runtime-core/src/components/KeepAlive.ts
View File

@ -360,14 +360,16 @@ function registerKeepAliveHook(
}
function injectToKeepAliveRoot(
hook: Function,
hook: Function & { __weh?: Function },
type: LifecycleHooks,
target: ComponentInternalInstance,
keepAliveRoot: ComponentInternalInstance
) {
injectHook(type, hook, keepAliveRoot, true /* prepend */)
// injectHook wraps the original for error handling, so make sure to remove
// the wrapped version.
const injected = injectHook(type, hook, keepAliveRoot, true /* prepend */)
onUnmounted(() => {
remove(keepAliveRoot[type]!, hook)
remove(keepAliveRoot[type]!, injected)
}, target)
}