Bug 278011 - [WorkingSets] Incorrect change event firing in WorkbenchPage.setWorkingSets(..)
Summary: [WorkingSets] Incorrect change event firing in WorkbenchPage.setWorkingSets(..)
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-27 05:41 EDT by Hitesh CLA
Modified: 2019-09-06 16:04 EDT (History)
2 users (show)

See Also:

Attachments
Patch v01 (1.19 KB, patch)
2009-07-22 07:28 EDT, Hitesh 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 Hitesh CLA 2009-05-27 05:41:01 EDT 
Just spotted this. In WorkbenchPage.setWorkingSets(..) the Property Change Event code is as follows

if (!Arrays.equals(oldWorkingSets, newWorkingSets)) {
			firePropertyChange(CHANGE_WORKING_SETS_REPLACE, oldWorkingSets,
					newWorkingSets);
			if (aggregateWorkingSet != null) {
				aggregateWorkingSet.setComponents(workingSets);
			}
		}
The doc for CHANGE_WORKING_SETS_REPLACE event : "Change event id when the page working set list was replaced". The event should be fired after the aggregateWorkingSet has been updated.
Comment 1 Hitesh CLA 2009-07-22 07:28:06 EDT 
Created attachment 142246 [details]
Patch v01
Comment 2 Eclipse Webmaster CLA 2019-09-06 16:04:31 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.