[wfs] Proper handling of DCP POST Uri

qgis · Feb 25, 2018 · 9083ec6 · 9083ec6
1 parent ef24e65
commit 9083ec6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/providers/wfs/qgswfsdatasourceuri.cpp b/src/providers/wfs/qgswfsdatasourceuri.cpp
@@ -182,7 +182,7 @@ QUrl QgsWFSDataSourceURI::requestUrl( const QString &request, const Method &meth
   }
   QUrl url( endpoint );
   url.addQueryItem( QStringLiteral( "SERVICE" ), QStringLiteral( "WFS" ) );
-  if ( ! request.isEmpty() )
+  if ( method == Method::Get && ! request.isEmpty() )
     url.addQueryItem( QStringLiteral( "REQUEST" ), request );
   return url;
 }

diff --git a/src/providers/wfs/qgswfstransactionrequest.cpp b/src/providers/wfs/qgswfstransactionrequest.cpp
@@ -23,9 +23,7 @@ QgsWFSTransactionRequest::QgsWFSTransactionRequest( const QgsWFSDataSourceURI &u
 
 bool QgsWFSTransactionRequest::send( const QDomDocument &doc, QDomDocument &serverResponse )
 {
-  QUrl url( mUri.requestUrl( QStringLiteral( "Transaction" ) ) );
-  // We don't want/need REQUEST=Transaction
-  url.removeQueryItem( QStringLiteral( "REQUEST" ) );
+  QUrl url( mUri.requestUrl( QStringLiteral( "Transaction" ), QgsWFSDataSourceURI::Method::Post ) );
 
   QgsDebugMsgLevel( doc.toString(), 4 );