Merge lp:~openerp-dev/openobject-addons/trunk-import-bank-statements into lp:openobject-addons

[DJ Patel (OpenERP)]

Hello,

I have created a generic wizard for ofx, qif and coda file format.
It is used to import these types of bank statements into openerp.

Thanks,
Divyesh

[lp:~openerp-dev/openobject-addons/trunk-import-bank-statements]

9257. By DJ Patel (OpenERP) on 2014-03-27

[IMP] account : Improved the code to return an action.

9258. By DJ Patel (OpenERP) on 2014-03-27

[FIX] account, account_qif, l10n_be_coda : Fixed the test cases.

9259. By DJ Patel (OpenERP) on 2014-03-28

[Merge] Merged with trunk.

9260. By DJ Patel (OpenERP) on 2014-03-31

[Merge] Merged with trunk.

9261. By DJ Patel (OpenERP) on 2014-04-01

[Merge] Merged with trunk.

9262. By DJ Patel (OpenERP) on 2014-04-02

[Merge] Merged with trunk.

Unmerged revisions

9262. By DJ Patel (OpenERP) on 2014-04-02

[Merge] Merged with trunk.

9261. By DJ Patel (OpenERP) on 2014-04-01

[Merge] Merged with trunk.

9260. By DJ Patel (OpenERP) on 2014-03-31

[Merge] Merged with trunk.

9259. By DJ Patel (OpenERP) on 2014-03-28

[Merge] Merged with trunk.

9258. By DJ Patel (OpenERP) on 2014-03-27

[FIX] account, account_qif, l10n_be_coda : Fixed the test cases.

9257. By DJ Patel (OpenERP) on 2014-03-27

[IMP] account : Improved the code to return an action.

9256. By DJ Patel (OpenERP) on 2014-03-27

[Merge] Merged the branch: lp:~openerp-dev/openobject-addons/trunk-import-bank-statement with improvements.

=== modified file 'account/__init__.py'
--- account/__init__.py	2012-11-29 22:26:45 +0000
+++ account/__init__.py	2014-04-02 04:28:41 +0000
@@ -38,5 +38,6 @@
 import res_currency
 import edi
 import res_config
+import account_bank_statement_import
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/__openerp__.py'
--- account/__openerp__.py	2014-02-13 17:39:10 +0000
+++ account/__openerp__.py	2014-04-02 04:28:41 +0000
@@ -124,7 +124,7 @@
         'account_bank_view.xml',
         'res_config_view.xml',
         'account_pre_install.yml',
