Remove some event-based errors.

src/com/dmdirc/config/ConfigBinder.java

 
 package com.dmdirc.config;
 
-import com.dmdirc.DMDircMBassador;
-import com.dmdirc.events.AppErrorEvent;
 import com.dmdirc.interfaces.config.AggregateConfigProvider;
 import com.dmdirc.interfaces.config.ConfigChangeListener;
 import com.dmdirc.interfaces.config.ReadOnlyConfigProvider;
-import com.dmdirc.logger.ErrorLevel;
 
 import com.google.common.collect.ArrayListMultimap;
 import com.google.common.collect.Multimap;
 
 import javax.annotation.Nonnull;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import static com.dmdirc.util.LogUtils.APP_ERROR;
+
 /**
  * Facilitates automatically binding fields or methods annotated with a {@link ConfigBinding}
  * element to a configuration value.
  */
 public class ConfigBinder {
 
+    private static final Logger LOG = LoggerFactory.getLogger(ConfigBinder.class);
+
     /** A map of instances to created listeners. */
     private final Multimap<Object, ConfigChangeListener> listeners = ArrayListMultimap.create();
     /** The default domain to use. */
     private final Optional<String> defaultDomain;
     /** The configuration manager to use to retrieve settings. */
     private final AggregateConfigProvider manager;
-    /** The event but used to raise error events. */
-    private final DMDircMBassador eventBus;
 
-    ConfigBinder(final AggregateConfigProvider manager, final DMDircMBassador eventBus) {
+    ConfigBinder(final AggregateConfigProvider manager) {
         this.manager = manager;
-        this.eventBus = eventBus;
         this.defaultDomain = Optional.empty();
     }
 
-    ConfigBinder(final AggregateConfigProvider manager, final DMDircMBassador eventBus,
-            @Nonnull final String domain) {
+    ConfigBinder(final AggregateConfigProvider manager, @Nonnull final String domain) {
         this.manager = manager;
-        this.eventBus = eventBus;
         this.defaultDomain = Optional.of(domain);
     }
 
         try {
             binding.invocation().newInstance().invoke(element, instance, value);
         } catch (ReflectiveOperationException ex) {
-            eventBus.publish(new AppErrorEvent(ErrorLevel.HIGH, ex,
-                    "Exception when updating bound setting", ""));
+            LOG.error(APP_ERROR, "Exception when updating bound setting", ex);
         }
     }
 
      * @param newListeners The listeners to be added
      */
     private void addListeners(final Object instance,
-            final Collection<ConfigChangeListener> newListeners) {
+            final Iterable<ConfigChangeListener> newListeners) {
         synchronized (listeners) {
             listeners.putAll(instance, newListeners);
         }
      * @return A config binder with the specified default domain.
      */
     public ConfigBinder withDefaultDomain(@Nonnull final String domain) {
-        return new ConfigBinder(manager, eventBus, domain);
+        return new ConfigBinder(manager, domain);
     }
 
 }

src/com/dmdirc/config/ConfigFileBackedConfigProvider.java

 
 package com.dmdirc.config;
 
-import com.dmdirc.DMDircMBassador;
-import com.dmdirc.events.UserErrorEvent;
 import com.dmdirc.interfaces.config.ConfigChangeListener;
 import com.dmdirc.interfaces.config.ConfigProvider;
-import com.dmdirc.logger.ErrorLevel;
 import com.dmdirc.util.io.ConfigFile;
 import com.dmdirc.util.io.InvalidConfigFileException;
 import com.dmdirc.util.validators.Validator;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import static com.dmdirc.util.LogUtils.USER_ERROR;
+
 /**
  * Provides configuration settings from a {@link ConfigFile}.
  */
     /** The config change listeners for this source. */
     protected final List<WeakReference<ConfigChangeListener>> listeners =
             new CopyOnWriteArrayList<>();
-    /** The event bus to post error events to. */
-    private final DMDircMBassador eventBus;
     /** Whether this identity needs to be saved. */
     protected boolean needSave;
 
      * @throws InvalidIdentityFileException Missing required properties
      * @throws IOException                  Input/output exception
      */
-    public ConfigFileBackedConfigProvider(@Nullable final IdentityManager identityManager,
-            final DMDircMBassador eventBus, final Path file, final boolean forceDefault)
-            throws IOException, InvalidIdentityFileException {
+    public ConfigFileBackedConfigProvider(
+            @Nullable final IdentityManager identityManager, final Path file,
+            final boolean forceDefault) throws IOException, InvalidIdentityFileException {
         this.identityManager = identityManager;
         this.file = new ConfigFile(file);
         this.file.setAutomake(true);
-        this.eventBus = eventBus;
         initFile(forceDefault);
         myTarget = getTarget(forceDefault);
     }
      * @throws InvalidIdentityFileException Missing required properties
      * @throws IOException                  Input/output exception
      */
-    public ConfigFileBackedConfigProvider(final DMDircMBassador eventBus, final InputStream stream,
+    public ConfigFileBackedConfigProvider(final InputStream stream,
             final boolean forceDefault) throws IOException, InvalidIdentityFileException {
         this.identityManager = null;
         this.file = new ConfigFile(stream);
         this.file.setAutomake(true);
-        this.eventBus = eventBus;
         initFile(forceDefault);
         myTarget = getTarget(forceDefault);
     }
      * @param target          The target of this identity
      */
     public ConfigFileBackedConfigProvider(@Nullable final IdentityManager identityManager,
-            final DMDircMBassador eventBus, final ConfigFile configFile,
-            final ConfigTarget target) {
+            final ConfigFile configFile, final ConfigTarget target) {
         this.identityManager = identityManager;
         this.file = configFile;
-        this.eventBus = eventBus;
         this.file.setAutomake(true);
         this.myTarget = target;
     }
 
                 needSave = false;
             } catch (IOException ex) {
-                eventBus.publish(new UserErrorEvent(ErrorLevel.MEDIUM, ex,
-                        "Unable to save identity file: " + ex.getMessage(), ""));
+                LOG.warn(USER_ERROR, "Unable to save identity file", ex);
             }
         }
     }

src/com/dmdirc/config/ConfigManager.java

         this.server = server;
         this.channel = chanName;
 
-        binder = new ConfigBinder(this, eventBus);
+        binder = new ConfigBinder(this);
     }
 
     @Override

