Fix for smarter attribute insertion in QgsOgrProvider::addFeature. Th…

…is is e.g. usefull for copying/pasting features git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@5719 c8812cc2-4d05-0410-92ff-de0c093fc19c
qgis · Aug 21, 2006 · 42dcd4a · 42dcd4a
1 parent d4626da
commit 42dcd4a
Showing 1 changed file with 15 additions and 9 deletions.
diff --git a/src/providers/ogr/qgsogrprovider.cpp b/src/providers/ogr/qgsogrprovider.cpp
@@ -1027,26 +1027,32 @@ bool QgsOgrProvider::addFeature(QgsFeature* f)
       }
   }
 
-  //todo: choose i such that it doesn't get larger than fdef->GetFieldCount()
-  //todo: only add the attribute if it has the same name as fdef->GetFieldDefn(i)->GetNameRef()
-
   //add possible attribute information
   for(int i=0;i<f->attributeMap().size();++i)
   {
     QString s=(f->attributeMap())[i].fieldValue();
+
+    //find a matching field for the new attribute
+    QString newAttribute = (f->attributeMap())[i].fieldName();
+    int targetAttributeId = fdef->GetFieldIndex(newAttribute);
+    if(targetAttributeId == -1)
+      {
+	continue;
+      }
+
     if(!s.isEmpty())
     {
-      if(fdef->GetFieldDefn(i)->GetType()==OFTInteger)
+      if(fdef->GetFieldDefn(targetAttributeId)->GetType()==OFTInteger)
       {
-        feature->SetField(i,s.toInt());
+        feature->SetField(targetAttributeId,s.toInt());
       }
-      else if(fdef->GetFieldDefn(i)->GetType()==OFTReal)
+      else if(fdef->GetFieldDefn(targetAttributeId)->GetType()==OFTReal)
       {
-        feature->SetField(i,s.toDouble());
+        feature->SetField(targetAttributeId,s.toDouble());
       }
-      else if(fdef->GetFieldDefn(i)->GetType()==OFTString)
+      else if(fdef->GetFieldDefn(targetAttributeId)->GetType()==OFTString)
       {
-	  feature->SetField(i,s.ascii());
+	  feature->SetField(targetAttributeId,s.ascii());
       }
       else
       {