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

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 142
Proposed branch: lp:~therp-nl/banking-addons/6.1-check_statement_date_period_id
Merge into: lp:banking-addons/6.1
Diff against target: 20 lines (+10/-0)
1 file modified
account_banking/wizard/bank_import.py (+10/-0)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/6.1-check_statement_date_period_id
Reviewer Review Type Date Requested Status
Alexandre Fayolle - camptocamp Approve
Review via email: mp+137990@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Seems good to me.

Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== 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 @@
263 ('date_stop','>=',statement.date),263 ('date_stop','>=',statement.date),
264 ('special', '=', False)])264 ('special', '=', False)])
265265
266 if not period_ids:
267 results.log.append(
268 _('No period found covering statement date %(date)s, '
269 'statement %(id)s skipped') % {
270 'date': statement.date,
271 'id': statement.id,
272 }
273 )
274 continue
275
266 # Create the bank statement record276 # Create the bank statement record
267 statement_id = statement_obj.create(cursor, uid, dict(277 statement_id = statement_obj.create(cursor, uid, dict(
268 name = statement.id,278 name = statement.id,

Subscribers

People subscribed via source and target branches