Skip to content
Permalink
Browse files

leave last attribute empty instead of first

because though fid appears to be the first field it's added in the end and has the last index.

fixes #20276

(cherry-picked from d173b70)
  • Loading branch information
signedav committed Nov 22, 2018
1 parent 06d5545 commit 20ea3d6df164cbddc624c00c11c90b1989370363
Showing with 2 additions and 4 deletions.
  1. +2 −4 src/core/qgsofflineediting.cpp
@@ -763,11 +763,9 @@ QgsVectorLayer *QgsOfflineEditing::copyVectorLayer( QgsVectorLayer *layer, sqlit
// NOTE: SpatiaLite provider ignores position of geometry column
// fill gap in QgsAttributeMap if geometry column is not last (WORKAROUND)
QgsAttributes attrs = f.attributes();
int column = 0;
// on GPKG newAttrs has an addition FID attribute, so we have to add a dummy in the original set
if ( containerType == GPKG )
column++;
QgsAttributes newAttrs( attrs.count() + column );
QgsAttributes newAttrs( containerType == GPKG ? attrs.count() + 1 : attrs.count() );
int column = 0;
for ( int it = 0; it < attrs.count(); ++it )
{
newAttrs[column++] = attrs.at( it );

0 comments on commit 20ea3d6

Please sign in to comment.
You can’t perform that action at this time.