Merge lp:~openbig/bigconsulting/fix_bug_account_id into lp:bigconsulting

Proposed by gpa(OpenERP)
Status: Merged
Merged at revision: 74
Proposed branch: lp:~openbig/bigconsulting/fix_bug_account_id
Merge into: lp:bigconsulting
Diff against target: 63 lines (+10/-6)
1 file modified
account_invoice_cash_discount/account_invoice_cash_discount.py (+10/-6)
To merge this branch: bzr merge lp:~openbig/bigconsulting/fix_bug_account_id
Reviewer Review Type Date Requested Status
openbig Pending
Review via email: mp+32570@code.launchpad.net

Description of the change

Fixed the problem of account_id when pay from simple pay invoice wizard.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_invoice_cash_discount/account_invoice_cash_discount.py'
2--- account_invoice_cash_discount/account_invoice_cash_discount.py 2010-08-13 09:34:37 +0000
3+++ account_invoice_cash_discount/account_invoice_cash_discount.py 2010-08-13 10:37:40 +0000
4@@ -348,7 +348,7 @@
5 tax_value = invoice_tax_obj.browse(cr, uid, tax_id)
6 tax_account_id = tax_value.account_id.id
7 tax_code_id = tax_value.tax_code_id.id
8-
9+
10 if invoice.company_id.currency_id.id<>invoice.currency_id.id:
11 tax_amount_currency = cur_obj.compute(cr, uid, invoice.currency_id.id, invoice.company_id.currency_id.id, tax_real_amt, context=context)
12 l3 = {
13@@ -361,7 +361,7 @@
14 'currency_id':currency_id,
15 'amount_currency':tax_amount_currency and direction * tax_amount_currency or 0.0,
16 'tax_code_id':tax_code_id,
17- 'tax_amount':tax_real_amt,
18+ 'tax_amount':-tax_real_amt,
19 }
20 l3['name'] = name
21 lines.append((0, 0, l3))
22@@ -382,7 +382,7 @@
23 'amount_currency':move_line_data.amount_currency and direction * move_line_data.amount_currency or 0.0,
24 'name':move_line_data.name,
25 'tax_code_id':move_line_data.tax_code_id.id,
26- 'tax_amount':move_line_data.tax_amount,
27+ 'tax_amount':-move_line_data.tax_amount,
28 }
29 lines.append((0, 0, l4))
30 else:
31@@ -390,6 +390,11 @@
32 account_id = False
33 if 'account_id' in context and context['account_id']:
34 account_id = context['account_id']
35+ else:
36+ account_id = self._get_account(cr, uid, ids , pay_amount, invoice.payment_term.id, context=context)
37+ fpos = invoice.fiscal_position or False
38+ if account_id:
39+ account_id = self.pool.get('account.fiscal.position').map_account(cr, uid, fpos, account_id)
40
41 for line in invoice.invoice_line:
42 tax_account_id = False
43@@ -422,7 +427,7 @@
44 tax_value = invoice_tax_obj.browse(cr, uid, tax_id)
45 tax_account_id = tax_value.account_id.id
46 base_code_id = tax_value.base_code_id.id
47-
48+
49 if invoice.company_id.currency_id.id<>invoice.currency_id.id:
50 discount_amount = discount_ratio - total_tax_amount
51 discount_amount_currency = cur_obj.compute(cr, uid, invoice.currency_id.id, invoice.company_id.currency_id.id, discount_amount, context=context)
52@@ -436,11 +441,10 @@
53 'currency_id':currency_id,
54 'amount_currency':discount_amount_currency and direction * discount_amount_currency or 0.0,
55 'tax_code_id':base_code_id,
56- 'tax_amount':(discount_ratio - total_tax_amount),
57+ 'tax_amount':-(discount_ratio - total_tax_amount),
58 }
59 l4['name'] = name
60 lines.append((0, 0, l4))
61-
62 move = {'ref': ref, 'line_id': lines, 'journal_id': pay_journal_id, 'period_id': period_id, 'date': date}
63 move_id = move_obj.create(cr, uid, move, context=context)
64

Subscribers

People subscribed via source and target branches