[Geometry checker] Don't recollect errors with empty recheck list

qgis · Oct 19, 2017 · f507246d3bcf6b9f5595980cbece605c88ab0f62 · f507246
1 parent 15b5da8
commit f507246d3bcf6b9f5595980cbece605c88ab0f62
Unified Split

Showing with 8 additions and 2 deletions.

+8 −2 src/analysis/vector/geometry_checker/qgsgeometrychecker.cpp
diff --git a/src/analysis/vector/geometry_checker/qgsgeometrychecker.cpp b/src/analysis/vector/geometry_checker/qgsgeometrychecker.cpp
@@ -179,11 +179,17 @@ bool QgsGeometryChecker::fixError( QgsGeometryCheckError *error, int method, boo
   {
     if ( check->getCheckType() == QgsGeometryCheck::LayerCheck )
     {
-      check->collectErrors( recheckErrors, mMessages, nullptr, recheckAreaFeatures );
+      if ( !recheckAreaFeatures.isEmpty() )
+      {
+        check->collectErrors( recheckErrors, mMessages, nullptr, recheckAreaFeatures );
+      }
     }
     else
     {
-      check->collectErrors( recheckErrors, mMessages, nullptr, recheckFeatures );
+      if ( !recheckFeatures.isEmpty() )
+      {
+        check->collectErrors( recheckErrors, mMessages, nullptr, recheckFeatures );
+      }
     }
   }