Show errors when loading frame managers

Change-Id: I7965c1569a86fa288b532169ff2ef44dc04881b3
Reviewed-on: http://gerrit.dmdirc.com/1198
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Chris Smith <chris@dmdirc.com>

src/com/dmdirc/addons/ui_swing/MainFrame.java

@@ -38,6 +38,8 @@ import com.dmdirc.interfaces.ConfigChangeListener;
 import com.dmdirc.ui.IconManager;
 import com.dmdirc.addons.ui_swing.framemanager.FrameManager;
 import com.dmdirc.addons.ui_swing.framemanager.FramemanagerPosition;
+import com.dmdirc.logger.ErrorLevel;
+import com.dmdirc.logger.Logger;
 import com.dmdirc.ui.interfaces.MainWindow;
 import com.dmdirc.ui.interfaces.Window;
 import com.dmdirc.ui.CoreUIUtils;
@@ -48,6 +50,7 @@ import java.awt.Dimension;
 import java.awt.event.WindowEvent;
 import java.awt.event.WindowFocusListener;
 import java.awt.event.WindowListener;
+import java.lang.reflect.InvocationTargetException;
 
 import javax.swing.ImageIcon;
 import javax.swing.JDesktopPane;
@@ -100,7 +103,7 @@ public final class MainFrame extends JFrame implements WindowListener,
 
     /**
      * Creates new form MainFrame.
-     * 
+     *
      * @param controller Swing controller
      */
     protected MainFrame(final SwingController controller) {
@@ -155,7 +158,7 @@ public final class MainFrame extends JFrame implements WindowListener,
 
     /**
      * Returns the status bar for this frame.
-     * 
+     *
      * @return Status bar
      */
     public SwingStatusBar getStatusBar() {
@@ -259,16 +262,16 @@ public final class MainFrame extends JFrame implements WindowListener,
 
     /**
      * Returns the desktop pane for the frame.
-     * 
+     *
      * @return JDesktopPane for the frame
      */
     public JDesktopPane getDesktopPane() {
         return desktopPane;
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -276,9 +279,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         //ignore
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -286,9 +289,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         quit(exitCode);
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -303,9 +306,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         }, "Quit thread").start();
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -313,9 +316,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         ActionManager.processEvent(CoreActionType.CLIENT_MINIMISED, null);
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -323,9 +326,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         ActionManager.processEvent(CoreActionType.CLIENT_UNMINIMISED, null);
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -333,9 +336,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         //ignore
     }
 
-    /** 
+    /**
      * {@inheritDoc}.
-     * 
+     *
      * @param windowEvent Window event
      */
     @Override
@@ -351,16 +354,38 @@ public final class MainFrame extends JFrame implements WindowListener,
             @Override
             public void run() {
                 final String manager = IdentityManager.getGlobalConfig().
-                        getOption("ui",
-                        "framemanager");
-
+                        getOption("ui", "framemanager");
                 try {
                     mainFrameManager = (FrameManager) Class.forName(manager).
                             getConstructor().newInstance();
-                } catch (Exception ex) {
-                    // Throws craploads of exceptions and we want to handle them all
-                    // the same way, so we might as well catch Exception
-                    mainFrameManager = new TreeFrameManager();
+                } catch (InvocationTargetException ex) {
+                    Logger.appError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (InstantiationException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (NoSuchMethodException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (SecurityException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (IllegalAccessException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (IllegalArgumentException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (ClassNotFoundException ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } catch (LinkageError ex) {
+                    Logger.userError(ErrorLevel.MEDIUM, "Unable to load frame "
+                            + "manager, falling back to default.", ex);
+                } finally {
+                    if (mainFrameManager == null) {
+                        mainFrameManager = new TreeFrameManager();
+                    }
                 }
                 mainFrameManager.setController(controller);
                 mainFrameManager.setParent(frameManagerPanel);
@@ -376,7 +401,8 @@ public final class MainFrame extends JFrame implements WindowListener,
     private void initComponents() {
         statusBar = new SwingStatusBar(controller, this);
         frameManagerPanel = new JPanel();
-        desktopPane = new DMDircDesktopPane(controller, this, controller.getDomain());
+        desktopPane = new DMDircDesktopPane(controller, this, controller.
+                getDomain());
 
         initFrameManagers();
 
@@ -399,7 +425,7 @@ public final class MainFrame extends JFrame implements WindowListener,
 
     /**
      * Initialises the split pane.
-     * 
+     *
      * @return Returns the initialised split pane
      */
     private JSplitPane initSplitPane() {
@@ -476,9 +502,9 @@ public final class MainFrame extends JFrame implements WindowListener,
         quit(0);
     }
 
-    /** 
-     * Exit code call to quit. 
-     * 
+    /**
+     * Exit code call to quit.
+     *
      * @param exitCode Exit code
      */
     public void quit(final int exitCode) {
@@ -577,7 +603,7 @@ public final class MainFrame extends JFrame implements WindowListener,
 
     /**
      * Adds a window to this frame manager.
-     * 
+     *
      * @param window The server to be added
      * @param index Index of the window to be added
      */
@@ -609,5 +635,4 @@ public final class MainFrame extends JFrame implements WindowListener,
             }
         });
     }
-
-}
+}