account-payment-team team mailing list archive

[Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>]

Pedro Manuel Baeza has proposed merging lp:~pedro.baeza/account-payment/6.1-account_payment_extension-fix-amount-sign into lp:account-payment.

Requested reviews:
  Account Payment (account-payment-team)

For more details, see:
https://code.launchpad.net/~pedro.baeza/account-payment/6.1-account_payment_extension-fix-amount-sign/+merge/192377

[FIX] account_payment_extension: Correct sign for amount and amount_currency when adding lines manually in a receivable payment order.

This is the same change as for 7.0:

https://code.launchpad.net/~pedro.baeza/account-payment/7.0-account_payment_extension-fix-amount-sign/+merge/192373
-- 
https://code.launchpad.net/~pedro.baeza/account-payment/6.1-account_payment_extension-fix-amount-sign/+merge/192377
Your team Account Payment is requested to review the proposed merge of lp:~pedro.baeza/account-payment/6.1-account_payment_extension-fix-amount-sign into lp:account-payment.

=== modified file 'account_payment_extension/payment.py'
--- account_payment_extension/payment.py	2013-06-25 20:17:09 +0000
+++ account_payment_extension/payment.py	2013-10-23 17:41:33 +0000
@@ -380,6 +380,12 @@
             if line.name != '/':
                 res['value']['communication'] = res['value']['communication'] + '. ' + line.name
             res['value']['account_id'] = line.account_id.id
+            if context.get('order_id'):
+                payment_order = self.pool.get('payment.order').browse(
+                    cr, uid, context['order_id'], context=context)
+                if payment_order.type == 'receivable':
+                    res['value']['amount'] *= -1
+                    res['value']['amount_currency'] *= -1
         return res
 
 payment_line()