-
+        'account_bank_statement_import_view.xml',
         'views/report_vat.xml',
     ],
     'js': [
 
=== added file 'account/account_bank_statement_import.py'
--- account/account_bank_statement_import.py	1970-01-01 00:00:00 +0000
+++ account/account_bank_statement_import.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,118 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    Copyright (C) 2012-Today OpenERP SA (<http://www.openerp.com>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+import logging
+import base64
+import os
+import time
+
+from openerp.osv import fields, osv
+from openerp.tools.translate import _
+from openerp import tools
+
+_logger = logging.getLogger(__name__)
+
+try:
+    from ofxparse import OfxParser as ofxparser
+except ImportError:
+    _logger.warning("OFX parser partially unavailable because the `ofxparse` Python library cannot be found. "
+                    "It can be easily download and install from this `https://pypi.python.org/pypi/ofxparse`.")
+    ofxparser = None
+
+_IMPORT_FILE_TYPE = [('ofx', 'OFX')]
+
+class account_bank_statement_import(osv.TransientModel):
+    _name = 'account.bank.statement.import'
+    _description = 'Import Bank Statement' 
+    _columns = {
+        'data_file': fields.binary('Bank Statement File', required=True, help='Select bank statement file to import in OpenERP. .OFX, .QIF or CODA are accepted.'),
+        'file_type': fields.selection(_IMPORT_FILE_TYPE, 'File Type'),
+        'journal_id': fields.many2one('account.journal', 'Journal', required=True),
+        'account_id': fields.many2one('account.account', 'Account', domain="[('type', '!=', 'view')]", help=" Choose the account to be credited/debited for the entries of the statement.", required=True),
+    }
+
+    def _get_default_journal(self, cr, uid, context=None):
+        company_id = self.pool.get('res.company')._company_default_get(cr, uid, 'account.bank.statement', context=context)
+        journal_ids = self.pool.get('account.journal').search(cr, uid, [('type', '=', 'bank'), ('company_id', '=', company_id)], context=context)
+        return journal_ids and journal_ids[0] or False
+
+    _defaults = {
+        'file_type': 'ofx',
+        'journal_id': _get_default_journal,
+    }
+
+    def import_bank_statement(self, cr, uid, bank_statement_vals=False, context=None):
+        statement_id = self.pool.get('account.bank.statement').create(cr, uid, bank_statement_vals, context=context)
+        return statement_id
+
+    def process_ofx(self, cr, uid, data_file, journal_id=False, account_id=False, context=None):
+        try:
+            tempfile = open( "temp.ofx", "w+" )
+            tempfile.write(base64.decodestring(data_file))
+            tempfile.read()
+            pathname = os.path.dirname('temp.ofx')
+            path = os.path.join(os.path.abspath(pathname), 'temp.ofx')
+            ofx = ofxparser.parse(file(path))
+        except:
+            raise osv.except_osv(_('Import Error!'), _('Please check OFX file format is proper or not.'))
+        line_ids = []
+        total_amt = 0.00
+        try:
+            for transaction in ofx.account.statement.transactions:
+                vals_line = {
+                    'date' : transaction.date,
+                    'name' : transaction.memo,
+                    'ref': transaction.id,
+                    'amount' : transaction.amount,
+                    'account_id': account_id
+                }
+                total_amt += float(transaction.amount)
+                line_ids.append((0, 0, vals_line))
+        except Exception,e:
+            raise osv.except_osv(_('Error!'), _("Following problem has been occurred while importing your file, Please verify the file is proper or not.\n\n %s" % e.message))
+        st_start_date = ofx.account.statement.start_date or False
+        st_end_date = ofx.account.statement.end_date or False
+        period_obj = self.pool.get('account.period')
+        if st_end_date:
+            period_ids = period_obj.find(cr, uid, st_end_date, context=context)
+        else:
+            period_ids = period_obj.find(cr, uid, st_start_date, context=context)
+        vals_bank_statement = {
+            'name': ofx.account.routing_number,
+            'balance_start': ofx.account.statement.balance,
+            'balance_end_real': float(ofx.account.statement.balance) + total_amt,
+            'period_id': period_ids and period_ids[0] or False,
+            'journal_id': journal_id
+        }
+        vals_bank_statement.update({'line_ids': line_ids})
+        os.remove(path)
+        return vals_bank_statement
+
+    def parse_file(self, cr, uid, ids, context=None):
+        data = self.browse(cr, uid, ids[0], context=context)
+        vals = getattr(self, "process_%s" % data.file_type)(cr, uid, data.data_file, data.journal_id.id, data.account_id.id, context=context)
+        statement_id = self.import_bank_statement(cr, uid, vals, context=context)
+        model, action_id = self.pool.get('ir.model.data').get_object_reference(cr, uid, 'account', 'action_bank_statement_tree')
+        action = self.pool[model].read(cr, uid, action_id, context=context)
+        action['domain'] = "[('id', 'in', ["+', '.join(map(str, [statement_id]))+"])]"
+        return action
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== added file 'account/account_bank_statement_import_view.xml'
--- account/account_bank_statement_import_view.xml	1970-01-01 00:00:00 +0000
+++ account/account_bank_statement_import_view.xml	2014-04-02 04:28:41 +0000
@@ -0,0 +1,47 @@
+<?xml version="1.0" ?>
+<openerp>
+    <data>
+
+        <record id="account_bank_statement_import_view" model="ir.ui.view">
+            <field name="name">Import Bank Statements</field>
+            <field name="model">account.bank.statement.import</field>
+            <field name="priority">1</field>
+            <field name="arch" type="xml">
+                <form string="Import Bank Statements" version="7.0">
+                    <group>
+                        <group>
+                            <field name="file_type" required="1"/>
+                            <field name="data_file" attrs="{'invisible':[('file_type','=',False)]}"/>
+                            <field name="journal_id" domain="[('type', '=', 'bank')]" attrs="{'invisible':[('file_type','=',False)]}" context="{'default_type':'bank'}"/>
+                            <field name="account_id" attrs="{'invisible':[('file_type','=',False)]}"/>
+                        </group>
+                        <group>
+                            <b colspan="2"> How to import your bank statement in OpenERP.</b>
+                            <label string= "1. Go to your bank account website." colspan="2"/>
+                            <label string= "2. Download your bank statements in the right format. (.OFX, .QIF or CODA are accepted)" colspan="2"/>
+                            <label string= "3. Upload right here the bank statements file into OpenERP. Click Import." colspan="2"/>
+                        </group>
+                    </group>
+                    <footer>
+                        <button name="parse_file" string="_Import" type="object" class="oe_highlight"/>
+                        or
+                        <button string="Cancel" class="oe_link" special="cancel"/>
+                    </footer>
+                </form>
+            </field>
+        </record>
+
+        <record id="action_account_bank_statement_import" model="ir.actions.act_window">
+            <field name="name">Import Bank Statements</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="res_model">account.bank.statement.import</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">form</field>
+            <field name="target">new</field>
+            <field name="view_id" ref="account_bank_statement_import_view"/>
+        </record>
+
+        <menuitem parent="account.menu_finance_bank_and_cash" id="menu_account_bank_statement_import" action="action_account_bank_statement_import" sequence="11"/>
+
+    </data>
+</openerp>
 
=== modified file 'account/tests/__init__.py'
--- account/tests/__init__.py	2013-12-06 17:00:12 +0000
+++ account/tests/__init__.py	2014-04-02 04:28:41 +0000
@@ -1,7 +1,11 @@
 from . import test_tax
+from . import test_import_bank_statement
 from . import test_search
 
 fast_suite = [
 	test_tax,
 	test_search,
 ]
+checks = [
+        test_import_bank_statement
+]
 
=== added file 'account/tests/test_import_bank_statement.py'
--- account/tests/test_import_bank_statement.py	1970-01-01 00:00:00 +0000
+++ account/tests/test_import_bank_statement.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,73 @@
+from openerp.tests.common import TransactionCase
+
+ofx_file = """PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iQVNDSUkiPz4NCjw/
+        T0ZYIE9GWEhFQURFUj0iMjAwIiBWRVJTSU9OPSIyMTEiIFNFQ1VSSVRZPSJOT05FIiBPTE
+        RGSUxFVUlEPSJOT05FIiBORVdGSUxFVUlEPSJOT05FIj8+DQo8T0ZYPg0KICA8U0lHTk9O
+        TVNHU1JTVjE+DQogICAgPFNPTlJTPg0KICAgICAgPFNUQVRVUz4NCiAgICAgICAgPENPRE
+        U+MDwvQ09ERT4NCiAgICAgICAgPFNFVkVSSVRZPklORk88L1NFVkVSSVRZPg0KICAgICAg
+        PC9TVEFUVVM+DQogICAgICA8RFRTRVJWRVI+MjAwNTA4MzExNjUxNTMuMDAwWy04OlBTVF
+        08L0RUU0VSVkVSPg0KICAgICAgPExBTkdVQUdFPkVORzwvTEFOR1VBR0U+DQogICAgPC9T
+        T05SUz4NCiAgPC9TSUdOT05NU0dTUlNWMT4NCiAgPEJBTktNU0dTUlNWMT4NCiAgICA8U1
+        RNVFRSTlJTPg0KICAgICAgPFRSTlVJRD4wPC9UUk5VSUQ+DQogICAgICA8U1RBVFVTPg0K
+        ICAgICAgICA8Q09ERT4wPC9DT0RFPg0KICAgICAgICA8U0VWRVJJVFk+SU5GTzwvU0VWRV
+        JJVFk+DQogICAgICA8L1NUQVRVUz4NCiAgICAgIDxTVE1UUlM+DQogICAgICAgIDxDVVJE
+        RUY+VVNEPC9DVVJERUY+DQogICAgICAgIDxCQU5LQUNDVEZST00+DQogICAgICAgICAgPE
+        JBTktJRD4wMDAwMDAxMjM8L0JBTktJRD4NCiAgICAgICAgICA8QUNDVElEPjEyMzQ1Njwv
+        QUNDVElEPg0KICAgICAgICAgIDxBQ0NUVFlQRT5DSEVDS0lORzwvQUNDVFRZUEU+DQogIC
+        AgICAgIDwvQkFOS0FDQ1RGUk9NPg0KICAgICAgICA8QkFOS1RSQU5MSVNUPg0KICAgICAg
+        ICAgIDxEVFNUQVJUPjIwMTQwODAxPC9EVFNUQVJUPg0KICAgICAgICAgIDxEVEVORD4yMD
+        E0MDgzMTE2NTE1My4wMDBbLTg6UFNUXTwvRFRFTkQ+DQogICAgICAgICAgPFNUTVRUUk4+
+        DQogICAgICAgICAgICA8VFJOVFlQRT5QT1M8L1RSTlRZUEU+DQogICAgICAgICAgICA8RF
+        RQT1NURUQ+MjAwNTA4MjQwODAwMDA8L0RUUE9TVEVEPg0KICAgICAgICAgICAgPFRSTkFN
+        VD4tODA8L1RSTkFNVD4NCiAgICAgICAgICAgIDxGSVRJRD4yMTkzNzg8L0ZJVElEPg0KIC
+        AgICAgICAgICAgPE5BTUU+RnJvZ0tpY2sgU2N1YmEgR2VhcjwvTkFNRT4NCiAgICAgICAg
+        ICA8L1NUTVRUUk4+DQogICAgICAgIDwvQkFOS1RSQU5MSVNUPg0KICAgICAgICA8TEVER0
+        VSQkFMPg0KICAgICAgICAgIDxCQUxBTVQ+MjE1Ni41NjwvQkFMQU1UPg0KICAgICAgICAg
+        IDxEVEFTT0Y+MjAwNTA4MzExNjUxNTM8L0RUQVNPRj4NCiAgICAgICAgPC9MRURHRVJCQU
+        w+DQogICAgICA8L1NUTVRSUz4NCiAgICA8L1NUTVRUUk5SUz4NCiAgPC9CQU5LTVNHU1JT
+        VjE+DQogIDxDUkVESVRDQVJETVNHU1JTVjE+DQogICAgPENDU1RNVFRSTlJTPg0KICAgIC
+        AgPFRSTlVJRD4wPC9UUk5VSUQ+DQogICAgICA8U1RBVFVTPg0KICAgICAgICA8Q09ERT4w
+        PC9DT0RFPg0KICAgICAgICA8U0VWRVJJVFk+SU5GTzwvU0VWRVJJVFk+DQogICAgICA8L1
+        NUQVRVUz4NCiAgICAgIDxDQ1NUTVRSUz4NCiAgICAgICAgPENVUkRFRj5VU0Q8L0NVUkRF
+        Rj4NCiAgICAgICAgPENDQUNDVEZST00+DQogICAgICAgICAgPEFDQ1RJRD4xMjM0MTIzND
+        EyMzQ8L0FDQ1RJRD4NCiAgICAgICAgPC9DQ0FDQ1RGUk9NPg0KICAgICAgICA8QkFOS1RS
+        QU5MSVNUPg0KICAgICAgICAgIDxEVFNUQVJUPjIwMTQwODAxPC9EVFNUQVJUPg0KICAgIC
+        AgICAgIDxEVEVORD4yMDE0MDgzMTE2NTE1My4wMDBbLTg6UFNUXTwvRFRFTkQ+DQogICAg
+        ICAgICAgPFNUTVRUUk4+DQogICAgICAgICAgICA8VFJOVFlQRT5JTlQ8L1RSTlRZUEU+DQ
+        ogICAgICAgICAgICA8RFRQT1NURUQ+MjAwNTA4MTEwODAwMDA8L0RUUE9TVEVEPg0KICAg
+        ICAgICAgICAgPFRSTkFNVD4tMjMuMDA8L1RSTkFNVD4NCiAgICAgICAgICAgIDxGSVRJRD
+        4yMTk4Njc8L0ZJVElEPg0KICAgICAgICAgICAgPE5BTUU+SW50ZXJlc3QgQ2hhcmdlPC9O
+        QU1FPg0KICAgICAgICAgIDwvU1RNVFRSTj4NCiAgICAgICAgICA8U1RNVFRSTj4NCiAgIC
+        AgICAgICAgIDxUUk5UWVBFPkNSRURJVDwvVFJOVFlQRT4NCiAgICAgICAgICAgIDxEVFBP
+        U1RFRD4yMDA1MDgxMTA4MDAwMDwvRFRQT1NURUQ+DQogICAgICAgICAgICA8VFJOQU1UPj
+        M1MC4wMDwvVFJOQU1UPg0KICAgICAgICAgICAgPEZJVElEPjIxOTg2ODwvRklUSUQ+DQog
+        ICAgICAgICAgICA8TkFNRT5QYXltZW50IC0gVGhhbmsgWW91PC9OQU1FPg0KICAgICAgIC
+        AgIDwvU1RNVFRSTj4NCiAgICAgICAgPC9CQU5LVFJBTkxJU1Q+DQogICAgICAgIDxMRURH
+        RVJCQUw+DQogICAgICAgICAgPEJBTEFNVD4tNTYyLjAwPC9CQUxBTVQ+DQogICAgICAgIC
+        AgPERUQVNPRj4yMDA1MDgzMTE2NTE1MzwvRFRBU09GPg0KICAgICAgICA8L0xFREdFUkJB
+        TD4NCiAgICAgIDwvQ0NTVE1UUlM+DQogICAgPC9DQ1NUTVRUUk5SUz4NCiAgPC9DUkVESV
+        RDQVJETVNHU1JTVjE+DQo8L09GWD4NCg=="""
+
+class TestOfxFile(TransactionCase):
+    """Tests for import bank statement ofx file format (account.bank.statement.import)
+    """
+
+    def setUp(self):
+        super(TestOfxFile, self).setUp()
+        self.statement_import_model = self.registry('account.bank.statement.import')
+        self.bank_statement_model = self.registry('account.bank.statement')
+
+    def test_ofx_file_import(self):
+        cr, uid = self.cr, self.uid
+        bank_temp_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'account', 'conf_bnk')
+        self.bank_temp_id = bank_temp_ref and bank_temp_ref[1] or False
+        bank_statement_id = self.statement_import_model.create(cr, uid, dict(
+                            file_type = 'ofx',
+                            data_file = ofx_file,
+                            account_id = self.bank_temp_id
+                             ))
+        self.statement_import_model.parse_file(cr, uid, [bank_statement_id])
+        statement_id = self.bank_statement_model.search(cr, uid, [('name', '=', '000000123')])[0]
+        bank_st_record = self.bank_statement_model.browse(cr, uid, statement_id)
+        self.assertEquals(bank_st_record.balance_start, 2156.56)
+        self.assertEquals(bank_st_record.balance_end_real, 2076.56)
 
=== modified file 'account/wizard/account_financial_report.py'
--- account/wizard/account_financial_report.py	2013-10-27 12:31:04 +0000
+++ account/wizard/account_financial_report.py	2014-04-02 04:28:41 +0000
@@ -2,7 +2,7 @@
 ##############################################################################
 #
 #    OpenERP, Open Source Management Solution
-#    Copyright (C) 2004-2010 Tiny SPRL (<http://tiny.be>).
+#    Copyright (C) 2012-Today OpenERP SA (<http://www.openerp.com>).
 #
 #    This program is free software: you can redistribute it and/or modify
 #    it under the terms of the GNU Affero General Public License as
 
=== added directory 'account_qif'
=== added file 'account_qif/__init__.py'
--- account_qif/__init__.py	1970-01-01 00:00:00 +0000
+++ account_qif/__init__.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    Copyright (C) 2012-Today OpenERP SA (<http://www.openerp.com>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+import account_qif
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== added file 'account_qif/__openerp__.py'
--- account_qif/__openerp__.py	1970-01-01 00:00:00 +0000
+++ account_qif/__openerp__.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,49 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    Copyright (C) 2012-Today OpenERP SA (<http://www.openerp.com>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+{
+    'name': 'Import QIF Bank Statement',
+    'version': '1.1',
+    'author': 'OpenERP SA',
+    'category': 'Accounting & Finance',
+    'description': '''
+Module to import QIF bank statements.
+======================================
+
+The machine readable QIF Files are parsed and stored in human readable format in 
+Bank Statements. Also Bank Statements are generated containing a subset of 
+the QIF information (only those transaction lines that are required for the 
+creation of the Financial Accounting records). The Bank Statement is a 
+'read-only' object, hence remaining a reliable representation of the original
+QIF file whereas the Bank Statement will get modified as required by accounting 
+business processes.
+
+QIF Bank Accounts configured as type 'QIF' will only generate QIF Bank Statements.
+''',
+    'images' : [],
+    'depends': ['account'],
+    'demo': [],
+    'data': [],
+    'auto_install': False,
+    'installable': True,
+}
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== added file 'account_qif/account_qif.py'
--- account_qif/account_qif.py	1970-01-01 00:00:00 +0000
+++ account_qif/account_qif.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,96 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    Copyright (C) 2012-Today OpenERP SA (<http://www.openerp.com>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+import dateutil.parser
+import base64
+import logging
+from tempfile import TemporaryFile
+
+from openerp.tools.translate import _
+from openerp.osv import fields, osv
+from openerp import tools
+
+_logger = logging.getLogger(__name__)
+
+from openerp.addons.account import account_bank_statement_import as ibs
+
+ibs._IMPORT_FILE_TYPE.append(('qif', 'QIF'))
+
+class account_bank_statement_import(osv.TransientModel):
+    _inherit = "account.bank.statement.import"
+
+    _columns = {
+        'file_type': fields.selection(ibs._IMPORT_FILE_TYPE, 'File Type'),
+    }
+
+    def process_qif(self, cr, uid, data_file, journal_id=False, account_id=False, context=None):
+        try:
+            fileobj = TemporaryFile('wb+')
+            fileobj.write(base64.b64decode(data_file))
+            fileobj.seek(0)
+            file_data = ""
+            for line in fileobj.readlines():
+                file_data += line
+            fileobj.close()
+            if '\r' in file_data:
+                data_list = file_data.split('\r')
+            else:
+                data_list = file_data.split('\n')
+            header = data_list[0].strip()
+            header = header.split(":")[1]
+        except:
+            raise osv.except_osv(_('Import Error!'), _('Please check QIF file format is proper or not.'))
+        line_ids = []
+        vals_line = {}
+        total = 0
+        if header == "Bank":
+            vals_bank_statement = {}
+            for line in data_list:
+                line = line.strip()
+                if not line: continue
+                if line[0] == 'D': # date of transaction 
+                    vals_line['date'] = dateutil.parser.parse(line[1:], fuzzy=True).date()
+                    if vals_line.get('date'):
+                        period_ids = self.pool.get('account.period').find(cr, uid, vals_line['date'], context=context)
+                        vals_bank_statement.update({'period_id': period_ids and period_ids[0] or False})
+                elif line[0] == 'T': # Total amount
+                    total += float(line[1:].replace(',', ''))
+                    vals_line['amount'] = float(line[1:].replace(',', ''))
+                elif line[0] == 'N': # Check number
+                    vals_line['ref'] = line[1:]
+                elif line[0] == 'P': # Payee
+                    vals_line['name'] = line[1:]
+                elif line[0] == '^': # end of item
+                    vals_line['account_id'] = account_id
+                    line_ids.append((0, 0, vals_line))
+                    vals_line = {}
+                elif line[0] == '\n':
+                    line_ids = []
+                else:
+                    pass
+        else:
+            raise osv.except_osv(_('Error!'), _('Cannot support this Format !Type:%s.') % (header,))
+        vals_bank_statement.update({'balance_end_real': total,
+                                    'line_ids': line_ids,
+                                    'journal_id': journal_id})
+        return vals_bank_statement
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== added directory 'account_qif/tests'
=== added file 'account_qif/tests/__init__.py'
--- account_qif/tests/__init__.py	1970-01-01 00:00:00 +0000
+++ account_qif/tests/__init__.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,4 @@
+from . import test_import_bank_statement
+checks = [
+    test_import_bank_statement
+]
 
=== added file 'account_qif/tests/test_import_bank_statement.py'
--- account_qif/tests/test_import_bank_statement.py	1970-01-01 00:00:00 +0000
+++ account_qif/tests/test_import_bank_statement.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,30 @@
+from openerp.tests.common import TransactionCase
+
+qif_file = """IVR5cGU6QmFuawpEOC8xMi8xNApULTEsMDAwLjAwClBGcmFua3MgUGx1bWJpbmcKXgpEOC8xNS8xNApULTc1L
+              jQ2ClBXYWx0cyBEcnVncwpeCkQzLzMvMTQKVC0zNzkuMDAKUENJVFkgT0YgU1BSSU5HRklFTEQKXgpEMy80LzE0ClQtMjAuM
+              jgKUFlPVVIgTE9DQUwgU1VQRVJNQVJLRVQKXgpEMy8zLzE0ClQtNDIxLjM1ClBTUFJJTkdGSUVMRCBXQVRFUiBVVElMSVRZCl4K"""
+
+class TestQifFile(TransactionCase):
+    """Tests for import bank statement qif file format (account.bank.statement.import)
+    """
+
+    def setUp(self):
+        super(TestQifFile, self).setUp()
+        self.statement_import_model = self.registry('account.bank.statement.import')
+        self.bank_statement_model = self.registry('account.bank.statement')
+        self.bank_statement_line_model = self.registry('account.bank.statement.line')
+
+    def test_qif_file_import(self):
+        cr, uid = self.cr, self.uid
+        bank_temp_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'account', 'conf_bnk')
+        self.bank_temp_id = bank_temp_ref and bank_temp_ref[1] or False
+        bank_statement_id = self.statement_import_model.create(cr, uid, dict(
+                            file_type = 'qif',
+                            data_file = qif_file,
+                            account_id = self.bank_temp_id
+                             ))
+        self.statement_import_model.parse_file(cr, uid, [bank_statement_id])
+        line_id = self.bank_statement_line_model.search(cr, uid, [('name', '=', 'YOUR LOCAL SUPERMARKET')])[0]
+        statement_id = self.bank_statement_line_model.browse(cr, uid, line_id).statement_id.id
+        bank_st_record = self.bank_statement_model.browse(cr, uid, statement_id)
+        self.assertEquals(bank_st_record.balance_end_real, -1896.09)
 
=== modified file 'l10n_be_coda/__openerp__.py'
--- l10n_be_coda/__openerp__.py	2012-12-06 11:18:16 +0000
+++ l10n_be_coda/__openerp__.py	2014-04-02 04:28:41 +0000
@@ -93,7 +93,6 @@
     'depends': ['account_voucher','base_iban', 'l10n_be_invoice_bba',],
     'demo': [],
     'data': [
-        'l10n_be_coda_wizard.xml',
         'l10n_be_coda_view.xml',
     ],
     'auto_install': False,
 
=== modified file 'l10n_be_coda/l10n_be_coda_view.xml'
--- l10n_be_coda/l10n_be_coda_view.xml	2013-10-27 12:31:04 +0000
+++ l10n_be_coda/l10n_be_coda_view.xml	2014-04-02 04:28:41 +0000
@@ -93,6 +93,5 @@
 		</record>
 
         <menuitem name="Bank Statement Lines" parent="account.menu_finance_bank_and_cash" id="menu_account_bank_statement_line_coda" action="action_account_bank_statement_line_coda" sequence="8" groups="base.group_no_one"/>
-    	<menuitem name="Import CODA File" parent="account.menu_finance_bank_and_cash" id="menu_account_coda_import" action="action_account_coda_import" sequence="10"/>
 	</data> 
-</openerp>
 \ No newline at end of file
+</openerp>
 
=== removed file 'l10n_be_coda/l10n_be_coda_wizard.xml'
--- l10n_be_coda/l10n_be_coda_wizard.xml	2012-12-20 14:52:27 +0000
+++ l10n_be_coda/l10n_be_coda_wizard.xml	1970-01-01 00:00:00 +0000
@@ -1,35 +0,0 @@
-<?xml version="1.0" ?>
-<openerp>
-    <data>
-
-    <record id="account_coda_import_view" model="ir.ui.view">
-      <field name="name">Import CODA File</field>
-      <field name="model">account.coda.import</field>
-      <field name="priority">1</field>
-      <field name="arch" type="xml">
-        <form string="Import CODA File" version="7.0">
-            <group col="2">
-                <field name="coda_data" filename="coda_fname"/>
-                <field name="temporary_account_id" />
-            </group>
-            <footer>
-                <button name="coda_parsing" string="_Import" type="object" class="oe_highlight"/>
-                or
-                <button string="Cancel" class="oe_link" special="cancel"/>
-            </footer>
-        </form>
-      </field>
-    </record>
-
-    <record id="action_account_coda_import" model="ir.actions.act_window">
-      <field name="name">Import CODA File</field>
-      <field name="type">ir.actions.act_window</field>
-      <field name="res_model">account.coda.import</field>
-      <field name="view_type">form</field>
-      <field name="view_mode">form</field>
-      <field name="target">new</field>
-      <field name="view_id" ref="account_coda_import_view"/>
-    </record>
-    
-  </data> 
-</openerp>
 
=== renamed file 'l10n_be_coda/test_coda_file/Ontvangen_CODA.2011-01-11-18.59.15.txt' => 'l10n_be_coda/test_coda_file/Ontvangen_CODA.2013-01-11-18.59.15.txt'
--- l10n_be_coda/test_coda_file/Ontvangen_CODA.2011-01-11-18.59.15.txt	2012-01-31 13:36:57 +0000
+++ l10n_be_coda/test_coda_file/Ontvangen_CODA.2013-01-11-18.59.15.txt	2014-04-02 04:28:41 +0000
@@ -1,24 +1,24 @@
-0000011011172505        00178299  DE MEYER LUC              KREDBEBB   00820512013 00000                                       2
+0000011011472505        00178299  DE MEYER LUC              KREDBEBB   00820512014 00000                                       2
 12135BE33737018595246                  EUR0000000011812700270710NOVIAT NV                 KBC-Business Comfortrekening       003
-2100010000OL44483FW SCTOFBIONLO1000000000435000110111001010000MEDEDELING                                           11011113501 0
[phone]OL44483FW SCTOFBIONLO1000000000435000110114001010000MEDEDELING                                           11011413501 0
 2200010000                                                                                        GKCCBEBB                   1 0
 2300010000BE41063012345610                     PARTNER 1                                                                     0 1
 3100010001OL44483FW SCTOFBIONLO001010001001PARTNER 1                                                                         0 0
-2100020000OL4414AC8BOVSOVSOVERS00000000030444501101110015000002010237                                              11011113501 0
[phone]OL4414AC8BOVSOVSOVERS00000000030444501101140015000002010237                                              11011413501 0
 2200020000                                                                                        BBRUBEBB                   1 0
 2300020000BE61310126985517                     PARTNER 2                                                                     0 1
 3100020001OL4414AC8BOVSOVSOVERS001500001001PARTNER 2                                                                         1 0
 3200020001MOLENSTRAAT 60                     9340    LEDE                                                                    0 0
-2100030000AFECA0CVA IKLINNINNIG1000000000479040110111313410000              KBC-INVESTERINGSKREDIET 737-6543210-21 11011113510 0
-2100030001AFECA0CVA IKLINNINNIG1000000000419920110111813410660                                                     11011113500 0
-2100030002AFECA0CVA IKLINNINNIG1000000000059120110111813410020                                                     11011113510 0
-2100040000AFECA0CVA IKLINNINNIG1000000000479040110111313410000              KBC-INVESTERINGSKREDIET 737-6543210-21 11011113510 0
-2100040001AFECA0CVA IKLINNINNIG1000000000419920110111813410660                                                     11011113500 0
-2100040002AFECA0CVA IKLINNINNIG1000000000059120110111813410020                                                     11011113510 0
-2100050000AOGM00160BSCTOBOGOVER0000000000063740110111001500000TERUGGAVE 37232481 8400083296 .                      11011113501 0
[phone]AFECA0CVA IKLINNINNIG1000000000479040110114313410000              KBC-INVESTERINGSKREDIET 737-6543210-21 11011413510 0
[phone]AFECA0CVA IKLINNINNIG1000000000419920110114813410660                                                     11011413500 0
[phone]AFECA0CVA IKLINNINNIG1000000000059120110114813410020                                                     11011413510 0
[phone]AFECA0CVA IKLINNINNIG1000000000479040110114313410000              KBC-INVESTERINGSKREDIET 737-6543210-21 11011413510 0
[phone]AFECA0CVA IKLINNINNIG1000000000419920110114813410660                                                     11011413500 0
[phone]AFECA0CVA IKLINNINNIG1000000000059120110114813410020                                                     11011413510 0
[phone]AOGM00160BSCTOBOGOVER0000000000063740110114001500000TERUGGAVE 37232481 8400083296 .                      11011413501 0
 2200050000                                                     362/363                            KREDBEBB                   1 0
 2300050000BE43730004200601                     KBC VERZEKERINGEN NV                                                          0 1
 3100050001AOGM00160BSCTOBOGOVER001500001001KBC VERZEKERINGEN NV                                                              1 0
 3200050001VAN OVERSTRAETENPLEIN 2            3000    LEUVEN                                                                  0 0
-8135BE44734024486445                  EUR0000000013527810110111                                                                0
+8135BE44734024486445                  EUR0000000013527810110114                                                                0
 9               000022000000001393080000000003108190                                                                           2
 
=== added directory 'l10n_be_coda/tests'
=== added file 'l10n_be_coda/tests/__init__.py'
--- l10n_be_coda/tests/__init__.py	1970-01-01 00:00:00 +0000
+++ l10n_be_coda/tests/__init__.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,4 @@
+from . import test_import_bank_statement
+checks = [
+    test_import_bank_statement
+]
 
=== added file 'l10n_be_coda/tests/test_import_bank_statement.py'
--- l10n_be_coda/tests/test_import_bank_statement.py	1970-01-01 00:00:00 +0000
+++ l10n_be_coda/tests/test_import_bank_statement.py	2014-04-02 04:28:41 +0000
@@ -0,0 +1,39 @@
+from openerp.tests.common import TransactionCase
+from openerp.modules.module import get_module_resource
+
+class TestCodaFile(TransactionCase):
+    """Tests for import bank statement coda file format (account.bank.statement.import)
+    """
+
+    def setUp(self):
+        super(TestCodaFile, self).setUp()
+        self.statement_import_model = self.registry('account.bank.statement.import')
+        self.bank_statement_model = self.registry('account.bank.statement')
+
+    def test_coda_file_import(self):
+        cr, uid = self.cr, self.uid
+        bank_temp_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'account', 'conf_bnk')
+        partner_id_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'base', 'main_partner')
+        company_id_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'base', 'main_company')
+        self.bank_temp_id = bank_temp_ref and bank_temp_ref[1] or False
+        self.partner_id = partner_id_ref and partner_id_ref[1] or False
+        self.company_id = company_id_ref and company_id_ref[1] or False
+        coda_file_path = get_module_resource('l10n_be_coda', 'test_coda_file', 'Ontvangen_CODA.2013-01-11-18.59.15.txt')
+        coda_file = open(coda_file_path, 'rb').read().encode('base64')
+        bank_account_id = self.registry('res.partner.bank').create(cr, uid, dict(
+                        state = 'bank',
+                        acc_number = 'BE33737018595246',
+                        bank_name = 'Reserve',
+                        partner_id = self.partner_id,
+                        company_id = self.company_id
+                        ))
+        bank_statement_id = self.statement_import_model.create(cr, uid, dict(
+                        file_type = 'coda',
+                        data_file = coda_file,
+                        account_id = self.bank_temp_id
+                         ))
+        self.statement_import_model.parse_file(cr, uid, [bank_statement_id])
+        statement_id = self.bank_statement_model.search(cr, uid, [('name', '=', '135')])[0]
+        bank_st_record = self.bank_statement_model.browse(cr, uid, statement_id)
+        self.assertEquals(bank_st_record.balance_start, 11812.70)
+        self.assertEquals(bank_st_record.balance_end_real, 13527.81)
 
