HHH-18036 Truncate time-related milliseconds when creating a java.sql.Date

This commit is contained in:
Christian Beikov 2024-04-29 21:04:13 +02:00
parent 59f7209fd6
commit 3578ed845f
2 changed files with 39 additions and 9 deletions

View File

@ -145,7 +145,7 @@ public class JdbcDateJavaType extends AbstractTemporalJavaType<Date> {
} }
if ( java.sql.Timestamp.class.isAssignableFrom( type ) ) { if ( java.sql.Timestamp.class.isAssignableFrom( type ) ) {
return new java.sql.Timestamp( value.getTime() ); return new java.sql.Timestamp( unwrapDateEpoch( value ) );
} }
if ( java.sql.Time.class.isAssignableFrom( type ) ) { if ( java.sql.Time.class.isAssignableFrom( type ) ) {
@ -158,18 +158,31 @@ public class JdbcDateJavaType extends AbstractTemporalJavaType<Date> {
private LocalDate unwrapLocalDate(Date value) { private LocalDate unwrapLocalDate(Date value) {
return value instanceof java.sql.Date return value instanceof java.sql.Date
? ( (java.sql.Date) value ).toLocalDate() ? ( (java.sql.Date) value ).toLocalDate()
: new java.sql.Date( value.getTime() ).toLocalDate(); : new java.sql.Date( unwrapDateEpoch( value ) ).toLocalDate();
} }
private java.sql.Date unwrapSqlDate(Date value) { private java.sql.Date unwrapSqlDate(Date value) {
return value instanceof java.sql.Date if ( value instanceof java.sql.Date ) {
? (java.sql.Date) value final java.sql.Date sqlDate = (java.sql.Date) value;
: new java.sql.Date( value.getTime() ); final long dateEpoch = toDateEpoch( sqlDate.getTime() );
return dateEpoch == sqlDate.getTime() ? sqlDate : new java.sql.Date( dateEpoch );
}
return new java.sql.Date( unwrapDateEpoch( value ) );
} }
private static long unwrapDateEpoch(Date value) { private static long unwrapDateEpoch(Date value) {
return value.getTime(); return toDateEpoch( value.getTime() );
}
private static long toDateEpoch(long value) {
Calendar calendar = Calendar.getInstance();
calendar.setTimeInMillis( value );
calendar.set(Calendar.HOUR_OF_DAY, 0);
calendar.clear(Calendar.MINUTE);
calendar.clear(Calendar.SECOND);
calendar.clear(Calendar.MILLISECOND);
return calendar.getTimeInMillis();
} }
@Override @Override
@ -183,15 +196,15 @@ public class JdbcDateJavaType extends AbstractTemporalJavaType<Date> {
} }
if ( value instanceof Long ) { if ( value instanceof Long ) {
return new java.sql.Date( (Long) value ); return new java.sql.Date( toDateEpoch( (Long) value ) );
} }
if ( value instanceof Calendar ) { if ( value instanceof Calendar ) {
return new java.sql.Date( ( (Calendar) value ).getTimeInMillis() ); return new java.sql.Date( toDateEpoch( ( (Calendar) value ).getTimeInMillis() ) );
} }
if ( value instanceof Date ) { if ( value instanceof Date ) {
return new java.sql.Date( ( (Date) value ).getTime() ); return unwrapSqlDate( (Date) value );
} }
if ( value instanceof LocalDate ) { if ( value instanceof LocalDate ) {

View File

@ -1,10 +1,13 @@
package org.hibernate.orm.test.type.descriptor.java; package org.hibernate.orm.test.type.descriptor.java;
import java.sql.Date; import java.sql.Date;
import java.time.ZoneOffset;
import java.time.ZonedDateTime;
import org.hibernate.type.descriptor.java.JdbcDateJavaType; import org.hibernate.type.descriptor.java.JdbcDateJavaType;
import org.hibernate.testing.orm.junit.BaseUnitTest; import org.hibernate.testing.orm.junit.BaseUnitTest;
import org.hibernate.testing.orm.junit.JiraKey;
import org.junit.jupiter.api.Test; import org.junit.jupiter.api.Test;
import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat; import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
@ -32,6 +35,7 @@ public class JdbcDateJavaTypeDescriptorTest {
} }
@Test @Test
@JiraKey("HHH-18036")
public void testWrap() { public void testWrap() {
final JdbcDateJavaType javaType = JdbcDateJavaType.INSTANCE; final JdbcDateJavaType javaType = JdbcDateJavaType.INSTANCE;
@ -42,5 +46,18 @@ public class JdbcDateJavaTypeDescriptorTest {
final java.util.Date utilDate = new java.util.Date( 0 ); final java.util.Date utilDate = new java.util.Date( 0 );
final java.util.Date wrappedUtilDate = javaType.wrap( utilDate, null ); final java.util.Date wrappedUtilDate = javaType.wrap( utilDate, null );
assertThat( wrappedUtilDate ).isInstanceOf( java.sql.Date.class ); assertThat( wrappedUtilDate ).isInstanceOf( java.sql.Date.class );
final java.util.Date utilDateWithTime = java.util.Date.from( ZonedDateTime.of(
2000,
1,
1,
12,
0,
0,
0,
ZoneOffset.UTC
).toInstant() );
final java.util.Date wrappedUtilDateWithTime = javaType.wrap( utilDateWithTime, null );
assertThat( wrappedUtilDateWithTime ).isEqualTo( java.sql.Date.valueOf( "2000-01-01" ) );
} }
} }