georeferencer: Fix parameter reporting when enabling/disabling GCPs.

Updates the model before emiting pointEnabled signal, so that downstream handlers work with up-to-date transform parameters. git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@13554 c8812cc2-4d05-0410-92ff-de0c093fc19c
qgis · May 23, 2010 · b7e1bdd · b7e1bdd
1 parent e3d87c8
commit b7e1bdd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 12 deletions.
diff --git a/src/plugins/georeferencer/qgsgcplistwidget.cpp b/src/plugins/georeferencer/qgsgcplistwidget.cpp
@@ -109,15 +109,14 @@ void QgsGCPListWidget::itemClicked( QModelIndex index )
     if ( item->checkState() == Qt::Checked )
     {
       p->setEnabled( true );
-      emit pointEnabled( p, index.row() );
     }
     else // Qt::Unchecked
     {
       p->setEnabled( false );
-      emit pointEnabled( p, index.row() );
     }
 
     mGCPListModel->updateModel();
+    emit pointEnabled( p, index.row() );
     adjustTableContent();
   }
 

diff --git a/src/plugins/georeferencer/qgsgeorefplugingui.cpp b/src/plugins/georeferencer/qgsgeorefplugingui.cpp
@@ -1345,16 +1345,6 @@ bool QgsGeorefPluginGui::writePDFReportFile( const QString& fileName, const QgsG
     parameterLabel->setSceneRect( QRectF( 2, composerMap->rect().bottom() + composerMap->transform().dy() + 5, composition->paperWidth(), 8 ) );
     parameterLabel->setFrame( false );
 
-    int nPointsEnabled = 0;
-    QgsGCPList::const_iterator gcpIt = mPoints.constBegin();
-    for ( ; gcpIt != mPoints.constEnd(); ++gcpIt )
-    {
-      if (( *gcpIt )->isEnabled() )
-      {
-        ++nPointsEnabled;
-      }
-    }
-
     //calculate mean error
     double meanError = 0;
     calculateMeanError( meanError );