Bug 231920 - New Entity wizard generating "@IdClass" for compound keys
Summary: New Entity wizard generating "@IdClass" for compound keys
Status: NEW
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: JPA (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: Future   Edit
Assignee: Nan Li CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2008-05-13 15:32 EDT by Chuck Bridgham CLA
Modified: 2013-10-08 11:08 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chuck Bridgham CLA 2008-05-13 15:32:37 EDT
To be consistent with bottom-up support, and also using the best practice for compound key classes...

@Embeddable needs to be used.
Comment 1 Neil Hauge CLA 2008-05-13 16:09:46 EDT
That makes sense, but it might be too late to make this change for 2.0.  Copying Dimiter so he can take a look.
Comment 2 Dimiter Dimitrov CLA 2008-05-16 03:09:03 EDT
I thought about embeddable ID class generation, but leave it because it generation seems to be separated process. EmbeddedId class consists of fields/properties, not introduced in entity itself, so in my opinion there are (mainly) two approaches to provide support for it generation:
1. Different wizard for the creation of Embeddable object intended to be EmbeddedId. After this generation, it could be allowed to be set as PK in the entity wizard
2. Additional page in entity wizard, for EmbeddedId settings. This approach seems to be clearer, but my fear is that Entity wizard will become very intricate with all these settings
However I agree that the request make sense and it would be fine to be implemented for the next release. In this context every comments/notes above my thoughts, as well as every new proposal for support of Embeddable objects as PK will be appreciated

Comment 3 Neil Hauge CLA 2011-07-01 16:25:28 EDT
Moving JPA specific bugs to new JPA component in bugzilla.