moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

Apply fix reported in bug 47661 and add unit tests

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1680642 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Dominik Stadler 2015-05-20 18:00:19 +00:00
parent 589b189151
commit 001cdeb936
2 changed files with 22 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/java/org/apache/poi/ss/usermodel/DateUtil.java
View File

@ -567,7 +567,7 @@ public class DateUtil {
private static int daysInPriorYears(int yr, boolean use1904windowing)
{
if ((!use1904windowing && yr < 1900) || (use1904windowing && yr < 1900)) {
if ((!use1904windowing && yr < 1900) || (use1904windowing && yr < 1904)) {
throw new IllegalArgumentException("'year' must be 1900 or greater");
}

24
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFDateUtil.java
View File

@ -17,9 +17,7 @@
package org.apache.poi.hssf.usermodel;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.*;
import java.io.IOException;
import java.text.ParseException;
@ -349,6 +347,7 @@ public final class TestHSSFDateUtil {
HSSFWorkbook workbook = HSSFTestDataSamples.openSampleWorkbook("DateFormats.xls");
HSSFSheet sheet = workbook.getSheetAt(0);
InternalWorkbook wb = workbook.getWorkbook();
assertNotNull(wb);
HSSFRow row;
HSSFCell cell;
@ -481,6 +480,25 @@ public final class TestHSSFDateUtil {
assertEquals("Checking absolute day (1 Jan 1901)", 366, HSSFDateUtil.absoluteDay(calendar, false));
}
@Test
public void absoluteDayYearTooLow() {
GregorianCalendar calendar = new GregorianCalendar(1899, 0, 1);
try {
HSSFDateUtil.absoluteDay(calendar, false);
fail("Should fail here");
} catch (IllegalArgumentException e) {
// expected here
}
try {
calendar = new GregorianCalendar(1903, 0, 1);
HSSFDateUtil.absoluteDay(calendar, true);
fail("Should fail here");
} catch (IllegalArgumentException e) {
// expected here
}
}
@Test
public void convertTime() {