Issue 11059

Title
Remove SignalEvent
Priority
feature
Status
resolved
Nosy list
acaubet, ced, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2021-12-15.17:00:45 by ced, last changed 2 months ago by roundup-bot.

Messages

New changeset b55eb1e83f3a by Cédric Krier in branch 'default':
Restore test of modified_fields with isEmptyObject
https://hg.tryton.org/tryton-env/rev/b55eb1e83f3a
New changeset 7bfae4f854b5 by Cédric Krier in branch 'default':
Restore test of modified_fields with isEmptyObject
https://hg.tryton.org/sao/rev/7bfae4f854b5
New changeset 1d8ed6db6044 by Cédric Krier in branch 'default':
Remove SignalEvent usage
https://hg.tryton.org/tryton-env/rev/1d8ed6db6044
New changeset d0ffba2a59a8 by Cédric Krier in branch 'default':
Remove SignalEvent usage
https://hg.tryton.org/tryton/rev/d0ffba2a59a8
New changeset 403bc7a95b0a by Cédric Krier in branch 'default':
Use same names to register windows and screens as tryton
https://hg.tryton.org/sao/rev/403bc7a95b0a
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-15.17:11:21

I updated also sao to follow the same naming and design to ease the maintenance. And bonus point I implemented the missing send_modified and the activate_save feature.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-15.17:00:45

This design has always been a source of problem and we did not replicate it to sao (on purpose).
I think it is time to remove it and follow the same design as sao which is to register parent object on Screen.windows and Group.screens.
This design will simplify the addition of new feature that needs to push information from the record to the screen or window (ex: https://discuss.tryton.org/t/allow-the-form-view-to-emit-informative-popup-messages/4611/7).

History
Date User Action Args
2022-04-02 18:18:54roundup-botsetmessages: + msg74934
2022-04-02 18:18:51roundup-botsetmessages: + msg74933
2022-01-02 13:55:26roundup-botsetmessages: + msg73046
2022-01-02 13:55:21roundup-botsetmessages: + msg73045
2022-01-02 13:55:18roundup-botsetmessages: + msg73044
nosy: + roundup-bot
status: testing -> resolved
2022-01-01 19:24:50reviewbotsetmessages: + msg73042
2022-01-01 19:24:41reviewbotsetmessages: + msg73041
2021-12-16 10:40:32cedlinkissue11065 superseder
2021-12-15 23:10:56cedlinkissue11063 superseder
2021-12-15 18:27:15reviewbotsetmessages: + msg72568

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