HHH-9343 - Case/when in criteria with string literal result fails type checking

Author: dreab8

hibernate-entitymanager/src/main/java/org/hibernate/jpa/criteria/ValueHandlerFactory.java

@@ -68,6 +68,10 @@ public static boolean isNumeric(Object value) {
 				|| Double.TYPE.isInstance( value );
 	}
 
+	public static boolean isBoolean(Object value) {
+		return Boolean.class.isInstance( value );
+	}
+
 	public static class ByteValueHandler extends BaseValueHandler<Byte> implements Serializable {
 		public static final ByteValueHandler INSTANCE = new ByteValueHandler();
 		@SuppressWarnings({ "UnnecessaryBoxing" })
@@ -230,6 +234,23 @@ public String render(BigDecimal value) {
 		}
 	}
 
+	public static class BooleanValueHandler extends BaseValueHandler<Boolean> implements Serializable {
+		public static final BooleanValueHandler INSTANCE = new BooleanValueHandler();
+
+		public Boolean convert(Object value) {
+			if ( value == null ) {
+				return null;
+			}
+			if ( Boolean.class.isInstance( value ) ) {
+				return (Boolean) value;
+			}
+			if ( String.class.isInstance( value ) ) {
+				return Boolean.getBoolean( (String) value );
+			}
+			throw unknownConversion( value, Boolean.class );
+		}
+	}
+
 	public static class StringValueHandler extends BaseValueHandler<String> implements Serializable {
 		public static final StringValueHandler INSTANCE = new StringValueHandler();
 		public String convert(Object value) {
@@ -308,6 +329,9 @@ public static <T> ValueHandler<T> determineAppropriateHandler(Class<T> targetTyp
 		if ( BigDecimal.class.equals( targetType ) ) {
 			return (ValueHandler<T>) BigDecimalValueHandler.INSTANCE;
 		}
+		if ( Boolean.class.equals( targetType ) ) {
+			return (ValueHandler<T>) BooleanValueHandler.INSTANCE;
+		}
 		return null;
 	}
 }

hibernate-entitymanager/src/main/java/org/hibernate/jpa/criteria/expression/LiteralExpression.java

@@ -49,6 +49,9 @@ public String render(RenderingContext renderingContext) {
 		if ( ValueHandlerFactory.isNumeric( literal ) ) {
 			return ValueHandlerFactory.determineAppropriateHandler( (Class) literal.getClass() ).render( literal );
 		}
+		else if ( ValueHandlerFactory.isBoolean( literal ) ) {
+			return ValueHandlerFactory.determineAppropriateHandler( (Class) literal.getClass() ).render( literal );
+		}
 
 		// else...
 		final String parameterName = renderingContext.registerLiteralParameterBinding( getLiteral(), getJavaType() );