=== modified file 'l10n_be_coda/wizard/account_coda_import.py'
--- l10n_be_coda/wizard/account_coda_import.py	2013-10-27 12:31:04 +0000
+++ l10n_be_coda/wizard/account_coda_import.py	2014-04-02 04:28:41 +0000
@@ -30,44 +30,21 @@
 
 _logger = logging.getLogger(__name__)
 
-class account_coda_import(osv.osv_memory):
-    _name = 'account.coda.import'
+from openerp.addons.account import account_bank_statement_import as coda_ibs
+
+coda_ibs._IMPORT_FILE_TYPE.append(('coda', 'CODA'))
+
+class account_bank_statement_import(osv.TransientModel):
+    _inherit = "account.bank.statement.import"
     _description = 'Import CODA File'
     _columns = {
-        'coda_data': fields.binary('CODA File', required=True),
-        'coda_fname': fields.char('CODA Filename', size=128, required=True),
+        'file_type': fields.selection(coda_ibs._IMPORT_FILE_TYPE, 'File Type'),
         'note': fields.text('Log'),
-        'temporary_account_id': fields.many2one('account.account', 'Temporary Account', domain="[('type','!=','view')]", help="It acts as a temporary account for general amount", required=True),
-    }
-
-    def _get_default_tmp_account(self, cr, uid, context):
-        tmp_accounts = self.pool.get('account.account').search(cr, uid, [('code', '=', '490000')])
-        if tmp_accounts and len(tmp_accounts) > 0:
-            tmp_account_id = tmp_accounts[0]
-        else:
-            tmp_account_id = False
-        return tmp_account_id
-
-    _defaults = {
-        'coda_fname': lambda *a: '',
-        'temporary_account_id': _get_default_tmp_account,
-    }
-
-    def coda_parsing(self, cr, uid, ids, context=None, batch=False, codafile=None, codafilename=None):
+    }
+
+    def process_coda(self, cr, uid, codafile, journal_id=False, account_id=False, context=None):
         if context is None:
             context = {}
