Fix attribute table remove row, Followup 8eca38c

m-kuhn · m-kuhn · commit a02e69481888 · 2015-05-28T11:25:51.000+02:00
Was an off-by-one error emitted by the attribute table model
diff --git a/src/gui/attributetable/qgsattributetablemodel.cpp b/src/gui/attributetable/qgsattributetablemodel.cpp
@@ -131,7 +131,7 @@ void QgsAttributeTableModel::featuresDeleted( QgsFeatureIds fids )
 
 bool QgsAttributeTableModel::removeRows( int row, int count, const QModelIndex &parent )
 {
-  beginRemoveRows( parent, row, row + count );
+  beginRemoveRows( parent, row, row + count - 1 );
 #ifdef QGISDEBUG
   if ( 3 > QgsLogger::debugLevel() )
     QgsDebugMsgLevel( QString( "remove %2 rows at %1 (rows %3, ids %4)" ).arg( row ).arg( count ).arg( mRowIdMap.size() ).arg( mIdRowMap.size() ), 3 );

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ void QgsAttributeTableModel::featuresDeleted( QgsFeatureIds fids )`
`131`	`131`
`132`	`132`	`bool QgsAttributeTableModel::removeRows( int row, int count, const QModelIndex &parent )`
`133`	`133`	`{`
`134`		`- beginRemoveRows( parent, row, row + count );`
	`134`	`+ beginRemoveRows( parent, row, row + count - 1 );`
`135`	`135`	`#ifdef QGISDEBUG`
`136`	`136`	`if ( 3 > QgsLogger::debugLevel() )`
`137`	`137`	`QgsDebugMsgLevel( QString( "remove %2 rows at %1 (rows %3, ids %4)" ).arg( row ).arg( count ).arg( mRowIdMap.size() ).arg( mIdRowMap.size() ), 3 );`