fix(comp:tree): correct param checked of onCheck (#1404)

`checked` should be the state of check operation instead of state of node
IDuxFE · Jan 13, 2023 · 2858da6 · 2858da6
1 parent 5d604f9
commit 2858da6
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/packages/components/tree/src/composables/useCheckable.ts b/packages/components/tree/src/composables/useCheckable.ts
@@ -80,14 +80,15 @@ export function useCheckable(props: TreeProps, mergedNodeMap: ComputedRef<Map<VK
     const cascaderEnabled = cascaderStrategy !== 'off'
     const childrenKeys = cascaderEnabled ? getChildrenKeys(node, disabledKeys) : []
     const index = allCheckedKeys.value.indexOf(currKey)
-    const checked = index > -1
+
     let tempKeys = [...allCheckedKeys.value]
 
-    if (
-      checked ||
-      (childrenKeys.length &&
+    const checked =
+      index > -1 ||
+      (!!childrenKeys.length &&
         childrenKeys.every(key => tempKeys.includes(key) || indeterminateKeys.value.includes(key)))
-    ) {
+
+    if (checked) {
       const parentKeys = cascaderEnabled ? getParentKeys(nodeMap, node, disabledKeys) : []
       tempKeys.splice(index, 1)
       tempKeys = tempKeys.filter(key => !parentKeys.includes(key) && !childrenKeys.includes(key))