← Back to team overview

banking-addons-team team mailing list archive

Re: banking_addons: Upgrade to 6.1

 

On 22-02-12 09:42, James Jesudason wrote:

Having looked at what's involved, I think there's a simpler approach that we could take that will work require much less code. Instead of calculating and generating the journals ourselves, we could create an account.voucher record (since all the information needed for that has already been collected). Then let the account.voucher handle all of the multi-currency considerations, since it already handles single and multi-currency scenarios. There have been extra development in the 6.1 account_voucher to handle things like currency revaluation.

The benefits of this approach are:
1. We're using the standard approach for generating the bank rec journals, but using a simpler screen. 2. We don't have to rebuild a lot of logic that is already in the standard module. 3. We reduce complexity of the account_banking module, reducing the number of lines-of-code that we need to maintain.

What do you think?


Hi James,

full integration with account_voucher is something that we've know for some time to be the way forward for the Banking Addons, but we just never got round to do it. Your arguments sum it up all the more convincingly and I would be all in favour of it.

Regards,
Stefan.

--
Therp - Maatwerk in open ontwikkeling

Stefan Rijnhart - Ontwerp en implementatie

mail: stefan@xxxxxxxx
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan



References