Bug 167548 - Faces config editor does not initialize components, converters and validators properly
Summary: Faces config editor does not initialize components, converters and validators...
Status: NEW
Alias: None
Product: Java Server Faces
Classification: WebTools
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: Future   Edit
Assignee: Debajit Adhikary CLA
QA Contact:
URL:
Whiteboard: faces-config editor
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2006-12-12 02:50 EST by Cameron Bateman CLA
Modified: 2008-04-25 16:47 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 Cameron Bateman CLA 2006-12-12 02:50:49 EST
In the "Component" tab of the faces config editor, the user can open the sectios for "Components", "Converters" or "Validators" and select the "Add" button.  Doing so creates a new empty instance of the respective configuration.  However, some of the fields of these configuration elements need to be populated for  the  configuration to be correct.  This is even more noticable in 1.2 because schema validation flags these elements as errors.
Comment 1 Raghunathan Srinivasan CLA 2007-01-05 20:28:33 EST
Triaged for WP 2.0.
Comment 2 Ian Trimble CLA 2007-05-16 16:38:33 EDT
I believe this would be better handled with a wizard on hitting the "Add" button, much like what we currently have for managed beans. By using a wizard, we can validate that we have correct information before updating the model. This change effectively pushed this out of scope for the WTP 2.0 release.
Comment 3 Cameron Bateman CLA 2007-05-16 16:52:43 EDT
A wizard may be the best way but for now I think reasonable non-breaking defaults are good enough.  Each of the types of elements that need defaulting have impls available on the classpath if a runtime is installed (i.e. UIInput for components).  This will at least ensure that the editor is not creating malformed configurations.
Comment 4 Raghunathan Srinivasan CLA 2008-04-25 16:47:46 EDT
Deferred due to lack of resources