openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henry Lai (JIRA)" <j...@apache.org>
Subject [jira] Created: (OPENJPA-329) Generic interface may cause the openjpa Reflection code to pickup the wrong getter method
Date Wed, 22 Aug 2007 22:12:31 GMT
Generic interface may cause the openjpa Reflection code to pickup the wrong getter method
-----------------------------------------------------------------------------------------

                 Key: OPENJPA-329
                 URL: https://issues.apache.org/jira/browse/OPENJPA-329
             Project: OpenJPA
          Issue Type: Bug
          Components: jpa
            Reporter: Henry Lai
             Fix For: 1.0.0


The test case

	public void testFind(){
		PersistenceProviderImpl openJPA = new PersistenceProviderImpl();
		EntityManagerFactory factory = 
			openJPA.createEntityManagerFactory("test", "ptp/test/issue6/persistence.xml",
						System.getProperties() );
		
        EntityManager em = factory.createEntityManager();
        em.getTransaction().begin();
        
		String query="select obj from T6Entity obj where obj.name=?1";
        Query queryObject = em.createQuery(query);
        queryObject.setParameter(1, "Mom");
        List resultList = queryObject.getResultList();
        
        em.getTransaction().commit();
        em.close();
	}


produces the following exception

<1.0.0-SNAPSHOT-SNAPSHOT fatal user error> org.apache.openjpa.persistence.ArgumentException:
Collection field "ptp.test.issue6.T6Entity.children" declares that it is mapped by "ptp.test.issue6.T6Entity.parent",
but this is not a valid inverse relation.
	at org.apache.openjpa.jdbc.meta.MappingRepository.useInverseKeyMapping(MappingRepository.java:903)
	at org.apache.openjpa.jdbc.meta.MappingRepository.defaultTypeStrategy(MappingRepository.java:838)
	at org.apache.openjpa.jdbc.meta.MappingRepository.defaultStrategy(MappingRepository.java:771)
	at org.apache.openjpa.jdbc.meta.RuntimeStrategyInstaller.installStrategy(RuntimeStrategyInstaller.java:62)

public interface ITree <T extends ITree> {
	public List<T> getChildren();
	public T getParent();
}


public class T6Entity implements ITree {

	private Long id;
	T6Entity parent;
	List<T6Entity> children;

	public T6Entity getParent() {
		return parent;
	}
	public Long getId() {
		return id;
	}
	public void setId(Long id) {
		this.id = id;
	}
	public List<T6Entity> getChildren() {
		return children;
	}
	public void setChildren(List<T6Entity> children) {
		this.children = children;
	}
	public void setParent(T6Entity parent) {
		this.parent = parent;
	}
	public void addChild(T6Entity aChild) {
		if ( children == null ){
			children = new ArrayList<T6Entity>();
		}
		children.add( aChild );
		aChild.setParent(this);
	}

}


	<entity class="T6Entity">
		<table name="T6Entity" />
		<attributes>
			<id name="id">
				<column name="S_ID" />
				<generated-value strategy="SEQUENCE" generator="seqGen1" />
				<sequence-generator name="seqGen1"	sequence-name="SEQ_OBJECT_ID" />
			</id>

			<many-to-one name="parent" fetch="EAGER">
				<join-column name="PARENT_ID" />
				<cascade>
					<cascade-persist />
					<cascade-merge />
				</cascade>
			</many-to-one>
			<one-to-many name="children"
				target-entity="T6Entity"
				mapped-by="parent" fetch="LAZY">
				<cascade>
					<cascade-persist />
					<cascade-merge />
				</cascade>
			</one-to-many>
		</attributes>
	</entity>



The java compiler generate the following 2 method for getParent()

public volatile ptp.test.issue6.ITree ptp.test.issue6.T6Entity.getParent()
public ptp.test.issue6.T6Entity ptp.test.issue6.T6Entity.getParent()

and depending on the order of the methods return from the reflection getDeclaredMethod(),
 it cause the wrong return type to be infer as the inverse relationship. 

There is a  work around, by specifying the target-entity in the many-to-one mapping.

But may be something that should be fix.




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message