BSpline Path Effect: 'Change only selected nodes' first edits previously selected node

[Hachmann]

When using the BSpline LPE, and checking 'Change only selected nodes', the 'make cusp' and 'change weight' buttons are - when used for the first time - applied to the previously selected node in the path, not to the currently selected one.

1. Draw BSpline path.
2. In LPE dialog, check 'Change only selected nodes'
3. Select a node
4. Change its weight, so you can see the bug
5. Select another node.
6. Click on 'default weight' or 'make cusp'
7. Result: the previously selected node with the non-default weight is changed

Inkscape trunk ppa version of Oct. 4th 2016.

[su_v (suv-lp)]

Reproduced with lp:inkscape/0.92.x r15094 (on OS X 10.7.5).

Based on tests with recent archived builds:
- not reproduced with lp:inkscape/0.92.x rev <= 15051
- reproduced with lp:inkscape/0.92.x rev >= 15053
this regression possibly has been exposed with the changes from:

Revision 15053: Allow update helper paths from LPE widget interaction
https://bazaar.launchpad.net/~inkscape.dev/inkscape/0.92.x/revision/15053

The same changes had been committed earlier to trunk lp:inkscape in revision 15086.

[Hachmann (marenhachmann)]

Thanks for testing!

[Jabiertxof (jabiertxof)]

I fix the bug. I commit it when finish compiling.

[Jabiertxof (jabiertxof)]

Fixes on trunk r.15149
And in 0.92.x in 15095

[Jabiertxof (jabiertxof)]

Thanks for fill the bug and tests.

[Hachmann (marenhachmann)]

Ah, that was quick, Jabier! Thx, too!