Topology check integration in validation service

Author: m-kuhn

src/app/qgsgeometryvalidationservice.cpp

@@ -52,6 +52,10 @@ void QgsGeometryValidationService::onLayersAdded( const QList<QgsMapLayer *> &la
       {
         onFeatureDeleted( vectorLayer, fid );
       } );
+      connect( vectorLayer, &QgsVectorLayer::beforeCommitChanges, this, [this, vectorLayer]()
+      {
+        onBeforeCommitChanges( vectorLayer );
+      } );
 
       enableLayerChecks( vectorLayer );
     }
@@ -76,6 +80,14 @@ void QgsGeometryValidationService::onFeatureDeleted( QgsVectorLayer *layer, QgsF
   cancelChecks( layer, fid );
 }
 
+void QgsGeometryValidationService::onBeforeCommitChanges( QgsVectorLayer *layer )
+{
+  if ( !mTopologyChecksOk.value( layer ) )
+  {
+    triggerTopologyChecks( layer );
+  }
+}
+
 void QgsGeometryValidationService::enableLayerChecks( QgsVectorLayer *layer )
 {
   // TODO: finish all ongoing checks
@@ -145,7 +157,9 @@ void QgsGeometryValidationService::processFeature( QgsVectorLayer *layer, QgsFea
   }
 
   emit geometryCheckCompleted( layer, fid, allErrors );
-  // TODO: this is a bit hardcore
-  // const auto errors = mIsValidGeometryCheck->processGeometry( feature.geometry() );
-  // emit geometryCheckCompleted( layer, fid, errors );
+}
+
+void QgsGeometryValidationService::triggerTopologyChecks( QgsVectorLayer *layer )
+{
+
 }

src/app/qgsgeometryvalidationservice.h

@@ -71,6 +71,7 @@ class QgsGeometryValidationService : public QObject
     void onFeatureAdded( QgsVectorLayer *layer, QgsFeatureId fid );
     void onGeometryChanged( QgsVectorLayer *layer, QgsFeatureId fid, const QgsGeometry &geometry );
     void onFeatureDeleted( QgsVectorLayer *layer, QgsFeatureId fid );
+    void onBeforeCommitChanges( QgsVectorLayer *layer );
 
   private:
     void enableLayerChecks( QgsVectorLayer *layer );
@@ -79,9 +80,12 @@ class QgsGeometryValidationService : public QObject
 
     void processFeature( QgsVectorLayer *layer, QgsFeatureId fid );
 
+    void triggerTopologyChecks( QgsVectorLayer *layer );
+
     QgsProject *mProject = nullptr;
 
     QHash<QgsVectorLayer *, QList< QgsSingleGeometryCheck * > > mSingleFeatureChecks;
+    QHash<QgsVectorLayer *, bool > mTopologyChecksOk;
 };
 
 #endif // QGSGEOMETRYVALIDATIONSERVICE_H