prevent potential misuse and remove duplicate nodes for some topologi…

…cal cases
qgis · Aug 13, 2019 · 5077a78 · 5077a78
1 parent cf08114
commit 5077a78
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/app/qgsmaptooltrimextendfeature.cpp b/src/app/qgsmaptooltrimextendfeature.cpp
@@ -109,14 +109,23 @@ void QgsMapToolTrimExtendFeature::canvasMoveEvent( QgsMapMouseEvent *e )
 
       QgsMapLayer *currentLayer = mCanvas->currentLayer();
       if ( !currentLayer )
+      {
+        mIsModified = false;
         break;
+      }
 
       mVlayer = qobject_cast<QgsVectorLayer *>( currentLayer );
       if ( !mVlayer )
+      {
+        mIsModified = false;
         break;
+      }
 
       if ( !mVlayer->isEditable() )
+      {
+        mIsModified = false;
         break;
+      }
 
       filter.setLayer( mVlayer );
       match = mCanvas->snappingUtils()->snapToMap( mMapPoint, &filter );
@@ -189,6 +198,7 @@ void QgsMapToolTrimExtendFeature::canvasMoveEvent( QgsMapMouseEvent *e )
 
           if ( mIsModified )
           {
+            mGeom.removeDuplicateNodes();
             mRubberBandExtend->setToGeometry( mGeom );
             mRubberBandExtend->show();
           }