-        if batch:
-            codafile = str(codafile)
-            codafilename = codafilename
-        else:
-            data = self.browse(cr, uid, ids)[0]
-            try:
-                codafile = data.coda_data
-                codafilename = data.coda_fname
-                temporaryaccount = data.temporary_account_id.id
-            except:
-                raise osv.except_osv(_('Error'), _('Wizard in incorrect state. Please hit the Cancel button'))
-                return {}
         recordlist = unicode(base64.decodestring(codafile), 'windows-1252', 'strict').split('\n')
         statements = []
         for line in recordlist:
@@ -254,6 +231,7 @@
                     statement['balance_end_real'] = statement['balance_start'] + statement['balancePlus'] - statement['balanceMin']
         for i, statement in enumerate(statements):
             statement['coda_note'] = ''
+            statement_line = []
             balance_start_check_date = (len(statement['lines']) > 0 and statement['lines'][0]['entryDate']) or statement['date']
             cr.execute('SELECT balance_end_real \
                 FROM account_bank_statement \
@@ -278,7 +256,6 @@
                 'balance_start': statement['balance_start'],
                 'balance_end_real': statement['balance_end_real'],
             }
-            statement['id'] = self.pool.get('account.bank.statement').create(cr, uid, data, context=context)
             for line in statement['lines']:
                 if line['type'] == 'information':
                     statement['coda_note'] = "\n".join([statement['coda_note'], line['type'].title() + ' with Ref. ' + str(line['ref']), 'Date: ' + str(line['entryDate']), 'Communication: ' + line['communication'], ''])
