Make avoid overlap work with more digitizing tools (fixes #50433)

src/app/qgsmaptoolmovefeature.cpp

@@ -15,6 +15,7 @@
 
 #include "qgisapp.h"
 #include "qgsadvanceddigitizingdockwidget.h"
+#include "qgsavoidintersectionsoperation.h"
 #include "qgsfeatureiterator.h"
 #include "qgsgeometry.h"
 #include "qgslogger.h"
@@ -217,6 +218,20 @@ void QgsMapToolMoveFeature::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
         request.setFilterFids( mMovedFeatures ).setNoAttributes();
         QgsFeatureIterator fi = vlayer->getFeatures( request );
         QgsFeature f;
+
+        QgsAvoidIntersectionsOperation avoidIntersections;
+        connect( &avoidIntersections, &QgsAvoidIntersectionsOperation::messageEmitted, this, &QgsMapTool::messageEmitted );
+
+        // when removing intersections don't check for intersections with selected features
+        QSet<QgsFeatureId> ignoreFeatureIds;
+        for ( const auto &feat : mMovedFeatures )
+        {
+          ignoreFeatureIds.insert( feat );
+        }
+
+        QHash<QgsVectorLayer *, QSet<QgsFeatureId> > ignoreFeatures;
+        ignoreFeatures.insert( vlayer, ignoreFeatureIds );
+
         while ( fi.nextFeature( f ) )
         {
           if ( !f.hasGeometry() )
@@ -227,6 +242,23 @@ void QgsMapToolMoveFeature::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
             continue;
 
           const QgsFeatureId id = f.id();
+
+          if ( vlayer->geometryType() == Qgis::GeometryType::Polygon )
+          {
+            const QgsAvoidIntersectionsOperation::Result res = avoidIntersections.apply( vlayer, id, geom, ignoreFeatures );
+
+            if ( res.operationResult == Qgis::GeometryOperationResult::InvalidInputGeometryType || geom.isEmpty() )
+            {
+              QString errorMessage = ( geom.isEmpty() ) ?
+                                     tr( "Resulting geometry would be empty" ) :
+                                     tr( "An error was reported during intersection removal" );
+
+              emit messageEmitted( errorMessage, Qgis::MessageLevel::Warning );
+              vlayer->destroyEditCommand();
+              return;
+            }
+          }
+
           vlayer->changeGeometry( id, geom );
 
           if ( QgsProject::instance()->topologicalEditing() )

src/app/qgsmaptooloffsetcurve.cpp

@@ -17,6 +17,7 @@
 #include <QGridLayout>
 #include <QLabel>
 
+#include "qgsavoidintersectionsoperation.h"
 #include "qgsdoublespinbox.h"
 #include "qgsfeatureiterator.h"
 #include "qgsmaptooloffsetcurve.h"
@@ -346,6 +347,28 @@ void QgsMapToolOffsetCurve::applyOffset( double offset, Qt::KeyboardModifiers mo
 
   destLayer->beginEditCommand( tr( "Offset curve" ) );
 
+  QgsAvoidIntersectionsOperation avoidIntersections;
+
+  connect( &avoidIntersections, &QgsAvoidIntersectionsOperation::messageEmitted, this, &QgsMapTool::messageEmitted );
+
+  QHash<QgsVectorLayer *, QSet<QgsFeatureId> > ignoreFeatures;
+  ignoreFeatures.insert( destLayer, QSet<QgsFeatureId> { mModifiedFeature } );
+
+  const QgsAvoidIntersectionsOperation::Result res = avoidIntersections.apply( destLayer, mModifiedFeature, mModifiedGeometry, ignoreFeatures );
+
+  if ( res.operationResult == Qgis::GeometryOperationResult::InvalidInputGeometryType || mModifiedGeometry.isEmpty() )
+  {
+    QString errorMessage = ( mModifiedGeometry.isEmpty() ) ?
+                           tr( "Resulting geometry would be empty" ) :
+                           tr( "An error was reported during intersection removal" );
+
+    emit messageEmitted( errorMessage, Qgis::MessageLevel::Warning );
+    destLayer->destroyEditCommand();
+    deleteRubberBandAndGeometry();
+    deleteUserInputWidget();
+    return;
+  }
+
   bool editOk = true;
   if ( !mCtrlHeldOnFirstClick && !( modifiers & Qt::ControlModifier ) )
   {

src/app/qgsmaptoolrotatefeature.cpp

@@ -23,6 +23,7 @@
 #include <cmath>
 
 #include "qgsadvanceddigitizingdockwidget.h"
+#include "qgsavoidintersectionsoperation.h"
 #include "qgsmaptoolrotatefeature.h"
 #include "qgsfeatureiterator.h"
 #include "qgsgeometry.h"
@@ -414,11 +415,44 @@ void QgsMapToolRotateFeature::applyRotation( double rotation )
   request.setFilterFids( mRotatedFeatures ).setNoAttributes();
   QgsFeatureIterator fi = vlayer->getFeatures( request );
   QgsFeature f;
+
+  QgsAvoidIntersectionsOperation avoidIntersections;
+  connect( &avoidIntersections, &QgsAvoidIntersectionsOperation::messageEmitted, this, &QgsMapTool::messageEmitted );
+
+  // when removing intersections don't check for intersections with selected features
+  QSet<QgsFeatureId> ignoreFeatureIds;
+  for ( const auto &feat : mRotatedFeatures )
+  {
+    ignoreFeatureIds.insert( feat );
+  }
+
+  QHash<QgsVectorLayer *, QSet<QgsFeatureId> > ignoreFeatures;
+  ignoreFeatures.insert( vlayer, ignoreFeatureIds );
+
   while ( fi.nextFeature( f ) )
   {
     const QgsFeatureId id = f.id();
     QgsGeometry geom = f.geometry();
     geom.rotate( mRotation, anchorPoint );
+
+    if ( vlayer->geometryType() == Qgis::GeometryType::Polygon )
+    {
+      const QgsAvoidIntersectionsOperation::Result res = avoidIntersections.apply( vlayer, id, geom, ignoreFeatures );
+
+      if ( res.operationResult == Qgis::GeometryOperationResult::InvalidInputGeometryType || geom.isEmpty() )
+      {
+        QString errorMessage = ( geom.isEmpty() ) ?
+                               tr( "Resulting geometry would be empty" ) :
+                               tr( "An error was reported during intersection removal" );
+
+        emit messageEmitted( errorMessage, Qgis::MessageLevel::Warning );
+        vlayer->destroyEditCommand();
+        deleteRotationWidget();
+        deleteRubberband();
+        return;
+      }
+    }
+
     vlayer->changeGeometry( id, geom );
   }
 

src/app/qgsmaptoolscalefeature.cpp

@@ -23,6 +23,7 @@
 #include <cmath>
 
 #include "qgsadvanceddigitizingdockwidget.h"
+#include "qgsavoidintersectionsoperation.h"
 #include "qgsmaptoolscalefeature.h"
 #include "qgsfeatureiterator.h"
 #include "qgsgeometry.h"
@@ -368,6 +369,20 @@ void QgsMapToolScaleFeature::applyScaling( double scale )
   request.setFilterFids( mScaledFeatures ).setNoAttributes();
   QgsFeatureIterator fi = vlayer->getFeatures( request );
   QgsFeature feat;
+
+  QgsAvoidIntersectionsOperation avoidIntersections;
+  connect( &avoidIntersections, &QgsAvoidIntersectionsOperation::messageEmitted, this, &QgsMapTool::messageEmitted );
+
+  // when removing intersections don't check for intersections with selected features
+  QSet<QgsFeatureId> ignoreFeatureIds;
+  for ( const auto &f : mScaledFeatures )
+  {
+    ignoreFeatureIds.insert( f );
+  }
+
+  QHash<QgsVectorLayer *, QSet<QgsFeatureId> > ignoreFeatures;
+  ignoreFeatures.insert( vlayer, ignoreFeatureIds );
+
   while ( fi.nextFeature( feat ) )
   {
     if ( !feat.hasGeometry() )
@@ -378,6 +393,25 @@ void QgsMapToolScaleFeature::applyScaling( double scale )
       continue;
 
     const QgsFeatureId id = feat.id();
+
+    if ( vlayer->geometryType() == Qgis::GeometryType::Polygon )
+    {
+      const QgsAvoidIntersectionsOperation::Result res = avoidIntersections.apply( vlayer, id, geom, ignoreFeatures );
+
+      if ( res.operationResult == Qgis::GeometryOperationResult::InvalidInputGeometryType || geom.isEmpty() )
+      {
+        QString errorMessage = ( geom.isEmpty() ) ?
+                               tr( "Resulting geometry would be empty" ) :
+                               tr( "An error was reported during intersection removal" );
+
+        emit messageEmitted( errorMessage, Qgis::MessageLevel::Warning );
+        vlayer->destroyEditCommand();
+        deleteScalingWidget();
+        deleteRubberband();
+        return;
+      }
+    }
+
     vlayer->changeGeometry( id, geom );
   }