[Geometry checker] Only add to feature pool spatial indices features …

…which are to be processed
qgis · Oct 23, 2017 · e089b3b · e089b3b
1 parent e00f767
commit e089b3b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/src/analysis/vector/geometry_checker/qgsfeaturepool.cpp b/src/analysis/vector/geometry_checker/qgsfeaturepool.cpp
@@ -48,7 +48,7 @@ QgsFeaturePool::QgsFeaturePool( QgsVectorLayer *layer, double layerToMapUnits, c
   QgsFeatureIterator it = layer->getFeatures( req );
   while ( it.nextFeature( feature ) )
   {
-    if ( feature.geometry() )
+    if ( mFeatureIds.contains( feature.id() ) && feature.geometry() )
     {
       mIndex.insertFeature( feature );
     }

diff --git a/src/analysis/vector/geometry_checker/qgsgeometrychecker.cpp b/src/analysis/vector/geometry_checker/qgsgeometrychecker.cpp
@@ -166,11 +166,6 @@ bool QgsGeometryChecker::fixError( QgsGeometryCheckError *error, int method, boo
     QgsFeaturePool *featurePool = mContext->featurePools[layerId];
     QgsCoordinateTransform t = QgsCoordinateTransformCache::instance()->transform( mContext->mapCrs, featurePool->getLayer()->crs().authid() );
     recheckAreaFeatures[layerId] = featurePool->getIntersects( t.transform( recheckArea ) );
-    // If only selected features were checked, confine the recheck areas to the selected features
-    if ( featurePool->getSelectedOnly() )
-    {
-      recheckAreaFeatures[layerId] = recheckAreaFeatures[layerId].intersect( featurePool->getLayer()->selectedFeatureIds() );
-    }
   }
 
   // Recheck feature / changed area to detect new errors