[TINKERPOP-1832] TraversalHelper.replaceStep sets previousStep to the wrong step

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.3.0
• Fix Version/s: 3.2.7, 3.3.1
• Component/s: process
• Assignee: Daniel Kuppitz
• Reporter: pieter martin
• Created: 11/Nov/17 12:00
• Updated: 27/Nov/17 13:01
• Resolved: 27/Nov/17 13:01

Description

The bug is described over here

I have tested dkuppitz fix for it and all seems well.

    public static <S, E> void replaceStep(final Step<S, E> removeStep, final Step<S, E> insertStep, final Traversal.Admin<?, ?> traversal) {
        final int i;
        traversal.removeStep(i = stepIndex(removeStep, traversal));
        traversal.addStep(i, insertStep);
    }

Issue Links

• links to: GitHub Pull Request #751
• links to: GitHub Pull Request #756