Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338893 - notification link opens unexpected plan when build is in progress
Summary: notification link opens unexpected plan when build is in progress
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P2 normal (vote)
Target Milestone: 0.9   Edit
Assignee: Steffen Pingel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-04 02:27 EST by Steffen Pingel CLA
Modified: 2012-02-25 13:08 EST (History)
0 users

See Also:


Attachments
mylyn/context/zip (108.23 KB, application/octet-stream)
2012-02-25 13:08 EST, Steffen Pingel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Steffen Pingel CLA 2011-03-04 02:27:53 EST
Steps:
1. Refresh Builds view
2. Fail a build
3. Run build
4. Refresh view

A notification for the failed plan is displayed as expected. When the link in the notification popup is clicked the latest running build is opened and not the failed one.
Comment 1 Steffen Pingel CLA 2011-03-07 19:46:43 EST
The notification only references the plan but would need to include a reference to the corresponding build object to make this work. This should get fixed but will require more thought.
Comment 2 Steffen Pingel CLA 2011-08-27 13:13:18 EDT
RefreshSession.refreshPlans() runs before the last build for a plan is updated and hence the notification is displayed before the latest build is available in the model. If the link is clicked before the refresh completes the previously cached build is shown.
Comment 3 Steffen Pingel CLA 2012-02-25 13:08:40 EST
The framework code has been changed to always store the last build and download all details in case the link is clicked before it has been fully received.
Comment 4 Steffen Pingel CLA 2012-02-25 13:08:44 EST
Created attachment 211626 [details]
mylyn/context/zip