[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[platform-ui-dev] Bug in Navigator Framework and Fix

Hi all,

hope it is fine, when I post this issue here...

I had a problem with the Common Navigator Framework. I have debugged the system in detail and also read the documentation. I feel that there is a bug and may be I also have a fix for that.

The problem occurs when overriding another navigator content extension. The NavigatorContentProvider is not invoked although according to the documentation it should.
Enclosed the an excerpt of the plugin.xml:

      <navigatorContent
            activeByDefault="true"
            contentProvider="org.oomega.core.eclipse.navigator.NavigatorContentProvider"
            icon="icons/oomega.gif"
            id="org.oomega.core.eclipse.modelContent"
            labelProvider="org.oomega.core.eclipse.navigator.NavigatorLabelProvider"
            name="Model Repository Content"
            priority="high"
            providesSaveables="true">
         <override
               policy="InvokeAlwaysRegardlessOfSuppressedExt"
               suppressedExtensionId="org.eclipse.ui.navigator.resourceContent">
         </override>

         ...

      </navigatorContent>

When debugging the the code a got the feeling that the wrong OverridePolicy is checked. When I replace InvokeAlwaysRegardlessOfSuppressedExt by InvokeOnlyIfSuppressedExtAlsoVisibleAndActive everything works fine (as shown).

private boolean isOverridingExtensionInSet(INavigatorContentDescriptor aDescriptor, Set theEnabledExtensions) {

if (aDescriptor.getSuppressedExtensionId() != null /*
 * The descriptor is
 * an override
 * descriptor
 */
&& aDescriptor.getOverridePolicy() == OverridePolicy.InvokeOnlyIfSuppressedExtAlsoVisibleAndActive) {
/*
 * if the policy is set as such, it can lead to this extension being
 * invoked twice; once as a first class extension, and once an
 * overriding extension.
 */
if (theEnabledExtensions.contains(contentService.getExtension(aDescriptor.getOverriddenDescriptor()))) {
return true;
}
}
return false;
}

Do I misunderstand something or am I right?!?

Kind regards,
stefano


OOMEGA GbR | Stefano Merenda
Fingerhutstraße 6, 80995 Munich, Germany

Phone +49 89 829097-70 | Fax +49 89 829097-80