Fix LPE for groups bounding box calculation by using the SPItem->getBounds method.

[inkscape.git] / src / live_effects / lpegroupbbox.cpp

diff --git a/src/live_effects/lpegroupbbox.cpp b/src/live_effects/lpegroupbbox.cpp
index 50f7599c0700d26f4a43b79b55f5159618fee4a4..f797d03582ef379acead9573222f6bc9a6443954 100644 (file)
--- a/src/live_effects/lpegroupbbox.cpp
+++ b/src/live_effects/lpegroupbbox.cpp
@@ -7,80 +7,34 @@
  */
 
 #include "live_effects/lpegroupbbox.h"
-#include "sp-shape.h"
+
 #include "sp-item.h"
-#include "sp-path.h"
-#include "sp-item-group.h"
+#include "libnr/nr-matrix-fns.h"
 #include "libnr/n-art-bpath-2geom.h"
-#include "svg/svg.h"
-#include "ui/widget/scalar.h"
-
-#include <2geom/sbasis.h>
-#include <2geom/sbasis-geometric.h>
-#include <2geom/bezier-to-sbasis.h>
-#include <2geom/sbasis-to-bezier.h>
-#include <2geom/d2.h>
-#include <2geom/piecewise.h>
-
-#include <algorithm>
-
-using std::vector;
 
 namespace Inkscape {
 namespace LivePathEffect {
 
 void
-GroupBBoxEffect::recursive_original_bbox(SPGroup *group, Geom::Piecewise<Geom::D2<Geom::SBasis> > & pwd2, std::vector<Geom::Path> & temppath)
+GroupBBoxEffect::original_bbox(SPLPEItem *lpeitem, bool absolute)
 {
-    std::vector<Geom::Path> tempsubpath;
-    GSList const *item_list = sp_item_group_item_list(group);
-
-    for ( GSList const *iter = item_list; iter; iter = iter->next )
-    {
-        SPObject *subitem = static_cast<SPObject *>(iter->data);
-        if (SP_IS_PATH(subitem))
-        {
-            //if there is not an original-d, just take the d
-            if(SP_OBJECT_REPR(subitem)->attribute("inkscape:original-d") != NULL)      
-                tempsubpath = SVGD_to_2GeomPath(SP_OBJECT_REPR(subitem)->attribute("inkscape:original-d"));
-            else
-                tempsubpath = SVGD_to_2GeomPath(SP_OBJECT_REPR(subitem)->attribute("d")); 
-            
-            temppath.insert(temppath.end(), tempsubpath.begin(), tempsubpath.end());
-        } 
-        else if (SP_IS_GROUP(subitem))
-        {
-            recursive_original_bbox(SP_GROUP(subitem), pwd2, temppath);
-        }
+    // Get item bounding box
+    SPItem* item = SP_ITEM(lpeitem);
+    
+    NR::Matrix transform;
+    if (absolute) {
+        transform = sp_item_i2doc_affine(item);
     }
-}
-
-void
-GroupBBoxEffect::original_bbox(SPLPEItem *lpeitem)
-{
-
-    using namespace Geom;
-    Piecewise<D2<SBasis> > pwd2;
-    std::vector<Geom::Path> temppath;  
-
-
-    if (SP_IS_PATH(lpeitem))
-    {
-    //TODO : this won't work well with LPE stacking
-        temppath = SVGD_to_2GeomPath( SP_OBJECT_REPR(lpeitem)->attribute("inkscape:original-d"));
-    }
-    else if (SP_IS_GROUP(lpeitem))
-    {
-        recursive_original_bbox(SP_GROUP(lpeitem), pwd2, temppath);
-    }
-
-    for (unsigned int i=0; i < temppath.size(); i++) {
-        pwd2.concat( temppath[i].toPwSb() );
+    else {
+        transform = NR::identity();
     }
+    
+    NR::Maybe<NR::Rect> itemBBox = item->getBounds(transform, SPItem::GEOMETRIC_BBOX);
 
-    D2<Piecewise<SBasis> > d2pw = make_cuts_independant(pwd2);
-    boundingbox_X = bounds_exact(d2pw[0]);
-    boundingbox_Y = bounds_exact(d2pw[1]);
+    // NR to Geom glue
+    Geom::Rect geomBBox = Geom::Rect(itemBBox->min().to_2geom(), itemBBox->max().to_2geom());
+    boundingbox_X = geomBBox[Geom::X];
+    boundingbox_Y = geomBBox[Geom::Y];
 }
 
 } // namespace LivePathEffect