Change a couple of event bus error reportings.

src/com/dmdirc/Main.java

@@ -175,7 +175,7 @@ public class Main {
      * Initialises the client.
      */
     public void init() {
-        Thread.setDefaultUncaughtExceptionHandler(new DMDircExceptionHandler(eventBus));
+        Thread.setDefaultUncaughtExceptionHandler(new DMDircExceptionHandler());
         setupLogback();
         migrators.stream().filter(Migrator::needsMigration).forEach(Migrator::migrate);
         commands.forEach(c -> commandManager.registerCommand(c.getCommand(), c.getInfo()));

src/com/dmdirc/logger/DMDircExceptionHandler.java

@@ -22,28 +22,22 @@
 
 package com.dmdirc.logger;
 
-import com.dmdirc.DMDircMBassador;
-import com.dmdirc.events.AppErrorEvent;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import static com.dmdirc.util.LogUtils.APP_ERROR;
+import static com.dmdirc.util.LogUtils.FATAL_APP_ERROR;
 
 /**
  * Passes uncaught exceptions to the logger.
  */
-public final class DMDircExceptionHandler implements
-        Thread.UncaughtExceptionHandler {
-
-    private final DMDircMBassador eventBus;
+public final class DMDircExceptionHandler implements Thread.UncaughtExceptionHandler {
 
-    public DMDircExceptionHandler(final DMDircMBassador eventBus) {
-        this.eventBus = eventBus;
-    }
+    private static final Logger LOG = LoggerFactory.getLogger(DMDircExceptionHandler.class);
 
     @Override
     public void uncaughtException(final Thread t, final Throwable e) {
-        if (e instanceof Error) {
-            eventBus.publish(new AppErrorEvent(ErrorLevel.FATAL, e, e.toString(), ""));
-        } else {
-            eventBus.publish(new AppErrorEvent(ErrorLevel.HIGH, e, e.toString(), ""));
-        }
+        LOG.error(e instanceof Error ? FATAL_APP_ERROR : APP_ERROR, "Uncaught exception", e);
     }
 
 }

src/com/dmdirc/logger/ModeAliasReporter.java

@@ -22,9 +22,7 @@
 
 package com.dmdirc.logger;
 
-import com.dmdirc.DMDircMBassador;
 import com.dmdirc.MissingModeAliasException;
-import com.dmdirc.events.AppErrorEvent;
 import com.dmdirc.events.ServerConnectedEvent;
 import com.dmdirc.interfaces.config.AggregateConfigProvider;
 import com.dmdirc.parser.interfaces.Parser;
@@ -32,10 +30,15 @@ import com.dmdirc.parser.interfaces.Parser;
 import javax.inject.Inject;
 import javax.inject.Singleton;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import net.engio.mbassy.listener.Handler;
 import net.engio.mbassy.listener.Listener;
 import net.engio.mbassy.listener.References;
 
+import static com.dmdirc.util.LogUtils.APP_ERROR;
+
 /**
  * Watches for newly connected servers and raises errors about their mode aliases.
  */
@@ -43,14 +46,13 @@ import net.engio.mbassy.listener.References;
 @Listener(references = References.Strong)
 public class ModeAliasReporter {
 
+    private static final Logger LOG = LoggerFactory.getLogger(ModeAliasReporter.class);
+
     /** The name of the server domain. */
     private static final String DOMAIN_SERVER = "server";
 
-    private final DMDircMBassador eventBus;
-
     @Inject
-    public ModeAliasReporter(final DMDircMBassador eventBus) {
-        this.eventBus = eventBus;
+    public ModeAliasReporter() {
     }
 
     @Handler
@@ -99,13 +101,12 @@ public class ModeAliasReporter {
             }
 
 
-            eventBus.publish(new AppErrorEvent(ErrorLevel.LOW,
-                    new MissingModeAliasException(
+            LOG.info(APP_ERROR, "Missing mode aliases", new MissingModeAliasException(
                             event.getConnection().getNetwork(),
                             parser,
                             configManager.getOption("identity", "modealiasversion"),
                             missing.toString()),
-                    missing + " [" + parser.getServerSoftwareType() + ']', ""));
+                    missing + " [" + parser.getServerSoftwareType() + ']', "");
         }
     }
 }

src/com/dmdirc/plugins/PluginInfo.java

@@ -25,13 +25,10 @@ package com.dmdirc.plugins;
 import com.dmdirc.DMDircMBassador;
 import com.dmdirc.config.ConfigFileBackedConfigProvider;
 import com.dmdirc.config.InvalidIdentityFileException;
-import com.dmdirc.events.AppErrorEvent;
 import com.dmdirc.events.PluginLoadedEvent;
 import com.dmdirc.events.PluginUnloadedEvent;
-import com.dmdirc.events.UserErrorEvent;
 import com.dmdirc.interfaces.config.ConfigProvider;
 import com.dmdirc.interfaces.config.IdentityController;
-import com.dmdirc.logger.ErrorLevel;
 import com.dmdirc.util.validators.ValidationResponse;
 
 import java.io.IOException;
@@ -57,6 +54,9 @@ import org.slf4j.LoggerFactory;
 
 import dagger.ObjectGraph;
 
+import static com.dmdirc.util.LogUtils.APP_ERROR;
+import static com.dmdirc.util.LogUtils.USER_ERROR;
+
 /**
  * Stores plugin metadata and handles loading of plugin resources.
  */
@@ -253,8 +253,8 @@ public class PluginInfo implements ServiceProvider {
                 directoryStream.forEach(this::loadIdentity);
             }
         } catch (final IOException ioe) {
-            eventBus.publish(new UserErrorEvent(ErrorLevel.MEDIUM, ioe,
-                    "Error finding identities in plugin '" + metaData.getName() + '\'', ""));
+            LOG.warn(USER_ERROR, "Error finding identities in plugin '{}'", metaData.getName(),
+                    ioe);
         }
     }
 
@@ -271,14 +271,12 @@ public class PluginInfo implements ServiceProvider {
                 identityController.addConfigProvider(configProvider);
                 configProviders.add(configProvider);
             }
-        } catch (final InvalidIdentityFileException ex) {
-            eventBus.publish(new UserErrorEvent(ErrorLevel.MEDIUM, ex,
-                    "Error with identity file '" + path.getFileName() + "' in plugin '" +
-                            metaData.getName() + '\'', ""));
+        } catch (InvalidIdentityFileException ex) {
+            LOG.warn(USER_ERROR, "Error with identity file '{}' in plugin '{}'",
+                    path.getFileName(), metaData.getName(), ex);
         } catch (IOException ex) {
-            eventBus.publish(new UserErrorEvent(ErrorLevel.MEDIUM, ex,
-                    "Unable to load identity file '" + path.getFileName() + "' in plugin '" +
-                            metaData.getName() + '\'', ""));
+            LOG.warn(USER_ERROR, "Unable to load identity file '{}' in plugin '{}'",
+                    path.getFileName(), metaData.getName(), ex);
         }
     }
 
@@ -330,10 +328,10 @@ public class PluginInfo implements ServiceProvider {
      * files.
      */
     public void pluginUpdated() throws PluginException {
-            updateClassList();
-            updateMetaData();
-            updateProvides();
-            getDefaults();
+        updateClassList();
+        updateMetaData();
+        updateProvides();
+        getDefaults();
     }
 
     /**
@@ -597,23 +595,23 @@ public class PluginInfo implements ServiceProvider {
             lastError = "Class not found ('" + filename + ':' + classname + ':'
                     + classname.equals(metaData.getMainClass()) + "') - "
                     + cnfe.getMessage();
-            eventBus.publishAsync(new UserErrorEvent(ErrorLevel.LOW, cnfe, lastError, ""));
+            LOG.info(USER_ERROR, lastError, cnfe);
         } catch (NoClassDefFoundError ncdf) {
             lastError = "Unable to instantiate plugin ('" + filename + ':'
                     + classname + ':'
                     + classname.equals(metaData.getMainClass())
                     + "') - Unable to find class: " + ncdf.getMessage();
-            eventBus.publishAsync(new UserErrorEvent(ErrorLevel.LOW, ncdf, lastError, ""));
+            LOG.info(USER_ERROR, lastError, ncdf);
         } catch (VerifyError ve) {
             lastError = "Unable to instantiate plugin ('" + filename + ':'
                     + classname + ':'
                     + classname.equals(metaData.getMainClass())
                     + "') - Incompatible: " + ve.getMessage();
-            eventBus.publishAsync(new UserErrorEvent(ErrorLevel.LOW, ve, lastError, ""));
+            LOG.info(USER_ERROR, lastError, ve);
         } catch (IllegalArgumentException | ReflectiveOperationException ex) {
             lastError = "Unable to instantiate class for plugin " + metaData.getName()
                     + ": " + classname;
-            eventBus.publishAsync(new AppErrorEvent(ErrorLevel.LOW, ex, lastError, ""));
+            LOG.info(APP_ERROR, lastError, ex);
         }
     }
 
@@ -626,7 +624,7 @@ public class PluginInfo implements ServiceProvider {
                 lastError = "Prerequisites for plugin not met. ('"
                         + filename + ':' + metaData.getMainClass()
                         + "' -> '" + prerequisites.getFailureReason() + "') ";
-                eventBus.publish(new UserErrorEvent(ErrorLevel.LOW, null, lastError, ""));
+                LOG.info(USER_ERROR, lastError);
             } else {
                 initialisePlugin((Plugin) temp);
             }
@@ -640,7 +638,7 @@ public class PluginInfo implements ServiceProvider {
             plugin.onLoad();
         } catch (LinkageError | Exception e) {
             lastError = "Error in onLoad for " + metaData.getName() + ':' + e.getMessage();
-            eventBus.publishAsync(new AppErrorEvent(ErrorLevel.MEDIUM, e, lastError, ""));
+            LOG.warn(APP_ERROR, lastError, e);
             unloadPlugin();
         }
     }
@@ -698,7 +696,7 @@ public class PluginInfo implements ServiceProvider {
             } catch (Exception | LinkageError e) {
                 lastError = "Error in onUnload for " + metaData.getName()
                         + ':' + e + " - " + e.getMessage();
-                eventBus.publishAsync(new AppErrorEvent(ErrorLevel.MEDIUM, e, lastError, ""));
+                LOG.warn(APP_ERROR, lastError, e);
             }
 
             //TODO plugin unloading shouldn't be done from here, event bus shouldn't be here.

test/com/dmdirc/logger/DMDircExceptionHandlerTest.java

@@ -1,73 +0,0 @@
-/*
- * Copyright (c) 2006-2015 DMDirc Developers
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
- * SOFTWARE.
- */
-package com.dmdirc.logger;
-
-
-import com.dmdirc.DMDircMBassador;
-import com.dmdirc.events.AppErrorEvent;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.ArgumentCaptor;
-import org.mockito.Captor;
-import org.mockito.Mock;
-import org.mockito.runners.MockitoJUnitRunner;
-
-import static junit.framework.TestCase.assertEquals;
-import static org.mockito.Mockito.verify;
-
-@RunWith(MockitoJUnitRunner.class)
-public class DMDircExceptionHandlerTest {
-
-    @Mock private DMDircMBassador eventBus;
-    @Captor private ArgumentCaptor<AppErrorEvent> event;
-    private DMDircExceptionHandler eh;
-
-    @Before
-    public void setup() {
-        eh = new DMDircExceptionHandler(eventBus);
-    }
-
-    @Test
-    public void testUncaughtException() {
-        final Exception exception = new UnsupportedOperationException();
-        eh.uncaughtException(null, exception);
-        verify(eventBus).publish(event.capture());
-        assertEquals(ErrorLevel.HIGH, event.getValue().getLevel());
-        assertEquals(exception, event.getValue().getThrowable());
-        assertEquals(exception.toString(), event.getValue().getMessage());
-        assertEquals("", event.getValue().getDetails());
-    }
-
-    @Test
-    public void testUncaughtError() {
-        final Error error = new OutOfMemoryError();
-        eh.uncaughtException(null, error);
-        verify(eventBus).publish(event.capture());
-        assertEquals(ErrorLevel.FATAL, event.getValue().getLevel());
-        assertEquals(error, event.getValue().getThrowable());
-        assertEquals(error.toString(), event.getValue().getMessage());
-        assertEquals("", event.getValue().getDetails());
-    }
-
-}