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

Bug 46852

Summary:	[Viewers] remove should not try to preserve selection for removed elements
Product:	[Eclipse Project] Platform	Reporter:	Nick Edgar <n.a.edgar>
Component:	UI	Assignee:	Platform UI Triaged <platform-ui-triaged>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	normal
Priority:	P3	CC:	eclipse, tom.schindl
Version:	3.0	Keywords:	helpwanted
Target Milestone:	---
Hardware:	PC
OS:	Windows 2000
Whiteboard:	stalebug

Description Nick Edgar

2003-11-18 12:18:58 EST

build N20031117

- When calling AbstractTreeViewer.remove, it calls preservingSelection, which 
tries to preserve the selection and is also responsible for notifying listeners 
of changes to the selection.
- It is often the case that the selection contains the elements being removed.
- It should not try to restore the selection for these elements, since this 
causes extra work, and may lead to population of the parent from the model, 
which can lead to inconsistencies (see bug 44081).

Should fix up all viewers, not just AbstractTreeViewer.
Need to be careful not to break contracts for protected API when fixing this.

Comment 1 Michael Van Meekeren

2004-05-25 15:01:39 EDT

Deferring

Comment 2 Nick Edgar

2006-03-15 11:47:44 EST

Reassigning bugs in component areas that are changing ownership.

Comment 3 Boris Bokowski

2009-11-26 09:48:26 EST

Hitesh is now responsible for watching bugs in the [Viewers] component area.

Comment 4 Eclipse Webmaster

2019-09-06 16:17:28 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

Comment 5 Eclipse Genie

2021-10-07 16:42:48 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.