Only enable reset track button when a track exists

qgis · Nov 7, 2022 · 7ea1224 · 7ea1224
1 parent 5152259
commit 7ea1224
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 1 deletion.
diff --git a/src/app/gps/qgsappgpsdigitizing.cpp b/src/app/gps/qgsappgpsdigitizing.cpp
@@ -93,8 +93,9 @@ void QgsAppGpsDigitizing::addVertex()
   // we store the capture list in wgs84 and then transform to layer crs when
   // calling close feature
   const QgsPoint pointWgs84 = QgsPoint( mLastGpsPositionWgs84.x(), mLastGpsPositionWgs84.y(), mLastElevation );
-  mCaptureListWgs84.push_back( pointWgs84 );
 
+  const bool trackWasEmpty = mCaptureListWgs84.empty();
+  mCaptureListWgs84.push_back( pointWgs84 );
 
   // we store the rubber band points in map canvas CRS so transform to map crs
   // potential problem with transform errors and wrong coordinates if map CRS is changed after points are stored - SLM
@@ -118,14 +119,22 @@ void QgsAppGpsDigitizing::addVertex()
   }
 
   mRubberBand->addPoint( mapPoint );
+
+  if ( trackWasEmpty )
+    emit trackIsEmptyChanged( false );
 }
 
 void QgsAppGpsDigitizing::resetFeature()
 {
   mBlockGpsStateChanged++;
   createRubberBand(); //deletes existing rubberband
+
+  const bool trackWasEmpty = mCaptureListWgs84.isEmpty();
   mCaptureListWgs84.clear();
   mBlockGpsStateChanged--;
+
+  if ( !trackWasEmpty )
+    emit trackIsEmptyChanged( true );
 }
 
 void QgsAppGpsDigitizing::addFeature()
@@ -282,6 +291,7 @@ void QgsAppGpsDigitizing::addFeature()
 
       f.setGeometry( g );
       QgsFeatureAction action( tr( "Feature added" ), f, vlayer, QUuid(), -1, this );
+
       if ( action.addFeature( attrMap ) )
       {
         if ( QgsProject::instance()->gpsSettings()->automaticallyCommitFeatures() )
@@ -304,6 +314,10 @@ void QgsAppGpsDigitizing::addFeature()
       } // action.addFeature()
 
       mBlockGpsStateChanged--;
+
+      if ( mCaptureListWgs84.empty() )
+        emit trackIsEmptyChanged( true );
+
       break;
     }
 

diff --git a/src/app/gps/qgsappgpsdigitizing.h b/src/app/gps/qgsappgpsdigitizing.h
@@ -62,6 +62,11 @@ class APP_EXPORT QgsAppGpsDigitizing: public QObject
 
   signals:
 
+    /**
+     * Emitted whenever the current track changes from being empty to non-empty or vice versa.
+     */
+    void trackIsEmptyChanged( bool isEmpty );
+
     void timeStampDestinationChanged( const QString &fieldName );
 
   private slots:

diff --git a/src/app/gps/qgsgpstoolbar.cpp b/src/app/gps/qgsgpstoolbar.cpp
@@ -121,6 +121,7 @@ QgsGpsToolBar::QgsGpsToolBar( QgsAppGpsConnection *connection, QgsMapCanvas *can
   mRecenterAction->setEnabled( false );
   mAddFeatureAction->setEnabled( false );
   mAddTrackVertexAction->setEnabled( false );
+  mResetFeatureAction->setEnabled( false );
   connect( mConnection, &QgsAppGpsConnection::statusChanged, this, [ = ]( Qgis::GpsConnectionStatus status )
   {
     switch ( status )
@@ -170,6 +171,11 @@ void QgsGpsToolBar::setAddVertexButtonEnabled( bool enabled )
   mAddTrackVertexAction->setEnabled( mEnableAddVertexButton && mConnection->isConnected() );
 }
 
+void QgsGpsToolBar::setResetTrackButtonEnabled( bool enabled )
+{
+  mResetFeatureAction->setEnabled( enabled );
+}
+
 void QgsGpsToolBar::updateLocationLabel( const QgsPoint &point )
 {
   if ( point.isEmpty() )

diff --git a/src/app/gps/qgsgpstoolbar.h b/src/app/gps/qgsgpstoolbar.h
@@ -45,6 +45,7 @@ class QgsGpsToolBar : public QToolBar
   public slots:
 
     void setAddVertexButtonEnabled( bool enabled );
+    void setResetTrackButtonEnabled( bool enabled );
 
   private slots:
 

diff --git a/src/app/qgisapp.cpp b/src/app/qgisapp.cpp
@@ -1420,6 +1420,7 @@ QgisApp::QgisApp( QSplashScreen *splash, bool restorePlugins, bool skipBadLayers
   connect( mGpsSettingsMenu, &QgsAppGpsSettingsMenu::nmeaLogFileChanged, mGpsDigitizing, &QgsAppGpsDigitizing::setNmeaLogFile );
   connect( mGpsSettingsMenu, &QgsAppGpsSettingsMenu::timeStampDestinationChanged, mGpsDigitizing, &QgsAppGpsDigitizing::setTimeStampDestination );
   connect( mGpsDigitizing, &QgsAppGpsDigitizing::timeStampDestinationChanged, mGpsSettingsMenu, &QgsAppGpsSettingsMenu::setCurrentTimeStampField );
+  connect( mGpsDigitizing, &QgsAppGpsDigitizing::trackIsEmptyChanged, mGpsToolBar, [ = ]( bool isEmpty ) { mGpsToolBar->setResetTrackButtonEnabled( !isEmpty ); } );
 
   mpGpsWidget = new QgsGpsInformationWidget( mGpsConnection, mMapCanvas );
   QgsPanelWidgetStack *gpsStack = new QgsPanelWidgetStack();