Merge lp:~richard-willowit/domsense-agilebg-addons/no-lpr into lp:~deepak-willowit/domsense-agilebg-addons/no-lpr

Proposed by Richard deMeester
Status: Merged
Approved by: Deepak Seshadri
Approved revision: 161
Merged at revision: 161
Proposed branch: lp:~richard-willowit/domsense-agilebg-addons/no-lpr
Merge into: lp:~deepak-willowit/domsense-agilebg-addons/no-lpr
Diff against target: 12 lines (+2/-0)
1 file modified
base_report_to_printer/printing.py (+2/-0)
To merge this branch: bzr merge lp:~richard-willowit/domsense-agilebg-addons/no-lpr
Reviewer Review Type Date Requested Status
Deepak Seshadri Approve
Review via email: mp+104496@code.launchpad.net

Description of the change

raise exception after sending report

To post a comment you must log in.
Revision history for this message
Deepak Seshadri (deepak-willowit) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'base_report_to_printer/printing.py'
2--- base_report_to_printer/printing.py 2012-03-05 06:43:02 +0000
3+++ base_report_to_printer/printing.py 2012-05-03 07:07:21 +0000
4@@ -354,6 +354,8 @@
5 and self._reports[report_id].get('format', False)):
6 report_obj.print_direct(cr, uid, base64.encodestring(self._reports[report_id]['result']),
7 self._reports[report_id]['format'], printer)
8+ cr.close()
9+ raise osv.except_osv(_('Printed !'), _("Your print job has been sent to printer '%s'." % (printer.name,)))
10 cr.close()
11
12 res = super(virtual_report_spool, self).exp_report_get(db, uid, report_id)

Subscribers

People subscribed via source and target branches