Issue 11033

Title
Using ID field as notification email recipient does not work
Priority
bug
Status
resolved
Nosy list
ced, pokoli, reviewbot, roundup-bot
Assigned to
pokoli
Keywords
review

Created on 2021-12-09.10:45:31 by pokoli, last changed 2 months ago by roundup-bot.

Messages

New changeset 5b238116083d by Sergi Almacellas Abellana in branch '6.2':
Use record as recipient for id fields
https://hg.tryton.org/modules/notification_email/rev/5b238116083d
New changeset 11c08b72afa6 by Sergi Almacellas Abellana in branch 'default':
Use record as recipient for id fields
https://hg.tryton.org/tryton-env/rev/11c08b72afa6
New changeset cce8a58fa33c by Sergi Almacellas Abellana in branch 'default':
Use record as recipient for id fields
https://hg.tryton.org/modules/notification_email/rev/cce8a58fa33c

New changeset 9d0273e50446 by Sergi Almacellas Abellana in branch 'default':
Add test using id as recipient
https://hg.tryton.org/modules/notification_email/rev/9d0273e50446
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2022-01-04.22:38:14

Indeed it is since issue9449.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-09.10:58:37

I do not understand the rational nor why it is a bug. An ID is an integer.

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2021-12-09.10:45:30

When using an id field as recipient (for example by setting a notification on the party.party module the pyson evaluation returns a integer field (not a model instance) so the _get_addresses and _get_languages fail to pick the right email.

We should browse the instance in such case.

History
Date User Action Args
2022-02-26 11:47:11roundup-botsetkeyword: - backport
messages: + msg74350
2022-02-18 13:18:26roundup-botsetmessages: + msg74229
2022-02-18 13:18:21roundup-botsetmessages: + msg74228
nosy: + roundup-bot
status: testing -> resolved
2022-02-18 13:16:10pokolisetkeyword: + backport
2022-02-07 09:40:42reviewbotsetmessages: + msg73906
2022-01-27 16:43:23reviewbotsetmessages: + msg73583
2022-01-27 16:14:26reviewbotsetmessages: + msg73582
2022-01-04 22:38:15cedsetmessages: + msg73114
2021-12-09 11:36:02reviewbotsetmessages: + msg72068
2021-12-09 11:01:46reviewbotsetmessages: + msg72067
nosy: + reviewbot

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