src/com/dmdirc/config/IdentityManager.java

 
 import com.dmdirc.DMDircMBassador;
 import com.dmdirc.Precondition;
-import com.dmdirc.events.AppErrorEvent;
 import com.dmdirc.events.UserErrorEvent;
 import com.dmdirc.interfaces.config.AggregateConfigProvider;
 import com.dmdirc.interfaces.config.ConfigProvider;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import static com.dmdirc.util.LogUtils.FATAL_APP_ERROR;
 import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Preconditions.checkNotNull;
 
         addonSettings.put("name", "Addon defaults");
         addonConfigFile.addDomain("identity", addonSettings);
 
-        addonConfig = new ConfigFileBackedConfigProvider(this, eventBus, addonConfigFile, target);
+        addonConfig = new ConfigFileBackedConfigProvider(this, addonConfigFile, target);
         addConfigProvider(addonConfig);
     }
 
             loadIdentity(FileUtils.getPathForResource(getClass().getResource(
                     "defaults/default/formatter")));
         } catch (URISyntaxException ex) {
-            eventBus.publishAsync(new AppErrorEvent(ErrorLevel.FATAL, ex,
-                    "Unable to load settings", ""));
+            LOG.error(FATAL_APP_ERROR, "Unable to load settings", ex);
         }
 
         final Path file = identitiesDirectory.resolve("modealiases");
         }
 
         try {
-            final ConfigProvider provider = new ConfigFileBackedConfigProvider(this, eventBus,
-                    file, false);
+            final ConfigProvider provider = new ConfigFileBackedConfigProvider(this, file, false);
             addConfigProvider(provider);
             configProvidersByPath.put(file, provider);
         } catch (InvalidIdentityFileException ex) {
     @Override
     public void loadVersionIdentity() {
         try {
-            versionConfig = new ConfigFileBackedConfigProvider(eventBus, IdentityManager.class.
+            versionConfig = new ConfigFileBackedConfigProvider(IdentityManager.class.
                     getResourceAsStream("/com/dmdirc/version.config"), false);
             addConfigProvider(versionConfig);
         } catch (IOException | InvalidIdentityFileException ex) {
                 Files.createFile(file);
             }
 
-            config = new ConfigFileBackedConfigProvider(this, eventBus, file, true);
+            config = new ConfigFileBackedConfigProvider(this, file, true);
             config.setOption("identity", "name", "Global config");
             configProvidersByPath.put(file, config);
             addConfigProvider(config);
         configFile.write();
 
         final ConfigFileBackedConfigProvider identity = new ConfigFileBackedConfigProvider(this,
-                eventBus, file, false);
+                file, false);
         addConfigProvider(identity);
 
         return identity;

src/com/dmdirc/plugins/PluginInfo.java

         try (final InputStream stream = Files.newInputStream(path)) {
             synchronized (configProviders) {
                 final ConfigProvider configProvider =
-                        new ConfigFileBackedConfigProvider(eventBus, stream, false);
+                        new ConfigFileBackedConfigProvider(stream, false);
                 identityController.addConfigProvider(configProvider);
                 configProviders.add(configProvider);
             }

src/com/dmdirc/ui/themes/Theme.java

 
         if (stream != null) {
             try {
-                identity = new ThemeIdentity(eventBus, stream, this);
+                identity = new ThemeIdentity(stream, this);
                 identityController.addConfigProvider(identity);
             } catch (InvalidIdentityFileException | IOException ex) {
                 eventBus.publish(new UserErrorEvent(ErrorLevel.MEDIUM,

src/com/dmdirc/ui/themes/ThemeIdentity.java

 
 package com.dmdirc.ui.themes;
 
-import com.dmdirc.DMDircMBassador;
 import com.dmdirc.config.ConfigFileBackedConfigProvider;
 import com.dmdirc.config.InvalidIdentityFileException;
 import com.dmdirc.util.validators.Validator;
      * @throws InvalidIdentityFileException Missing required properties
      * @throws IOException                  Input/output exception
      */
-    public ThemeIdentity(final DMDircMBassador eventBus, final InputStream stream,
-            final Theme theme) throws IOException, InvalidIdentityFileException {
-        super(eventBus, stream, true);
+    public ThemeIdentity(final InputStream stream, final Theme theme)
+            throws IOException, InvalidIdentityFileException {
+        super(stream, true);
 
         myTarget.setTheme();
         this.theme = theme;

test/com/dmdirc/config/ConfigBinderTest.java

 
 package com.dmdirc.config;
 
-import com.dmdirc.DMDircMBassador;
 import com.dmdirc.interfaces.config.AggregateConfigProvider;
 
 import org.junit.Before;
 public class ConfigBinderTest {
 
     @Mock private AggregateConfigProvider configProvider;
-    @Mock private DMDircMBassador eventBus;
     private ConfigBinder binder;
 
     @Before
     public void setup() {
-        binder = new ConfigBinder(configProvider, eventBus);
+        binder = new ConfigBinder(configProvider);
     }
 
     @Test

test/com/dmdirc/config/ConfigFileBackedConfigProviderTest.java

 
 package com.dmdirc.config;
 
-import com.dmdirc.DMDircMBassador;
 import com.dmdirc.interfaces.config.ConfigChangeListener;
 import com.dmdirc.interfaces.config.ConfigProvider;
 import com.dmdirc.util.io.InvalidConfigFileException;
 
     @Mock private IdentityManager identityManager;
     @Mock private ConfigChangeListener changeListener;
-    @Mock private DMDircMBassador eventBus;
 
     private FileSystem fs;
 
 
     @Test(expected = InvalidIdentityFileException.class)
     public void testNoName() throws IOException, InvalidIdentityFileException {
-        new ConfigFileBackedConfigProvider(identityManager, eventBus, fs.getPath("no-name"), false);
+        new ConfigFileBackedConfigProvider(identityManager, fs.getPath("no-name"), false);
     }
 
     @Test(expected = InvalidIdentityFileException.class)
     public void testNoTarget() throws IOException, InvalidIdentityFileException {
-        new ConfigFileBackedConfigProvider(identityManager,  eventBus,fs.getPath("no-target"), false);
+        new ConfigFileBackedConfigProvider(identityManager, fs.getPath("no-target"), false);
     }
 
     @Test(expected = InvalidIdentityFileException.class)
     public void testInvalidConfigFile() throws IOException, InvalidIdentityFileException {
-        new ConfigFileBackedConfigProvider(identityManager, eventBus,
-                fs.getPath("invalid-config-file"), false);
+        new ConfigFileBackedConfigProvider(identityManager, fs.getPath("invalid-config-file"), false);
     }
 
     @Test
 
     private ConfigFileBackedConfigProvider getProvider(final String file)
             throws IOException, InvalidIdentityFileException {
-        return new ConfigFileBackedConfigProvider(identityManager, eventBus,
-                fs.getPath(file), false);
+        return new ConfigFileBackedConfigProvider(identityManager, fs.getPath(file), false);
     }
 
 }