Fix crash in attributetable when removing more than one column

signedav · m-kuhn · commit fd9213ce7223 · 2017-07-18T09:59:21.000+02:00
References #16746
diff --git a/src/gui/attributetable/qgsattributetablefiltermodel.cpp b/src/gui/attributetable/qgsattributetablefiltermodel.cpp
@@ -197,7 +197,8 @@ void QgsAttributeTableFilterModel::setAttributeTableConfig( const QgsAttributeTa
     {
       if ( newColumnMapping.size() == mColumnMapping.size() - removedColumnCount )
       {
-        beginRemoveColumns( QModelIndex(), firstRemovedColumn, firstRemovedColumn );
+        //the amount of removed column in the model need to be equal removedColumnCount
+        beginRemoveColumns( QModelIndex(), firstRemovedColumn, firstRemovedColumn+removedColumnCount-1 );
         mColumnMapping = newColumnMapping;
         endRemoveColumns();
       }

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,8 @@ void QgsAttributeTableFilterModel::setAttributeTableConfig( const QgsAttributeTa`
`197`	`197`	`{`
`198`	`198`	`if ( newColumnMapping.size() == mColumnMapping.size() - removedColumnCount )`
`199`	`199`	`{`
`200`		`- beginRemoveColumns( QModelIndex(), firstRemovedColumn, firstRemovedColumn );`
	`200`	`+ //the amount of removed column in the model need to be equal removedColumnCount`
	`201`	`+ beginRemoveColumns( QModelIndex(), firstRemovedColumn, firstRemovedColumn+removedColumnCount-1 );`
`201`	`202`	`mColumnMapping = newColumnMapping;`
`202`	`203`	`endRemoveColumns();`
`203`	`204`	`}`