Fix setting the current GDAL profile options temporarily overwrites

the definition of one of the stored GDAL profiles Fixes #41378 Fixes #40235
qgis · Feb 9, 2021 · 7ff0187 · 7ff0187
1 parent ccd4fb6
commit 7ff0187
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/src/gui/qgsrasterformatsaveoptionswidget.cpp b/src/gui/qgsrasterformatsaveoptionswidget.cpp
@@ -217,6 +217,7 @@ void QgsRasterFormatSaveOptionsWidget::updateProfiles()
 
 void QgsRasterFormatSaveOptionsWidget::updateOptions()
 {
+  mBlockOptionUpdates++;
   QString myOptions = mOptionsMap.value( currentProfileKey() );
   QStringList myOptionsList = myOptions.trimmed().split( ' ', QString::SkipEmptyParts );
 
@@ -248,6 +249,7 @@ void QgsRasterFormatSaveOptionsWidget::updateOptions()
     mOptionsLineEdit->setCursorPosition( 0 );
   }
 
+  mBlockOptionUpdates--;
   emit optionsChanged();
 }
 
@@ -363,6 +365,9 @@ QString QgsRasterFormatSaveOptionsWidget::validateOptions( bool gui, bool report
 
 void QgsRasterFormatSaveOptionsWidget::optionsTableChanged()
 {
+  if ( mBlockOptionUpdates )
+    return;
+
   QTableWidgetItem *key, *value;
   QString options;
   for ( int i = 0; i < mOptionsTable->rowCount(); i++ )
@@ -591,30 +596,32 @@ void QgsRasterFormatSaveOptionsWidget::showEvent( QShowEvent *event )
 
 void QgsRasterFormatSaveOptionsWidget::setOptions( const QString &options )
 {
-  mOptionsTable->blockSignals( true );
+  mBlockOptionUpdates++;
   mOptionsTable->clearContents();
 
-  QStringList values;
-  QStringList optionsList = options.trimmed().split( ' ', QString::SkipEmptyParts );
-  const auto constOptionsList = optionsList;
-  for ( const QString &opt : constOptionsList )
+  const QStringList optionsList = options.trimmed().split( ' ', QString::SkipEmptyParts );
+  for ( const QString &opt : optionsList )
   {
     int rowCount = mOptionsTable->rowCount();
     mOptionsTable->insertRow( rowCount );
 
-    values = opt.split( '=' );
+    const QStringList values = opt.split( '=' );
     if ( values.count() == 2 )
     {
       QTableWidgetItem *nameItem = new QTableWidgetItem( values.at( 0 ) );
       mOptionsTable->setItem( rowCount, 0, nameItem );
       QTableWidgetItem *valueItem = new QTableWidgetItem( values.at( 1 ) );
-      mOptionsTable->setItem( rowCount, 0, valueItem );
+      mOptionsTable->setItem( rowCount, 1, valueItem );
     }
   }
 
+  // reset to no profile index, otherwise we are changing the definition of whichever profile
+  // is currently selected...
+  mProfileComboBox->setCurrentIndex( 0 );
+
   mOptionsMap[ currentProfileKey()] = options.trimmed();
   mOptionsLineEdit->setText( options.trimmed() );
   mOptionsLineEdit->setCursorPosition( 0 );
 
-  mOptionsTable->blockSignals( false );
+  mBlockOptionUpdates--;
 }
diff --git a/src/gui/qgsrasterformatsaveoptionswidget.h b/src/gui/qgsrasterformatsaveoptionswidget.h
@@ -145,6 +145,7 @@ class GUI_EXPORT QgsRasterFormatSaveOptionsWidget: public QWidget, private Ui::Q
     static QMap< QString, QStringList > sBuiltinProfiles;
     bool mPyramids = false;
     QgsRaster::RasterPyramidsFormat mPyramidsFormat = QgsRaster::PyramidsGTiff;
+    int mBlockOptionUpdates = 0;
 
     QString settingsKey( QString profile ) const SIP_FORCE;
     QString currentProfileKey() const SIP_FORCE;