Banking Addons

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

  1. 6.1-empty_message_direct_debit
  2. Merge into 6.1
Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 178
Proposed branch: lp:~therp-nl/banking-addons/6.1-empty_message_direct_debit
Merge into: lp:banking-addons/6.1
Diff against target: 14 lines (+4/-0)
1 file modified
account_banking_nl_girotel/girotel.py (+4/-0)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/6.1-empty_message_direct_debit
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp Approve
Stefan Rijnhart (Opener) Approve
Review via email: mp+175082@code.launchpad.net

Commit message

[FIX] cope with empty messages in direct debits

To post a comment you must log in.
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

LGTM

review: Approve
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) :
review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Merged with 6.1 and 7.0

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_banking_nl_girotel/girotel.py'
2--- account_banking_nl_girotel/girotel.py 2013-06-10 13:47:38 +0000
3+++ account_banking_nl_girotel/girotel.py 2013-07-16 17:14:26 +0000
4@@ -242,6 +242,10 @@
5 for i in range(0, len(self.message), 32)
6 ]
7 self.reference = self.remote_owner
8+
9+ if not parts:
10+ return
11+
12 if self.reference.startswith('KN: '):
13 self.reference = self.reference[4:]
14 if parts[0] == self.reference:

Subscribers

People subscribed via source and target branches