/*
 * Contributions to FindBugs
 * Copyright (C) 2008, Andrei Loskutov
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Lesser General Public
 * License as published by the Free Software Foundation; either
 * version 2.1 of the License, or (at your option) any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this library; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 */
package de.tobject.findbugs.view;

import org.eclipse.core.resources.IMarker;
import org.eclipse.jface.viewers.ISelection;
import org.eclipse.ui.ISelectionListener;
import org.eclipse.ui.IWorkbenchPart;
import org.eclipse.ui.views.properties.PropertySheet;

import de.tobject.findbugs.reporter.MarkerUtil;

class MarkerSelectionListener implements ISelectionListener {
	private final IMarkerSelectionHandler handler;

	public MarkerSelectionListener(IMarkerSelectionHandler handler) {
		this.handler = handler;
	}

	public void selectionChanged(IWorkbenchPart thePart,
			ISelection theSelection) {
		if (thePart == handler || !handler.isVisible()) {
			return;
		}
		IMarker marker = MarkerUtil.getMarkerFromSingleSelection(theSelection);
		// ignore selections from property view: if this view shows our bug,
		// we simply can't switch to any other Property view pane or select some text...
		if (marker == null && thePart instanceof PropertySheet) {
			return;
		}
		handler.markerSelected(thePart, marker);
	}
}

@@ -1037,63 +1037,8 @@
 rt;%0D
-%0Aimport org.eclipse.ui.views.properties.PropertySheet;%0D
 %0A%0D%0Ai
@@ -1542,157 +1542,362 @@
 %09// 
-ignore selections from property view: if this view shows our bug,%0D%0A%09%09// we simply can't switch to any other Property view pane or select some text...
+only handle a null if it's from the bug explorer. this way if the user%0D%0A%09%09// selects a bug pattern or other %22folder%22 in the tree, the annotation view%0D%0A%09%09// and the property view are cleared. BUT if the user clicks into %0D%0A%09%09// the editor, or some other view, it stays up. not quite sure what the%0D%0A%09%09// best behavior is here but for now this works. -Keiths
 %0D%0A%09%09
@@ -1918,16 +1918,18 @@
 null && 
+!(
 thePart 
@@ -1943,21 +1943,24 @@
 eof 
-PropertySheet
+BugExplorerView)
 ) %7B%0D

improve bug comments tab behavior when user switches between Views.