Correctly invalidate single geometry results on delete

m-kuhn · m-kuhn · commit c0088f73057b · 2018-10-15T14:28:07.000+02:00
diff --git a/src/app/qgsgeometryvalidationservice.cpp b/src/app/qgsgeometryvalidationservice.cpp
@@ -127,6 +127,8 @@ void QgsGeometryValidationService::onFeatureDeleted( QgsVectorLayer *layer, QgsF
     invalidateTopologyChecks( layer );
   }
 
+  mLayerChecks[layer].singleFeatureCheckErrors.remove( fid );
+
   // There should be no geometry errors on an inexistent feature, right?
   emit geometryCheckCompleted( layer, fid, QList<std::shared_ptr<QgsSingleGeometryCheckError>>() );
 }

Original file line number	Diff line number	Diff line change
`@@ -127,6 +127,8 @@ void QgsGeometryValidationService::onFeatureDeleted( QgsVectorLayer *layer, QgsF`
`127`	`127`	`invalidateTopologyChecks( layer );`
`128`	`128`	`}`
`129`	`129`
	`130`	`+ mLayerChecks[layer].singleFeatureCheckErrors.remove( fid );`
	`131`	`+`
`130`	`132`	`// There should be no geometry errors on an inexistent feature, right?`
`131`	`133`	`emit geometryCheckCompleted( layer, fid, QList<std::shared_ptr<QgsSingleGeometryCheckError>>() );`
`132`	`134`	`}`