Spiro and path operations fails

Bug #629081 reported by Mark Jensen on 2010-09-02
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Low
Matthew Petroff

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.

Related branches

Mark Jensen (mark-markjensen) wrote :
Mark Jensen (mark-markjensen) wrote :

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

~suv (suv-lp) on 2010-09-02
tags: added: boolops livepatheffects
~suv (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
~suv (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

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

~suv (suv-lp) on 2013-04-26
Changed in inkscape:
assignee: nobody → Matthew Petroff (matthewpetroff)
status: Confirmed → In Progress
jazzynico (jazzynico) wrote :

Looks good on Debian testing, Inkscape trunk revision 12305.

tags: added: backport-proposed
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?

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

jazzynico (jazzynico) wrote :

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

Changed in inkscape:
milestone: none → 0.49
~suv (suv-lp) wrote :

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

jazzynico (jazzynico) wrote :

Patch committed in the trunk, revision 12314.

Changed in inkscape:
status: In Progress → Fix Committed
~suv (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).

jazzynico (jazzynico) wrote :

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

~suv (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) on 2013-12-11
tags: removed: backport-proposed
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Bug attachments