Stefan Rijnhart (Therp) has proposed merging 
lp:~therp-nl/banking-addons/ba61-fix_surplus_return 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-fix_surplus_return/+merge/164808

When creating a voucher in account_banking to use in the reconciliation of bank 
statement lines, the voucher line type should not depend on the sign of the 
booking that it is to be reconciled with, but on the sign of the bank statement 
line. Otherwise the voucher line gets the wrong sign and the wrong move lines 
are being created.

Case: A sales invoice of €150 is paid with €200. The surplus amount of €50 is 
returned in a separate transaction. The two transactions together should be 
reconcilable with the original invoice.

-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba61-fix_surplus_return/+merge/164808
Your team CLEARCORP development team is subscribed to branch lp:banking-addons.
=== modified file 'account_banking/banking_import_transaction.py'
--- account_banking/banking_import_transaction.py	2013-05-20 08:07:48 +0000
+++ account_banking/banking_import_transaction.py	2013-05-20 21:21:24 +0000
@@ -468,10 +468,12 @@
         journal = st_line.statement_id.journal_id
         if st_line.amount < 0.0:
             voucher_type = 'payment'
+            voucher_line_type = 'dr'
             account_id = (journal.default_debit_account_id and
                           journal.default_debit_account_id.id or False)
         else:
             voucher_type = 'receipt'
+            voucher_line_type = 'cr'
             account_id = (journal.default_credit_account_id and
                           journal.default_credit_account_id.id or False)
 
@@ -533,7 +535,7 @@
             'reconcile': True,
             'amount': line_amount,
             'account_id': transaction.move_line_id.account_id.id,
-            'type': transaction.move_line_id.credit and 'dr' or 'cr',
+            'type': voucher_line_type,
             }
         voucher['line_ids'] = [(0, 0, vch_line)]
         voucher_id = self.pool.get('account.voucher').create(

_______________________________________________
Mailing list: https://launchpad.net/~clearcorp
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~clearcorp
More help   : https://help.launchpad.net/ListHelp

Reply via email to