← Back to team overview

banking-addons-team team mailing list archive

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

 

Holger Brunn (Therp) has proposed merging lp:~therp-nl/banking-addons/6.1-empty_message_direct_debit into lp:banking-addons.

Commit message:
[FIX] cope with empty messages in direct debits

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/6.1-empty_message_direct_debit/+merge/175082
-- 
https://code.launchpad.net/~therp-nl/banking-addons/6.1-empty_message_direct_debit/+merge/175082
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/6.1-empty_message_direct_debit into lp:banking-addons.
=== 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 @@
                      for i in range(0, len(self.message), 32)
                     ]
             self.reference = self.remote_owner
+
+            if not parts:
+                return
+
             if self.reference.startswith('KN: '):
                 self.reference = self.reference[4:]
                 if parts[0] == self.reference:


Follow ups