[GitHub] [fineract] vorburger commented on a change in pull request #1191: FINERACT-1087: Fixing date issues with bulk import templates

[GitBox <gi...@apache.org>]

vorburger commented on a change in pull request #1191:
URL: https://github.com/apache/fineract/pull/1191#discussion_r457228525



##########
File path: fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/populator/AbstractWorkbookPopulator.java
##########
@@ -66,9 +66,9 @@ protected void writeDate(int colIndex, Row row, String value, CellStyle dateCell
         try {
             SimpleDateFormat formatinDB = null;
             if (value.matches("\\d{4}-\\d{1,2}-\\d{1,2}")) {
-                formatinDB = new SimpleDateFormat("yyyy-mm-dd");
+                formatinDB = new SimpleDateFormat("yyyy-MM-dd");

Review comment:
       https://docs.oracle.com/javase/7/docs/api/java/text/SimpleDateFormat.html:
   
   * m = Minute in hour
   * M = Month in year




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org