From dd651d93be123ca5f55e3b13712ebe403da9e856 Mon Sep 17 00:00:00 2001 From: Magnus Ellinge Date: Tue, 2 Apr 2019 00:27:21 +0200 Subject: [PATCH] fix: Code smells --- src/tree/index.js | 3 +-- src/utils/nodeVisitor.js | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/tree/index.js b/src/tree/index.js index aac75916..69e8c8e1 100644 --- a/src/tree/index.js +++ b/src/tree/index.js @@ -68,8 +68,7 @@ class Tree extends Component { const currentId = props.activeDescendant.replace(/_li$/, '') const focusIndex = this.allVisibleNodes.findIndex(n => n.key === currentId) + 1 this.currentPage = focusIndex > 0 ? Math.ceil(focusIndex / this.props.pageSize) : 1 - } - else { + } else { this.currentPage = 1 } } diff --git a/src/utils/nodeVisitor.js b/src/utils/nodeVisitor.js index 4d5b4772..194cc7c1 100644 --- a/src/utils/nodeVisitor.js +++ b/src/utils/nodeVisitor.js @@ -25,7 +25,7 @@ const getNodesMatching = (tree, nodePredicate) => { return nodes } -const getVisibleNodes = (tree, getItemById, markSubTreeOnNonExpanded) => +const getVisibleNodes = (tree, getItemById, markSubTreeOnNonExpanded) => getNodesMatching(tree, (node, key, visited) => { if (markSubTreeOnNonExpanded && node._children && node._children.length && node.expanded !== true) { markSubTreeVisited(node, visited, getItemById)