Merge pull request #8692 from m-kuhn/fixSingleLayerErrors

Allow saving a layer after fixing invalid geometries

Author: m-kuhn

src/app/qgsgeometryvalidationservice.cpp

@@ -145,6 +145,10 @@ void QgsGeometryValidationService::onFeatureDeleted( QgsVectorLayer *layer, QgsF
 
 void QgsGeometryValidationService::onBeforeCommitChanges( QgsVectorLayer *layer )
 {
+  if ( mLayerChecks[layer].topologyChecks.empty() && !layer->allowCommit() )
+  {
+    showMessage( tr( "Geometry errors have been found. Please fix the errors before saving the layer." ) );
+  }
   if ( !mBypassChecks && !mLayerChecks[layer].topologyChecks.empty() )
   {
     if ( !layer->allowCommit() )
@@ -359,6 +363,8 @@ void QgsGeometryValidationService::processFeature( QgsVectorLayer *layer, QgsFea
 
   if ( !mLayerChecks[layer].singleFeatureCheckErrors.empty() )
     layer->setAllowCommit( false );
+  else if ( mLayerChecks[layer].topologyChecks.empty() )
+    layer->setAllowCommit( true );
 
   emit geometryCheckCompleted( layer, fid, allErrors );
 }