Amit Dodiya (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.0-opw-573996-ado into 
lp:openobject-addons/6.0.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-573996-ado/+merge/102814

Hello,

"[FIX] the checks default_iban_check always fail and returns The IBAN is 
invalid"

Steps:
1). Install base_iban
2). Add Bank details in customer form

The iban number you have entered that has been set to upper case letters and 
that are compared only with lower case letters in def 
_construct_constraint_msg() so the checks always fail and returns 'The IBAN is 
invalid, It should begin with the country code'

Regards,
Amit
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-573996-ado/+merge/102814
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-573996-ado.
=== modified file 'base_iban/base_iban.py'
--- base_iban/base_iban.py	2011-01-14 09:34:28 +0000
+++ base_iban/base_iban.py	2012-04-20 08:34:28 +0000
@@ -114,7 +114,7 @@
         def default_iban_check(iban_cn):
             return iban_cn[0] in string.ascii_lowercase and iban_cn[1] in string.ascii_lowercase
 
-        iban_country = self.browse(cr, uid, ids)[0].iban[:2]
+        iban_country = self.browse(cr, uid, ids)[0].iban[:2].lower()
         if default_iban_check(iban_country):
             iban_example = iban_country in _ref_iban and _ref_iban[iban_country] + ' \nWhere A = Account number, B = National bank code, S = Branch code, C = account No, N = branch No, K = National check digits....' or ''
             return _('The IBAN does not seem to be correct. You should have entered something like this %s'), (iban_example)

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

Reply via email to