This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 132244 - support Column/JoinColumn annotation insertable and updatable
Summary: support Column/JoinColumn annotation insertable and updatable
Status: CLOSED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: General (show other bugs)
Version: 0.5   Edit
Hardware: PC Windows XP
: P2 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Brian Vosburgh CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-16 15:40 EST by Karen Butzke CLA
Modified: 2007-03-14 18:21 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karen Butzke CLA 2006-03-16 15:40:49 EST
The default for these two elements is true.  If both are false this is the equivalent of a read only mapping.  I recommend adding the following to the UI anywhere a Column or JoinColumn is specified


 _[] Read Only __________
|                        |
|     [] Insertable      |
|     [] Updatable       |
|________________________|

If 'Read Only' is checked, then the subpanel would be disables and both insertable and updatabase would be set to false.  The Insertable and Updatable widgets should be combos with the options:  Default (true), true, false.

With multiple joinColumns, can they each have different settings for these?  If not, should we only have one widget in the UI and how do we handle if they set things differently on each joinColumn directly in the source.
Comment 1 Kevin Sutter CLA 2006-04-12 09:46:18 EDT
Oops, I just created a separate bug (#136321) that covers the "updateable" request for the @Column support.  (The reason I didn't immediately find this report is because I only looked at the reported bugs and not the enhancement requests.)
Comment 2 Brian Vosburgh CLA 2006-06-03 00:24:12 EDT
We can't use check-boxes for Insertable and Updatable. We have to use combo-boxes to support the 3 options: Default, True, False. As a result, we are not implementing the suggested "Read Only" check box. The current implementation will also take up less screen space. We can re-visit this idea in the future if it still looks desirable after using the current implementation for a while.
Comment 3 Karen Butzke CLA 2006-06-28 20:10:41 EDT
verified this is fixed in build R-0.5RC2-200606280020
Comment 4 Neil Hauge CLA 2007-03-14 18:21:29 EDT
Closing Verified Fixed bugs from 0.5.