Merge lp:~therp-nl/banking-addons/6.1-empty_message_direct_debit into lp:banking-addons/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
=== modified file 'account_banking_nl_girotel/girotel.py'
--- account_banking_nl_girotel/girotel.py 2013-06-10 13:47:38 +0000
+++ account_banking_nl_girotel/girotel.py 2013-07-16 17:14:26 +0000
@@ -242,6 +242,10 @@
242 for i in range(0, len(self.message), 32)242 for i in range(0, len(self.message), 32)
243 ]243 ]
244 self.reference = self.remote_owner244 self.reference = self.remote_owner
245
246 if not parts:
247 return
248
245 if self.reference.startswith('KN: '):249 if self.reference.startswith('KN: '):
246 self.reference = self.reference[4:]250 self.reference = self.reference[4:]
247 if parts[0] == self.reference:251 if parts[0] == self.reference:

Subscribers

People subscribed via source and target branches