Revision: 6384
          http://sourceforge.net/p/jump-pilot/code/6384
Author:   michaudm
Date:     2020-08-20 07:17:48 +0000 (Thu, 20 Aug 2020)
Log Message:
-----------
Revert the fallback to FlexibleDateParser if rs.getTimestamp fails (useful for 
sqlite ?)

Modified Paths:
--------------
    core/trunk/ChangeLog
    
core/trunk/src/com/vividsolutions/jump/datastore/jdbc/ValueConverterFactory.java

Modified: core/trunk/ChangeLog
===================================================================
--- core/trunk/ChangeLog        2020-08-20 06:59:27 UTC (rev 6383)
+++ core/trunk/ChangeLog        2020-08-20 07:17:48 UTC (rev 6384)
@@ -5,7 +5,7 @@
 #<-------------------------------- 80 chars 
---------------------------------->#
 
 2020-08-20 mmichaud <m.michael.mich...@orange.fr>
-  * Fix regression : read dates from database as java.util.Date, not String
+  * #497 : read dates from database as java.util.Date, not String
   * update postgresql and sqlite jdbc drivers
 
 2020-08-17 Giuseppe Aruta

Modified: 
core/trunk/src/com/vividsolutions/jump/datastore/jdbc/ValueConverterFactory.java
===================================================================
--- 
core/trunk/src/com/vividsolutions/jump/datastore/jdbc/ValueConverterFactory.java
    2020-08-20 06:59:27 UTC (rev 6383)
+++ 
core/trunk/src/com/vividsolutions/jump/datastore/jdbc/ValueConverterFactory.java
    2020-08-20 07:17:48 UTC (rev 6384)
@@ -119,22 +119,22 @@
         public Object getValue(ResultSet rs, int columnIndex) throws 
SQLException {
               // always return string for dates and let FlexibleFeature 
convert later during runtime
               //return rs.getString(columnIndex);
-//            Object ret = null;
-//            try {
-              return rs.getTimestamp(columnIndex);
-//                if (rs.wasNull()) return null;
-//            } catch (Exception e) {
-//                // try to read date from string, as some SpatialDatabases 
like SQLite
-//                // can store DATE type in string
-//                FlexibleDateParser parser = new FlexibleDateParser();
-//                try {
-//                    ret = parser.parse(rs.getString(columnIndex), false);
-//                } catch (Exception ee) {
-//                    System.err.println("cannot parse date value: \"" + 
rs.getString(columnIndex)
-//                    + "\" Defaulting to null.\n" + ee.getMessage());
-//                }
-//            }
-//            return ret;
+              Object ret = null;
+              try {
+                  ret = rs.getTimestamp(columnIndex);
+                  if (rs.wasNull()) return null;
+              } catch (Exception e) {
+                  // try to read date from string, as some SpatialDatabases 
like SQLite
+                  // can store DATE type in string
+                  FlexibleDateParser parser = new FlexibleDateParser();
+                  try {
+                      ret = parser.parse(rs.getString(columnIndex), false);
+                  } catch (Exception ee) {
+                      System.err.println("cannot parse date value: \"" + 
rs.getString(columnIndex)
+                      + "\" Defaulting to null.\n" + ee.getMessage());
+                  }
+              }
+              return ret;
         }
     }
 



_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to