bencripps / react-redux-grid

src/util/moveTreeNode.js

import { List } from 'immutable';

import { findTreeNode } from './findTreeNode';

export const moveTreeNode = (
    treeData,
    currentIndex,
    currentPath,
    nextIndex,
    nextPath,
    childIdentifier = 'children',
    rootIdentifier = 'root'
) => {

    const originalTreeData = treeData;

    const { node: currentParent, indexPath: currentIndexPath } = findTreeNode(
        treeData,
        currentPath,
        childIdentifier,
        rootIdentifier
    );

    if (!currentParent) {

If you intend to check if the variable currentParent is declared in the current environment, consider using typeof currentParent === "undefined" instead. This is safe if the variable is not actually declared.

        return originalTreeData;
    }

    currentIndexPath.push(...[childIdentifier, currentIndex]);

The variable currentIndexPath seems to be never declared. If this is a global, consider adding a /** global: currentIndexPath */ comment.

    let node = treeData.getIn(currentIndexPath);

You seem to be assigning a new value to the const node. Consider using a var instead.

The variable node already seems to be declared on line 17. Consider using another variable name or omitting the var keyword.

    treeData = treeData.deleteIn(currentIndexPath);

    const { node: nextParent, indexPath: nextIndexPath } = findTreeNode(
        treeData,
        nextPath,
        childIdentifier,
        rootIdentifier
    );

    if (!nextParent) {

If you intend to check if the variable nextParent is declared in the current environment, consider using typeof nextParent === "undefined" instead. This is safe if the variable is not actually declared.

        return originalTreeData;
    }

    nextIndexPath.push(childIdentifier);

The variable nextIndexPath seems to be never declared. If this is a global, consider adding a /** global: nextIndexPath */ comment.

    if (!List.isList(treeData.getIn(nextIndexPath))) {
        treeData = treeData.setIn(nextIndexPath, List());
    }

    node = node.set('parentId', nextPath.last());

    treeData = treeData.setIn(
        nextIndexPath,
        treeData.getIn(nextIndexPath).insert(nextIndex, node)
    );

    return treeData;
};