Error on importing ING bank statements

Bug #1310841 reported by Kevin van der Draai
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Banking Addons
Invalid
Undecided
Unassigned

Bug Description

I configured OpenERP with Account Banking + ING (NL) bank Statements Import. I was able to import a few bank statements, but after the latest update of Account Banking, I receive the following error after starting the import:

DataError: invalid input syntax for type date: ""
LINE 1: ...e_uid,create_date,write_uid,write_date) values (2,'','','','...
                                                                                                                   ^
The first value (I assume create_date_uid), is sequentially rising after each attempt.
I use the 'komma gescheiden dd-mm-jjjj' format, but this also happened while using 'komma gescheiden jjjjmmdd' and 'Girotel kommagescheiden'.

I tried to reinstall the add-on, this to no success.
The CSV files contain the right data. (i.e. it has the same structure as the previous files, which I could successfully import).

When I remove the bank account from the standard settings (Finance > Settings > Banking > Standard settings), the import add-on reports that the csv file contains bank statements for account ING *******, but that there is no bank account defined in the standard settings.

Attached you will find the complete error message/error log.

Revision history for this message
Kevin van der Draai (info-2t) wrote :
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

From the fact that it happened after an update, I reckon it's due to rev. 241, which changes some date fields around. My first question is, "which bazaar revision are you running", but I'm guessing you installed from the apps menu with which I have no experience. I'm also guessing that the apps menu indicated that account_banking was upgradable because its version number was increased.

Now, the code in the ing module was changed accordingly but I did not increase that module's version number because there was no database scheme changed involved in that module. It could be that the apps menu therefore does not give you access to the required, latest version of the module (even if you 'reinstall' the addon).

Then again, the exact error message does not directly support my guesses. These database query error messages are always truncated, which is nothing you can help but it makes it difficult to find the actual problem.

My first suggestion would be for you to try out account_banking_nl_ing_mt940 instead, at least if you can get mt940 statements for your bank account. I'd very much like to retire the old csv module now that we have an mt940 one.

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Setting to incomplete to indicate I am waiting for a response from the submitter.

Changed in banking-addons:
status: New → Incomplete
Revision history for this message
Kevin van der Draai (info-2t) wrote :

The first time I did install the module from the apps menu. I uninstalled the complete module and reinstalled from the LP branch (also on a clean environment). The CSV import returns again returns the same error.

Unfortunately I can't download MT940 statements. However, I'm building a small CSV to MT940 converter.
Importing an empty MT940 transaction file completes without any errors (other than the file is empty).

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks for the response. You don't indicate that you can download camt.053 statements, but I guess not. That leaves a place for the csv module I suppose.

Changed in banking-addons:
status: Incomplete → New
Revision history for this message
Kevin van der Draai (info-2t) wrote :

I can only download text files, CSV 'kommagescheiden' and Girotel 'kommagescheiden' files.
By retiring the old CSV module I assume a fix/patch will not be created?

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Can't reproduce. Can you send me a sample line privately

https://launchpad.net/~stefan-therp/+contactuser

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Kevin, I received your sample statement. Good news is, I could import it without any problem. Bad news is of course that this does not solve your problem. Are you sure the apps version of the module was removed entirely from the OpenERP addons path (and that OpenERP was restarted after installing the new version of the module, etc.)?

Revision history for this message
Kevin van der Draai (info-2t) wrote :

Hi Stefan,
I used the following set of command (linux):

apt-get install bzr
cd /usr/lib/openerp-7.0*/openerp/addons/
bzr branch lp:banking-addons
reboot

Then, in OpenERP:
Configuration > Update module list (> Update);
Configuration > Installed Modules > remove "installed" from the search box;
Navigate to Account_Banking and install.
After that I performed the same action to install the ING import template.

Is there anything I did wrong in this process?

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Subdirectories in the addons path are not taken into account. You should move the banking-addons directory to something like /usr/local/share/, and add /usr/local/share/banking-addons to the addons path in the server configuration file. You should also remove the existing banking modules from /usr/lib/openerp-7.0*/openerp/addons/.

Changed in banking-addons:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.