Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316915 - CMPPolicy.createPrimaryKeyInstanceFromPrimaryKeyValues does not support PK class
Summary: CMPPolicy.createPrimaryKeyInstanceFromPrimaryKeyValues does not support PK class
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-15 10:39 EDT by Gordon Yorke CLA
Modified: 2022-06-09 10:19 EDT (History)
1 user (show)

See Also:

Attachments
Proposed Patch (1.67 KB, application/octet-stream)
2010-06-15 10:49 EDT, Gordon Yorke CLA 		no flags Details
Intermediate patch (2.50 KB, patch)
2010-06-17 16:07 EDT, Gordon Yorke CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Yorke CLA 2010-06-15 10:39:40 EDT 
This code has a problem where the ElementIndex is not incremented correctly so compound PK classes are not processed correctly.
Comment 1 Gordon Yorke CLA 2010-06-15 10:49:01 EDT 
Created attachment 171937 [details]
Proposed Patch

Simple patch to ensure the index is incremented.
Comment 2 Gordon Yorke CLA 2010-06-17 15:44:06 EDT 
This patch requires an update to support EmbeddedIds and nested EmbeddedIds.
Comment 3 Gordon Yorke CLA 2010-06-17 16:07:36 EDT 
Created attachment 172155 [details]
Intermediate patch

This upload includes some changes that are required for EmbeddedId support but this patch should not yet be considered complete as it does not include support for nested EmbeddedIds.
Comment 4 Gordon Yorke CLA 2010-06-29 16:27:41 EDT 
Patch back ported to 2.1.1
Comment 5 Gordon Yorke CLA 2010-07-22 15:30:20 EDT 
Nested Embeddables are not allowed by the JPA specification.  The patch is complete.  Nested EmbeddedIds can be added at a later date.
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:19:55 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink