[QGIS Commit] r14007 - trunk/qgis/src/providers/wfs

[svn_qgis at osgeo.org]

Author: mhugent
Date: 2010-08-04 15:21:06 +0000 (Wed, 04 Aug 2010)
New Revision: 14007

Modified:
   trunk/qgis/src/providers/wfs/qgswfsprovider.cpp
Log:
Don't add empty attribute fields to the insert statement in wfs-t (some servers complain about it)

Modified: trunk/qgis/src/providers/wfs/qgswfsprovider.cpp
===================================================================
--- trunk/qgis/src/providers/wfs/qgswfsprovider.cpp	2010-08-04 14:45:09 UTC (rev 14006)
+++ trunk/qgis/src/providers/wfs/qgswfsprovider.cpp	2010-08-04 15:21:06 UTC (rev 14007)
@@ -261,10 +261,14 @@
       QgsAttributeMap::const_iterator valueIt = featureAttributes.find( fieldIt.key() );
       if ( valueIt != featureAttributes.constEnd() )
       {
-        QDomElement fieldElem = transactionDoc.createElementNS( mWfsNamespace, fieldIt.value().name() );
-        QDomText fieldText = transactionDoc.createTextNode( valueIt.value().toString() );
-        fieldElem.appendChild( fieldText );
-        featureElem.appendChild( fieldElem );
+        QVariant fieldValue = valueIt.value();
+        if ( fieldValue.isValid() && !fieldValue.isNull() )
+        {
+          QDomElement fieldElem = transactionDoc.createElementNS( mWfsNamespace, fieldIt.value().name() );
+          QDomText fieldText = transactionDoc.createTextNode( fieldValue.toString() );
+          fieldElem.appendChild( fieldText );
+          featureElem.appendChild( fieldElem );
+        }
       }
     }
 
@@ -2003,6 +2007,7 @@
   QByteArray response = reply->readAll();
   reply->deleteLater();
   serverResponse.setContent( response, true );
+
   return true;
 }