Merge lp:~acsone-openerp/banking-addons/7.0-bug-1280822-base_import into lp:banking-addons/bank-statement-reconcile-70

Proposed by Laurent Mignon (Acsone)
Status: Merged
Approved by: Guewen Baconnier @ Camptocamp
Approved revision: 119
Merged at revision: 125
Proposed branch: lp:~acsone-openerp/banking-addons/7.0-bug-1280822-base_import
Merge into: lp:banking-addons/bank-statement-reconcile-70
Diff against target: 14 lines (+4/-0)
1 file modified
account_statement_base_import/statement.py (+4/-0)
To merge this branch: bzr merge lp:~acsone-openerp/banking-addons/7.0-bug-1280822-base_import
Reviewer Review Type Date Requested Status
Alexandre Fayolle - camptocamp code review, no test Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+206618@code.launchpad.net

Description of the change

[FIX] lp:1280822 define a default value for import_type to avoid breaking tests of completions addons

To post a comment you must log in.
Revision history for this message
Laurent Mignon (Acsone) (lmi) wrote :
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM

Thanks!

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

LGTM

review: Approve (code review, no test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_statement_base_import/statement.py'
2--- account_statement_base_import/statement.py 2014-01-21 16:46:36 +0000
3+++ account_statement_base_import/statement.py 2014-02-16 15:47:24 +0000
4@@ -54,6 +54,10 @@
5
6 }
7
8+ _defaults = {
9+ 'import_type': 'generic_csvxls_so'
10+ }
11+
12 def _write_extra_statement_lines(
13 self, cr, uid, parser, result_row_list, profile, statement_id, context):
14 """Insert extra lines after the main statement lines.

Subscribers

People subscribed via source and target branches