Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345011 - Cache size default not updating properly
Summary: Cache size default not updating properly
Status: RESOLVED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: JPA (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.2   Edit
Assignee: Pascal Filion CLA
QA Contact:
URL:
Whiteboard: fix_ready
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-06 14:38 EDT by Leslie Davis CLA
Modified: 2012-05-09 14:26 EDT (History)
2 users (show)

See Also:
neil.hauge: review? (brian.vosburgh)

Attachments
Pascal's proposed patch. (1.49 KB, patch)
2011-05-10 11:50 EDT, Neil Hauge CLA 		no flags Details | Diff
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 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.