Disable current cell on editing off

signedav · signedav · commit 8a47643aa797 · 2018-12-04T07:40:45.000+01:00
because otherwise on single cell change the change would not been stored in mEditBuffer fixes #15975 (cherry-picked from 51ee713)
diff --git a/src/app/qgsattributetabledialog.cpp b/src/app/qgsattributetabledialog.cpp
@@ -874,6 +874,12 @@ void QgsAttributeTableDialog::mActionToggleEditing_toggled( bool )
 {
   if ( !mLayer )
     return;
+
+  //this has to be done, because in case only one cell has been changed and is still enabled, the change
+  //would not be added to the mEditBuffer. By disabling, it looses focus and the change will be stored.
+  if ( mLayer->isEditable() && mMainView->tableView()->indexWidget( mMainView->tableView()->currentIndex() ) )
+    mMainView->tableView()->indexWidget( mMainView->tableView()->currentIndex() )->setEnabled( false );
+
   if ( !QgisApp::instance()->toggleEditing( mLayer ) )
   {
     // restore gui state if toggling was canceled or layer commit/rollback failed

Original file line number	Diff line number	Diff line change
`@@ -874,6 +874,12 @@ void QgsAttributeTableDialog::mActionToggleEditing_toggled( bool )`
`874`	`874`	`{`
`875`	`875`	`if ( !mLayer )`
`876`	`876`	`return;`
	`877`	`+`
	`878`	`+ //this has to be done, because in case only one cell has been changed and is still enabled, the change`
	`879`	`+ //would not be added to the mEditBuffer. By disabling, it looses focus and the change will be stored.`
	`880`	`+ if ( mLayer->isEditable() && mMainView->tableView()->indexWidget( mMainView->tableView()->currentIndex() ) )`
	`881`	`+ mMainView->tableView()->indexWidget( mMainView->tableView()->currentIndex() )->setEnabled( false );`
	`882`	`+`
`877`	`883`	`if ( !QgisApp::instance()->toggleEditing( mLayer ) )`
`878`	`884`	`{`
`879`	`885`	`// restore gui state if toggling was canceled or layer commit/rollback failed`