[IMP] account_usability_akretion: simplify code for invoice attach del upon invoice back to draft

account_usability_akretion/models/account_move.py

@@ -205,33 +205,14 @@ class AccountMove(models.Model):
                 move.suitable_journal_ids = self.env['account.journal'].search(domain)
 
     def button_draft(self):
-        # Get report name before reset to draft because 'attachment' field of report
-        # is False when state != 'posted'
-        report_filenames = self._get_invoice_attachment_name()
-        super().button_draft()
         # Delete attached pdf invoice
-        if report_filenames:
         for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
-                attachments = self.env['ir.attachment'].search([
+            for report_xmlid in ('account.account_invoices', 'account.account_invoices_without_payment'):
-                    ('name', 'in', report_filenames[move.id]),
+                report = self.env.ref(report_xmlid)
-                    ('res_id', '=', move.id),
+                attach = report.retrieve_attachment(move)
-                    ('res_model', '=', self._name),
+                if attach:
-                    ('type', '=', 'binary'),
+                    attach.unlink()
-                    ])
+        super().button_draft()
-                if attachments:
-                    attachments.unlink()
-
-    def _get_invoice_attachment_name(self):
-        report_filenames = defaultdict(list)
-        for report_name in ('account.report_invoice', 'account.report_invoice_with_payments'):
-            try:
-                report_invoice = self.env['ir.actions.report']._get_report_from_name(report_name)
-            except IndexError:
-                report_invoice = False
-            if report_invoice and report_invoice.attachment:
-                for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
-                    report_filenames[move.id].append(safe_eval(report_invoice.attachment, {'object': self, 'time': time}))
-        return report_filenames
 
     def _get_accounting_date(self, invoice_date, has_tax):
         # On vendor bills/refunds, we want date = invoice_date unless