Merge pull request #1928 from ebean-orm/feature/1755

FIX: Regression of #1775 - set date for other platforms (non MySQL) broken, when ebean.dataTimeZone is set to GMT

src/main/java/io/ebeaninternal/server/core/timezone/DataTimeZone.java

@@ -8,7 +8,16 @@ import java.util.Calendar;
 public interface DataTimeZone {
 
   /**
-   * Return the Calendar to use for Timezone information.
+   * Return the Calendar to use for Timezone information when reading/writing timestamps.
    */
   Calendar getTimeZone();
+
+  /**
+   * Return the Calendar to use for Timezone information when reading/writing date.
+   * A 'date' only value has normally no timezone information, but some platforms (like MySQL)
+   * reqire this.
+   */
+  default Calendar getDateTimeZone() {
+    return null;
+  }
 }

src/main/java/io/ebeaninternal/server/core/timezone/LocalDataTimeZone.java

@@ -17,4 +17,9 @@ public class LocalDataTimeZone implements DataTimeZone {
   public Calendar getTimeZone() {
     return zone;
   }
+
+  @Override
+    public Calendar getDateTimeZone() {
+      return zone; // workaround for MySQL. TODO: rename class to MySqlDataTimeZone!?
+    }
 }

src/main/java/io/ebeaninternal/server/type/DataBind.java

@@ -141,7 +141,7 @@ public class DataBind {
   }
 
   public void setDate(java.sql.Date v) throws SQLException {
-    Calendar timeZone = dataTimeZone.getTimeZone();
+    Calendar timeZone = dataTimeZone.getDateTimeZone();
     if (timeZone != null) {
       pstmt.setDate(++pos, v, timeZone);
     } else {

src/main/java/io/ebeaninternal/server/type/RsetDataReader.java

@@ -106,7 +106,7 @@ public class RsetDataReader implements DataReader {
 
   @Override
   public Date getDate() throws SQLException {
-    Calendar cal = dataTimeZone.getTimeZone();
+    Calendar cal = dataTimeZone.getDateTimeZone();
     if (cal != null) {
       return rset.getDate(pos(), cal);
     } else {

src/test/java/org/tests/model/types/SomeNewTypesBean.java

@@ -48,6 +48,8 @@ public class SomeNewTypesBean {
 
   java.sql.Date sqlDate;
 
+  java.sql.Time sqlTime;
+
   LocalDate localDate;
 
   LocalDateTime localDateTime;
@@ -134,6 +136,14 @@ public class SomeNewTypesBean {
     this.sqlDate = sqlDate;
   }
 
+  public java.sql.Time getSqlTime() {
+    return sqlTime;
+  }
+
+  public void setSqlTime(java.sql.Time sqlTime) {
+    this.sqlTime = sqlTime;
+  }
+
   public LocalDate getLocalDate() {
     return localDate;
   }

src/test/java/org/tests/types/TestNewTypes.java

@@ -43,6 +43,7 @@ public class TestNewTypes extends BaseTestCase {
     bean.setZonedDateTime(ZonedDateTime.now());
     bean.setLocalTime(LocalTime.now());
     bean.setSqlDate(java.sql.Date.valueOf("2019-07-21"));
+    bean.setSqlTime(java.sql.Time.valueOf("12:34:56"));
     bean.setInstant(Instant.now());
     bean.setYear(Year.now());
     bean.setMonth(Month.APRIL);
@@ -81,6 +82,9 @@ public class TestNewTypes extends BaseTestCase {
     list = DB.find(SomeNewTypesBean.class).where().eq("sqlDate", bean.getSqlDate()).findList();
     assertTrue(!list.isEmpty());
 
+    list = DB.find(SomeNewTypesBean.class).where().eq("sqlTime", bean.getSqlTime()).findList();
+    assertTrue(!list.isEmpty());
+
     list = DB.find(SomeNewTypesBean.class).where().le("localTime", LocalTime.now()).findList();
     assertTrue(!list.isEmpty());
 
@@ -118,6 +122,7 @@ public class TestNewTypes extends BaseTestCase {
     assertEquals(bean.getMonth(), fetched.getMonth());
     assertEquals(bean.getYear(), fetched.getYear());
     assertEquals(bean.getSqlDate(), fetched.getSqlDate());
+    assertEquals(bean.getSqlTime(), fetched.getSqlTime());
     assertEquals(bean.getYearMonth(), fetched.getYearMonth());
     assertEquals(bean.getMonthDay(), fetched.getMonthDay());
     assertEquals(bean.getLocalDate(), fetched.getLocalDate());