Author: rob.stryker(a)jboss.com Date: 2008-12-01 20:53:58 -0500 (Mon, 01 Dec 2008) New Revision: 12191 Modified: trunk/as/plugins/org.jboss.ide.eclipse.archives.webtools/src/org/jboss/ide/eclipse/archives/webtools/ui/FilesetActionProvider.java Log: JBIDE-3292 - NPE, was not using the setSelection() call in all proper places, only some. Modified: trunk/as/plugins/org.jboss.ide.eclipse.archives.webtools/src/org/jboss/ide/eclipse/archives/webtools/ui/FilesetActionProvider.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.archives.webtools/src/org/jboss/ide/eclipse/archives/webtools/ui/FilesetActionProvider.java 2008-12-01 22:00:23 UTC (rev 12190) +++ trunk/as/plugins/org.jboss.ide.eclipse.archives.webtools/src/org/jboss/ide/eclipse/archives/webtools/ui/FilesetActionProvider.java 2008-12-02 01:53:58 UTC (rev 12191) @@ -15,9 +15,7 @@ import org.eclipse.jface.action.IMenuManager; import org.eclipse.jface.viewers.DoubleClickEvent; import org.eclipse.jface.viewers.IDoubleClickListener; -import org.eclipse.jface.viewers.IStructuredSelection; import org.eclipse.jface.viewers.ITreeSelection; -import org.eclipse.jface.viewers.StructuredSelection; import org.eclipse.jface.viewers.StructuredViewer; import org.eclipse.jface.viewers.TreePath; import org.eclipse.jface.window.Window; @@ -78,12 +76,8 @@ public void fillContextMenu(IMenuManager menu) { ICommonViewerSite site = actionSite.getViewSite(); - IStructuredSelection selection = null; if (site instanceof ICommonViewerWorkbenchSite) { - ICommonViewerWorkbenchSite wsSite = (ICommonViewerWorkbenchSite) site; - selection = (IStructuredSelection) wsSite.getSelectionProvider() - .getSelection(); - selected = selection.toArray(); + setSelection(); if( selected == null ) return; if( selected.length == 1 && selected[0] instanceof ServerWrapper ) {