banking-addons-team team mailing list archive

["Laurent Mignon \(Acsone\)" <laurent.mignon@xxxxxxxxx>]

Laurent Mignon (Acsone) has proposed merging lp:~acsone-openerp/banking-addons/7.0-bug-1280822-base_import into lp:banking-addons/bank-statement-reconcile-7.0.

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)
Related bugs:
  Bug #1280822 in Banking Addons: " [7.0] account_statement_base_completion - Tests fails if the account_statement_base_import module is installed"
  https://bugs.launchpad.net/banking-addons/+bug/1280822

For more details, see:
https://code.launchpad.net/~acsone-openerp/banking-addons/7.0-bug-1280822-base_import/+merge/206618

[FIX] lp:1280822 define a default value for import_type to avoid breaking tests of completions addons
-- 
https://code.launchpad.net/~acsone-openerp/banking-addons/7.0-bug-1280822-base_import/+merge/206618
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~acsone-openerp/banking-addons/7.0-bug-1280822-base_import into lp:banking-addons/bank-statement-reconcile-7.0.

=== modified file 'account_statement_base_import/statement.py'
--- account_statement_base_import/statement.py	2014-01-21 16:46:36 +0000
+++ account_statement_base_import/statement.py	2014-02-16 15:47:24 +0000
@@ -54,6 +54,10 @@
 
     }
 
+    _defaults = {
+                'import_type': 'generic_csvxls_so'
+            }
+
     def _write_extra_statement_lines(
             self, cr, uid, parser, result_row_list, profile, statement_id, context):
         """Insert extra lines after the main statement lines.