Bugzilla – Full Text Bug Listing
| Summary: | Can't open perspective (NPE) | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Dani Megert <daniel_megert> | ||||
| Component: | UI | Assignee: | Eric Moffatt <emoffatt> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | critical | ||||||
| Priority: | P3 | CC: | emoffatt, john.arthorne, remy.suen, tom.schindl | ||||
| Version: | 3.8 | ||||||
| Target Milestone: | 4.2 M3 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows 7 | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Dani Megert
Could be introduced by bug 355750 Created attachment 203594 [details]
Workspace
1. extract the zip into a new folder
2. start Eclipse using that folder as workspace
3. Window > Open Perspective > Java Browsing
==> NPE
The problem is really caused by the fix for bug 355750 - it looks like page.getPerspective() returns null. I think the difference is that in this workspace config perspectives are opened always in new windows and so it fails when restoring. I20110915-1308 1. Start in a new workspace. 2. Close the 'Welcome' screen. 3. Window > Save Perspective As... > Some name > OK 4. Window > New Window 5. Nothing happens, the error is logged, check the 'Error Log' view. 6. Close your window. 7. Eclipse will _not_ prompt you. 8. Check your processes, Eclipse is actually still running. If you try to run Eclipse again it will tell you the workspace is still in use. Eric asked me whether this deserves a respin of M2 (M2a since M2 is already available). Arguments in favour of a respin: + This is really bad end user behaviour. Essentially Window->New Window is broken for perspectives that have been customized and saved. + Failure to shutdown properly after the failure makes it worse Arguments against respin: - We are past our release train date so this impacts other projects' schedules. Some +1 projects would need to react to this change. - Perspective customization and Window->New Window are not the most common workflows. For example nobody hit this in the three weeks since the bug was introduced. - There is a workaround: Window > Reset Perspective seems to fix it. - This "only" affects 3.8 M2. The problem does not occur in 4.2 M2. This means release train and EPP packages are not affected by the problem. Overall I could go either way, but I think this does not deserve an M2a. This is lost end user behaviour rather than something that affects downstream projects. There will be an integration build tomorrow towards M3 that will contain a fix, so if end users can't live without this workflow they can upgrade to tomorrow's build to get the fix. commit cac34ecf178dc898ef1db0badbd67da7301ffc0f This commit (on R3_development) reverts the changes in the ViewSashContainer made by the fix fir bug 355750 (which I'll re-open and tag for a revisit in M3). I've re-opened bug 355750 for more work in M3. We should likely verify that this is fixed in tomorrow's I-build and close it if it is... > Overall I could go either way, but I think this does not deserve an M2a. I agree. > There is a workaround: Window > Reset Perspective seems to fix it. If one does not want to loose the settings one can also use M1 to open the perspective and then switch to M2 again. We should have a test to ensure that this problem doesn't occur again. I also opened bug 358229 for the shutdown problem described by comment 5. I'll mark this as fixed since we've reverted the original 'fix', we're continuing to work on the issue in bug 355750. |