Description
The move-limiting logic contains an issue in an inner for loop where the wrong variable is being incremented. This can lead to incorrect iteration behavior and potentially skip or mis-handle elements during move restriction checks.
How to reproduce the issue?
1.Open the iD editor
2.Perform an action that triggers move-limiting (e.g., moving features with constraints)
3.Observe inconsistent or incorrect behavior in how movement is restricted
Expected Behavior
The loop increments the wrong variable, which may:
Skip elements
Cause incorrect validations
Lead to inconsistent move-limiting results
Proposed Fix
Update the inner loop to increment the correct variable. I can submit a PR with the fix and a small regression test to prevent this issue in the future.
Description
The move-limiting logic contains an issue in an inner for loop where the wrong variable is being incremented. This can lead to incorrect iteration behavior and potentially skip or mis-handle elements during move restriction checks.
How to reproduce the issue?
1.Open the iD editor
2.Perform an action that triggers move-limiting (e.g., moving features with constraints)
3.Observe inconsistent or incorrect behavior in how movement is restricted
Expected Behavior
The loop increments the wrong variable, which may:
Skip elements
Cause incorrect validations
Lead to inconsistent move-limiting results
Proposed Fix
Update the inner loop to increment the correct variable. I can submit a PR with the fix and a small regression test to prevent this issue in the future.