Clean up code and appease the linter

Author: fritz-c

src/examples/basicExample/app.js

@@ -197,7 +197,7 @@ class App extends Component {
                             treeData={this.state.treeData}
                             updateTreeData={this.updateTreeData}
                             maxDepth={5}
-                            generateNodeProps={(rowInfo) => ({
+                            generateNodeProps={rowInfo => ({
                                 buttons: [
                                     <button
                                         style={{

src/node-renderer-default.js

@@ -139,7 +139,6 @@ NodeRendererDefault.propTypes = {
     node:               PropTypes.object.isRequired,
     path:               PropTypes.arrayOf(PropTypes.oneOfType([ PropTypes.string, PropTypes.number ])).isRequired,
     treeIndex:          PropTypes.number.isRequired,
-    lowerSiblingCounts: PropTypes.arrayOf(PropTypes.number).isRequired,
 
     scaffoldBlockPxWidth:     PropTypes.number.isRequired,
     toggleChildrenVisibility: PropTypes.func,

src/react-sortable-tree.js

@@ -16,6 +16,9 @@ import {
     insertNode,
     getDescendantCount,
 } from './utils/tree-data-utils';
+import {
+    swapRows,
+} from './utils/generic-utils';
 import {
     defaultGetNodeKey,
     defaultToggleChildrenVisibility,
@@ -89,19 +92,6 @@ class ReactSortableTree extends Component {
         });
     }
 
-    swapRows(rows, fromIndex, toIndex, count = 1) {
-        const rowsWithoutMoved = [
-            ...rows.slice(0, fromIndex),
-            ...rows.slice(fromIndex + count),
-        ];
-
-        return [
-            ...rowsWithoutMoved.slice(0, toIndex),
-            ...rows.slice(fromIndex, fromIndex + count),
-            ...rowsWithoutMoved.slice(toIndex),
-        ];
-    }
-
     startDrag({ path }) {
         const draggingTreeData = removeNodeAtPath({
             treeData: this.props.treeData,
@@ -126,7 +116,7 @@ class ReactSortableTree extends Component {
         const rows = this.getRows(addedResult.treeData);
         const expandedParentPath = rows[addedResult.treeIndex].path;
         this.setState({
-            rows: this.swapRows(
+            rows: swapRows(
                 rows,
                 addedResult.treeIndex,
                 minimumTreeIndex,
@@ -254,7 +244,6 @@ class ReactSortableTree extends Component {
                 <NodeContentRenderer
                     node={node}
                     path={path}
-                    lowerSiblingCounts={lowerSiblingCounts}
                     treeIndex={treeIndex}
                     startDrag={this.startDrag}
                     endDrag={this.endDrag}
@@ -268,8 +257,7 @@ class ReactSortableTree extends Component {
 }
 
 ReactSortableTree.propTypes = {
-    treeData:   PropTypes.arrayOf(PropTypes.object).isRequired,
-    changeData: PropTypes.func,
+    treeData: PropTypes.arrayOf(PropTypes.object).isRequired,
 
     // Callback for move operation.
     // Called as moveNode({ node, path, parentPath, minimumTreeIndex })
@@ -295,7 +283,6 @@ ReactSortableTree.propTypes = {
     getNodeKey:                PropTypes.func,
     updateTreeData:            PropTypes.func,
     toggleChildrenVisibility:  PropTypes.func,
-    loadCollapsedLazyChildren: PropTypes.bool,
 };
 
 ReactSortableTree.defaultProps = {

src/utils/generic-utils.js

@@ -0,0 +1,12 @@
+export function swapRows(rows, fromIndex, toIndex, count = 1) {
+    const rowsWithoutMoved = [
+        ...rows.slice(0, fromIndex),
+        ...rows.slice(fromIndex + count),
+    ];
+
+    return [
+        ...rowsWithoutMoved.slice(0, toIndex),
+        ...rows.slice(fromIndex, fromIndex + count),
+        ...rowsWithoutMoved.slice(toIndex),
+    ];
+}

src/utils/tree-data-utils.js

@@ -760,7 +760,7 @@ export function getTreeFromFlatData({
     }
 
     const childrenToParents = {};
-    flatData.forEach(child => {
+    flatData.forEach((child) => {
         const parentKey = getParentKey(child);
 
         if (parentKey in childrenToParents) {

src/utils/tree-data-utils.test.js

@@ -1317,7 +1317,7 @@ describe('insertNode', () => {
 
 describe('walk', () => {
     it('should handle empty data', () => {
-        [[], null, undefined].forEach(treeData => {
+        [[], null, undefined].forEach((treeData) => {
             expect(() => walk({
                 treeData,
                 getNodeKey: keyFromTreeIndex,