Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312646 - empty feature list when 1 is selected
Summary: empty feature list when 1 is selected
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6 RC2   Edit
Assignee: Ankur Sharma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 313381
  Show dependency tree
 
Reported: 2010-05-12 11:20 EDT by Darin Wright CLA
Modified: 2010-05-21 11:25 EDT (History)
3 users (show)

See Also:
curtis.windatt.public: review+
darin.eclipse: review+

Attachments
Patch (3.67 KB, patch)
2010-05-13 16:05 EDT, Ankur Sharma 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 Darin Wright CLA 2010-05-12 11:20:30 EDT 
Using HEAD:

* New Eclipse Application launch configuration
* Choose "features selected below"
* De-select all features
* check "Only show selected"
* Select "org.eclipse.pde" feature (appears in list)
* Select "plug-ins selected below only"
* Select "features selected below"

The selection count shows 1 of N, but the list/tree is empty

At this point, switching back to "plug-ins selected below" and then back to "features selected below", the selection count is now 0... so the select gets removed somehow.
Comment 1 Curtis Windatt CLA 2010-05-12 17:49:03 EDT 
We should look into this for 3.6.
Comment 2 Ankur Sharma CLA 2010-05-13 16:05:58 EDT 
Created attachment 168466 [details]
Patch
Comment 3 Darin Wright CLA 2010-05-18 09:47:52 EDT 
+1
Comment 4 Curtis Windatt CLA 2010-05-18 12:02:22 EDT 
I separated the extra code changes included in the patch into bug 313381 (and modified the fix to change the text and remove an unecessary import that was added).

Ankur, when you attach a patch for review, please provide a very brief description of what is fixing and/or how it fixes the reported problem.  If a patch contains additional changes that could affect behaviour they need to be reviewed as well.

Fixed in HEAD.
Comment 5 Darin Wright CLA 2010-05-21 11:25:58 EDT 
Verified.