diff options
| author | Christophe Simonis <chs@odoo.com> | 2014-10-16 15:50:26 +0200 |
|---|---|---|
| committer | Christophe Simonis <chs@odoo.com> | 2014-10-16 15:50:26 +0200 |
| commit | 2b192bef2242c71d0ff3a45b9efb9e717fd364c4 (patch) | |
| tree | d3522e85fca12123cf8984de3ec81c2e10c3ea63 | |
| parent | acaab83756e7f0363cd0cd1a1789ade26033584c (diff) | |
[FIX] report: do not fail if PDF cannot be saved as attachment due to AccessError
| -rw-r--r-- | addons/report/models/report.py | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/addons/report/models/report.py b/addons/report/models/report.py index c2e1cd27a64..16adc510dcd 100644 --- a/addons/report/models/report.py +++ b/addons/report/models/report.py @@ -20,6 +20,7 @@ ############################################################################## from openerp import api +from openerp.exceptions import AccessError from openerp.osv import osv from openerp.tools import config, which from openerp.tools.translate import _ @@ -453,9 +454,14 @@ class Report(osv.Model): 'res_model': save_in_attachment.get('model'), 'res_id': reporthtml[0], } - self.pool['ir.attachment'].create(cr, uid, attachment) - _logger.info('The PDF document %s is now saved in the ' - 'database' % attachment['name']) + try: + self.pool['ir.attachment'].create(cr, uid, attachment) + except AccessError: + _logger.warning("Cannot save PDF report %r as attachment", + attachment['name']) + else: + _logger.info('The PDF document %s is now saved in the database', + attachment['name']) pdfdocuments.append(pdfreport_path) except: |