regression in load() of account.move.lines introduced in r9980

[Leonardo Pistone]

After r9980 of the v7 addons

http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/revision/9980

The import of account.move.lines breaks because it does not recognise the period column.

As a sidenode, I realized that just because I was launching the tests of the module in:

https://code.launchpad.net/~camptocamp/openerp-connector-file/adding-initial-lep/+merge/207676

And there happens to be a few tests that integrate with the standard load() function and fail.

The module linked above contains the csv files I am importing, which work with the standard demo data.

Thanks!

[Denis Ledoux (OpenERP) (dle-openerp)]

I cannot reproduce the problem at my end. Can you please provide the problematic csv file that you mention ? I can't find it in your branch.

Regards,

[Leonardo Pistone (lepistone)]

Denis,

http://bazaar.launchpad.net/~camptocamp/openerp-connector-file/adding-initial-lep/view/head:/connector_file/tests/test_data/one_chunk.csv

That worked in february, and a bisect in the addons pointed me to r9980.

Thanks!

[Denis Ledoux (OpenERP) (dle-openerp)]

Leonardo,

The actual problem is that the period name in your csv file is

02\/2014

Instead of
02/2014.

I just made this change and tested your csv file: The import work as expected.

If it worked before, it was just luck, as this period name is obviously not correct.

[Leonardo Pistone (lepistone)]

Denis,

agreed: the backslash in the test files should not be there and the name_search behaviour change in 9980 did show my bug.

Thank you and sorry for the noise.

leo