Bug 46410

Summary: [PATCH] Implementation of Excel TIME(h,m,s) function
Product: POI Reporter: Steven Butler <sebutler>
Component: HSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 3.5-dev   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Attachments: Patch to implement TIME() Formula - includes new unit test file TestTime.java

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.