diff --git a/hibernate-core/src/main/java/org/hibernate/loader/PropertyPath.java b/hibernate-core/src/main/java/org/hibernate/loader/PropertyPath.java
index 1eee1015f4..92c02f40aa 100644
--- a/hibernate-core/src/main/java/org/hibernate/loader/PropertyPath.java
+++ b/hibernate-core/src/main/java/org/hibernate/loader/PropertyPath.java
@@ -39,20 +39,28 @@ public class PropertyPath {
this.parent = parent;
this.property = property;
- final String prefix;
- if ( parent != null ) {
- final String resolvedParent = parent.getFullPath();
- if ( StringHelper.isEmpty( resolvedParent ) ) {
+ // the _identifierMapper is a "hidden" property on entities with composite keys.
+ // concatenating it will prevent the path from correctly being used to look up
+ // various things such as criteria paths and fetch profile association paths
+ if ("_identifierMapper".equals(property)) {
+ this.fullPath = parent != null ? parent.getFullPath() : "";
+ } else {
+ final String prefix;
+ if ( parent != null ) {
+ final String resolvedParent = parent.getFullPath();
+ if ( StringHelper.isEmpty( resolvedParent ) ) {
prefix = "";
+ }
+ else {
+ prefix = resolvedParent + '.';
+ }
}
else {
- prefix = resolvedParent + '.';
+ prefix = "";
}
+
+ this.fullPath = prefix + property;
}
- else {
- prefix = "";
- }
- this.fullPath = prefix + property;
}
public PropertyPath(String property) {
diff --git a/hibernate-core/src/test/java/org/hibernate/test/criteria/ComplexJoinAliasTest.java b/hibernate-core/src/test/java/org/hibernate/test/criteria/ComplexJoinAliasTest.java
new file mode 100644
index 0000000000..fdd10298c7
--- /dev/null
+++ b/hibernate-core/src/test/java/org/hibernate/test/criteria/ComplexJoinAliasTest.java
@@ -0,0 +1,182 @@
+/*
+ * Hibernate, Relational Persistence for Idiomatic Java
+ *
+ * Copyright (c) 2011, Red Hat Middleware LLC or third-party contributors as
+ * indicated by the @author tags or express copyright attribution
+ * statements applied by the authors. All third-party contributions are
+ * distributed under license by Red Hat Middleware LLC.
+ *
+ * This copyrighted material is made available to anyone wishing to use, modify,
+ * copy, or redistribute it subject to the terms and conditions of the GNU
+ * Lesser General Public License, as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License
+ * for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this distribution; if not, write to:
+ * Free Software Foundation, Inc.
+ * 51 Franklin Street, Fifth Floor
+ * Boston, MA 02110-1301 USA
+ *
+ */
+package org.hibernate.test.criteria;
+
+import java.util.Arrays;
+import javax.persistence.Column;
+import javax.persistence.Entity;
+import javax.persistence.EnumType;
+import javax.persistence.Enumerated;
+import javax.persistence.Id;
+import javax.persistence.IdClass;
+import javax.persistence.JoinColumn;
+import javax.persistence.ManyToOne;
+import javax.persistence.MappedSuperclass;
+import javax.persistence.Table;
+
+import org.junit.Test;
+
+import org.hibernate.Criteria;
+import org.hibernate.Session;
+import org.hibernate.criterion.Restrictions;
+import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
+
+import static org.junit.Assert.assertEquals;
+
+/**
+ * Hibernate generates the wrong alias on a Criteria query involving a join
+ * on a composite identifier. For example, in the test below without the fix
+ * to JoinWalker, it generates this SQL:
+ *
+ *
+ * select
+ * this_.role_code as role1_0_1_,
+ * this_.is_deleted as is2_0_1_,
+ * this_.record_version as record3_0_1_,
+ * complexjoi3_.code as code1_0_,
+ * complexjoi3_.is_deleted as is2_1_0_,
+ * complexjoi3_.record_version as record3_1_0_
+ * from
+ * list_action_roles this_
+ * left outer join
+ * roles complexjoi3_
+ * on this_.role_code=complexjoi3_.code
+ * where
+ * this_.is_deleted=?
+ * and complexjoi1_.is_deleted=?
+ *
+ *
+ * Which results in this error from the SQL server:
+ *
+ *
+ * Unknown column 'complexjoi1_.is_deleted' in 'where clause'
+ *
+ *
+ * Analysis of the problem:
+ *
+ * The entity persister class with the composite identifier has a fake
+ * property for it, called "_identifierMapper" (see HbmBinder.bindCompositeId()
+ * and similarly in Annotations). This property name ends up in the path
+ * used by JoinWalker.walkEntityTree() when it calls walkComponentTree().
+ * However that path is used by CriteriaJoinWalker.generateTableAlias()
+ * to look up the correct criteria (and hence the alias) from the
+ * translator, a CriteriaQueryTranslator.
+ *
+ * The alias was created in CriteriaQueryTranslator.createCriteriaSQLAliasMap
+ * for a Criteria without this extra _identifierMapper path component.
+ * So when CriteriaJoinWalker tries to use the path with _identifierMapper
+ * to look up the criteria to find the correct alias to use, in
+ * generateTableAlias(), it doesn't find one, so it generates a new alias.
+ *
+ * That new alias no longer matches the one that will be rendered out in
+ * the WHERE clause, so the WHERE clause will refer to table names using
+ * aliases that are not used anywhere else in the query, and the SQL server
+ * will fail to parse the statement.
+ *
+ * The solution appears to be to exclude "_identifierMapper" components in
+ * the alias path when building it. I don't know what other implications
+ * that might have, but it seems like an implementation nastiness that
+ * shouldn't be exposed.
+ *
+ * @author Chris Wilson
+ * @link http://opensource.atlassian.com/projects/hibernate/browse/HHH-4630
+ */
+
+public class ComplexJoinAliasTest extends BaseCoreFunctionalTestCase {
+ @MappedSuperclass
+ private static abstract class VersionedRecord
+ implements java.io.Serializable {
+ @Column(name = "record_version", nullable = false)
+ Long recordVersion;
+
+ @Column(name = "is_deleted", nullable = false, columnDefinition = "smallint")
+ Boolean isDeleted;
+ }
+
+ @Entity
+ @Table(name = "list_action_roles")
+ public static class ListActionRole extends VersionedRecord {
+ @Id
+ @Column(name = "role_code", length = 3)
+ @Enumerated(EnumType.STRING)
+ Role.Code roleCode;
+
+ @ManyToOne(targetEntity = Role.class)
+ @JoinColumn(name = "role_code", insertable = false, updatable = false)
+ @Id
+ Role role;
+
+ @Override
+ public String toString() {
+ return "ListActionRole.Id(roleCode=" + roleCode + ")";
+ }
+
+ @Override
+ public int hashCode() {
+ return toString().hashCode();
+ }
+ }
+
+ @Entity
+ @Table(name = "roles")
+ public static class Role extends VersionedRecord {
+ public enum Code {
+ ADM, CEN, RPA, RPP, PRJ, HUB, RQS, OAD, ORP, ORQ
+ }
+
+ @Id
+ @Column(name = "code", length = 3)
+ @Enumerated(EnumType.STRING)
+ Code code;
+ }
+
+ @Override
+ protected Class[] getAnnotatedClasses() {
+ return new Class[] {
+ ListActionRole.class,
+ Role.class
+ };
+ }
+
+ @Test
+ public void testCriteriaThroughCompositeId() throws Exception {
+ Session session = openSession();
+
+ Criteria listActionRoles = session.createCriteria( ListActionRole.class );
+ listActionRoles.add( Restrictions.eq( "isDeleted", false ) );
+
+ Criteria roles = listActionRoles.createCriteria( "role" );
+ roles.add( Restrictions.eq( "isDeleted", false ) );
+
+ assertEquals(
+ Arrays.asList( new ListActionRole[] { } ),
+ listActionRoles.list()
+ );
+
+ session.close();
+ }
+}
+
+