Bug 167192 - [faces-config editor] Type Selection Dialog in "Other" tab broken
Summary: [faces-config editor] Type Selection Dialog in "Other" tab broken
Status: CLOSED FIXED
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: 2.0 RC0   Edit
Assignee: Cameron Bateman CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 167190
Blocks:
  Show dependency tree
 
Reported: 2006-12-07 20:14 EST by Cameron Bateman CLA
Modified: 2007-04-13 19:35 EDT (History)
0 users

See Also:

Attachments
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 Cameron Bateman CLA 2006-12-07 20:14:13 EST 
This seems to be dependent on a problem in JDT (see 167190).  What is less clear is whether this has always been broken, whether JDT caused the breakage or whether the refactoring is broken.  

To reproduce:

1) Open the faces-config editor.
2) Switch to the "Other" tab.
3) Open the "Action Listener" item (works for any of them).
4) Click on "Add"
5) In the type selection dialog, type something.

The selection dialog won't find anything and exceptions get logged.
Comment 1 Cameron Bateman CLA 2006-12-07 20:15:39 EST 
By the "refactoring is broken" I meant the JSF plugin refactoring, which included elimination of related redundant classes.
Comment 2 Cameron Bateman CLA 2006-12-07 20:25:18 EST 
Looking at the pre-refactored code, I don't think this is a regression due to the plugin refactoring, since the old code also creates a custom IJavaSearchScope rather than using the default JavaSearchScope.
Comment 3 Raghunathan Srinivasan CLA 2007-01-05 20:24:26 EST 
Triaged for WP 2.0.
Comment 4 Cameron Bateman CLA 2007-01-26 14:41:21 EST 
We appear to have fix for the underlying JDT problem in 167190.  This version should be available by M5.  I have smoke tested the patch provided in 167190 so my  confidence is high that we will have this closed by M6.
Comment 5 John Lanuti CLA 2007-04-11 10:46:32 EDT 
Updating target to RC0.
Comment 6 Cameron Bateman CLA 2007-04-13 19:34:43 EDT 
Fixed.
Comment 7 Cameron Bateman CLA 2007-04-13 19:35:04 EDT 
Verified in WTP 2.0RC6.