Tryton - Issues

 

Issue6875

Title Autocompletion does work for M2O in editable list when modifying an existing value
Priority bug Status resolved
Superseder Nosy List nicoe, reviewbot, roundup-bot
Type behavior Components tryton
Assigned To nicoe Keywords review
Reviews 40771002
View: 40771002

Created on 2017-10-23.14:47:14 by nicoe, last changed by roundup-bot.

Messages
New changeset 753e78d0582e by C├ędric Krier in branch 'default':
Catch the edition event of cell widgets to reset the M2O field value
http://hg.tryton.org/tryton/rev/753e78d0582e
review40771002 updated at https://codereview.tryton.org/40771002/#ps60001
review40771002 updated at https://codereview.tryton.org/40771002/#ps40001
review40771002 updated at https://codereview.tryton.org/40771002/#ps20001
review40771002 updated at https://codereview.tryton.org/40771002/#ps1
msg36417 (view) Author: [hidden] (nicoe) (Tryton committer) (Tryton translator) Date: 2017-10-23.14:47:14
When the user change an existing value of a M2O field in an editable list the autocompletion search for this field is not triggered.

This is due to the fact that the underlying value in the field is set thus the call to _update_completion does not make the call. This process works for M2O fields in the form view because we're catching the changed signal and resetting the value of the underlying field.

The attached review adds the "changed" signal mechanism to GenericText and implements a simple version of it for M2O fields.
History
Date User Action Args
2017-12-09 17:45:26roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg37181
2017-11-20 17:16:15reviewbotsetmessages: + msg36960
2017-11-20 16:45:35reviewbotsetmessages: + msg36958
2017-11-06 13:32:12reviewbotsetmessages: + msg36724
2017-10-23 15:10:16reviewbotsetnosy: + reviewbot
messages: + msg36418
2017-10-23 14:47:14nicoecreate

Showing 10 items. Show all history (warning: this could be VERY long)