Slightly better "invalid plugin.yml" errors

src/main/java/org/bukkit/plugin/InvalidDescriptionException.java

@@ -1,36 +1,62 @@
-
+
-package org.bukkit.plugin;
+package org.bukkit.plugin;
-
+
-/**
+/**
- * Thrown when attempting to load an invalid PluginDescriptionFile
+ * Thrown when attempting to load an invalid PluginDescriptionFile
- */
+ */
-public class InvalidDescriptionException extends Exception {
+public class InvalidDescriptionException extends Exception {
-    private static final long serialVersionUID = 5721389122281775894L;
+    private static final long serialVersionUID = 5721389122281775894L;
-    private final Throwable cause;
+    private final Throwable cause;
-
+    private final String message;
-    /**
+
-     * Constructs a new InvalidDescriptionException based on the given Exception
+    /**
-     *
+     * Constructs a new InvalidDescriptionException based on the given Exception
-     * @param throwable Exception that triggered this Exception
+     *
-     */
+     * @param throwable Exception that triggered this Exception
-    public InvalidDescriptionException(Throwable throwable) {
+     */
-        cause = throwable;
+    public InvalidDescriptionException(Throwable throwable) {
-    }
+        this(throwable, "Invalid plugin.yml");
-
+    }
-    /**
+
-     * Constructs a new InvalidDescriptionException
+    /**
-     */
+     * Constructs a new InvalidDescriptionException with the given message
-    public InvalidDescriptionException() {
+     *
-        cause = null;
+     * @param message Brief message explaining the cause of the exception
-    }
+     */
-
+    public InvalidDescriptionException(final String message) {
-    /**
+        this(null, message);
-     * If applicable, returns the Exception that triggered this Exception
+    }
-     *
+
-     * @return Inner exception, or null if one does not exist
+    /**
-     */
+     * Constructs a new InvalidDescriptionException based on the given Exception
-    @Override
+     *
-    public Throwable getCause() {
+     * @param message Brief message explaining the cause of the exception
-        return cause;
+     * @param throwable Exception that triggered this Exception
-    }
+     */
-}
+    public InvalidDescriptionException(final Throwable throwable, final String message) {
+        this.cause = null;
+        this.message = message;
+    }
+
+    /**
+     * Constructs a new InvalidDescriptionException
+     */
+    public InvalidDescriptionException() {
+        this(null, "Invalid plugin.yml");
+    }
+
+    /**
+     * If applicable, returns the Exception that triggered this Exception
+     *
+     * @return Inner exception, or null if one does not exist
+     */
+    @Override
+    public Throwable getCause() {
+        return cause;
+    }
+
+    @Override
+    public String getMessage() {
+        return super.getMessage();
+    }
+}

src/main/java/org/bukkit/plugin/PluginDescriptionFile.java

@@ -20,11 +20,7 @@ public final class PluginDescriptionFile {
 
     @SuppressWarnings("unchecked")
     public PluginDescriptionFile(final InputStream stream) throws InvalidDescriptionException {
-        try {
+        loadMap((Map<String, Object>)yaml.load(stream));
-            loadMap((Map<String, Object>)yaml.load(stream));
-        } catch (ClassCastException ex) {
-            throw new InvalidDescriptionException(ex);
-        }
     }
 
     /**
@@ -32,7 +28,7 @@ public final class PluginDescriptionFile {
      * @param reader
      */
     @SuppressWarnings("unchecked")
-    public PluginDescriptionFile(final Reader reader) {
+    public PluginDescriptionFile(final Reader reader) throws InvalidDescriptionException {
         loadMap((Map<String, Object>)yaml.load(reader));
     }
 
@@ -84,10 +80,34 @@ public final class PluginDescriptionFile {
         return main;
     }
 
-    private void loadMap(Map<String, Object> map) throws ClassCastException {
+    private void loadMap(Map<String, Object> map) throws InvalidDescriptionException {
-        name = map.get("name").toString();
+        if (name == null) {
-        main = map.get("main").toString();
+            throw new InvalidDescriptionException("Name is not defined");
-        version = map.get("version").toString();
+        }
+
+        try {
+            name = map.get("name").toString();
+        } catch (NullPointerException ex) {
+            throw new InvalidDescriptionException(ex, "name is not defined");
+        } catch (ClassCastException ex) {
+            throw new InvalidDescriptionException(ex, "name is of wrong type");
+        }
+
+        try {
+            version = map.get("version").toString();
+        } catch (NullPointerException ex) {
+            throw new InvalidDescriptionException(ex, "version is not defined");
+        } catch (ClassCastException ex) {
+            throw new InvalidDescriptionException(ex, "version is of wrong type");
+        }
+
+        try {
+            main = map.get("main").toString();
+        } catch (NullPointerException ex) {
+            throw new InvalidDescriptionException(ex, "main is not defined");
+        } catch (ClassCastException ex) {
+            throw new InvalidDescriptionException(ex, "main is of wrong type");
+        }
     }
 
     private Map<String, Object> saveMap() {