@@ -372,12 +349,12 @@
                                     if partner.supplier:
                                         line['transaction_type'] = 'supplier'
                     if not partner and not invoice:
-                        line['account'] = temporaryaccount
+                        line['account'] = account_id
                     if 'communication' in line and line['communication'] != '':
                         note.append(_('Communication') + ': ' + line['communication'])
                     if 'voucher_id' not in line:
                         line['voucher_id'] = None
-                    data = {
+                    line_data = {
                         'name': line['name'],
                         'note':  "\n".join(note),
                         'date': line['entryDate'],
@@ -385,28 +362,16 @@
                         'type': line['transaction_type'],
                         'partner_id': partner_id,
                         'account_id': line['account'],
-                        'statement_id': statement['id'],
                         'ref': line['ref'],
                         'sequence': line['sequence'],
                         'voucher_id': line['voucher_id'],
                         'coda_account_number': line['counterpartyNumber'],
                     }
-                    self.pool.get('account.bank.statement.line').create(cr, uid, data, context=context)
+                    statement_line.append((0, 0, line_data))
             if statement['coda_note'] != '':
-                self.pool.get('account.bank.statement').write(cr, uid, [statement['id']], {'coda_note': statement['coda_note']}, context=context)
-        model, action_id = self.pool.get('ir.model.data').get_object_reference(cr, uid, 'account', 'action_bank_statement_tree')
-        action = self.pool[model].browse(cr, uid, action_id, context=context)
-        return {
-            'name': action.name,
-            'view_type': action.view_type,
-            'view_mode': action.view_mode,
-            'res_model': action.res_model,
-            'domain': action.domain,
-            'context': action.context,
-            'type': 'ir.actions.act_window',
-            'search_view_id': action.search_view_id.id,
-            'views': [(v.view_id.id, v.view_mode) for v in action.view_ids]
-        }
+                data.update({'coda_note': statement['coda_note']})
+            data.update({'journal_id': journal_id, 'line_ids' : statement_line})
+        return data
 
 
 def rmspaces(s):