From 5151c0569bb266461c3b6a6325c0d4be60b41a67 Mon Sep 17 00:00:00 2001
From: Jabiertxof <jtx@jtx>
Date: Mon, 29 May 2017 02:09:29 +0200
Subject: Fix bug #1694111 also fixes noumerous bugfixes on LPE undo. And
 shapes are improved to only perform path effet one time each

Fixed bugs:
  - https://launchpad.net/bugs/1694111

(bzr r15713)
---
 src/live_effects/effect.cpp | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

(limited to 'src/live_effects/effect.cpp')

diff --git a/src/live_effects/effect.cpp b/src/live_effects/effect.cpp
index 3a628b243..21136353c 100644
--- a/src/live_effects/effect.cpp
+++ b/src/live_effects/effect.cpp
@@ -452,10 +452,13 @@ Effect::processObjects(LpeAction lpe_action)
             std::vector<SPItem*> item_selected;
             SPCSSAttr *css;
             Glib::ustring css_str;
+            SPItem *item = SP_ITEM(elemref);
             switch (lpe_action){
             case LPE_TO_OBJECTS:
-                if (SP_ITEM(elemref)->isHidden()) {
-                    elemref->deleteObject();
+                if (item->isHidden()) {
+                    sp_object_ref(item, 0 );
+                    item->deleteObject(true);
+                    sp_object_unref(item);
                 } else {
                     if (elemnode->attribute("inkscape:path-effect")) {
                         sp_item_list_to_curves(item_list, item_selected, item_to_select);
@@ -465,7 +468,9 @@ Effect::processObjects(LpeAction lpe_action)
                 break;
 
             case LPE_ERASE:
-                elemref->deleteObject();
+                sp_object_ref(item, 0 );
+                item->deleteObject(true);
+                sp_object_unref(item);
                 break;
 
             case LPE_VISIBILITY:
-- 
cgit v1.2.3