Excel Export - provides numbers as characters

Bug #844569 reported by Ferdinand
24
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Fix Released
Wishlist
OpenERP's Framework R&D
Odoo Web (MOVED TO GITHUB)
Invalid
Undecided
Unassigned

Bug Description

just tried because of Antony's remark
Btw the new export system was just merged few days ago in commit:
http://bazaar.launchpad.net/~openerp/openerp-web/trunk/revision/912

using
http://client-web-trunk.runbot.openerp.com:9406/web/webclient/home#

opening an export in Excel ( I tried assets) provides numbers as characters makeing the export not very useful for further processing

numbers must be numbers

BTW
1) IMHO export to CSV MUST at least allow to set /overwrite decimal separator !!!
it's a source of permanent annoyance because an export must be done according to the (localized) specs of the import program

in Openoffice/Libre Office all numeric columns have to be changed manually to "US English" which I do not consider as userfriendly at all.

2) IMHO Default Export Type should be "all data"

3) export Type "Import compatible" only shows ID ??

Related branches

affects: openobject-client-web → openerp-web
Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

Please, don't report 4 different issues in the same bug, it's very unwieldy.

> opening an export in Excel ( I tried assets) provides numbers as characters makeing the export not very useful for further processing

The web client can't really do anything about that, short of reprocessing each and every field: the export_data API currently returns lists of strings, instead of lists of the correct representation for the field. If this is fixed, the Excel export will automatically behave as expected: the excel export just uses xlwt.Worksheet.write with (almost) no preparation, and xlwt handles its own conversions from native Python datatypes to sensible Excel representations.

> IMHO export to CSV MUST at least allow to set /overwrite decimal separator !!!

Things would be a bit more complex than that, this may require adding variability for the separator as well for instance. I also don't think this is supported by Python's CSV *API* (as far as I can tell, csv.Dialect provides option for changing the fields delimiter or the quoting character, but not the formatting of non-string types). But feel free to open a feature request for that.

> 2) IMHO Default Export Type should be "all data"

I'm pretty sure this conversation already happened with the old web client and the GTK client, and fp rejected it then. With the new import/export, we considered removing import_compatible altogether but fp rejected the proposal, considering that they fit different flows (export for import, and export for stats analysis) and that the default should remain import compatible (wording may change to make these roles clearer, but you get the point).

If you think you can marshall enough arguments to try and sway him, feel free to bring this up on e.g. the mailing lists or in private conversations with him (opening new bugs without sufficient arguments will likely yield the same answer as this one: that fp has decided to keep things as is)

> 3) export Type "Import compatible" only shows ID ??

Yes, in order to try and simplify the export system we've decided on the following changes to the export:
* external ids (xmlid, formerly id field) are always exported, they're added by default on the main object and selecting a relational field (not its subfields but the field itself) will export its external id instead of its name_get (or whatever it used to be). This results in removing the children of m2m and m2o fields in import_compatible
* database ids (formerly .id) have been renamed to ID, and can be exported explicitly if the user so desires.

I hope these answers are satisfactory, I can't help more than that with the main gist of the bug so I'll close it as wontfix (we can't fix it in the web client, and most of the description would be irrelevant to the server).

https://secure.simplistix.co.uk/svn/xlwt/trunk/xlwt/doc/xlwt.html#xlwt.Worksheet.write-method

Changed in openerp-web:
status: New → Won't Fix
status: Won't Fix → Invalid
Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

correction: closed as invalid, since the main bug is actually invalid for this project

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Ferdinand,

As XMO said, your only point#1 and point#2 are valid, but it's improvement as well as your suggestion.

So I am setting this as a "Wishlist" for only point#1 and #2.

Thanks!

Changed in openobject-server:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Ferdinand (office-chricar) wrote :

please take care of these issues. users are frustrated

see attachment
even without thousand separators numbers are exported as characters

Revision history for this message
Martin Trigaux (OpenERP) (mat-openerp) wrote :

Hello,

We have added this feature into OpenERP trunk. Excell export now use raw data.

Server:
revno: 5159 [merge]
revision-id: <email address hidden>

Web:
revno: 3963 [merge]
revision-id: <email address hidden>

Changed in openobject-server:
status: Confirmed → Fix Released
Revision history for this message
Ferdinand (office-chricar) wrote :

great to see a working excel export

Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Martin Trigaux (OpenERP) (mat-openerp) wrote :

Hello,

How is it not solved ? Please avoid creating duplicates of bugs.
Thanks

Revision history for this message
Ferdinand (office-chricar) wrote :

please read the other bug report - and look at the screenshot there

Revision history for this message
Martin Trigaux (OpenERP) (mat-openerp) wrote :

I see, this export is using another method so not affected by the previous revision, I confirm it needs to be solved.

Regards

PS: being a bit explicit is always helpful.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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