banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/banking-addons/6.1-check_statement_date_period_id into lp:banking-addons.

Requested reviews:
  Banking Addons Team (banking-addons-team)

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/6.1-check_statement_date_period_id/+merge/137990
-- 
https://code.launchpad.net/~therp-nl/banking-addons/6.1-check_statement_date_period_id/+merge/137990
Your team Banking Addons Team is requested to review the proposed merge of lp:~therp-nl/banking-addons/6.1-check_statement_date_period_id into lp:banking-addons.

=== modified file 'account_banking/wizard/bank_import.py'
--- account_banking/wizard/bank_import.py	2012-06-26 09:46:07 +0000
+++ account_banking/wizard/bank_import.py	2012-12-04 20:51:33 +0000
@@ -263,6 +263,16 @@
                                                          ('date_stop','>=',statement.date),
                                                          ('special', '=', False)])
 
+            if not period_ids:
+                results.log.append(
+                    _('No period found covering statement date %(date)s, '
+                      'statement %(id)s skipped') % {
+                        'date': statement.date,
+                        'id': statement.id,
+                    }
+                )
+                continue
+
             # Create the bank statement record
             statement_id = statement_obj.create(cursor, uid, dict(
                 name = statement.id,