diff options
| author | David Beguin <dbe@odoo.com> | 2020-06-03 10:17:13 +0000 |
|---|---|---|
| committer | David Beguin <dbe@odoo.com> | 2020-06-03 11:55:08 +0000 |
| commit | 859537d77cafc6edc6411e6838ce5c09038172f2 (patch) | |
| tree | 6ec7d6868e5d7a5b374224d0e9a5da76fa2ea99e /addons/web/static/src/xml/debug.xml | |
| parent | 8a48aee8eb6ac6539dad8e6191273726cd179016 (diff) | |
[FIX] mail: Reserve mis-referenced trackings to system users
Note : Manual forward port recovering from commit ae1e70eba10112170283cfc17fa95d94dd948d2b
Purpose
=======
Let's say that the field 'foo' is tracked and defined with a res.group.
When the field is modified, a mail.tracking.value is generated, but the
reference to the field name is a char field.
When displaying the mail.tracking.values on the chatter, a check is done
according to the field group to decide whether we should display it or not
to the user. See: c7aa8c5#diff-ad8b6db158187579d2208f233d993c3cR43
So if I rename the field, and if the mail.tracking.value is not modified,
the mail.tracking.value magically appears to the users who shouldn't
access it before.
Note: If the migration is correctly handled, this shouldn't be the case.
But manual manipulations on the database could lead to this issue.
Specification
=============
If the field referenced by the mail_tracking_value doesn't seem to exist,
then display its value to system users only, by security.
closes #39016
closes odoo/odoo#52348
Taskid: 2088634
X-original-commit: 19bc081e4a71042c3f009e4af641da18fc16bdfe
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Diffstat (limited to 'addons/web/static/src/xml/debug.xml')
0 files changed, 0 insertions, 0 deletions
