Merge lp:~openerp-dev/openobject-addons/6.1-opw-576447-ado into lp:openobject-addons/6.1

[Amit Dodiya (OpenERP)]

Hello,

"[FIX]: while paying the invoice of selected journal in voucher has inactive sequence that will throw the traceback it should raise warning"

Stpes to reproduce:
1). Goto : Accounting/Configuration/Financial Accounting/Journals/Journals, Select "Bank Journal - (test)" journal
2). Open Entry sequence of above journal ,inactive it and save it.
3). Create invoice, validate and click on payment button the voucher will be opened.
4). Now select the above journal which has inactive sequence and try to validate the voucher
You will get the trace-back it should raise the warning message.

Regards,
Amit Dodiya

[lp:~openerp-dev/openobject-addons/6.1-opw-576447-ado]

6873. By Amit Dodiya<email address hidden> on 2012-07-04

[FIX]: while paying the invoice of selected journal in voucher has inactive sequence that will throw the traceback it should raise warning

[Naresh(OpenERP) (nch-openerp)]

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-576447-port-mma/+merge/132488 and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6873. By Amit Dodiya<email address hidden> on 2012-07-04

[FIX]: while paying the invoice of selected journal in voucher has inactive sequence that will throw the traceback it should raise warning

=== modified file 'account_voucher/account_voucher.py'
--- account_voucher/account_voucher.py	2012-03-30 11:07:16 +0000
+++ account_voucher/account_voucher.py	2012-07-04 09:25:22 +0000
@@ -890,6 +890,9 @@
         if voucher_brw.number:
             name = voucher_brw.number
         elif voucher_brw.journal_id.sequence_id:
+            if not voucher_brw.journal_id.sequence_id.active:
+                raise osv.except_osv(_('Configuration Error !'),
+                        _('Please activate the sequence of selected journal !'))
             name = seq_obj.next_by_id(cr, uid, voucher_brw.journal_id.sequence_id.id, context=context)
         else:
             raise osv.except_osv(_('Error !'),