diff --git a/src/java/org/apache/poi/ss/usermodel/DataFormatter.java b/src/java/org/apache/poi/ss/usermodel/DataFormatter.java index 61a5092f2..94740e9d2 100644 --- a/src/java/org/apache/poi/ss/usermodel/DataFormatter.java +++ b/src/java/org/apache/poi/ss/usermodel/DataFormatter.java @@ -582,6 +582,9 @@ public class DataFormatter implements Observer { } } else { + if (Character.isWhitespace(c)){ + ms.clear(); + } sb.append(c); } } diff --git a/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java b/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java index 50276ae28..8e6e38932 100644 --- a/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java +++ b/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java @@ -815,5 +815,17 @@ public class TestDataFormatter { assertEquals("5.6789", formatter.formatCellValue(cell, evaluator)); wb.close(); + } + + /** + * bug 60031: DataFormatter parses months incorrectly when put at the end of date segment + */ + @Test + public void testBug60031() { + // 23-08-2016 08:51:01 which is 42605.368761574071 as double will be parsed + // with format "yyyy-dd-MM HH:mm:ss" into "2016-23-51 08:51:01". + DataFormatter dfUS = new DataFormatter(Locale.US); + assertEquals("2016-23-08 08:51:01", dfUS.formatRawCellContents(42605.368761574071, -1, "yyyy-dd-MM HH:mm:ss")); + assertEquals("2017-12-01 January 09:54:33", dfUS.formatRawCellContents(42747.412892397523, -1, "yyyy-dd-MM MMMM HH:mm:ss")); } }