From e41c237bc0d7181d39dd8334da933cb1118418eb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Krier?= Date: Sat, 13 Apr 2024 15:00:52 +0200 Subject: [PATCH] Use function to not copy custom report content The Function fields are not copied so it is the storage field that must have a default value. Closes #13170 (grafted from 96d629fe11a710b395d9981881c5dfda1674becf) --HG-- branch : 7.0 --- trytond/trytond/ir/action.py | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/trytond/trytond/ir/action.py b/trytond/trytond/ir/action.py index 9cde0e3c02..fa4afbe3ad 100644 --- a/trytond/trytond/ir/action.py +++ b/trytond/trytond/ir/action.py @@ -675,16 +675,10 @@ class ActionReport(ActionMixin, ModelSQL, ModelView): default = {} default = default.copy() default.setdefault('module', None) - - new_reports = [] - for report in reports: - if report.report: - default['report_content'] = None - default['report'] = None - default['report_name'] = report.report_name - new_reports.extend(super(ActionReport, cls).copy([report], - default=default)) - return new_reports + default.setdefault( + 'report_content_custom', + lambda o: None if o['report'] else o['report_content_custom']) + return super().copy(reports, default=default) @classmethod def write(cls, reports, values, *args):