[Geometry snapper] Fix progress bar not working

qgis · Jan 21, 2016 · b00950b · b00950b
1 parent 7afdecd
commit b00950b
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 2 deletions.
diff --git a/src/plugins/geometry_snapper/qgsgeometrysnapper.cpp b/src/plugins/geometry_snapper/qgsgeometrysnapper.cpp
@@ -49,11 +49,13 @@ QgsGeometrySnapper::QgsGeometrySnapper( QgsVectorLayer *adjustLayer, QgsVectorLa
 
 QFuture<void> QgsGeometrySnapper::processFeatures()
 {
+  emit progressRangeChanged( 0, mFeatures.size() );
   return QtConcurrent::map( mFeatures, ProcessFeatureWrapper( this ) );
 }
 
 void QgsGeometrySnapper::processFeature( QgsFeatureId id )
 {
+  emit progressStep();
   // Get current feature
   QgsFeature feature;
   if ( !getFeature( mAdjustLayer, mAdjustLayerMutex, id, feature ) )

diff --git a/src/plugins/geometry_snapper/qgsgeometrysnapper.h b/src/plugins/geometry_snapper/qgsgeometrysnapper.h
@@ -35,6 +35,10 @@ class QgsGeometrySnapper : public QObject
     QFuture<void> processFeatures();
     const QStringList& getErrors() const { return mErrors; }
 
+  signals:
+    void progressRangeChanged( int min, int max );
+    void progressStep();
+
   private:
     struct ProcessFeatureWrapper
     {

diff --git a/src/plugins/geometry_snapper/qgsgeometrysnapperdialog.cpp b/src/plugins/geometry_snapper/qgsgeometrysnapperdialog.cpp
@@ -280,8 +280,6 @@ void QgsGeometrySnapperDialog::run()
   /** Run **/
   QEventLoop evLoop;
   QFutureWatcher<void> futureWatcher;
-  connect( &futureWatcher, SIGNAL( progressRangeChanged( int, int ) ), progressBar, SLOT( setRange( int, int ) ) );
-  connect( &futureWatcher, SIGNAL( progressValueChanged( int ) ), progressBar, SLOT( setValue( int ) ) );
   connect( &futureWatcher, SIGNAL( finished() ), &evLoop, SLOT( quit() ) );
   connect( buttonBox->button( QDialogButtonBox::Abort ), SIGNAL( clicked() ), &futureWatcher, SLOT( cancel() ) );
 
@@ -294,6 +292,8 @@ void QgsGeometrySnapperDialog::run()
   widgetInputs->setEnabled( false );
 
   QgsGeometrySnapper snapper( layer, referenceLayer, selectedOnly, doubleSpinBoxMaxDistance->value(), &mIface->mapCanvas()->mapSettings() );
+  connect( &snapper, SIGNAL( progressRangeChanged( int, int ) ), progressBar, SLOT( setRange( int, int ) ) );
+  connect( &snapper, SIGNAL( progressStep() ), this, SLOT( progressStep() ) );
   futureWatcher.setFuture( snapper.processFeatures() );
   evLoop.exec();
 
@@ -317,3 +317,8 @@ void QgsGeometrySnapperDialog::run()
   hide() ;
 }
 
+void QgsGeometrySnapperDialog::progressStep()
+{
+  progressBar->setValue( progressBar->value() + 1 );
+}
+
diff --git a/src/plugins/geometry_snapper/qgsgeometrysnapperdialog.h b/src/plugins/geometry_snapper/qgsgeometrysnapperdialog.h
@@ -46,6 +46,7 @@ class QgsGeometrySnapperDialog: public QDialog, private Ui::QgsGeometrySnapperDi
     void updateLayers();
     void validateInput();
     void selectOutputFile();
+    void progressStep();
 };
 
 #endif // QGS_GEOMETRY_SNAPPER_DIALOG_H