Merge lp:~therp-nl/banking-addons/ba70-IE_BBAN_format into lp:banking-addons

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 196
Proposed branch: lp:~therp-nl/banking-addons/ba70-IE_BBAN_format
Merge into: lp:banking-addons
Diff against target: 12 lines (+1/-1)
1 file modified
account_banking/sepa/iban.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/ba70-IE_BBAN_format
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Review via email: mp+187799@code.launchpad.net

Description of the change

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

To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== 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 @@
214 'GR': BBANFormat('BBBCCCCAAAAAAAAAAAAAAAA', '%B-%C-%A', nolz=True),214 'GR': BBANFormat('BBBCCCCAAAAAAAAAAAAAAAA', '%B-%C-%A', nolz=True),
215 'HR': BBANFormat('BBBBBBBAAAAAAAAAA', '%B-%A'),215 'HR': BBANFormat('BBBBBBBAAAAAAAAAA', '%B-%A'),
216 'HU': BBANFormat('BBBCCCCXAAAAAAAAAAAAAAAV', '%B%C%X %A%V'),216 'HU': BBANFormat('BBBCCCCXAAAAAAAAAAAAAAAV', '%B%C%X %A%V'),
217 'IE': BBANFormat('BBBBCCCCCCAAAAAAAAV', '%C %A%V'),217 'IE': BBANFormat('BBBBCCCCCCAAAAAAAA', '%C %A'),
218 'IL': BBANFormat('BBBCCCAAAAAAAAAAAAA', '%C%A'),218 'IL': BBANFormat('BBBCCCAAAAAAAAAAAAA', '%C%A'),
219 # Iceland uses an extra identification number, split in two on219 # Iceland uses an extra identification number, split in two on
220 # display. Coded here as %P%V.220 # display. Coded here as %P%V.

Subscribers

People subscribed via source and target branches

to status/vote changes: