Merge pull request #184 from Parabot/bugfix/issue-183

[BUGFIX] Keep Tools menu

src/main/java/org/parabot/core/Context.java

@@ -164,7 +164,6 @@ public class Context {
      * Loads the game
      */
     public void load() {
-        BotUI.getInstance().getJMenuBar().remove(2);
         Core.verbose(TranslationHelper.translate("PARSING_SERVER_JAR"));
         serverProvider.init();
         serverProvider.parseJar();

src/main/java/org/parabot/core/ui/BotUI.java

@@ -34,9 +34,12 @@ public class BotUI extends JFrame implements ActionListener, ComponentListener,
     private static BotUI instance;
     private static JDialog dialog;
 
-    private JMenuItem run, pause, stop;
+    private JMenuBar menuBar;
+    private JMenu features, scripts, file;
+    private JMenuItem run, pause, stop, cacheClear, notifications;
     private boolean runScript, pauseScript;
 
+
     public BotUI(String username, String password) {
         if (instance != null) {
             throw new IllegalStateException("BotUI already created");
@@ -74,11 +77,11 @@ public class BotUI extends JFrame implements ActionListener, ComponentListener,
 
 
     private void createMenu() {
-        JMenuBar menuBar = new JMenuBar();
+        menuBar = new JMenuBar();
 
-        JMenu file = new JMenu("File");
+        file = new JMenu("File");
-        JMenu scripts = new JMenu("Script");
+        scripts = new JMenu("Script");
-        JMenu features = new JMenu("Features");
+        features = new JMenu("Features");
 
         JMenuItem screenshot = new JMenuItem("Create screenshot");
         JMenuItem proxy = new JMenuItem("Network");
@@ -104,10 +107,10 @@ public class BotUI extends JFrame implements ActionListener, ComponentListener,
         stop.setEnabled(false);
         stop.setIcon(new ImageIcon(Images.getResource("/storage/images/stop.png")));
 
-        JMenuItem cacheClear = new JMenuItem("Clear cache");
+        cacheClear = new JMenuItem("Clear cache");
         cacheClear.setIcon(new ImageIcon(Images.getResource("/storage/images/trash.png")));
 
-        JMenuItem notifications = new JMenuItem("Notifications");
+        notifications = new JMenuItem("Notifications");
         notifications.setIcon(new ImageIcon(Images.getResource("/storage/images/bell.png")));
 
         screenshot.addActionListener(this);
@@ -341,4 +344,23 @@ public class BotUI extends JFrame implements ActionListener, ComponentListener,
     public void windowOpened(WindowEvent arg0) {
     }
 
+    public JMenu getFeatures() {
+        return features;
+    }
+
+    public JMenu getScripts() {
+        return scripts;
+    }
+
+    public JMenu getFile() {
+        return file;
+    }
+
+    public JMenuItem getCacheClear() {
+        return cacheClear;
+    }
+
+    public JMenuItem getNotifications() {
+        return notifications;
+    }
 }

src/main/java/org/parabot/environment/servers/executers/ServerExecuter.java

@@ -1,39 +1,36 @@
 package org.parabot.environment.servers.executers;
 
 import org.parabot.core.Context;
-import org.parabot.core.Core;
+import org.parabot.core.ui.BotUI;
-import org.parabot.core.parsers.randoms.RandomParser;
 import org.parabot.core.ui.components.PaintComponent;
 import org.parabot.environment.servers.ServerProvider;
 
-import java.io.PrintStream;
-
 /**
- * 
  * Executes a server provider
  *
  * @author Everel
- *
  */
 public abstract class ServerExecuter {
 
-	public abstract void run();
+    public abstract void run();
 
-	public void finalize(final ServerProvider provider, final String serverName) {
+    public void finalize(final ServerProvider provider, final String serverName) {
-		new Thread(new Runnable() {
+        new Thread(new Runnable() {
-			@Override
+            @Override
-			public void run() {
+            public void run() {
-				try {
+                try {
-					Context context = Context.getInstance(provider);
+                    BotUI.getInstance().getCacheClear().setEnabled(false);
-					context.load();
-					PaintComponent.getInstance().startPainting(context);
 
-					Context.getInstance().getRandomHandler().init();
+                    Context context = Context.getInstance(provider);
-				} catch (Throwable t) {
+                    context.load();
-					t.printStackTrace();
+                    PaintComponent.getInstance().startPainting(context);
-				}
+
-			}
+                    Context.getInstance().getRandomHandler().init();
-		}).start();
+                } catch (Throwable t) {
-	}
+                    t.printStackTrace();
+                }
+            }
+        }).start();
+    }
 
 }