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

Bug 173831

Summary: [faces-config] Managed bean editor drops value data
Product: [WebTools] Java Server Faces Reporter: Cameron Bateman <cameron.bateman>
Component: UIAssignee: Raghunathan Srinivasan <raghunathan.srinivasan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: dmgloss, raghunathan.srinivasan
Version: 2.0Keywords: helpwanted
Target Milestone: 3.1.1   
Hardware: PC   
OS: Windows XP   
Whiteboard: faces-config editor
Attachments:
Description Flags
Proposed fix none

Description Cameron Bateman CLA 2007-02-12 00:39:45 EST 
1) Create a managed bean property on a normal Java bean.
2) Set the name to field "x" of type String (for the sake of demo) and enter a value, say "foo".
3) Save it by hitting Ok.
4) Select the property and hit "Edit".
5) Change the property name to a different bean property.
6) Don't change anything and hit Ok.
7) Notice, the value "foo" has been cleared from the bean property for no reason. The property is also now DTD invalid.
Comment 1 John Lanuti CLA 2007-04-11 10:46:36 EDT 
Updating target to RC0.
Comment 2 Raghunathan Srinivasan CLA 2007-04-13 17:36:00 EDT 
Consider for 3.0
Comment 3 Raghunathan Srinivasan CLA 2008-04-14 19:26:18 EDT 
Deferred due to lack of resources.
Comment 4 Vadim Dmitriev CLA 2008-07-12 16:30:20 EDT 
Created attachment 107261 [details]
Proposed fix

Recently I hit a bug looking very similar to this one. If editor dialog is opened for "value"-type managed property and immediately closed with "OK" button - "value" field will be cleared.

This patch should fix this issue.
Comment 5 Raghunathan Srinivasan CLA 2008-07-14 12:49:45 EDT 
(In reply to comment #4)
> Created an attachment (id=107261) [details]
> Proposed fix
> Recently I hit a bug looking very similar to this one. If editor dialog is
> opened for "value"-type managed property and immediately closed with "OK"
> button - "value" field will be cleared.
> This patch should fix this issue.

Thanks, Vadim for the patch. Ww will review and apply to HEAD.
Comment 6 Raghunathan Srinivasan CLA 2009-08-27 00:58:00 EDT 
Patch applied