Skip to content

Commit 9ce55ca

Browse files
committed
[postgres][db2] still save username when instructed too also with authcfg (backport 4ad3531)
1 parent 4009e4f commit 9ce55ca

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/providers/db2/qgsdb2newconnection.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,9 +120,9 @@ void QgsDb2NewConnection::accept()
120120
settings.setValue( baseKey + "/port", txtPort->text() );
121121
settings.setValue( baseKey + "/driver", txtDriver->text() );
122122
settings.setValue( baseKey + "/database", txtDatabase->text() );
123-
settings.setValue( baseKey + "/username", chkStoreUsername->isChecked() && !hasAuthConfigID ? txtUsername->text() : "" );
123+
settings.setValue( baseKey + "/username", chkStoreUsername->isChecked() ? txtUsername->text() : "" );
124124
settings.setValue( baseKey + "/password", chkStorePassword->isChecked() && !hasAuthConfigID ? txtPassword->text() : "" );
125-
settings.setValue( baseKey + "/saveUsername", chkStoreUsername->isChecked() && !hasAuthConfigID ? "true" : "false" );
125+
settings.setValue( baseKey + "/saveUsername", chkStoreUsername->isChecked() ? "true" : "false" );
126126
settings.setValue( baseKey + "/savePassword", chkStorePassword->isChecked() && !hasAuthConfigID ? "true" : "false" );
127127
settings.setValue( baseKey + "/authcfg", mAuthConfigSelect->configId() );
128128

src/providers/postgres/qgspgnewconnection.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,15 +146,15 @@ void QgsPgNewConnection::accept()
146146
settings.setValue( baseKey + "/host", txtHost->text() );
147147
settings.setValue( baseKey + "/port", txtPort->text() );
148148
settings.setValue( baseKey + "/database", txtDatabase->text() );
149-
settings.setValue( baseKey + "/username", chkStoreUsername->isChecked() && !hasAuthConfigID ? txtUsername->text() : "" );
149+
settings.setValue( baseKey + "/username", chkStoreUsername->isChecked() ? txtUsername->text() : "" );
150150
settings.setValue( baseKey + "/password", chkStorePassword->isChecked() && !hasAuthConfigID ? txtPassword->text() : "" );
151151
settings.setValue( baseKey + "/authcfg", mAuthConfigSelect->configId() );
152152
settings.setValue( baseKey + "/publicOnly", cb_publicSchemaOnly->isChecked() );
153153
settings.setValue( baseKey + "/geometryColumnsOnly", cb_geometryColumnsOnly->isChecked() );
154154
settings.setValue( baseKey + "/dontResolveType", cb_dontResolveType->isChecked() );
155155
settings.setValue( baseKey + "/allowGeometrylessTables", cb_allowGeometrylessTables->isChecked() );
156156
settings.setValue( baseKey + "/sslmode", cbxSSLmode->itemData( cbxSSLmode->currentIndex() ).toInt() );
157-
settings.setValue( baseKey + "/saveUsername", chkStoreUsername->isChecked() && !hasAuthConfigID ? "true" : "false" );
157+
settings.setValue( baseKey + "/saveUsername", chkStoreUsername->isChecked() ? "true" : "false" );
158158
settings.setValue( baseKey + "/savePassword", chkStorePassword->isChecked() && !hasAuthConfigID ? "true" : "false" );
159159
settings.setValue( baseKey + "/estimatedMetadata", cb_useEstimatedMetadata->isChecked() );
160160

0 commit comments

Comments
 (0)