Banking Addons

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

  1. 6.1-abnamro_payment_recognition
  2. Merge into 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
1=== modified file 'account_banking_nl_abnamro/abnamro.py'
2--- account_banking_nl_abnamro/abnamro.py 2013-04-26 09:18:37 +0000
3+++ account_banking_nl_abnamro/abnamro.py 2013-06-04 14:47:49 +0000
4@@ -89,6 +89,7 @@
5 'INTL': bt.ORDER, # international order
6 'UNKN': bt.ORDER, # everything else
7 'SEPA': bt.ORDER,
8+ 'PAYB': bt.PAYMENT_BATCH,
9 }
10
11 def __init__(self, line, *args, **kwargs):
12@@ -115,7 +116,7 @@
13 elif not self.execution_date:
14 self.error_message = "No execution date"
15 elif not self.remote_account and self.transfer_type not in [
16- 'BEA', 'GEA', 'COSTS', 'UNKN',
17+ 'BEA', 'GEA', 'COSTS', 'UNKN', 'PAYB',
18 ]:
19 self.error_message = _('No remote account for transaction type '
20 '%s') % self.transfer_type
21@@ -245,6 +246,8 @@
22 remote_owner = field[14:32].strip()
23 elif re.match("^EL[0-9]{13}I", field):
24 transfer_type = 'INTL'
25+ elif field.startswith("TOTAAL BETALINGEN"):
26+ transfer_type = 'PAYB'
27 return (transfer_type, remote_account, remote_owner)
28
29 fields = split_blob(self.blob)

Subscribers

People subscribed via source and target branches