diff --git a/account_bank_statement_import_save_file/models/account_bank_statement.py b/account_bank_statement_import_save_file/models/account_bank_statement.py index 4b94dc4..4d180d0 100644 --- a/account_bank_statement_import_save_file/models/account_bank_statement.py +++ b/account_bank_statement_import_save_file/models/account_bank_statement.py @@ -9,7 +9,9 @@ class AccountBankStatement(models.Model): import_file = fields.Many2one( 'ir.attachment', 'Import file', readonly=True) - import_date = fields.Datetime(related='import_file.create_date') - import_user = fields.Many2one(related='import_file.create_uid') + import_date = fields.Datetime( + related='import_file.create_date', string='Imported on') + import_user = fields.Many2one( + related='import_file.create_uid', string='Imported by') import_log = fields.Text( related='import_file.description', string='Import Warnings') diff --git a/account_bank_statement_import_save_file/models/account_bank_statement_import.py b/account_bank_statement_import_save_file/models/account_bank_statement_import.py index 7405c6a..ca80b85 100644 --- a/account_bank_statement_import_save_file/models/account_bank_statement_import.py +++ b/account_bank_statement_import_save_file/models/account_bank_statement_import.py @@ -24,11 +24,12 @@ class AccountBankStatementImport(models.TransientModel): @api.model def _prepare_import_file_attachment(self, data_file, statement_id, notifications, filename): + if not filename: + filename = '' return { 'name': filename, 'res_model': 'account.bank.statement', 'res_id': statement_id, - 'type': 'binary', 'datas': data_file, 'datas_fname': filename, 'description': '\n'.join( diff --git a/account_bank_statement_import_save_file/tests/__init__.py b/account_bank_statement_import_save_file/tests/__init__.py index 6086380..4859e86 100644 --- a/account_bank_statement_import_save_file/tests/__init__.py +++ b/account_bank_statement_import_save_file/tests/__init__.py @@ -1,5 +1 @@ -# -*- coding: utf-8 -*- -# © 2015 Therp BV (). -# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). - from . import test_save_file diff --git a/account_bank_statement_import_save_file/tests/test_save_file.py b/account_bank_statement_import_save_file/tests/test_save_file.py index b6fbd58..41b957c 100644 --- a/account_bank_statement_import_save_file/tests/test_save_file.py +++ b/account_bank_statement_import_save_file/tests/test_save_file.py @@ -1,6 +1,6 @@ -# -*- coding: utf-8 -*- -# © 2015 Therp BV (). -# © 2017 Today Mourad EL HADJ MIMOUNE +# Copyright 2015-2019 Therp BV (). +# Copyright 2017-Today Mourad EL HADJ MIMOUNE +# # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). import base64 @@ -19,7 +19,7 @@ class HelloWorldParser(models.TransientModel): @api.model def _parse_file(self, data_file): - if module_name in data_file: + if data_file == module_name.encode('utf-8'): return self._mock_parse(data_file) else: return super(HelloWorldParser, self)._parse_file(data_file) @@ -58,14 +58,12 @@ class TestSaveFile(TransactionCase): HelloWorldParser._build_model(self.registry, self.cr) import_wizard = self.env['account.bank.statement.import'] journal_id = self.bank_journal_euro.id + data_file = base64.b64encode(module_name.encode('utf-8')) import_wizard_id = import_wizard.with_context(journal_id=journal_id)\ - .create({ - 'data_file': base64.b64encode(bytes( - 'account_bank_statement_import_save_file: Hello world')) - }) + .create({'data_file': data_file, 'filename': 'test.ofx'}) action = import_wizard_id.import_file() for statement in self.env['account.bank.statement'].browse( action['context']['statement_ids']): self.assertEqual( base64.b64decode(statement.import_file.datas), - 'account_bank_statement_import_save_file: Hello world') + module_name.encode('utf-8'))