Merge lp:~openerp-dev/openobject-addons/trunk-bug-856751-bde into lp:openobject-addons

Proposed by Bharat Devnani (Open ERP)
Status: Rejected
Rejected by: qdp (OpenERP)
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-856751-bde
Merge into: lp:openobject-addons
Diff against target: 37 lines (+6/-5)
1 file modified
account_anglo_saxon/invoice.py (+6/-5)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-856751-bde
Reviewer Review Type Date Requested Status
Mustufa Rangwala (Open ERP) (community) Approve
qdp (OpenERP) Pending
Review via email: mp+77140@code.launchpad.net

Description of the change

Hello Sir,

I have fixed the bug by modifying the parameters of the function compute in def move_line_get of invoice.py in account_anglo_saxon module.

Thanks & Regards,
Devnani Bharat R.

To post a comment you must log in.
Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) :
review: Approve
Revision history for this message
qdp (OpenERP) (qdp) wrote :

how weird... i'm not able to reproduce the bug. I'm setting the merge prop as rejected in the meanwhile to clear the LP list.

thanks

Unmerged revisions

5220. By Bharat Devnani (Open ERP)

account_anglo_saxon: Wrong currency recorded from Sales Journal to Expense Acct

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_anglo_saxon/invoice.py'
2--- account_anglo_saxon/invoice.py 2011-09-24 10:33:29 +0000
3+++ account_anglo_saxon/invoice.py 2011-09-27 11:01:42 +0000
4@@ -1,8 +1,8 @@
5 ##############################################################################
6-#
7+#
8 # OpenERP, Open Source Management Solution
9-# Copyright (C)
10-# 2004-2010 Tiny SPRL (<http://tiny.be>).
11+# Copyright (C)
12+# 2004-2010 Tiny SPRL (<http://tiny.be>).
13 # 2009-2010 Veritos (http://veritos.nl).
14 # All Rights Reserved
15 #
16@@ -17,7 +17,7 @@
17 # GNU Affero General Public License for more details.
18 #
19 # You should have received a copy of the GNU Affero General Public License
20-# along with this program. If not, see <http://www.gnu.org/licenses/>.
21+# along with this program. If not, see <http://www.gnu.org/licenses/>.
22 #
23 ##############################################################################
24
25@@ -30,10 +30,11 @@
26 res = super(account_invoice_line,self).move_line_get(cr, uid, invoice_id, context=context)
27 inv = self.pool.get('account.invoice').browse(cr, uid, invoice_id, context=context)
28 company_currency = inv.company_id.currency_id.id
29+ company_currency_rate = inv.company_id.currency_id.rate
30 def get_price(cr, uid, inv, company_currency,i_line):
31 cur_obj = self.pool.get('res.currency')
32 if inv.currency_id.id != company_currency:
33- price = cur_obj.compute(cr, uid, company_currency, inv.currency_id.id, i_line.product_id.standard_price * i_line.quantity, context={'date': inv.date_invoice})
34+ price = cur_obj.compute(cr, uid, company_currency, inv.currency_id.id, i_line.product_id.standard_price * i_line.quantity * company_currency_rate, context={'date': inv.date_invoice})
35 else:
36 price = i_line.product_id.standard_price * i_line.quantity
37 return price

Subscribers

People subscribed via source and target branches

to all changes: