[#OPENJPA-280] Enhancement failure for inherited fields introduced with change 532137

OpenJPA

Enhancement failure for inherited fields introduced with change 532137

Created: 06/Jul/07 08:49 PM Updated: 13/Jul/07 08:03 PM

Return to search

Component/s:

kernel

Affects Version/s:

1.0.0

Fix Version/s:

1.0.0

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	PatchPCEnhancer	2007-07-06 08:50 PM	David Ezzio	0.7 kB

Environment:

The failure shows up when running the JDO 2. TCK on Kodo. Four test cases fail without the supplied patch.

Resolution Date:

13/Jul/07 08:03 PM

Description

« Hide

When the PCEnhancer.getPersistenceCapableOwner method was altered, the side effect on the "owner" parameter with the old code was removed with the change in 532137. The patch does the right thing by using f.getDeclaringClass() instead of "owner" when looking up the metadata.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

David Ezzio added a comment - 06/Jul/07 08:50 PM

SVN patch for the bug. (A one-line change.)

David Ezzio made changes - 06/Jul/07 08:50 PM

Field	Original Value	New Value
Attachment		PatchPCEnhancer [ 12361326 ]

Repository	Revision	Date	User	Message
ASF	#554083	Fri Jul 06 23:21:06 UTC 2007	pcl	~~OPENJPA-280~~. Committing for David Ezzio.
				Files Changed
				MODIFY /openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java

[ Permlink | « Hide ]

David Ezzio added a comment - 07/Jul/07 01:27 AM

The patch applied (svn 554083) is not the patch submitted here. I found that the tests (for mvn package) fail with the applied patch, but not with the submitted patch.

Repository	Revision	Date	User	Message
ASF	#554111	Sat Jul 07 01:47:21 UTC 2007	pcl	~~OPENJPA-280~~. Correcting my mistaken commit.
				Files Changed
				MODIFY /openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java

[ Permlink | « Hide ]

David Ezzio added a comment - 13/Jul/07 08:03 PM

Fixed by patch applied at 554111

David Ezzio made changes - 13/Jul/07 08:03 PM

Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Resolved [ 5 ]