Merge lp:~therp-nl/banking-addons/6.1-abnamro_payment_recognition into lp:banking-addons/6.1

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 173
Proposed branch: lp:~therp-nl/banking-addons/6.1-abnamro_payment_recognition
Merge into: lp:banking-addons/6.1
Diff against target: 29 lines (+4/-1)
1 file modified
account_banking_nl_abnamro/abnamro.py (+4/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/6.1-abnamro_payment_recognition
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) Approve
Review via email: mp+167299@code.launchpad.net

Commit message

[FIX] abnamro payment order recognition

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account_banking_nl_abnamro/abnamro.py'
--- account_banking_nl_abnamro/abnamro.py 2013-04-26 09:18:37 +0000
+++ account_banking_nl_abnamro/abnamro.py 2013-06-04 14:47:49 +0000
@@ -89,6 +89,7 @@
89 'INTL': bt.ORDER, # international order89 'INTL': bt.ORDER, # international order
90 'UNKN': bt.ORDER, # everything else90 'UNKN': bt.ORDER, # everything else
91 'SEPA': bt.ORDER,91 'SEPA': bt.ORDER,
92 'PAYB': bt.PAYMENT_BATCH,
92 }93 }
9394
94 def __init__(self, line, *args, **kwargs):95 def __init__(self, line, *args, **kwargs):
@@ -115,7 +116,7 @@
115 elif not self.execution_date:116 elif not self.execution_date:
116 self.error_message = "No execution date"117 self.error_message = "No execution date"
117 elif not self.remote_account and self.transfer_type not in [118 elif not self.remote_account and self.transfer_type not in [
118 'BEA', 'GEA', 'COSTS', 'UNKN',119 'BEA', 'GEA', 'COSTS', 'UNKN', 'PAYB',
119 ]:120 ]:
120 self.error_message = _('No remote account for transaction type '121 self.error_message = _('No remote account for transaction type '
121 '%s') % self.transfer_type122 '%s') % self.transfer_type
@@ -245,6 +246,8 @@
245 remote_owner = field[14:32].strip()246 remote_owner = field[14:32].strip()
246 elif re.match("^EL[0-9]{13}I", field):247 elif re.match("^EL[0-9]{13}I", field):
247 transfer_type = 'INTL'248 transfer_type = 'INTL'
249 elif field.startswith("TOTAAL BETALINGEN"):
250 transfer_type = 'PAYB'
248 return (transfer_type, remote_account, remote_owner)251 return (transfer_type, remote_account, remote_owner)
249 252
250 fields = split_blob(self.blob)253 fields = split_blob(self.blob)

Subscribers

People subscribed via source and target branches