Merge pull request #5403 from boundlessgeo/issue_17187_backportFrom3

pblottiere · web-flow · commit 6d50b37c628e · 2017-10-19T17:43:23.000+01:00
[bugfix] Attribute table model not updated when a feature is changed. fixes #17280 #17187
diff --git a/src/gui/attributetable/qgsattributetablemodel.cpp b/src/gui/attributetable/qgsattributetablemodel.cpp
@@ -289,7 +289,8 @@ void QgsAttributeTableModel::attributeValueChanged( QgsFeatureId fid, int idx, c
   // No filter request: skip all possibly heavy checks
   if ( mFeatureRequest.filterType() == QgsFeatureRequest::FilterNone )
   {
-    setData( index( idToRow( fid ), fieldCol( idx ) ), value, Qt::EditRole );
+    if ( loadFeatureAtId( fid ) )
+      setData( index( idToRow( fid ), fieldCol( idx ) ), value, Qt::EditRole );
   }
   else
   {

Original file line number	Diff line number	Diff line change
`@@ -289,7 +289,8 @@ void QgsAttributeTableModel::attributeValueChanged( QgsFeatureId fid, int idx, c`
`289`	`289`	`// No filter request: skip all possibly heavy checks`
`290`	`290`	`if ( mFeatureRequest.filterType() == QgsFeatureRequest::FilterNone )`
`291`	`291`	`{`
`292`		`- setData( index( idToRow( fid ), fieldCol( idx ) ), value, Qt::EditRole );`
	`292`	`+ if ( loadFeatureAtId( fid ) )`
	`293`	`+ setData( index( idToRow( fid ), fieldCol( idx ) ), value, Qt::EditRole );`
`293`	`294`	`}`
`294`	`295`	`else`
`295`	`296`	`{`