Bug 46410 - [PATCH] Implementation of Excel TIME(h,m,s) function
[PATCH] Implementation of Excel TIME(h,m,s) function
Status: RESOLVED FIXED
Product: POI
Classification: Unclassified
Component: HSSF
3.5-dev
PC Windows XP
: P2 normal (vote)
: ---
Assigned To: POI Developers List
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2008-12-17 03:58 UTC by Steven Butler
Modified: 2008-12-30 22:00 UTC (History)
0 users



Attachments
Patch to implement TIME() Formula - includes new unit test file TestTime.java (9.90 KB, patch)
2008-12-17 03:58 UTC, Steven Butler
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Steven Butler 2008-12-17 03:58:05 UTC
Created attachment 23033 [details]
Patch to implement TIME() Formula - includes new unit test file TestTime.java

I have tried to implement the Excel TIME() function in the attached patch.

Note that I had to include a precision parameter in the unit tests for the numeric comparisons in the unit tests because Excel seems to have higher internal precision than it is willing to display.

For this reason the unit tests also test the formatted time against the expected values as a secondary test.
Comment 1 Josh Micich 2008-12-30 21:59:31 UTC
Applied in svn r730309.