[jboss-svn-commits] JBL Code SVN: r25104 - labs/jbossrules/trunk/drools-eclipse/org.guvnor.tools/src/org/guvnor/tools/views.

[jboss-svn-commits at lists.jboss.org]

Author: john.graham at jboss.org
Date: 2009-02-04 11:06:59 -0500 (Wed, 04 Feb 2009)
New Revision: 25104

Modified:
   labs/jbossrules/trunk/drools-eclipse/org.guvnor.tools/src/org/guvnor/tools/views/RepositoryView.java
Log:
[JBIDE-3744] "remove option for BRMS repo may cause confusion over just is being deleted"

Modified: labs/jbossrules/trunk/drools-eclipse/org.guvnor.tools/src/org/guvnor/tools/views/RepositoryView.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.guvnor.tools/src/org/guvnor/tools/views/RepositoryView.java	2009-02-04 15:23:55 UTC (rev 25103)
+++ labs/jbossrules/trunk/drools-eclipse/org.guvnor.tools/src/org/guvnor/tools/views/RepositoryView.java	2009-02-04 16:06:59 UTC (rev 25104)
@@ -328,12 +328,25 @@
 		manager.add(new Separator());
 		manager.add(refreshAction);
 	}
-
+	
+	private boolean shouldAddDeleteAction() {
+		ISelection selection = viewer.getSelection();
+		if (selection == null) {
+			return false;
+		}
+		Object obj = ((IStructuredSelection)selection).getFirstElement();
+		if (obj instanceof TreeParent) {
+			return ((TreeParent)obj).getNodeType() == TreeObject.Type.REPOSITORY;
+		} else {
+			return false;
+		}
+	}
+	
 	private void fillContextMenu(IMenuManager manager) {
-		manager.add(deleteRepositoryLocAction);
-		manager.add(addRepositoryLocAction);
+		if (shouldAddDeleteAction()) {
+			manager.add(deleteRepositoryLocAction);
+		}
 		manager.add(refreshAction);
-		manager.add(new Separator());
 		drillDownAdapter.addNavigationActions(manager);
 		// Other plug-ins can contribute there actions here
 		manager.add(new Separator(IWorkbenchActionConstants.MB_ADDITIONS));