Add soft exceptions option... (#120)

Author: bensku

src/main/java/ch/njol/skript/Skript.java

@@ -1195,9 +1195,9 @@ public final static EmptyStacktraceException exception(@Nullable Throwable cause
 		logEx(info);
 		logEx();
 		logEx("If you're developing an add-on for Skript this likely means that you have done something wrong.");
-		logEx("If you're a server admin however please go to http://dev.bukkit.org/server-mods/skript/tickets/");
+		logEx("If you're a server admin however please go to https://github.com/bensku/Skript/issues/");
 		logEx("and check whether this error has already been reported.");
-		logEx("If not please create a new ticket with a meaningful title, copy & paste this whole error into it,");
+		logEx("If not please create a new ticket with a meaningful title, copy & paste this whole error into it (or use paste service),");
 		logEx("and describe what you did before it happened and/or what you think caused the error.");
 		logEx("If you think that it's a trigger that's causing the error please post the trigger as well.");
 		logEx("By following this guide fixing the error should be easy and done fast.");

src/main/java/ch/njol/skript/SkriptConfig.java

@@ -167,6 +167,8 @@ public EventPriority convert(final String s) {
 	public final static Option<Boolean> addonSafetyChecks = new Option<Boolean>("addon safety checks", false)
 			.optional(true);
 
+	public final static Option<Boolean> apiSoftExceptions = new Option<Boolean>("soft api exceptions", false);
+	
 	/**
 	 * This should only be used in special cases
 	 */

src/main/java/ch/njol/skript/registrations/Classes.java

@@ -85,16 +85,23 @@ private Classes() {}
 	 * @param info info about the class to register
 	 */
 	public static <T> void registerClass(final ClassInfo<T> info) {
-		Skript.checkAcceptRegistrations();
-		if (classInfosByCodeName.containsKey(info.getCodeName()))
-			throw new IllegalArgumentException("Can't register " + info.getC().getName() + " with the code name " + info.getCodeName() + " because that name is already used by " + classInfosByCodeName.get(info.getCodeName()));
-		if (exactClassInfos.containsKey(info.getC()))
-			throw new IllegalArgumentException("Can't register the class info " + info.getCodeName() + " because the class " + info.getC().getName() + " is already registered");
-		if (info.getCodeName().length() > DatabaseStorage.MAX_CLASS_CODENAME_LENGTH)
-			throw new IllegalArgumentException("The codename '" + info.getCodeName() + "' is too long to be saved in a database, the maximum length allowed is " + DatabaseStorage.MAX_CLASS_CODENAME_LENGTH);
-		exactClassInfos.put(info.getC(), info);
-		classInfosByCodeName.put(info.getCodeName(), info);
-		tempClassInfos.add(info);
+		try {
+			Skript.checkAcceptRegistrations();
+			if (classInfosByCodeName.containsKey(info.getCodeName()))
+				throw new IllegalArgumentException("Can't register " + info.getC().getName() + " with the code name " + info.getCodeName() + " because that name is already used by " + classInfosByCodeName.get(info.getCodeName()));
+			if (exactClassInfos.containsKey(info.getC()))
+				throw new IllegalArgumentException("Can't register the class info " + info.getCodeName() + " because the class " + info.getC().getName() + " is already registered");
+			if (info.getCodeName().length() > DatabaseStorage.MAX_CLASS_CODENAME_LENGTH)
+				throw new IllegalArgumentException("The codename '" + info.getCodeName() + "' is too long to be saved in a database, the maximum length allowed is " + DatabaseStorage.MAX_CLASS_CODENAME_LENGTH);
+			exactClassInfos.put(info.getC(), info);
+			classInfosByCodeName.put(info.getCodeName(), info);
+			tempClassInfos.add(info);
+		} catch (RuntimeException e) {
+			if (SkriptConfig.apiSoftExceptions.value())
+				Skript.warning("Ignored an exception due to user configuration: " + e.getMessage());
+			else
+				throw e;
+		}
 	}
 
 	public final static void onRegistrationsStop() {

src/main/resources/config.sk

@@ -151,8 +151,14 @@ disable variable conflict warnings: false
 # whenever you create new scripts with new variables.
 
 disable variable will not be saved warnings: false
-#Disables the "... i.e contents cannot be saved ..." warning when reloading and something in your scripts sets a variable(non local) to a value that is not serializable.
-#By Mirre.
+# Disables the "... i.e contents cannot be saved ..." warning when reloading and something in your scripts sets a variable(non local) to a value that is not serializable.
+# By Mirre.
+
+soft api exceptions: false
+# Allows Skript to ignore certain actions which would normally result in thrown exceptions.
+# If everything works correctly, you should keep this option disabled. It might cause problems in some cases.
+# However, if Skript or addons of it are not working correctly, this might help.
+# You may also get told to enable this by Skript or addon developers - then do so.
 
 # ==== Variables ====