[postgres][db2] username and authcfg are not mutually exclusive (foll…

…owup aae7a32)
qgis · Jul 4, 2018 · 47f6faa · 47f6faa
1 parent 53a25c7
commit 47f6faa
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/src/providers/db2/qgsdb2dataitems.cpp b/src/providers/db2/qgsdb2dataitems.cpp
@@ -80,22 +80,21 @@ bool QgsDb2ConnectionItem::ConnInfoFromParameters(
                + "dbname='" + database + "' ";
   }
 
+  if ( !username.isEmpty() )
+  {
+    connInfo += "user='" + username + "' ";
+  }
+
   if ( !authcfg.isEmpty() )
   {
     connInfo += "authcfg='" + authcfg + "' ";
   }
-  else  // include user and password if authcfg is empty
+  else  if ( !password.isEmpty() )
   {
-    if ( !username.isEmpty() )
+    // include user and password if authcfg is empty
-    {
+    connInfo += "password='" + password + "' ";
-      connInfo += "user='" + username + "' ";
-    }
-
-    if ( !password.isEmpty() )
-    {
-      connInfo += "password='" + password + "' ";
-    }
   }
+
   QgsDebugMsg( "connInfo: '" + connInfo + "'" );
   return true;
 }

diff --git a/src/providers/postgres/qgspostgresconn.cpp b/src/providers/postgres/qgspostgresconn.cpp
@@ -1790,7 +1790,6 @@ QgsDataSourceUri QgsPostgresConn::connUri( const QString &connName )
 
   if ( !authcfg.isEmpty() )
   {
-    username.clear();
     password.clear();
   }