Tree: remove arrayContains function

ElemeFE · Jun 10, 2019 · 2a58f4d · 2a58f4d
1 parent 74191d0
commit 2a58f4d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/packages/tree/src/model/node.js b/packages/tree/src/model/node.js
@@ -1,6 +1,6 @@
 import objectAssign from 'element-ui/src/utils/merge';
 import { markNodeData, NODE_KEY } from './util';
-import { arrayContains } from 'element-ui/src/utils/util';
+import { arrayFindIndex } from 'element-ui/src/utils/util';
 
 export const getChildState = node => {
   let all = true;
@@ -437,7 +437,7 @@ export default class Node {
 
     newData.forEach((item, index) => {
       const key = item[NODE_KEY];
-      const isNodeExists = !!key && arrayContains(oldData, data => data[NODE_KEY] === key);
+      const isNodeExists = !!key && arrayFindIndex(oldData, data => data[NODE_KEY] === key) >= 0;
       if (isNodeExists) {
         newDataMap[key] = { index, data: item };
       } else {

diff --git a/src/utils/util.js b/src/utils/util.js
@@ -97,10 +97,6 @@ export const arrayFindIndex = function(arr, pred) {
   return -1;
 };
 
-export const arrayContains = function(arr, pred) {
-  return arrayFindIndex(arr, pred) >= 0;
-};
-
 export const arrayFind = function(arr, pred) {
   const idx = arrayFindIndex(arr, pred);
   return idx !== -1 ? arr[idx] : undefined;