Add topological points on background layers for qgsmaptooladdfeature

qgis · Nov 20, 2019 · 47bc117 · 47bc117
1 parent bcdbc3a
commit 47bc117
Showing 1 changed file with 23 additions and 11 deletions.
diff --git a/src/app/qgsmaptooladdfeature.cpp b/src/app/qgsmaptooladdfeature.cpp
@@ -65,28 +65,40 @@ void QgsMapToolAddFeature::digitized( const QgsFeature &f )
   QgsVectorLayer *vlayer = currentVectorLayer();
   bool res = addFeature( vlayer, f, false );
 
-  if ( res && ( mode() == CaptureLine || mode() == CapturePolygon ) )
+  if ( res )
   {
     //add points to other features to keep topology up-to-date
     bool topologicalEditing = QgsProject::instance()->topologicalEditing();
+    if ( mode() == CaptureLine || mode() == CapturePolygon )
+    {
 
-    //use always topological editing for avoidIntersection.
-    //Otherwise, no way to guarantee the geometries don't have a small gap in between.
-    const QList<QgsVectorLayer *> intersectionLayers = QgsProject::instance()->avoidIntersectionsLayers();
+      //use always topological editing for avoidIntersection.
+      //Otherwise, no way to guarantee the geometries don't have a small gap in between.
+      const QList<QgsVectorLayer *> intersectionLayers = QgsProject::instance()->avoidIntersectionsLayers();
 
-    if ( !intersectionLayers.isEmpty() ) //try to add topological points also to background layers
-    {
-      for ( QgsVectorLayer *vl : intersectionLayers )
+      if ( !intersectionLayers.isEmpty() ) //try to add topological points also to background layers
       {
-        //can only add topological points if background layer is editable...
-        if ( vl->geometryType() == QgsWkbTypes::PolygonGeometry && vl->isEditable() )
+        for ( QgsVectorLayer *vl : intersectionLayers )
         {
-          vl->addTopologicalPoints( f.geometry() );
+          //can only add topological points if background layer is editable...
+          if ( vl->geometryType() == QgsWkbTypes::PolygonGeometry && vl->isEditable() )
+          {
+            vl->addTopologicalPoints( f.geometry() );
+          }
         }
       }
     }
-    else if ( topologicalEditing )
+    if ( topologicalEditing )
     {
+      QList<QgsPointLocator::Match> sm = snappingMatches();
+      Q_ASSERT( f.geometry().constGet()->vertexCount() == sm.size() );
+      for ( int i = 0; i < sm.size() ; ++i )
+      {
+        if ( sm.at( i ).layer() )
+        {
+          sm.at( i ).layer()->addTopologicalPoints( f.geometry().vertexAt( i ) );
+        }
+      }
       vlayer->addTopologicalPoints( f.geometry() );
     }
   }