banking-addons-team team mailing list archive

["Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>]

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

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

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/7.0-girotel_sepa_payment_batches/+merge/209682
-- 
https://code.launchpad.net/~therp-nl/banking-addons/7.0-girotel_sepa_payment_batches/+merge/209682
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/7.0-girotel_sepa_payment_batches into lp:banking-addons.

=== modified file 'account_banking_nl_girotel/girotel.py'
--- account_banking_nl_girotel/girotel.py	2013-08-02 12:51:00 +0000
+++ account_banking_nl_girotel/girotel.py	2014-03-06 13:48:30 +0000
@@ -106,6 +106,7 @@
         self.date = str2date(self.date, '%Y%m%d')
         if self.direction == 'A':
             self.transferred_amount = -float(self.transferred_amount)
+            #payment batch done via clieop
             if (self.transfer_type == 'VZ'
                     and (not self.remote_account or self.remote_account == '0')
                     and (not self.message or re.match('^\s*$', self.message))
@@ -113,6 +114,14 @@
                 self.transfer_type = 'PB'
                 self.message = self.remote_owner
                 self.remove_owner = False
+            #payment batch done via sepa
+            if self.transfer_type == 'VZ'\
+                    and not self.remote_account\
+                    and not self.remote_owner\
+                    and re.match(
+                        '^Verzamel Eurobetaling .* TOTAAL \d+ POSTEN\s*$',
+                        self.message):
+                self.transfer_type = 'PB'
         else:
             self.transferred_amount = float(self.transferred_amount)
         self.local_account = self.local_account.zfill(10)