Summary: | Quickfix to solve restriction access problem | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] PDE | Reporter: | Pascal Rapicault <pascal> | ||||||||||
Component: | UI | Assignee: | Adam Archer <agarcher> | ||||||||||
Status: | RESOLVED FIXED | QA Contact: | |||||||||||
Severity: | enhancement | ||||||||||||
Priority: | P3 | CC: | caniszczyk | ||||||||||
Version: | 3.1 | Keywords: | contributed, noteworthy | ||||||||||
Target Milestone: | 3.4 M2 | Flags: | caniszczyk:
review+
|
||||||||||
Hardware: | PC | ||||||||||||
OS: | Windows XP | ||||||||||||
Whiteboard: | |||||||||||||
Attachments: |
|
Description
Pascal Rapicault
2005-05-27 09:10:03 EDT
a quickfix is not major. It is an enhancement at best. I agree that what I want is an enhancement, however it is a major enhancement, because this error will cause the users to scratch their heads many times before they can actually figure out what needs to be changed. adam adam adam :D Created attachment 77409 [details]
patch
Adds a quickfix when:
1) The source project is a plug-in project.
2) The unaccessible node resolves to a plug-in project in the workspace.
3) The package containing the unaccessible node is not exported in the MANIFEST.MF.
The one thing I'm not sure about is if JDT will dispose the quickfix icon that I provide. I expect that they should and, therefore, have not bothered to do it myself, but we need to confirm this.
Chris, can you review this one? will do, this one looks like fun. Created attachment 77492 [details]
patch
Turns out JDT does not dispose the Image for us. Unfortunately, because there is no life cycle management on the proposal interface, we have to use the ImageRegistry.
Created attachment 77493 [details]
patch
In the last patch, I changed the other use of the same image to use the copy in the registry, but I forgot to remove the code that disposes it. This one's good.
Great job Adam. This is easily one of the greatest PDE quickfixes ever. I only found a small issue where you would have multiple quickfixes if you had two access restriction notices on something like MyClass.MyStaticField. We want to make sure that we don't present duplicate quickfixes for exporting the same package. I also did some patch cleanup. Thanks again Adam, this one was a good one. Created attachment 77605 [details]
mylyn/context/zip
|