diff options
3 files changed, 14 insertions, 15 deletions
diff --git a/bundles/org.eclipse.ui.navigator/META-INF/MANIFEST.MF b/bundles/org.eclipse.ui.navigator/META-INF/MANIFEST.MF index 46d9ae09d16..02599169c5c 100644 --- a/bundles/org.eclipse.ui.navigator/META-INF/MANIFEST.MF +++ b/bundles/org.eclipse.ui.navigator/META-INF/MANIFEST.MF @@ -2,7 +2,7 @@ Manifest-Version: 1.0 Bundle-ManifestVersion: 2 Bundle-Name: %Plugin.name Bundle-SymbolicName: org.eclipse.ui.navigator; singleton:=true -Bundle-Version: 3.5.300.qualifier +Bundle-Version: 3.5.400.qualifier Bundle-Activator: org.eclipse.ui.internal.navigator.NavigatorPlugin Bundle-Vendor: %Plugin.providerName Bundle-Localization: plugin diff --git a/bundles/org.eclipse.ui.navigator/pom.xml b/bundles/org.eclipse.ui.navigator/pom.xml index 1c267d2f8a0..c88d236c329 100644 --- a/bundles/org.eclipse.ui.navigator/pom.xml +++ b/bundles/org.eclipse.ui.navigator/pom.xml @@ -19,6 +19,6 @@ </parent> <groupId>org.eclipse.ui</groupId> <artifactId>org.eclipse.ui.navigator</artifactId> - <version>3.5.300-SNAPSHOT</version> + <version>3.5.400-SNAPSHOT</version> <packaging>eclipse-plugin</packaging> </project> diff --git a/bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/extensions/Binding.java b/bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/extensions/Binding.java index ec34b6f6fca..6023363e24d 100644 --- a/bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/extensions/Binding.java +++ b/bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/extensions/Binding.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2006, 2010 IBM Corporation and others. + * Copyright (c) 2006-2014 IBM Corporation and others. * All rights reserved. This program and the accompanying materials * are made available under the terms of the Eclipse Public License v1.0 * which accompanies this distribution, and is available at @@ -7,6 +7,7 @@ * * Contributors: * IBM Corporation - initial API and implementation + * Matthieu Wipliez <matthieu.wipliez@synflow.com> (Synflow SAS) - [CommonNavigator] Implementation of Binding isVisibleExtension not excluding as expected - http://bugs.eclipse.org/425867 ******************************************************************************/ package org.eclipse.ui.internal.navigator.extensions; @@ -48,36 +49,34 @@ class Binding { * @see org.eclipse.ui.internal.navigator.extensions.INavigatorViewerDescriptor#isVisibleExtension(java.lang.String) */ boolean isVisibleExtension(String anExtensionId) { - - // Have we seen this pattern before? if (knownIds.containsKey(anExtensionId)) { // we have, don't recompute return ((Boolean) knownIds.get(anExtensionId)).booleanValue(); } - - for (Iterator itr = includePatterns.iterator(); itr.hasNext();) { + + for (Iterator itr = excludePatterns.iterator(); itr.hasNext();) { Pattern pattern = (Pattern) itr.next(); if (pattern.matcher(anExtensionId).matches()) { - // keep track of the result for next time - knownIds.put(anExtensionId, Boolean.TRUE); + knownIds.put(anExtensionId, Boolean.FALSE); if (Policy.DEBUG_RESOLUTION) { - System.out.println("Viewer Binding: " + TAG_EXTENSION +//$NON-NLS-1$ + System.out.println("Viewer Binding: EXCLUDED: " + TAG_EXTENSION +//$NON-NLS-1$ " to: " + anExtensionId); //$NON-NLS-1$ } - return true; + return false; } } - for (Iterator itr = excludePatterns.iterator(); itr.hasNext();) { + for (Iterator itr = includePatterns.iterator(); itr.hasNext();) { Pattern pattern = (Pattern) itr.next(); if (pattern.matcher(anExtensionId).matches()) { - knownIds.put(anExtensionId, Boolean.FALSE); + // keep track of the result for next time + knownIds.put(anExtensionId, Boolean.TRUE); if (Policy.DEBUG_RESOLUTION) { - System.out.println("Viewer Binding: EXCLUDED: " + TAG_EXTENSION +//$NON-NLS-1$ + System.out.println("Viewer Binding: " + TAG_EXTENSION +//$NON-NLS-1$ " to: " + anExtensionId); //$NON-NLS-1$ } - return false; + return true; } } |