Fix for bug #263620 comment 5,

Duplicating LPE path more like clone

1 files changed, 2 insertions, 12 deletions

diff --git a/src/object/sp-star.cpp b/src/object/sp-star.cpp
index a960d2018..b0c9a8197 100644
--- a/src/object/sp-star.cpp
+++ b/src/object/sp-star.cpp
@@ -494,21 +494,14 @@ void SPStar::snappoints(std::vector<Inkscape::SnapCandidatePoint> &p, Inkscape::
 Geom::Affine SPStar::set_transform(Geom::Affine const &xform)
 {
     bool opt_trans = (randomized == 0);
-    // TODO: try to remove CLONE_ORIGINAL from here
-    if (hasPathEffect() && pathEffectsEnabled() && 
-        this->hasPathEffectOfType(Inkscape::LivePathEffect::CLONE_ORIGINAL))
+    if (hasPathEffect() && pathEffectsEnabled())
     {
-        // if path has this LPE applied, don't write the transform to the pathdata, but write it 'unoptimized'
-        // also if the effect is type BEND PATH to fix bug #179842
+        // Inverse it to compensate
         this->adjust_livepatheffect(xform.inverse());
         return xform;
     }
     // Only set transform with proportional scaling
     if (!xform.withoutTranslation().isUniformScale()) {
-        // Adjust livepatheffect
-        if (hasPathEffect() && pathEffectsEnabled()) {
-            this->adjust_livepatheffect(xform.inverse());
-        }
         return xform;
     }
 
@@ -549,9 +542,6 @@ Geom::Affine SPStar::set_transform(Geom::Affine const &xform)
     // Adjust gradient fill
     this->adjust_gradient(xform * ret.inverse());
 
-    // Adjust livepatheffect
-    this->adjust_livepatheffect(xform);
-
     return ret;
 }