Inkscape

Spiro and path operations fails

Bug #629081 reported by Mark Jensen
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Matthew Petroff
Inkscape 0.48.5

Bug Description

When using Spiro Splines and a path operation (I haven't tried them all, but those I have tried all have the same problem), Inkscape uses the "underlying" bezier curve instead of converting the object to a non-LPE version (Object to path, ctrl+shift+c).

Steps to reproduce:

1. Make two Spiro splines where this and the underlying bezier curve are not equal.
2. Use a path operation.

or try my attached file.

Workaround:
Use Object to path (ctrl+shift+c) on all objects before using path operations.

Revision history for this message
Mark Jensen (mark-markjensen) wrote :
Revision history for this message
Mark Jensen (mark-markjensen) wrote :

Oh, and by the way. Inkscape 0.48 and Windows Vista

su_v (suv-lp)
tags: added: boolops livepatheffects
Revision history for this message
su_v (suv-lp) wrote :

Reproduced with Inkscape 0.47 and 0.48+devel r9736 on OS X 10.5.8.

Changed in inkscape:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
su_v (suv-lp) wrote :

Adding Johan's tag from the duplicate report bug #1041806 here as well so that this report is found by launchpad's normal searches (duplicate reports are hidden by default).

tags: added: easy-fix
Revision history for this message
Matthew Petroff (matthewpetroff) wrote :

The attached patch applies live path effects prior to performing boolean operations to fix the bug.

su_v (suv-lp)
Changed in inkscape:
assignee: nobody → Matthew Petroff (matthewpetroff)
status: Confirmed → In Progress
Revision history for this message
jazzynico (jazzynico) wrote :

Looks good on Debian testing, Inkscape trunk revision 12305.

tags: added: backport-proposed
Revision history for this message
Johan Engelen (johanengelen) wrote :

Why the check
   if (SP_IS_PATH(l->data) && SP_SHAPE(l->data)->_curve_before_lpe) {
instead of only checking for SP_IS_LPE_ITEM?

Revision history for this message
Matthew Petroff (matthewpetroff) wrote :

I didn't know SP_IS_LPE_ITEM exisited, else I would have used it. I've attached a revised patch.

Revision history for this message
jazzynico (jazzynico) wrote :

Patch from comment #8 tested successfully on Windows XP, Inkscape trunk revision 12312.

Changed in inkscape:
milestone: none → 0.49
Revision history for this message
su_v (suv-lp) wrote :

Patch from comment #8 tested successfully on OS X 10.7.5, Inkscape trunk revision 12312.

Revision history for this message
jazzynico (jazzynico) wrote :

Patch committed in the trunk, revision 12314.

Changed in inkscape:
status: In Progress → Fix Committed
Revision history for this message
su_v (suv-lp) wrote :

Attached backport of r12312 to the stable branch tested successfully with 0.48.x r9963 on OS X 10.7.5.

Please review (the revision didn't merge without conflicts into 0.48.x).

Revision history for this message
jazzynico (jazzynico) wrote :

Backport patch tested successfully on Windows XP, 0.48.x branch revision 9663.

Revision history for this message
su_v (suv-lp) wrote :

Backport committed to lp:inkscape/0.48.x in rev 9968

Changed in inkscape:
milestone: 0.49 → 0.48.5
Kris (kris-degussem)
tags: removed: backport-proposed
Alex Valavanis (valavanisalex)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.