[hibernate-commits] Hibernate SVN: r16537 - in core/branches/Branch_3_2: test/org/hibernate/test/sql/hand and 1 other directories.
hibernate-commits at lists.jboss.org
hibernate-commits at lists.jboss.org
Mon May 11 17:15:24 EDT 2009
Author: steve.ebersole at jboss.com
Date: 2009-05-11 17:15:24 -0400 (Mon, 11 May 2009)
New Revision: 16537
Added:
core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Group.java
Modified:
core/branches/Branch_3_2/src/org/hibernate/loader/custom/sql/SQLQueryReturnProcessor.java
core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Person.java
core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueries.hbm.xml
core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueriesTest.java
Log:
HHH-2745 - NullPointerException when eager fetching joined many-to-many with native SQL query
Modified: core/branches/Branch_3_2/src/org/hibernate/loader/custom/sql/SQLQueryReturnProcessor.java
===================================================================
--- core/branches/Branch_3_2/src/org/hibernate/loader/custom/sql/SQLQueryReturnProcessor.java 2009-05-11 21:13:36 UTC (rev 16536)
+++ core/branches/Branch_3_2/src/org/hibernate/loader/custom/sql/SQLQueryReturnProcessor.java 2009-05-11 21:15:24 UTC (rev 16537)
@@ -374,7 +374,7 @@
alias2CollectionSuffix.put( alias, suffix );
collectionPropertyResultMaps.put( alias, propertyResults );
- if ( collectionPersister.isOneToMany() ) {
+ if ( collectionPersister.isOneToMany() || collectionPersister.isManyToMany() ) {
SQLLoadable persister = ( SQLLoadable ) collectionPersister.getElementPersister();
addPersister( alias, filter( propertyResults ), persister );
}
Added: core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Group.java
===================================================================
--- core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Group.java (rev 0)
+++ core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Group.java 2009-05-11 21:15:24 UTC (rev 16537)
@@ -0,0 +1,42 @@
+package org.hibernate.test.sql.hand;
+
+import java.util.ArrayList;
+import java.util.List;
+
+public class Group {
+ private Long id;
+ private List persons = new ArrayList();
+ private String name;
+
+ public Group() {
+ }
+
+ public Group(String name) {
+ this.name = name;
+ }
+
+ public Long getId() {
+ return id;
+ }
+
+ public void setId(Long id) {
+ this.id = id;
+ }
+
+ public List getPersons() {
+ return persons;
+ }
+
+ public void setPersons(List persons) {
+ this.persons = persons;
+ }
+
+ public String getName() {
+ return name;
+ }
+
+ public void setName(String name) {
+ this.name = name;
+ }
+
+}
Modified: core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Person.java
===================================================================
--- core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Person.java 2009-05-11 21:13:36 UTC (rev 16536)
+++ core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/Person.java 2009-05-11 21:15:24 UTC (rev 16537)
@@ -1,37 +1,61 @@
-//$Id: Person.java 4316 2004-08-14 13:12:03Z oneovthafew $
+/*
+ * Copyright (c) 2009, 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.sql.hand;
/**
* @author Gavin King
*/
public class Person {
- private long id;
+ private Long id;
private String name;
public Person(String name) {
this.name = name;
}
-
- public Person() {}
-
-/**
+
+ public Person() {
+ }
+
+ /**
* @return Returns the id.
*/
- public long getId() {
+ public Long getId() {
return id;
}
+
/**
* @param id The id to set.
*/
- public void setId(long id) {
+ public void setId(Long id) {
this.id = id;
}
+
/**
* @return Returns the name.
*/
public String getName() {
return name;
}
+
/**
* @param name The name to set.
*/
Modified: core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueries.hbm.xml
===================================================================
--- core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueries.hbm.xml 2009-05-11 21:13:36 UTC (rev 16536)
+++ core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueries.hbm.xml 2009-05-11 21:15:24 UTC (rev 16537)
@@ -31,6 +31,19 @@
</class>
+ <class name="Group" table="GROUPP">
+ <id name="id" unsaved-value="0" column="ID">
+ <generator class="increment"/>
+ </id>
+ <property name="name" column="NAME" not-null="true"/>
+ <list name="persons" table="GROUP_PERSON"
+ cascade="none" inverse="false" lazy="true">
+ <key column="GROUP_ID" />
+ <list-index column="POS" />
+ <many-to-many class="Person" column="PERSON_ID" />
+ </list>
+ </class>
+
<class name="Employment" table="EMPLOYMENT">
<id name="employmentId" unsaved-value="0" column="EMPID">
<generator class="increment"/>
@@ -259,4 +272,31 @@
LEFT OUTER JOIN EMPLOYMENT emp ON org.ORGID = emp.EMPLOYER
</sql-query>
+ <sql-query name="manyToManyFetch">
+ <![CDATA[
+ SELECT groupp.ID as group_id,
+ groupp.NAME as group_name,
+ group_person.PERSON_ID as group_person_personId,
+ group_person.GROUP_ID as group_person_groupId,
+ group_person.POS as group_person_pos,
+ person.PERID as person_id,
+ person.NAME as person_name
+ FROM GROUPP groupp,
+ GROUP_PERSON group_person,
+ PERSON person
+ WHERE groupp.ID = group_person.GROUP_ID
+ and person.PERID = group_person.PERSON_ID
+ ]]>
+ <return alias="groupp" class="Group">
+ <return-property name="id" column="group_id" />
+ <return-property name="name" column="group_name" />
+ </return>
+ <return-join alias="group_person" property="groupp.persons">
+ <return-property name="key" column="group_person_groupId" />
+ <return-property name="index" column="group_person_pos" />
+ <return-property name="element" column="person_id" />
+ <return-property name="element.id" column="person_id" />
+ <return-property name="element.name" column="person_name" />
+ </return-join>
+ </sql-query>
</hibernate-mapping>
\ No newline at end of file
Modified: core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueriesTest.java
===================================================================
--- core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueriesTest.java 2009-05-11 21:13:36 UTC (rev 16536)
+++ core/branches/Branch_3_2/test/org/hibernate/test/sql/hand/query/NativeSQLQueriesTest.java 2009-05-11 21:15:24 UTC (rev 16537)
@@ -14,21 +14,22 @@
import org.hibernate.Query;
import org.hibernate.Session;
import org.hibernate.Transaction;
+import org.hibernate.cfg.Configuration;
+import org.hibernate.cfg.Environment;
+import org.hibernate.junit.functional.FunctionalTestCase;
+import org.hibernate.junit.functional.FunctionalTestClassTestSuite;
+import org.hibernate.test.sql.hand.Dimension;
+import org.hibernate.test.sql.hand.Employment;
+import org.hibernate.test.sql.hand.Group;
+import org.hibernate.test.sql.hand.Order;
import org.hibernate.test.sql.hand.Organization;
import org.hibernate.test.sql.hand.Person;
-import org.hibernate.test.sql.hand.Employment;
import org.hibernate.test.sql.hand.Product;
-import org.hibernate.test.sql.hand.Order;
-import org.hibernate.test.sql.hand.Dimension;
import org.hibernate.test.sql.hand.SpaceShip;
import org.hibernate.test.sql.hand.Speech;
-import org.hibernate.cfg.Configuration;
-import org.hibernate.cfg.Environment;
-import org.hibernate.junit.functional.FunctionalTestCase;
-import org.hibernate.junit.functional.FunctionalTestClassTestSuite;
+import org.hibernate.transform.AliasToEntityMapResultTransformer;
import org.hibernate.transform.DistinctRootEntityResultTransformer;
import org.hibernate.transform.Transformers;
-import org.hibernate.transform.AliasToEntityMapResultTransformer;
/**
* Tests of various features of native SQL queries.
@@ -608,6 +609,63 @@
}
}
+ public void testAddJoinForManyToMany() {
+ Session s = openSession();
+ Transaction t = s.beginTransaction();
+ Person gavin = new Person( "Gavin" );
+ Person max = new Person( "Max" );
+ Person pete = new Person( "Pete" );
+
+ Group hibernate = new Group( "Hibernate" );
+ Group seam = new Group( "Seam" );
+
+ s.persist( gavin );
+ s.persist( max );
+ s.persist( pete );
+ s.persist( seam );
+ s.persist( hibernate );
+
+ hibernate.getPersons().add( gavin );
+ hibernate.getPersons().add( max );
+ seam.getPersons().add( gavin );
+ seam.getPersons().add( pete );
+
+ s.flush();
+ s.clear();
+
+ // todo : see http://opensource.atlassian.com/projects/hibernate/browse/HHH-3908
+// String sqlStr = "SELECT {groupp.*} , {gp.*} " +
+// "FROM GROUPP groupp, GROUP_PERSON gp, PERSON person WHERE groupp.ID = gp.GROUP_ID and person.PERID = gp.PERSON_ID";
+//
+// List l = s.createSQLQuery( sqlStr )
+// .addEntity("groupp", Group.class)
+// .addJoin("gp","groupp.persons")
+// .list();
+ List l = s.getNamedQuery( "manyToManyFetch" ).list();
+ //assertEquals( 2, l.size() );
+
+ t.commit();
+ s.close();
+
+ s = openSession();
+ t = s.beginTransaction();
+
+ seam.getPersons().remove( gavin );
+ seam.getPersons().remove( pete );
+
+ hibernate.getPersons().remove( gavin );
+ hibernate.getPersons().remove( max );
+
+ s.delete( seam );
+ s.delete( hibernate );
+ s.delete( gavin );
+ s.delete( max );
+ s.delete( pete );
+
+ t.commit();
+ s.close();
+ }
+
private static class UpperCasedAliasToEntityMapResultTransformer extends AliasToEntityMapResultTransformer {
public Object transformTuple(Object[] tuple, String[] aliases) {
String[] ucAliases = new String[aliases.length];
More information about the hibernate-commits
mailing list