Some LPE fails on apply to original, but cloned elements (rev >= 11212)

[Jabiertxof]

Steps to reproduce:
Create some path.
Clone.
Select original path.
Apply envelope deformation.

I think it happens in all eefect than use doEffect_pwd2.
Tested with: concatenate_before_pwd2 = true;

[Alvin Penner (apenner)]

confirmed on Windows XP, Inkscape rev 13123

- draw path
- use Edit->Clone->Create Clone
- drag clone to right
- select original object
- save as clone_before.svg

[Alvin Penner (apenner)]

- add Path Effects->Envelope Deformation
- both objects collapse to very small size
- original path is at origin (0, 1052)
- clone is at top middle (322, 1040)
- both objects are less than a pixel in size
- save as clone_after.svg

- not reproduced on Inkscape 0.48.4

[su_v (suv-lp)]

Reproduced as described in comment #2 with r13229 on OS X 10.7.5

Based on tests with archived builds:
- not reproduced with rev <= 11208,
- reproduced with rev >= 11212;
the regression could be related to the changes in r11212 (?):
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/changes/11212>

[Johan Engelen (johanengelen)]

It is caused by r11212, the change to sbasis.cpp on line 282.
However, this was part of a fix to 2geom because of some bug with paths where handles are at same location as nodes. (zero-length handle)... It has been quite a while since, and I no longer remember the details.

[Johan Engelen (johanengelen)]

Symptom: bounding box is wrong after lpe-enveloppe.cpp:line 232.

[Johan Engelen (johanengelen)]

The problem is that the clone wants to update too early, during construction of the patheffect setup. Originally this worked, because before the 2geom change, LPE Envelope threw an exception. After the fix, 2geom no longer throws and the LPE calculation returns some supersmall item, so the path (not clone) is updated *during* LPE set up.

[Jabiertxof (jabiertxof)]

fixed in master in 0aba9576

[Jabiertxof (jabiertxof)]

backported to 0.92.x on e12c9962

[Qantas94Heavy (qantas94heavy)]

Fix released with 0.92.4.