Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 345011

Summary: Cache size default not updating properly
Product: [WebTools] Dali JPA Tools Reporter: Leslie Davis <les.davis>
Component: JPAAssignee: Pascal Filion <pascal.filion>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: neil.hauge, paul.fullbright
Version: 3.0Flags: neil.hauge: review? (brian.vosburgh)
Target Milestone: 3.2   
Hardware: PC   
OS: Windows 7   
Whiteboard: fix_ready
Attachments:
Description Flags
Pascal's proposed patch. none

Description Leslie Davis CLA 2011-05-06 14:38:38 EDT 
Steps to reproduce:

1.  Create JPA 2.0 project
2.  Generate entities from tables
3.  Select one of the generated entities and expand Caching
4.  Select the persistence.xml Caching tab
5.  Change the cache size to anything
6.  Return to the entity caching tab and note the default value is unchanged

Note, if you then click on the drop-down the correct updated value will be visible and once selected the value updates properly afterward.
Comment 1 Paul Fullbright CLA 2011-05-06 15:52:32 EDT 
I don't believe this bug actually depends on bug 245431, but the ability to duplicate this behavior does depend on the functionality added by its patch.

Removing bug as a dependency, but maintaining the link with this comment.
Comment 2 Pascal Filion CLA 2011-05-10 09:21:25 EDT 
It seems Combo.setText() does not update the selected index. When changing the selected item, getSelectionIndex() returns -1 causing the text to not be updated. So the fix uses Combo.select() instead if the selected item is one of the combo's items, otherwise setText() is used.
Comment 3 Neil Hauge CLA 2011-05-10 11:50:34 EDT 
Created attachment 195236 [details]
Pascal's proposed patch.

Proposed patch from Pascal.
Comment 4 Neil Hauge CLA 2011-05-10 11:51:45 EDT 
Reopening to go through review process.
Comment 5 Neil Hauge CLA 2011-07-01 16:23:10 EDT 
Moving JPA specific bugs to new JPA component in bugzilla.
Comment 6 Neil Hauge CLA 2011-10-25 13:31:28 EDT 
Brian hasn't had a chance to look at this yet due to more pressing issues...deferring to Juno.
Comment 7 Neil Hauge CLA 2012-05-09 14:26:26 EDT 
This appears to be fixed via some other change.  Closing.