fix for HHH-7487 org.hibernate.type.EnumType incorrectly logs binded values, trace logging was done as it is done in BasicBinder with the String.format

This commit is contained in:
Ivan Sopov 2012-09-14 13:03:26 +03:00 committed by brmeyer
parent 500e0222b5
commit 8462677f53
1 changed files with 6 additions and 6 deletions

View File

@ -282,14 +282,14 @@ public class EnumType implements EnhancedUserType, DynamicParameterizedType, Ser
if ( jdbcValue == null ) { if ( jdbcValue == null ) {
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Binding null to parameter: {0}", index ); LOG.trace(String.format("Binding null to parameter: [%s]", index));
} }
st.setNull( index, getSqlType() ); st.setNull( index, getSqlType() );
return; return;
} }
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Binding '{0}' to parameter: '{1}", jdbcValue, index ); LOG.trace(String.format("Binding [%s] to parameter: [%s]", jdbcValue, index));
} }
st.setObject( index, jdbcValue, EnumType.this.sqlType ); st.setObject( index, jdbcValue, EnumType.this.sqlType );
} }
@ -308,14 +308,14 @@ public class EnumType implements EnhancedUserType, DynamicParameterizedType, Ser
final int ordinal = rs.getInt( names[0] ); final int ordinal = rs.getInt( names[0] );
if ( rs.wasNull() ) { if ( rs.wasNull() ) {
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Returning null as column {0}", names[0] ); LOG.trace(String.format("Returning null as column [%s]", names[0]));
} }
return null; return null;
} }
final Enum enumValue = fromOrdinal( ordinal ); final Enum enumValue = fromOrdinal( ordinal );
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Returning '{0}' as column {1}", enumValue, names[0] ); LOG.trace(String.format("Returning [%s] as column [%s]", enumValue, names[0]));
} }
return enumValue; return enumValue;
} }
@ -378,14 +378,14 @@ public class EnumType implements EnhancedUserType, DynamicParameterizedType, Ser
if ( rs.wasNull() ) { if ( rs.wasNull() ) {
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Returning null as column {0}", names[0] ); LOG.trace(String.format("Returning null as column [%s]", names[0]));
} }
return null; return null;
} }
final Enum enumValue = fromName( value ); final Enum enumValue = fromName( value );
if ( LOG.isTraceEnabled() ) { if ( LOG.isTraceEnabled() ) {
LOG.tracev( "Returning '{0}' as column {1}", enumValue, names[0] ); LOG.trace(String.format("Returning [%s] as column [%s]", enumValue, names[0]));
} }
return enumValue; return enumValue;
} }