banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/banking-addons/ba61-clieop_amount_should_be_integer into lp:banking-addons.

Requested reviews:
  Banking Addons Team (banking-addons-team)

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/ba61-clieop_amount_should_be_integer/+merge/143124

Previous fix for lp:1098699 should return an integer
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba61-clieop_amount_should_be_integer/+merge/143124
Your team Banking Addons Team is requested to review the proposed merge of lp:~therp-nl/banking-addons/ba61-clieop_amount_should_be_integer into lp:banking-addons.

=== modified file 'account_banking_nl_clieop/wizard/clieop.py'
--- account_banking_nl_clieop/wizard/clieop.py	2013-01-11 19:42:19 +0000
+++ account_banking_nl_clieop/wizard/clieop.py	2013-01-14 15:24:31 +0000
@@ -278,7 +278,7 @@
         self.transaction.transactiontype = type_
         self.transaction.accountno_beneficiary = accountno_beneficiary
         self.transaction.accountno_payer = accountno_payer
-        self.transaction.amount = round(amount * 100)
+        self.transaction.amount = int(round(amount * 100))
         if reference:
             self.paymentreference.paymentreference = reference
         # Allow long message lines to redistribute over multiple message