fixed a bug in SpatiaLite provider (crash following 'ALTER TABLE ADD COLUMN')

esseffe · esseffe · commit 70d7b842ab57 · 2009-09-25T14:24:52.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@11712 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/providers/spatialite/qgsspatialiteprovider.cpp b/src/providers/spatialite/qgsspatialiteprovider.cpp
@@ -130,6 +130,7 @@ void QgsSpatiaLiteProvider::loadFields()
   char *errMsg = NULL;
   QString pkName;
   int pkCount = 0;
+  int fldNo = 0;
   char xSql[1024];
 
   attributeFields.clear();
@@ -177,7 +178,7 @@ void QgsSpatiaLiteProvider::loadFields()
           fieldType = QVariant::Double;
         }
 
-        attributeFields.insert( i - 1, QgsField( name, fieldType, type, 0, 0, "" ) );
+        attributeFields.insert( fldNo++, QgsField( name, fieldType, type, 0, 0, "" ) );
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,7 @@ void QgsSpatiaLiteProvider::loadFields()`
`130`	`130`	`char *errMsg = NULL;`
`131`	`131`	`QString pkName;`
`132`	`132`	`int pkCount = 0;`
	`133`	`+ int fldNo = 0;`
`133`	`134`	`char xSql[1024];`
`134`	`135`
`135`	`136`	`attributeFields.clear();`
`@@ -177,7 +178,7 @@ void QgsSpatiaLiteProvider::loadFields()`
`177`	`178`	`fieldType = QVariant::Double;`
`178`	`179`	`}`
`179`	`180`
`180`		`- attributeFields.insert( i - 1, QgsField( name, fieldType, type, 0, 0, "" ) );`
	`181`	`+ attributeFields.insert( fldNo++, QgsField( name, fieldType, type, 0, 0, "" ) );`
`181`	`182`	`}`
`182`	`183`	`}`
`183`	`184`	`}`