Change ErrorLevel.getIcon to use strings not swing Icons

Fixes CORE-9

Change-Id: Id43521640e66ee80962d3b2432f652c47b6223c1
Depends-On: Iee0aa1d166b301f56a92f0bc1d3b38c0ed1000dc
Reviewed-on: http://gerrit.dmdirc.com/1521
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: Greg Holmes <greg@dmdirc.com>

src/com/dmdirc/logger/ErrorLevel.java

@@ -22,27 +22,23 @@
 
 package com.dmdirc.logger;
 
-import com.dmdirc.ui.IconManager;
-
-import javax.swing.Icon;
-
 /** Specific error levels allowed by Logger. */
 public enum ErrorLevel {
     /** Fatal error. */
-    FATAL("Fatal", IconManager.getIconManager().getIcon("error")),
+    FATAL("Fatal", "error"),
     /** High priority error. */
-    HIGH("High", IconManager.getIconManager().getIcon("error")),
+    HIGH("High", "error"),
     /** Medium priority error. */
-    MEDIUM("Medium", IconManager.getIconManager().getIcon("warning")),
+    MEDIUM("Medium", "warning"),
     /** Low priority error. */
-    LOW("Low", IconManager.getIconManager().getIcon("info")),
+    LOW("Low", "info"),
     /** Unknown priority error. */
-    UNKNOWN("Unknown", IconManager.getIconManager().getIcon("info"));
+    UNKNOWN("Unknown", "info");
     
     /** Error level string. */
     private String value;
     /** Error level icon. */
-    private Icon icon;
+    private String icon;
     
     /** 
      * Instantiates the enum. 
@@ -50,7 +46,7 @@ public enum ErrorLevel {
      * @param value toString value
      * @param icon Error level icon
      */
-    ErrorLevel(final String value, final Icon icon) {
+    ErrorLevel(final String value, final String icon) {
         this.value = value;
         this.icon = icon;
     }
@@ -66,7 +62,7 @@ public enum ErrorLevel {
      * 
      * @return Error levels icon
      */
-    public Icon getIcon() {
+    public String getIcon() {
         return icon;
     }
     
@@ -75,7 +71,7 @@ public enum ErrorLevel {
      *
      * @param level Error level to compare
      *
-     * @return true iif the error is more important
+     * @return true iff the error is more important
      */
     public boolean moreImportant(final ErrorLevel level) {
         if (level == null) {

test/com/dmdirc/logger/ErrorLevelTest.java

@@ -21,7 +21,6 @@
  */
 package com.dmdirc.logger;
 
-import javax.swing.Icon;
 import org.junit.Test;
 import static org.junit.Assert.*;
 
@@ -36,15 +35,6 @@ public class ErrorLevelTest {
         assertTrue("Unknown".equalsIgnoreCase(ErrorLevel.UNKNOWN.toString()));
     }
 
-    @Test
-    public void testGetIcon() {
-        assertTrue(ErrorLevel.LOW.getIcon() instanceof Icon);
-        assertTrue(ErrorLevel.MEDIUM.getIcon() instanceof Icon);
-        assertTrue(ErrorLevel.HIGH.getIcon() instanceof Icon);
-        assertTrue(ErrorLevel.FATAL.getIcon() instanceof Icon);
-        assertTrue(ErrorLevel.UNKNOWN.getIcon() instanceof Icon);
-    }
-
     @Test
     public void testMoreImportantLow() {
         assertTrue(ErrorLevel.LOW.moreImportant(ErrorLevel.MEDIUM));