Merge lp:~openerp-dev/openobject-addons/7.0-opw-595923-chandni into lp:openobject-addons/7.0

[Chandni Machchhar(OpenERP)]

Hello,

 I have Fixed the Bug : Prevent that use of accounts of other chart of accounts while creating invoice.
 When you create invoice it will not allow you to set the account which is different than chart of account.

Thanks,
Chandni.

Unmerged revisions

9341. By Chandni Machchhar(OpenERP) on 2013-08-02

[FIX] Prevent that use of accounts of other chart of accounts.

=== modified file 'account/account_invoice.py'
--- account/account_invoice.py	2013-07-12 06:58:43 +0000
+++ account/account_invoice.py	2013-08-05 05:14:25 +0000
@@ -1042,6 +1042,19 @@
 
             ctx.update(invoice=inv)
             move_id = move_obj.create(cr, uid, move, context=ctx)
+            for move in move_obj.browse(cr, uid, [move_id], context=context):
+                top_common = None
+                for line in move.line_id:
+                    account = line.account_id
+                    top_account = account
+                    while top_account.parent_id:
+                        top_account = top_account.parent_id
+                    if not top_common:
+                        top_common = top_account
+                    elif top_account.id != top_common.id:
+                        raise osv.except_osv(_('Error!'),
+                                     _('You cannot validate this Invoice because account "%s" does not belong to chart of accounts "%s".') % (account.name, top_common.name))
+
             new_move_name = move_obj.browse(cr, uid, move_id, context=ctx).name
             # make the invoice point to that move
             self.write(cr, uid, [inv.id], {'move_id': move_id,'period_id':period_id, 'move_name':new_move_name}, context=ctx)