banking-addons-team team mailing list archive

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

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

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/ba70-IE_BBAN_format/+merge/187799

IE IBAN format for BBAN extraction is too long. Compare format specification on Wikipedia [1] and sample account IE29AIBK93115212345678

6.1 version here: https://code.launchpad.net/~therp-nl/banking-addons/ba61-IE_BBAN_format

[1] http://en.wikipedia.org/wiki/International_Bank_Account_Number#IBAN_formats_by_country


-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-IE_BBAN_format/+merge/187799
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/ba70-IE_BBAN_format into lp:banking-addons.

=== modified file 'account_banking/sepa/iban.py'
--- account_banking/sepa/iban.py	2013-04-15 13:59:50 +0000
+++ account_banking/sepa/iban.py	2013-09-26 14:28:25 +0000
@@ -214,7 +214,7 @@
         'GR': BBANFormat('BBBCCCCAAAAAAAAAAAAAAAA', '%B-%C-%A', nolz=True),
         'HR': BBANFormat('BBBBBBBAAAAAAAAAA', '%B-%A'),
         'HU': BBANFormat('BBBCCCCXAAAAAAAAAAAAAAAV', '%B%C%X %A%V'),
-        'IE': BBANFormat('BBBBCCCCCCAAAAAAAAV', '%C %A%V'),
+        'IE': BBANFormat('BBBBCCCCCCAAAAAAAA', '%C %A'),
         'IL': BBANFormat('BBBCCCAAAAAAAAAAAAA', '%C%A'),
         # Iceland uses an extra identification number, split in two on
         # display. Coded here as %P%V.