Summary: | [Team] NPE When Loading Team View | ||||||
---|---|---|---|---|---|---|---|
Product: | [Tools] Target Management | Reporter: | Kevin Doyle <kjdoyle> | ||||
Component: | RSE | Assignee: | David Dykstal <ddykstal.eclipse> | ||||
Status: | CLOSED FIXED | QA Contact: | Martin Oberhuber <mober.at+eclipse> | ||||
Severity: | normal | ||||||
Priority: | P3 | CC: | dmcknigh, kmunir | ||||
Version: | 2.0 | ||||||
Target Milestone: | 2.0.1 | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Kevin Doyle
2007-06-25 13:48:34 EDT
Created attachment 73329 [details]
Check to see if the ISystemProfile is null before attempting to obtain SystemTeamViewCategoryNode
This will resolve the issue of the NPE that is thrown on the restart. Although when investigating the issue, it seems that the state of the expanded tree in the Team View is not kept after the application is restarted. This could be another defect that might want to be opened?
I, Rupen Mardirossian, declare that I developed attached code from
scratch, without referencing any 3rd party materials except material licensed
under the EPL. I am authorized by my employer, IBM Canada Ltd. to make this
contribution under the EPL.
Rupen -- This is a result of changing root for the SystemTeamView. There used to be a RemoteSystemsProject there that the profiles were descended from. The problem is that the memento that is being stored is derived from the current structure of the view (as it should be) but the code reading that memento on restore is assuming the old structure. Thus, it is looking for a "RemoteSystemsProject" when there isn't any and misinterpreting the tokens present in the string. The code should tolerate the memento being delivered now but we should ensure that they are being decoded in the proper format. Anyway, please investigate to see if we are constructing mementos properly for this view when the workbench closes and that we are restoring from them properly when the view opens. We should probably also make sure that the mementos that currently exist don't cause problems but result in the view being "folded up" correctly. Not too sure if my patch was commited. Although, the NPE does not seem present anymore. There is still the persistence issue in which the team view does not have the profiles expanded as they were before the shutting of the app. When the app is restarted, all the profiles are collapsed. Should this be opened as a seperate bug? Apparently it was not committed. SystemTeamViewPart does not contain the patch and on the bug there is also no evidence that it was committed. This was fixed at the same time bug 187741 was fixed. Closing this as a result. Bug 201778 was opened to track the problem of restoring the state of the team view. Verified fixed with 2.0.1 RC1. Removing "contributed" kwd since this patch was not accepted and the issue was fixed differently. |