From cf1c65d531d509f1e07a2c50a9822898c9f9eed8 Mon Sep 17 00:00:00 2001 From: Sylvain GARANCHER Date: Tue, 13 Feb 2018 16:51:10 +0100 Subject: [PATCH] [FIX] Allow to generate printing options without a report --- base_report_to_printer/models/printing_printer.py | 3 --- base_report_to_printer/tests/test_printing_printer.py | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/base_report_to_printer/models/printing_printer.py b/base_report_to_printer/models/printing_printer.py index bccbb08..0e83e7e 100644 --- a/base_report_to_printer/models/printing_printer.py +++ b/base_report_to_printer/models/printing_printer.py @@ -164,9 +164,6 @@ class PrintingPrinter(models.Model): @api.multi def print_options(self, report=None, **print_opts): options = {} - if not report: - return options - for option, value in print_opts.items(): try: options.update(getattr( diff --git a/base_report_to_printer/tests/test_printing_printer.py b/base_report_to_printer/tests/test_printing_printer.py index 03ea2ad..84c437c 100644 --- a/base_report_to_printer/tests/test_printing_printer.py +++ b/base_report_to_printer/tests/test_printing_printer.py @@ -71,7 +71,7 @@ class TestPrintingPrinter(TransactionCase): # with tests in test_printing_printer_tray from when modules merged report = self.env['ir.actions.report'].search([], limit=1) self.assertEqual(self.Model.print_options( - report, doc_format='raw'), {'raw': 'True'} + doc_format='raw'), {'raw': 'True'} ) self.assertEqual(self.Model.print_options( report, doc_format='pdf', copies=2), {'copies': '2'}