audrius/reporting-engine
1
0
Fork 1
mirror of https://github.com/OCA/reporting-engine.git synced 2025-02-16 16:30:38 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] base_comment_template: Filter domain correctly

Browse Source 
1. Safe eval domain before checking if it is defined. This makes [] be "False" on the first check, and provides the proper use case for when the domain is empty
2. Use filtered_domain in record instead of a search in self. When on a compute, we are on a NewId context, so the search would always fail

TT29309
This commit is contained in:
João Marques
2021-05-25 09:04:26 +01:00
parent 7dbc2fe1ab
commit fee0f18049
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
base_comment_template/models/comment_template.py
View File

@@ -39,7 +39,6 @@ class CommentTemplate(models.AbstractModel):
] ]
) )
for template in templates: for template in templates:
if not template.domain or self in self.search( domain = safe_eval(template.domain)
safe_eval(template.domain) if not domain or record.filtered_domain(domain):
):
record.comment_template_ids = [(4, template.id)] record.comment_template_ids = [(4, template.id)]