Revision: 6483
          http://sourceforge.net/p/jump-pilot/code/6483
Author:   edso
Date:     2020-09-15 13:04:29 +0000 (Tue, 15 Sep 2020)
Log Message:
-----------
clean up plugin installation

Modified Paths:
--------------
    core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java
    core/trunk/src/org/openjump/core/ui/plugin/view/EasyButtonsPlugin.java

Modified: 
core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java
===================================================================
--- core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java 
2020-09-15 13:03:40 UTC (rev 6482)
+++ core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java 
2020-09-15 13:04:29 UTC (rev 6483)
@@ -66,30 +66,16 @@
 
 public class PythonToolsPlugIn extends ToolboxPlugIn {
 
-  private static String sName = "Python Console and Tools";
   private static final ImageIcon icon = IconLoader
       .icon("famfam/application_python.png");
 
   public String getName() {
-    return sName;
+    return I18N
+        
.get("org.openjump.core.ui.plugin.customize.PythonToolsPlugIn.Python-Console-and-Tools");
   }
 
   public void initialize(PlugInContext context) throws Exception {
-    this.sName = I18N
-        
.get("org.openjump.core.ui.plugin.customize.PythonToolsPlugIn.Python-Console-and-Tools");
-    // [sstein - old, access via button]
-    // 
context.getWorkbenchContext().getWorkbench().getFrame().getToolBar().addPlugIn(getIcon(),
-    // this, createEnableCheck(context.getWorkbenchContext()),
-    // context.getWorkbenchContext());
-    // [sstein - neu, access via menu]
-    FeatureInstaller featureInstaller = new FeatureInstaller(
-        context.getWorkbenchContext());
-    featureInstaller.addMainMenuItem(this, // exe
-        new String[] { MenuNames.CUSTOMIZE }, // menu path
-        this.sName + "...", // name methode .getName recieved by AbstractPlugIn
-        false, // checkbox
-        null, // icon
-        createEnableCheck(context.getWorkbenchContext())); // enable check
+    FeatureInstaller.getInstance().addMainMenuPlugin(this, new String[] { 
MenuNames.CUSTOMIZE });
   }
 
   protected void initializeToolbox(ToolboxDialog toolbox) {
@@ -155,7 +141,7 @@
   public MultiEnableCheck createEnableCheck(
       final WorkbenchContext workbenchContext) {
     EnableCheckFactory checkFactory = new EnableCheckFactory(workbenchContext);
-    return new MultiEnableCheck().add(checkFactory
+    return new MultiEnableCheck().add(super.getEnableCheck()).add(checkFactory
         .createTaskWindowMustBeActiveCheck());
   }
 }

Modified: core/trunk/src/org/openjump/core/ui/plugin/view/EasyButtonsPlugin.java
===================================================================
--- core/trunk/src/org/openjump/core/ui/plugin/view/EasyButtonsPlugin.java      
2020-09-15 13:03:40 UTC (rev 6482)
+++ core/trunk/src/org/openjump/core/ui/plugin/view/EasyButtonsPlugin.java      
2020-09-15 13:04:29 UTC (rev 6483)
@@ -44,6 +44,7 @@
 import com.vividsolutions.jump.workbench.ui.MenuNames;
 import com.vividsolutions.jump.workbench.ui.cursortool.CursorTool;
 import com.vividsolutions.jump.workbench.ui.images.IconLoader;
+import com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller;
 import com.vividsolutions.jump.workbench.ui.toolbox.ToolboxDialog;
 import com.vividsolutions.jump.workbench.ui.toolbox.ToolboxPlugIn;
 
@@ -57,8 +58,8 @@
   private EasyPanel buttonPanel = null;
 
   public void initialize(final PlugInContext context) throws Exception {
-    createMainMenuItem(new String[] { MenuNames.CUSTOMIZE }, getIcon(),
-        context.getWorkbenchContext());
+    FeatureInstaller.getInstance().addMainMenuPlugin(this, new String[] { 
MenuNames.CUSTOMIZE });
+
     // Wait 2 seconds because EasyButtonsPlugIn needs all menu items
     // to be initialized first
     // Initialization is done here rather than in initializeToolbox because



_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to