strip off superfluous parameters from WMS server url (for the common case that GetCapabilities request urls are advertised)

jef · jef · commit a0241d32ae51 · 2011-02-17T12:57:38.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@15183 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/app/qgsnewhttpconnection.cpp b/src/app/qgsnewhttpconnection.cpp
@@ -19,6 +19,7 @@
 #include "qgscontexthelp.h"
 #include <QSettings>
 #include <QMessageBox>
+#include <QUrl>
 
 QgsNewHttpConnection::QgsNewHttpConnection(
   QWidget *parent, const QString& baseKey, const QString& connName, Qt::WFlags fl ):
@@ -73,7 +74,21 @@ void QgsNewHttpConnection::accept()
     settings.remove( "/Qgis/WMS/" + mOriginalConnName );
   }
 
-  settings.setValue( key + "/url", txtUrl->text().trimmed() );
+  QUrl url( txtUrl->text().trimmed() );
+
+  QList< QPair<QByteArray, QByteArray> > params = url.encodedQueryItems();
+  for ( int i = 0; i < params.size(); i++ )
+  {
+    if ( params[i].first.toUpper() == "SERVICE" ||
+         params[i].first.toUpper() == "REQUEST" ||
+         params[i].first.toUpper() == "FORMAT" )
+    {
+      params.removeAt( i-- );
+    }
+  }
+  url.setEncodedQueryItems( params );
+
+  settings.setValue( key + "/url", url.toString() );
   settings.setValue( credentialsKey + "/username", txtUserName->text() );
   settings.setValue( credentialsKey + "/password", txtPassword->text() );
 
