Bug 55927 - [PATCH] poi xml export can't handle date types
Summary: [PATCH] poi xml export can't handle date types
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: XSSF (show other bugs)
Version: 3.9-FINAL
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2013-12-23 16:00 UTC by Marc
Modified: 2014-07-30 16:11 UTC (History)
0 users


Attachments
[PATCH] patch (9.73 KB, application/octet-stream)
2013-12-23 16:14 UTC, Marc 		Details
Exports formulas correctly to a xml document (8.19 KB, application/x-gzip)
2014-07-16 15:08 UTC, Marc 		Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc 2013-12-23 16:00:13 UTC 
It fails to export a date type because the cell value is interpreted as a double type.
Comment 1 Marc 2013-12-23 16:14:57 UTC 
Created attachment 31148 [details]
[PATCH] patch
Comment 2 Dominik Stadler 2014-02-03 19:22:12 UTC 
Thanks for the patch, committed in r1564011. This will not make it into 3.10 any more, but be included in the next release.
Comment 3 Marc 2014-07-16 15:08:41 UTC 
Created attachment 31821 [details]
Exports formulas correctly to a xml document
Comment 4 Marc 2014-07-16 15:10:11 UTC 
The patch I provied on december 23 does not take into account formulas which return dates.

E.g. POI does not handle excel cells correctly containing formulas like:
 =Date(2013;12;31) 
The POI XML exports results into a numeric value instead of a date string. 
I wrote a patch for the current trunk code which handles that correctly.
Comment 5 Marc 2014-07-16 15:12:17 UTC 
Reopenend because POI fails to export cells containing formulas which return a date value.
Comment 6 Nick Burch 2014-07-30 16:11:11 UTC 
Thanks for this, applied (with minor tweaks) in r1614696.