HHH-7945 xsd element with mixed set to true may return unexpected \n

hibernate-core/src/main/java/org/hibernate/hql/internal/ast/util/SyntheticAndFactory.java

@@ -29,6 +29,7 @@ import java.util.Map;
 import antlr.collections.AST;
 import org.jboss.logging.Logger;
 
+import org.hibernate.Filter;
 import org.hibernate.hql.internal.antlr.HqlSqlTokenTypes;
 import org.hibernate.hql.internal.ast.HqlSqlWalker;
 import org.hibernate.hql.internal.ast.tree.FromElement;
@@ -164,7 +165,7 @@ public class SyntheticAndFactory implements HqlSqlTokenTypes {
 	public void addDiscriminatorWhereFragment(
 			RestrictableStatement statement,
 			Queryable persister,
-			Map enabledFilters,
+			Map<String,Filter> enabledFilters,
 			String alias) {
 		String whereFragment = persister.filterFragment( alias, enabledFilters ).trim();
 		if ( "".equals( whereFragment ) ) {

hibernate-core/src/main/java/org/hibernate/internal/util/StringHelper.java

@@ -452,6 +452,10 @@ public final class StringHelper {
 		return string == null || string.length() == 0;
 	}
 
+	public static boolean isEmptyOrWhiteSpace(String string){
+		return isEmpty( string ) || isEmpty( string.trim() );
+	}
+
 	public static String qualify(String prefix, String name) {
 		if ( name == null || prefix == null ) {
 			throw new NullPointerException();

hibernate-core/src/main/java/org/hibernate/metamodel/internal/source/hbm/FilterDefinitionSourceImpl.java

@@ -28,6 +28,7 @@ import java.util.List;
 
 import javax.xml.bind.JAXBElement;
 
+import org.hibernate.internal.util.StringHelper;
 import org.hibernate.jaxb.spi.hbm.JaxbFilterDefElement;
 import org.hibernate.jaxb.spi.hbm.JaxbFilterParamElement;
 import org.hibernate.metamodel.spi.source.FilterDefinitionSource;
@@ -55,7 +56,10 @@ public class FilterDefinitionSourceImpl
 		final List<FilterParameterSource> parameterSources = new ArrayList<FilterParameterSource>();
 		for ( Object content : filterDefElement.getContent() ) {
 			if ( String.class.isInstance( content ) ){
-				conditionContent = (String) content;
+				final String str = content.toString();
+				if ( !StringHelper.isEmptyOrWhiteSpace( str ) ) {
+					conditionContent = str.trim();
+				}
 			}
 			else if ( JAXBElement.class.isInstance( content ) ) {
 				JAXBElement jaxbElement = JAXBElement.class.cast( content );

hibernate-core/src/main/java/org/hibernate/metamodel/internal/source/hbm/FilterSourceImpl.java

@@ -57,7 +57,10 @@ public class FilterSourceImpl
 
 		for ( Serializable content : filterElement.getContent() ) {
 			if ( String.class.isInstance( content ) ) {
-				conditionContent = String.class.cast( content );
+				final String str = content.toString();
+				if ( !StringHelper.isEmptyOrWhiteSpace( str ) ) {
+					conditionContent = str.trim();
+				}
 			}
 			else {
 				final JaxbFilterAliasMappingType aliasMapping = JaxbFilterAliasMappingType.class.cast( content );

hibernate-core/src/test/java/org/hibernate/test/filter/hql/BasicFilteredBulkManipulationTest.java

@@ -26,7 +26,6 @@ package org.hibernate.test.filter.hql;
 import org.junit.Test;
 
 import org.hibernate.Session;
-import org.hibernate.testing.FailureExpectedWithNewMetamodel;
 import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
 
 import static org.junit.Assert.assertEquals;
@@ -36,7 +35,6 @@ import static org.junit.Assert.assertEquals;
  *
  * @author Steve Ebersole
  */
-@FailureExpectedWithNewMetamodel
 public class BasicFilteredBulkManipulationTest extends BaseCoreFunctionalTestCase {
 	@Override
 	public String[] getMappings() {

hibernate-core/src/test/java/org/hibernate/test/filter/hql/JoinedFilteredBulkManipulationTest.java

@@ -29,7 +29,6 @@ import org.hibernate.testing.SkipForDialect;
 import org.junit.Test;
 
 import org.hibernate.Session;
-import org.hibernate.testing.FailureExpectedWithNewMetamodel;
 import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
 
 import static org.junit.Assert.assertEquals;
@@ -37,7 +36,6 @@ import static org.junit.Assert.assertEquals;
 /**
  * @author Steve Ebersole
  */
-@FailureExpectedWithNewMetamodel
 @SkipForDialect(
         value = CUBRIDDialect.class,
         comment = "As of verion 8.4.1 CUBRID doesn't support temporary tables. This test fails with" +