Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336005 - Temporal combo box does not get updated when choosing temporal radio button
Summary: Temporal combo box does not get updated when choosing temporal radio button
Status: RESOLVED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: JPA (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P4 normal (vote)
Target Milestone: ---   Edit
Assignee: Neil Hauge CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-01 13:55 EST by Nan Li CLA
Modified: 2012-09-18 14:10 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nan Li CLA 2011-02-01 13:55:58 EST
Build Identifier: I20101028-1441

Once specifying temporal option to a mapping of a mapping file, DATE type is added as the default value as <temporal>DATE</temporal>. No default value should be set.

Reproducible: Always

Steps to Reproduce:
1. Create a mapping file 
2. Define a basic or id mapping 
3. In the JPA Details view, select Type -> Temporal
Comment 1 Neil Hauge CLA 2011-02-08 09:55:04 EST
I think this is as-designed.  Temporal requires a setting, and we aren't really setting a Dali "default" in this case, but rather just inserting the first option in the list.  The implementation would probably be more like the java implementation, but XML-EMF translators likely made that difficult to achieve, so this seemed like reasonable behavior in the ORM XML.  

Perhaps Karen could comment on the history here, but either way, I don't think this is worth pursuing at this time.
Comment 2 Neil Hauge CLA 2011-07-01 16:24:31 EDT
Moving JPA specific bugs to new JPA component in bugzilla.
Comment 3 Nan Li CLA 2012-09-14 16:27:50 EDT
Just as what comment 1 says, we give a default value because the XML translator does not support element without value. This is a limitation of the translator. If we want to give a default value, the UI should also have it. Currently the temporal combo box does not get updated if we choose the temporal radio button even though the default value is already written to the source. I update the summary of this bug to track the real issue we want to fix.

PS: we need to fix the same issue of enumerated combo box.
Comment 4 Nan Li CLA 2012-09-17 17:32:07 EDT
This bug is fixed along with the changes of bug 335447.
Comment 5 Nan Li CLA 2012-09-18 14:10:08 EDT
.