Prevent undo/redo history when layer is being tracked

src/core/featurehistory.cpp

@@ -4,9 +4,11 @@
 #include <qgsvectorlayer.h>
 #include <qgsvectorlayereditbuffer.h>
 
+#include <tracker.h>
 
-FeatureHistory::FeatureHistory( const QgsProject *project )
+FeatureHistory::FeatureHistory( const QgsProject *project, TrackingModel *trackingModel )
   : mProject( project )
+  , mTrackingModel( trackingModel )
 {
   connect( mProject, &QgsProject::homePathChanged, this, &FeatureHistory::onHomePathChanged );
   connect( mProject, &QgsProject::layersAdded, this, &FeatureHistory::onLayersAdded );
@@ -72,6 +74,12 @@ void FeatureHistory::onBeforeCommitChanges()
     return;
   }
 
+  Tracker *layerTracker = mTrackingModel->trackerForLayer( vl );
+  if ( layerTracker && layerTracker->isActive() )
+  {
+    return;
+  }
+
   QgsVectorLayerEditBuffer *eb = vl->editBuffer();
 
   if ( !eb )
@@ -117,13 +125,19 @@ void FeatureHistory::onCommittedFeaturesAdded( const QString &localLayerId, cons
     return;
   }
 
-  const QgsVectorLayer *vl = qobject_cast<QgsVectorLayer *>( sender() );
+  QgsVectorLayer *vl = qobject_cast<QgsVectorLayer *>( sender() );
 
   if ( !vl )
   {
     return;
   }
 
+  Tracker *layerTracker = mTrackingModel->trackerForLayer( vl );
+  if ( layerTracker && layerTracker->isActive() )
+  {
+    return;
+  }
+
   qDebug() << "FeatureHistory::onCommittedFeaturesAdded: adding create committed features";
 
   FeatureModifications modifications = mTempHistoryStep.take( vl->id() );
@@ -160,6 +174,12 @@ void FeatureHistory::onAfterCommitChanges()
     return;
   }
 
+  Tracker *layerTracker = mTrackingModel->trackerForLayer( vl );
+  if ( layerTracker && layerTracker->isActive() )
+  {
+    return;
+  }
+
   const QString layerId = vl->id();
   FeatureModifications modifications = mTempHistoryStep.take( layerId );
   QMap<QgsFeatureId, QgsFeature> modifiedFeaturesOld = mTempModifiedFeaturesByLayerId.take( layerId );

src/core/featurehistory.h

@@ -5,9 +5,11 @@
 #include <QTimer>
 #include <qgsproject.h>
 
+#include <trackingmodel.h>
 
 typedef QPair<QgsFeature, QgsFeature> OldNewFeaturePair;
 
+
 class FeatureHistory : public QObject
 {
     Q_OBJECT
@@ -34,7 +36,7 @@ class FeatureHistory : public QObject
      *
      * @param project
      */
-    explicit FeatureHistory( const QgsProject *project );
+    explicit FeatureHistory( const QgsProject *project, TrackingModel *trackingModel );
 
     //! Perform undo of the most recent modification step
     Q_INVOKABLE bool undo();
@@ -96,6 +98,9 @@ class FeatureHistory : public QObject
     //! The current project instance.
     const QgsProject *mProject = nullptr;
 
+    //! Tracking model. Used to check if the currently modified layer is in tracking, so all changes should be ignored to prevent cluttering the undo history.
+    TrackingModel *mTrackingModel = nullptr;
+
     //! If currently applying undo or redo feature modifications.
     bool mIsApplyingModifications = false;
 

src/core/qgismobileapp.cpp

@@ -269,7 +269,7 @@ QgisMobileapp::QgisMobileapp( QgsApplication *app, QObject *parent )
   mTrackingModel = new TrackingModel();
   mGpkgFlusher = std::make_unique<QgsGpkgFlusher>( mProject );
   mLayerObserver = std::make_unique<LayerObserver>( mProject );
-  mFeatureHistory = std::make_unique<FeatureHistory>( mProject );
+  mFeatureHistory = std::make_unique<FeatureHistory>( mProject, mTrackingModel );
   mFlatLayerTree = new FlatLayerTreeModel( mProject->layerTreeRoot(), mProject, this );
   mLegendImageProvider = new LegendImageProvider( mFlatLayerTree->layerTreeModel() );
   mLocalFilesImageProvider = new LocalFilesImageProvider();