← Back to team overview

banking-addons-team team mailing list archive

Re: banking_addons: Upgrade to 6.1

 

On 02/16/2012 01:45 PM, James Jesudason wrote:
Hi Stefan,

I'm starting to look at upgrading the account_banking module to version 6.1 and have seen that base_iban now uses the acc_number field on the res.partner.bank record, instead of creating a separate iban field.

My thoughts are to update the IBAN functionality in banking_addons so that it uses acc_number, instead of having a separate field for the IBAN code. I would also move the validation, so that it still validates the IBAN number as in version 6.0.

Do you have any reason to retain the iban field so that it is separate from the acc_number field, or are you okay with my suggested approach?

Hi James,

(Replying to the list. Can you join it, please?)

Yes, this is problematic. For the Dutch region, there is a decent account number lookup that translates domestic account numbers to IBAN when a new account is entered. The lookup enables users in the Dutch territory to build up a collection of IBAN numbers for their business contacts, while still using the domestic format (which is still widely used). The matching mechanism queries both fields as well.

Slightly off topic, have you had a chance to look at https://code.launchpad.net/~therp-nl/banking-addons/6.0-interactive-matching-and-debit-orders? I had suggested in the merge request that this be the base for the 6.1 branch. If you need to keep using the current functionality, we could keep it as a 6.1-legacy branch.

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



Follow ups