Author: mes
Date: 2011-09-16 16:10:23 -0700 (Fri, 16 Sep 2011)
New Revision: 26847

Modified:
   core3/impl/branches/no-spring/table-browser-impl/osgi.bnd
   
core3/impl/branches/no-spring/table-browser-impl/src/main/java/org/cytoscape/browser/internal/PopupMenuHelper.java
   
core3/impl/branches/no-spring/vizmap-gui-impl/src/main/java/org/cytoscape/view/vizmap/gui/internal/VizMapperMenuManager.java
Log:
tweaks to fix various startup exceptions

Modified: core3/impl/branches/no-spring/table-browser-impl/osgi.bnd
===================================================================
--- core3/impl/branches/no-spring/table-browser-impl/osgi.bnd   2011-09-16 
23:08:57 UTC (rev 26846)
+++ core3/impl/branches/no-spring/table-browser-impl/osgi.bnd   2011-09-16 
23:10:23 UTC (rev 26847)
@@ -4,5 +4,5 @@
 
 
 Bundle-Activator: ${bundle.namespace}.CyActivator
-Private-Package: ${bundle.namespace},${bundle.namespace}.*
+Private-Package: ${bundle.namespace}.*
 

Modified: 
core3/impl/branches/no-spring/table-browser-impl/src/main/java/org/cytoscape/browser/internal/PopupMenuHelper.java
===================================================================
--- 
core3/impl/branches/no-spring/table-browser-impl/src/main/java/org/cytoscape/browser/internal/PopupMenuHelper.java
  2011-09-16 23:08:57 UTC (rev 26846)
+++ 
core3/impl/branches/no-spring/table-browser-impl/src/main/java/org/cytoscape/browser/internal/PopupMenuHelper.java
  2011-09-16 23:10:23 UTC (rev 26847)
@@ -58,19 +58,19 @@
  * A class that encapsulates the creation of JPopupMenus based
  * on TaskFactory services.
  */
-class PopupMenuHelper {
+public class PopupMenuHelper {
        private final TaskManager taskManager;
        private final Map<TableCellTaskFactory, Map> tableCellFactoryMap;
        private final Map<TableColumnTaskFactory, Map> tableColumnFactoryMap;
 
-       PopupMenuHelper(final TaskManager taskManager) {
+       public PopupMenuHelper(final TaskManager taskManager) {
                this.taskManager = taskManager;
 
                tableCellFactoryMap   = new HashMap<TableCellTaskFactory, 
Map>();
                tableColumnFactoryMap = new HashMap<TableColumnTaskFactory, 
Map>();
        }
 
-       void createColumnHeaderMenu(final CyColumn column, final Component 
invoker, final int x,
+       public void createColumnHeaderMenu(final CyColumn column, final 
Component invoker, final int x,
                                    final int y)
        {
                if (tableColumnFactoryMap.isEmpty())
@@ -87,7 +87,7 @@
                menu.show(invoker, x, y);
        }
 
-       void createTableCellMenu(final CyColumn column, final Object 
primaryKeyValue,
+       public void createTableCellMenu(final CyColumn column, final Object 
primaryKeyValue,
                                 final Component invoker, final int x, final 
int y)
        {
                if (tableCellFactoryMap.isEmpty())

Modified: 
core3/impl/branches/no-spring/vizmap-gui-impl/src/main/java/org/cytoscape/view/vizmap/gui/internal/VizMapperMenuManager.java
===================================================================
--- 
core3/impl/branches/no-spring/vizmap-gui-impl/src/main/java/org/cytoscape/view/vizmap/gui/internal/VizMapperMenuManager.java
        2011-09-16 23:08:57 UTC (rev 26846)
+++ 
core3/impl/branches/no-spring/vizmap-gui-impl/src/main/java/org/cytoscape/view/vizmap/gui/internal/VizMapperMenuManager.java
        2011-09-16 23:10:23 UTC (rev 26847)
@@ -146,11 +146,15 @@
        public void addTaskFactory(final TaskFactory taskFactory,
                        @SuppressWarnings("rawtypes") Map properties) {
 
-               final Object serviceType = properties.get(METADATA_MENU_KEY);
-               if (serviceType == null)
-                       throw new NullPointerException(
-                                       "Service Type metadata is null.  This 
value is required.");
+               // first filter the service...
+               final Object serviceType = properties.get("service.type");
+               if ( serviceType == null || !(serviceType instanceof String) || 
!((String)serviceType).equals("vizmapUI.taskFactory") )
+                       return;
 
+               final Object menuDef = properties.get(METADATA_MENU_KEY);
+               if (menuDef == null)
+                       throw new NullPointerException("Menu metadata is 
missing.");
+
                // This is a menu item for Main Command Button.
                final Object title = properties.get(METADATA_TITLE_KEY);
                if (title == null)
@@ -166,9 +170,9 @@
                        }
                });
 
-               if(serviceType.toString().equals(MAIN_MENU))
+               if(menuDef.toString().equals(MAIN_MENU))
                        mainMenu.add(menuItem);
-               else if(serviceType.toString().equals(CONTEXT_MENU))
+               else if(menuDef.toString().equals(CONTEXT_MENU))
                        edit.add(menuItem);
 
        }

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to