From 89b763d897e805a577f81dc0dccab4b75eb253cf Mon Sep 17 00:00:00 2001
From: Jabier Arraiza <jabier.arraiza@marker.es>
Date: Mon, 1 Oct 2018 03:10:05 +0200
Subject: Improvements to transforms in LPE

---
 src/object/sp-star.cpp | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

(limited to 'src/object/sp-star.cpp')

diff --git a/src/object/sp-star.cpp b/src/object/sp-star.cpp
index 40d38d4a8..a960d2018 100644
--- a/src/object/sp-star.cpp
+++ b/src/object/sp-star.cpp
@@ -494,23 +494,20 @@ 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) || 
-         this->hasPathEffectOfType(Inkscape::LivePathEffect::BEND_PATH) || 
-         this->hasPathEffectOfType(Inkscape::LivePathEffect::POWERCLIP) || 
-         this->hasPathEffectOfType(Inkscape::LivePathEffect::FILL_BETWEEN_MANY) ||
-         this->hasPathEffectOfType(Inkscape::LivePathEffect::FILL_BETWEEN_STROKES) ) )
+        this->hasPathEffectOfType(Inkscape::LivePathEffect::CLONE_ORIGINAL))
     {
         // 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
-        this->adjust_livepatheffect(xform);
+        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);
+            this->adjust_livepatheffect(xform.inverse());
         }
         return xform;
     }
-- 
cgit v1.2.3