Version 4.1.1-1.8.8-SNAPSHOT!

Bumped version to 4.1.1-SNAPSHOT for continued development.
(cherry picked from commit 1ba693ffd5)
2019-08-25 00:14:43 -04:00 · 2019-08-25 00:09:58 -04:00 · 2019-08-25 00:09:09 -04:00 · 2019-08-25 00:09:03 -04:00 · 2019-08-25 00:08:55 -04:00 · 2019-08-25 00:08:45 -04:00
67 changed files with 1744 additions and 1931 deletions
@@ -12,6 +12,7 @@

 # maven
 /target
+dependency-reduced-pom.xml

 # vim
 .*.sw[a-p]
@@ -1,7 +1,7 @@
-language: java
-jdk:
-  - openjdk6
-notifications:
-  email: false
-before_install:
-  - sed -i.bak -e 's|https://nexus.codehaus.org/snapshots/|https://oss.sonatype.org/content/repositories/codehaus-snapshots/|g' ~/.m2/settings.xml
+language: java
+jdk:
+  - oraclejdk8
+notifications:
+  email: false
+before_install:
+  - sed -i.bak -e 's|https://nexus.codehaus.org/snapshots/|https://oss.sonatype.org/content/repositories/codehaus-snapshots/|g' ~/.m2/settings.xml
@@ -13,3 +13,4 @@ Copyright (c) 2011, The Multiverse Team All rights reserved.
 Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met:

 Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution. Neither the name of the The Multiverse Team nor the names of its contributors may be used to endorse or promote products derived from this software without specific prior written permission. THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ 
@@ -0,0 +1,3 @@
+machine:
+  java:
+    version: oraclejdk8
@@ -115,7 +115,6 @@
            <property name="ignoreNumbers" value="-1, 0, 0.5, 1, 2, 3"/>
        </module>
        <module name="MissingSwitchDefault"/>
-        <module name="RedundantThrows"/>
        <module name="SimplifyBooleanExpression"/>
        <module name="SimplifyBooleanReturn"/>
        <!-- Don't like errors for `final` missing
@@ -3,7 +3,7 @@
    <modelVersion>4.0.0</modelVersion>
    <groupId>com.onarandombox.multiversecore</groupId>
    <artifactId>Multiverse-Core</artifactId>
-    <version>2.5</version>
+    <version>4.1.1-1.8.8-SNAPSHOT</version>
    <name>Multiverse-Core</name>
    <description>World Management Plugin</description>
    <properties>
@@ -13,10 +13,10 @@

    <repositories>
      <!-- Spigot -->
-    <repository>
-        <id>spigot-repo</id>
-        <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
-    </repository>
+        <repository>
+            <id>spigot-repo</id>
+            <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
+        </repository>
        <repository>
            <id>onarandombox</id>
            <url>http://repo.onarandombox.com/content/groups/public</url>
@@ -27,7 +27,16 @@
        </repository>
        <repository>
            <id>vault-repo</id>
-            <url>http://nexus.theyeticave.net/content/repositories/pub_releases</url>
+            <url>http://nexus.hc.to/content/repositories/pub_releases</url>
+        </repository>
+        <repository>
+            <id>minebench-repo</id>
+            <url>https://repo.minebench.de/</url>
+        </repository>
+        <!-- Has a copy of metrics R8-SNAPSHOT !-->
+        <repository>
+            <id>elmakers-repo</id>
+            <url>http://maven.elmakers.com/repository/</url>
        </repository>
    </repositories>

@@ -70,16 +79,16 @@
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-compiler-plugin</artifactId>
-                <version>2.3.2</version>
+                <version>3.7.0</version>
                <configuration>
-                    <source>1.6</source>
-                    <target>1.6</target>
+                    <source>1.8</source>
+                    <target>1.8</target>
                </configuration>
            </plugin>
            <plugin>
                <groupId>com.google.code.maven-replacer-plugin</groupId>
                <artifactId>maven-replacer-plugin</artifactId>
-                <version>1.3.8</version>
+                <version>1.4.1</version>
                <executions>
                    <execution>
                        <phase>prepare-package</phase>
@@ -102,7 +111,7 @@
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-jar-plugin</artifactId>
-                <version>2.3.1</version>
+                <version>3.0.2</version>
                <configuration>
                    <archive>
                        <manifestEntries>
@@ -113,7 +122,7 @@
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>
-                <version>2.11</version>
+                <version>3.0.0-M3</version>
                <configuration>
                    <parallel>methods</parallel>
                    <threadCount>10</threadCount>
@@ -126,14 +135,14 @@
                    <dependency>
                        <groupId>org.apache.maven.surefire</groupId>
                        <artifactId>surefire-junit47</artifactId>
-                        <version>2.11</version>
+                        <version>3.0.0-M3</version>
                    </dependency>
                </dependencies>
            </plugin>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-checkstyle-plugin</artifactId>
-                <version>2.10</version>
+                <version>2.17</version>
                <configuration>
                    <enableRulesSummary>true</enableRulesSummary>
                    <configLocation>${project.basedir}/config/mv_checks.xml</configLocation>
@@ -142,7 +151,7 @@
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-source-plugin</artifactId>
-                <version>2.1.2</version>
+                <version>3.0.1</version>
                <executions>
                    <execution>
                        <id>attach-sources</id>
@@ -155,7 +164,7 @@
            </plugin>
            <plugin>
                <artifactId>maven-javadoc-plugin</artifactId>
-                <version>2.8.1</version>
+                <version>2.10.4</version>
                <executions>
                    <execution>
                        <id>javadoc-jar</id>
@@ -163,13 +172,16 @@
                        <goals>
                            <goal>jar</goal>
                        </goals>
+                        <configuration>
+                            <failOnError>false</failOnError>
+                        </configuration>
                    </execution>
                </executions>
            </plugin>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-shade-plugin</artifactId>
-                <version>1.5</version>
+                <version>3.1.1</version>
                <executions>
                    <execution>
                        <phase>package</phase>
@@ -177,31 +189,22 @@
                            <goal>shade</goal>
                        </goals>
                        <configuration>
-                            <artifactSet>
-                                <includes>
-                                    <include>me.main__.util:SerializationConfig</include>
-                                    <include>com.pneumaticraft.commandhandler:CommandHandler</include>
-                                    <include>com.dumptruckman.minecraft:buscript</include>
-                                    <include>org.mcstats.bukkit:metrics</include>
-                                    <include>com.dumptruckman.minecraft:Logging</include>
-                                </includes>
-                            </artifactSet>
                            <relocations>
                                <relocation>
                                    <pattern>me.main__.util</pattern>
-                                    <shadedPattern>me.main__.util.multiverse</shadedPattern>
+                                    <shadedPattern>com.onarandombox.serializationconfig</shadedPattern>
                                </relocation>
                                <relocation>
                                    <pattern>com.pneumaticraft.commandhandler</pattern>
-                                    <shadedPattern>com.pneumaticraft.commandhandler.multiverse</shadedPattern>
+                                    <shadedPattern>com.onarandombox.commandhandler</shadedPattern>
                                </relocation>
                                <relocation>
                                    <pattern>buscript</pattern>
-                                    <shadedPattern>buscript.multiverse</shadedPattern>
+                                    <shadedPattern>com.onarandombox.buscript</shadedPattern>
                                </relocation>
                                <relocation>
                                    <pattern>org.mcstats</pattern>
-                                    <shadedPattern>org.mcstats.multiverse</shadedPattern>
+                                    <shadedPattern>com.onarandombox.mcstats</shadedPattern>
                                </relocation>
                                <relocation>
                                    <pattern>com.dumptruckman.minecraft.util.Logging</pattern>
@@ -211,29 +214,29 @@
                                    <pattern>com.dumptruckman.minecraft.util.DebugLog</pattern>
                                    <shadedPattern>com.onarandombox.MultiverseCore.utils.DebugFileLogger</shadedPattern>
                                </relocation>
+                                <relocation>
+                                    <pattern>org.codehaus.jettison</pattern>
+                                    <shadedPattern>com.onarandombox.jettison</shadedPattern>
+                                </relocation>
+                                <relocation>
+                                    <pattern>de.themoep.idconverter</pattern>
+                                    <shadedPattern>com.onarandombox.idconverter</shadedPattern>
+                                </relocation>
                            </relocations>
                        </configuration>
                    </execution>
+                </executions>
+            </plugin>
+            <plugin>
+                <groupId>se.eris</groupId>
+                <artifactId>notnull-instrumenter-maven-plugin</artifactId>
+                <version>0.6.8</version>
+                <executions>
                    <execution>
-                        <id>apache-commons-io</id>
-                        <phase>package</phase>
                        <goals>
-                            <goal>shade</goal>
+                            <goal>instrument</goal>
+                            <goal>tests-instrument</goal>
                        </goals>
-                        <configuration>
-                            <artifactSet>
-                                <includes>
-                                    <include>commons-io:commons-io</include>
-                                </includes>
-                            </artifactSet>
-                            <relocations>
-                                <relocation>
-                                    <pattern>org.apache.commons</pattern>
-                                    <shadedPattern>org.apache.commons.multiverse</shadedPattern>
-                                </relocation>
-                            </relocations>
-                            <minimizeJar>true</minimizeJar>
-                        </configuration>
                    </execution>
                </executions>
            </plugin>
@@ -242,70 +245,16 @@

    <dependencies>
        <dependency>
-            <groupId>commons-io</groupId>
-            <artifactId>commons-io</artifactId>
-            <version>2.4</version>
-            <type>jar</type>
-            <scope>compile</scope>
-        </dependency>
-    <!--Spigot API-->
-    <dependency>
-           <groupId>org.spigotmc</groupId>
-           <artifactId>spigot-api</artifactId>
+           <groupId>org.bukkit</groupId>
+           <artifactId>bukkit</artifactId>
           <version>1.8.8-R0.1-SNAPSHOT</version>
           <scope>provided</scope>
-    </dependency>
-        <!--Spigot API-->
-        <!-- Start of Spout (disabled because we aren't using it)
-        <dependency>
-            <groupId>org.getspout</groupId>
-            <artifactId>spoutpluginapi</artifactId>
-            <version>dev-SNAPSHOT</version>
        </dependency>
-             End of Spout -->
        <!-- SerializationConfig Dependency -->
        <dependency>
            <groupId>me.main__.util</groupId>
            <artifactId>SerializationConfig</artifactId>
            <version>1.7</version>
-            <type>jar</type>
-            <scope>compile</scope>
-        </dependency>
-        <!-- End of SerializationConfig Dependency -->
-        <!-- Start of Economy Dependency -->
-        <dependency>
-            <groupId>net.milkbowl.vault</groupId>
-            <artifactId>VaultAPI</artifactId>
-            <version>1.5</version>
-            <type>jar</type>
-            <scope>compile</scope>
-        </dependency>
-        <!-- End of Economy Dependency -->
-        <!-- Start of CommandHandler Dependency -->
-        <dependency>
-            <groupId>com.pneumaticraft.commandhandler</groupId>
-            <artifactId>CommandHandler</artifactId>
-            <version>8</version>
-            <type>jar</type>
-            <scope>compile</scope>
-        </dependency>
-        <!-- End of CommandHandler Dependency -->
-        <!-- Start of Buscript Dependency -->
-        <dependency>
-            <groupId>com.dumptruckman.minecraft</groupId>
-            <artifactId>buscript</artifactId>
-            <version>1.0</version>
-            <type>jar</type>
-            <scope>compile</scope>
-        </dependency>
-        <!-- End of Buscript Dependency -->
-        <!-- Start of Metrics Dependency -->
-        <dependency>
-            <groupId>org.mcstats.bukkit</groupId>
-            <artifactId>metrics</artifactId>
-            <version>R6</version>
-            <type>jar</type>
-            <scope>compile</scope>
            <exclusions>
                <exclusion>
                    <groupId>org.bukkit</groupId>
@@ -313,61 +262,102 @@
                </exclusion>
            </exclusions>
        </dependency>
-        <!-- End of Metrics Dependency -->
-        <!-- Start of Logging Dependency -->
+        <!-- End of SerializationConfig Dependency -->
+        <!-- Start of Economy Dependency -->
+        <dependency>
+            <groupId>net.milkbowl.vault</groupId>
+            <artifactId>VaultAPI</artifactId>
+            <version>1.5</version>
+            <scope>provided</scope>
+        </dependency>
+        <!-- End of Economy Dependency -->
+        <!-- Start of CommandHandler Dependency -->
+        <dependency>
+            <groupId>com.pneumaticraft.commandhandler</groupId>
+            <artifactId>CommandHandler</artifactId>
+            <version>11</version>
+        </dependency>
        <dependency>
            <groupId>com.dumptruckman.minecraft</groupId>
-            <artifactId>Logging</artifactId>
-            <version>1.0.9</version>
-            <type>jar</type>
-            <scope>compile</scope>
+            <artifactId>buscript</artifactId>
+            <version>2.0-SNAPSHOT</version>
+        </dependency>
+        <dependency>
+            <groupId>org.mcstats.bukkit</groupId>
+            <artifactId>metrics</artifactId>
+            <version>R6</version>
            <exclusions>
                <exclusion>
                    <groupId>org.bukkit</groupId>
-                    <artifactId>craftbukkit</artifactId>
+                    <artifactId>bukkit</artifactId>
                </exclusion>
            </exclusions>
        </dependency>
-        <!-- End of Logging Dependency -->
+        <dependency>
+            <groupId>com.dumptruckman.minecraft</groupId>
+            <artifactId>Logging</artifactId>
+            <version>1.1.1</version>
+        </dependency>
+        <dependency>
+            <groupId>org.jetbrains</groupId>
+            <artifactId>annotations</artifactId>
+            <version>16.0.2</version>
+        </dependency>
+        <dependency>
+            <groupId>de.themoep.idconverter</groupId>
+            <artifactId>mappings</artifactId>
+            <version>1.2-SNAPSHOT</version>
+        </dependency>
+
        <!-- Start of Test Dependencies -->
        <dependency>
            <groupId>junit</groupId>
            <artifactId>junit</artifactId>
-            <version>4.8.2</version>
+            <version>4.12</version>
+            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>org.powermock</groupId>
            <artifactId>powermock-module-junit4</artifactId>
-            <version>1.4.9</version>
+            <version>2.0.0</version>
            <type>jar</type>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>org.powermock</groupId>
            <artifactId>powermock-api-easymock</artifactId>
-            <version>1.4.9</version>
+            <version>2.0.0</version>
            <type>jar</type>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>org.powermock</groupId>
-            <artifactId>powermock-api-mockito</artifactId>
-            <version>1.4.9</version>
+            <artifactId>powermock-api-mockito2</artifactId>
+            <version>2.0.0</version>
            <type>jar</type>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>org.easymock</groupId>
            <artifactId>easymock</artifactId>
-            <version>3.0</version>
+            <version>4.0.2</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>commons-io</groupId>
+            <artifactId>commons-io</artifactId>
+            <version>2.4</version>
            <scope>test</scope>
        </dependency>
-        <!-- End of Test Dependencies -->
    </dependencies>
    <distributionManagement>
        <repository>
            <id>OnARandomBox</id>
            <url>http://repo.onarandombox.com/content/repositories/multiverse</url>
        </repository>
+        <snapshotRepository>
+            <id>OnARandomBox</id>
+            <url>http://repo.onarandombox.com/content/repositories/multiverse-snapshots/</url>
+        </snapshotRepository>
    </distributionManagement>
 </project>
@@ -12,6 +12,7 @@ import com.onarandombox.MultiverseCore.api.BlockSafety;
 import com.onarandombox.MultiverseCore.api.MultiverseWorld;
 import com.onarandombox.MultiverseCore.api.SafeTTeleporter;
 import com.onarandombox.MultiverseCore.configuration.SpawnLocation;
+import com.onarandombox.MultiverseCore.configuration.SpawnSettings;
 import com.onarandombox.MultiverseCore.configuration.WorldPropertyValidator;
 import com.onarandombox.MultiverseCore.enums.AllowedPortalType;
 import com.onarandombox.MultiverseCore.enums.EnglishChatColor;
@@ -23,6 +24,7 @@ import org.bukkit.ChatColor;
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
+import org.bukkit.Material;
 import org.bukkit.World;
 import org.bukkit.World.Environment;
 import org.bukkit.WorldType;
@@ -32,6 +34,7 @@ import org.bukkit.entity.Player;
 import org.bukkit.permissions.Permission;
 import org.bukkit.permissions.PermissionDefault;
 import org.bukkit.util.Vector;
+import org.jetbrains.annotations.Nullable;
 import org.json.simple.JSONObject;

 import java.util.Collections;
@@ -184,14 +187,22 @@ public class MVWorld implements MultiverseWorld {
        //this.props.validate();
    }

+    /**
+     * This method is here to provide a stopgap until the add/remove/clear methods are implemented with
+     * SerializationConfig.
+     */
+    public void validateEntitySpawns() {
+        setAllowAnimalSpawn(canAnimalsSpawn());
+        setAllowMonsterSpawn(canMonstersSpawn());
+    }
+
    private void validateProperties() {
        setPVPMode(isPVPEnabled());
        setDifficulty(getDifficulty());
        setKeepSpawnInMemory(isKeepingSpawnInMemory());
        setScaling(getScaling());
        setRespawnToWorld(this.props.getRespawnToWorld());
-        setAllowAnimalSpawn(canAnimalsSpawn());
-        setAllowMonsterSpawn(canMonstersSpawn());
+        validateEntitySpawns();
        setGameMode(getGameMode());
    }

@@ -246,10 +257,10 @@ public class MVWorld implements MultiverseWorld {
    /**
     * Used to apply the spawning-property.
     */
-    private final class SpawningPropertyValidator extends WorldPropertyValidator<Boolean> {
+    private final class SpawningPropertyValidator extends WorldPropertyValidator<SpawnSettings> {
        @Override
-        public Boolean validateChange(String property, Boolean newValue, Boolean oldValue,
-                MVWorld object) throws ChangeDeniedException {
+        public SpawnSettings validateChange(String property, SpawnSettings newValue, SpawnSettings oldValue,
+                                      MVWorld object) throws ChangeDeniedException {
            boolean allowMonsters, allowAnimals;
            if (getAnimalList().isEmpty()) {
                allowAnimals = canAnimalsSpawn();
@@ -271,7 +282,9 @@ public class MVWorld implements MultiverseWorld {
                }
                world.setSpawnFlags(allowMonsters, allowAnimals);
            }
-            plugin.getMVWorldManager().getTheWorldPurger().purgeWorld(MVWorld.this);
+            if (MultiverseCoreConfiguration.getInstance().isAutoPurgeEnabled()) {
+                plugin.getMVWorldManager().getTheWorldPurger().purgeWorld(MVWorld.this);
+            }
            return super.validateChange(property, newValue, oldValue, object);
        }
    }
@@ -542,6 +555,7 @@ public class MVWorld implements MultiverseWorld {
        if (list == null)
            return false;
        list.clear();
+        validateEntitySpawns();
        return true;
    }

@@ -557,6 +571,7 @@ public class MVWorld implements MultiverseWorld {
        if (list == null)
            return false;
        list.add(value);
+        validateEntitySpawns();
        return true;
    }

@@ -572,6 +587,7 @@ public class MVWorld implements MultiverseWorld {
        if (list == null)
            return false;
        list.remove(value);
+        validateEntitySpawns();
        return true;
    }

@@ -589,29 +605,6 @@ public class MVWorld implements MultiverseWorld {
        return null;
    }

-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated This is deprecated.
-     */
-    @Override
-    @Deprecated
-    public <T> com.onarandombox.MultiverseCore.configuration.MVConfigProperty<T> getProperty(String property,
-            Class<T> expected) throws PropertyDoesNotExistException {
-        throw new UnsupportedOperationException("'MVConfigProperty<T> getProperty(String,Class<T>)' is no longer supported!");
-    }
-
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated This is deprecated.
-     */
-    @Override
-    @Deprecated
-    public boolean setProperty(String name, String value, CommandSender sender) throws PropertyDoesNotExistException {
-        return this.setPropertyValue(name, value);
-    }
-
    /**
     * {@inheritDoc}
     */
@@ -931,7 +924,7 @@ public class MVWorld implements MultiverseWorld {
     * {@inheritDoc}
     */
    @Override
-    public int getCurrency() {
+    public Material getCurrency() {
        return this.props.getCurrency();
    }

@@ -939,7 +932,7 @@ public class MVWorld implements MultiverseWorld {
     * {@inheritDoc}
     */
    @Override
-    public void setCurrency(int currency) {
+    public void setCurrency(@Nullable Material currency) {
        this.props.setCurrency(currency);
    }

@@ -7,6 +7,23 @@

 package com.onarandombox.MultiverseCore;

+import java.io.BufferedReader;
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
+import java.io.UnsupportedEncodingException;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.logging.Level;
+
 import buscript.Buscript;
 import com.dumptruckman.minecraft.util.Logging;
 import com.onarandombox.MultiverseCore.MVWorld.NullLocation;
@@ -61,6 +78,7 @@ import com.onarandombox.MultiverseCore.destination.DestinationFactory;
 import com.onarandombox.MultiverseCore.destination.ExactDestination;
 import com.onarandombox.MultiverseCore.destination.PlayerDestination;
 import com.onarandombox.MultiverseCore.destination.WorldDestination;
+import com.onarandombox.MultiverseCore.event.MVDebugModeEvent;
 import com.onarandombox.MultiverseCore.event.MVVersionEvent;
 import com.onarandombox.MultiverseCore.listeners.MVAsyncPlayerChatListener;
 import com.onarandombox.MultiverseCore.listeners.MVChatListener;
@@ -77,6 +95,7 @@ import com.onarandombox.MultiverseCore.utils.MVEconomist;
 import com.onarandombox.MultiverseCore.utils.MVMessaging;
 import com.onarandombox.MultiverseCore.utils.MVPermissions;
 import com.onarandombox.MultiverseCore.utils.MVPlayerSession;
+import com.onarandombox.MultiverseCore.utils.MaterialConverter;
 import com.onarandombox.MultiverseCore.utils.SimpleBlockSafety;
 import com.onarandombox.MultiverseCore.utils.SimpleLocationManipulation;
 import com.onarandombox.MultiverseCore.utils.SimpleSafeTTeleporter;
@@ -90,7 +109,6 @@ import org.bukkit.ChatColor;
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
-import org.bukkit.Server;
 import org.bukkit.World.Environment;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
@@ -101,29 +119,16 @@ import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.configuration.file.YamlConfiguration;
 import org.bukkit.entity.Player;
 import org.bukkit.plugin.PluginDescriptionFile;
-import org.bukkit.plugin.PluginLoader;
 import org.bukkit.plugin.PluginManager;
 import org.bukkit.plugin.java.JavaPlugin;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.mcstats.Metrics;

-import java.io.File;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.LinkedHashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.logging.Level;
-
 /**
 * The implementation of the Multiverse-{@link Core}.
 */
 public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
-    private static final int PROTOCOL = 20;
+    private static final int PROTOCOL = 24;
    // TODO: Investigate if this one is really needed to be static.
    // Doubt it. -- FernFerret
    private static Map<String, String> teleportQueue = new HashMap<String, String>();
@@ -138,11 +143,13 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {

    /**
     * This is for unit testing.
-     * @deprecated
+     * @param loader The PluginLoader to use.
+     * @param description The Description file to use.
+     * @param dataFolder The folder that other datafiles can be found in.
+     * @param file The location of the plugin.
     */
-    @Deprecated
-    public MultiverseCore(PluginLoader loader, Server server, PluginDescriptionFile description, File dataFolder, File file) {
-        super(loader, server, description, dataFolder, file);
+    public MultiverseCore(JavaPluginLoader loader, PluginDescriptionFile description, File dataFolder, File file) {
+        super(loader, description, dataFolder, file);
    }

    /**
@@ -170,11 +177,6 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
        teleportQueue.put(teleportee, teleporter);
    }

-    @Override
-    public String toString() {
-        return "The Multiverse-Core Plugin";
-    }
-
    /**
     * {@inheritDoc}
     * @deprecated This is now deprecated, nobody needs it any longer.
@@ -229,7 +231,6 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
    private Buscript buscript;
    private int pluginCount;
    private DestinationFactory destFactory;
-    //private SpoutInterface spoutInterface = null;
    private MultiverseMessaging messaging;
    private BlockSafety blockSafety;
    private LocationManipulation locationManipulation;
@@ -304,7 +305,6 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
        // Setup & Load our Configuration files.
        loadConfigs();
        if (this.multiverseConfig != null) {
-            Logging.setDebugLevel(getMVConfig().getGlobalDebug());
            Logging.setShowingConfig(!getMVConfig().getSilentStart());
            this.worldManager.loadDefaultWorlds();
            this.worldManager.loadWorlds(true);
@@ -333,19 +333,15 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
            this.chatListener = new MVPlayerChatListener(this, this.playerListener);
        }
        getServer().getPluginManager().registerEvents(this.chatListener, this);
-        /*
-        // Check to see if spout was already loaded (most likely):
-        if (this.getServer().getPluginManager().getPlugin("Spout") != null) {
-            this.setSpout();
-            this.log(Level.INFO, "Spout integration enabled.");
-        }
-        */

        this.initializeBuscript();
        this.setupMetrics();

        // Output a little snippet to show it's enabled.
        Logging.config("Version %s (API v%s) Enabled - By %s", this.getDescription().getVersion(), PROTOCOL, getAuthors());
+
+        getLogger().info("Help dumptruckman keep this project alive. Become a patron! https://www.patreon.com/dumptruckman");
+        getLogger().info("One time donations are also appreciated: https://www.paypal.me/dumptruckman");
    }

    /**
@@ -354,7 +350,7 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
    private void initializeBuscript() {
        buscript = new Buscript(this);
        // Add global variable "multiverse" to javascript environment
-        buscript.getGlobalScope().put("multiverse", buscript.getGlobalScope(), this);
+        buscript.setScriptVariable("multiverse", this);
    }

    /**
@@ -479,8 +475,19 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
    public void loadConfigs() {
        // Now grab the Configuration Files.
        this.multiverseConfig = YamlConfiguration.loadConfiguration(new File(getDataFolder(), "config.yml"));
-        Configuration coreDefaults = YamlConfiguration.loadConfiguration(this.getClass().getResourceAsStream("/defaults/config.yml"));
-        this.multiverseConfig.setDefaults(coreDefaults);
+        InputStream resourceURL = this.getClass().getResourceAsStream("/defaults/config.yml");
+
+        // Read in our default config with UTF-8 now
+        Configuration coreDefaults;
+        try {
+            coreDefaults = YamlConfiguration.loadConfiguration(new BufferedReader(new InputStreamReader(resourceURL, "UTF-8")));
+            this.multiverseConfig.setDefaults(coreDefaults);
+        } catch (UnsupportedEncodingException e) {
+            Logging.severe("Couldn't load default config with UTF-8 encoding. Details follow:");
+            e.printStackTrace();
+            Logging.severe("Default configs NOT loaded.");
+        }
+
        this.multiverseConfig.options().copyDefaults(false);
        this.multiverseConfig.options().copyHeader(true);

@@ -504,6 +511,12 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
        // Old Config Format
        this.migrate22Values();
        this.saveMVConfigs();
+
+        int level = Logging.getDebugLevel();
+        Logging.setDebugLevel(getMVConfig().getGlobalDebug());
+        if (level != Logging.getDebugLevel()) {
+            getServer().getPluginManager().callEvent(new MVDebugModeEvent(level));
+        }
    }

    /**
@@ -624,8 +637,9 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
                // migrate entryfee
                if (section.isConfigurationSection("entryfee")) {
                    ConfigurationSection feeSection = section.getConfigurationSection("entryfee");
-                    if (feeSection.isInt("currency"))
-                        world.setCurrency(feeSection.getInt("currency"));
+                    if (feeSection.isInt("currency")) {
+                        world.setCurrency(MaterialConverter.convertConfigType(feeSection, "currency"));
+                    }

                    if (feeSection.isDouble("amount"))
                        world.setPrice(feeSection.getDouble("amount"));
@@ -871,17 +885,6 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
        }
    }

-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated This is deprecated.
-     */
-    @Override
-    @Deprecated
-    public com.onarandombox.MultiverseCore.utils.SafeTTeleporter getTeleporter() {
-        return new com.onarandombox.MultiverseCore.utils.SafeTTeleporter(this);
-    }
-
    /**
     * {@inheritDoc}
     */
@@ -1081,25 +1084,6 @@ public class MultiverseCore extends JavaPlugin implements MVPlugin, Core {
        this.serverFolder = newServerFolder;
    }

-    /*
-    /**
-     * Initializes Spout.
-     * /
-    public void setSpout() {
-        this.spoutInterface = new SpoutInterface();
-        this.commandHandler.registerCommand(new SpoutCommand(this));
-    }
-
-    /**
-     * Gets our {@link SpoutInterface}.
-     *
-     * @return The {@link SpoutInterface} we're using.
-     * /
-    public SpoutInterface getSpout() {
-        return this.spoutInterface;
-    }
-    */
-
    /**
     * {@inheritDoc}
     */
@@ -2,7 +2,9 @@ package com.onarandombox.MultiverseCore;

 import com.dumptruckman.minecraft.util.*;
 import com.onarandombox.MultiverseCore.api.*;
+import com.onarandombox.MultiverseCore.event.MVDebugModeEvent;
 import me.main__.util.SerializationConfig.*;
+import org.bukkit.Bukkit;

 import java.util.*;

@@ -67,6 +69,8 @@ public class MultiverseCoreConfiguration extends SerializationConfig implements
    private volatile boolean defaultportalsearch;
    @Property
    private volatile int portalsearchradius;
+    @Property
+    private volatile boolean autopurge;

    public MultiverseCoreConfiguration() {
        super();
@@ -98,6 +102,7 @@ public class MultiverseCoreConfiguration extends SerializationConfig implements
        silentstart = false;
        defaultportalsearch = false;
        portalsearchradius = 128;
+        autopurge = true;
        // END CHECKSTYLE-SUPPRESSION: MagicNumberCheck
    }

@@ -226,6 +231,7 @@ public class MultiverseCoreConfiguration extends SerializationConfig implements
    public void setGlobalDebug(int globalDebug) {
        this.globaldebug = globalDebug;
        Logging.setDebugLevel(globalDebug);
+        Bukkit.getPluginManager().callEvent(new MVDebugModeEvent(globalDebug));
    }

    /**
@@ -332,4 +338,14 @@ public class MultiverseCoreConfiguration extends SerializationConfig implements
    public int getPortalSearchRadius() {
        return portalsearchradius;
    }
+
+    @Override
+    public boolean isAutoPurgeEnabled() {
+        return autopurge;
+    }
+
+    @Override
+    public void setAutoPurgeEnabled(boolean autopurge) {
+        this.autopurge = autopurge;
+    }
 }
@@ -18,8 +18,10 @@ import org.bukkit.ChatColor;
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
+import org.bukkit.Material;
 import org.bukkit.World.Environment;
 import org.bukkit.configuration.serialization.SerializableAs;
+import org.jetbrains.annotations.Nullable;

 import java.util.ArrayList;
 import java.util.Collections;
@@ -261,7 +263,7 @@ public class WorldProperties extends SerializationConfig {
    @Property(description = "Sorry, 'animals' must either be: true or false.")
    private volatile SpawnSettings spawning;
    @Property
-    private volatile EntryFee entryfee;
+    volatile EntryFee entryfee;
    @Property(description = "Sorry, 'hunger' must either be: true or false.")
    private volatile boolean hunger;
    @Property(description = "Sorry, 'autoheal' must either be: true or false.")
@@ -496,11 +498,11 @@ public class WorldProperties extends SerializationConfig {
        return this.setPropertyValueUnchecked("respawnWorld", respawnToWorld);
    }

-    public int getCurrency() {
+    public Material getCurrency() {
        return this.entryfee.getCurrency();
    }

-    public void setCurrency(int currency) {
+    public void setCurrency(@Nullable Material currency) {
        this.setPropertyValueUnchecked("entryfee.currency", currency);
    }

@@ -70,17 +70,6 @@ public interface Core {
     */
    MVPlayerSession getPlayerSession(Player player);

-    /**
-     * Gets the instantiated Safe-T-Teleporter for performing
-     * safe teleports.
-     *
-     * @return A non-null {@link SafeTTeleporter}.
-     *
-     * @deprecated Use {@link #getSafeTTeleporter()} instead.
-     */
-    @Deprecated
-    com.onarandombox.MultiverseCore.utils.SafeTTeleporter getTeleporter();
-
    /**
     * Multiverse uses an advanced permissions setup, this object
     * simplifies getting/setting permissions.
@@ -199,7 +199,7 @@ public interface MVWorldManager {
    boolean isMVWorld(World world);

    /**
-     * Load the Worlds & Settings from the configuration file.
+     * Load the Worlds &amp; Settings from the configuration file.
     *
     * @param forceLoad If set to true, this will perform a total
     *                  reset and not just load new worlds.
@@ -207,7 +207,7 @@ public interface MVWorldManager {
    void loadWorlds(boolean forceLoad);

    /**
-     * Loads the Worlds & Settings for any worlds that bukkit loaded before us.
+     * Loads the Worlds &amp; Settings for any worlds that bukkit loaded before us.
     * <p>
     * This way people will _always_ have some worlds in the list.
     */
@@ -199,4 +199,18 @@ public interface MultiverseCoreConfig extends ConfigurationSerializable {
     * @return The portal search radius.
     */
    int getPortalSearchRadius();
+
+    /**
+     * Gets whether or not the automatic purge of entities is enabled.
+     *
+     * @return True if automatic purge is enabled.
+     */
+    boolean isAutoPurgeEnabled();
+
+    /**
+     * Sets whether or not the automatic purge of entities is enabled.
+     *
+     * @param autopurge True if automatic purge should be enabled.
+     */
+    void setAutoPurgeEnabled(boolean autopurge);
 }
@@ -13,10 +13,12 @@ import org.bukkit.ChatColor;
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
+import org.bukkit.Material;
 import org.bukkit.World;
 import org.bukkit.WorldType;
 import org.bukkit.command.CommandSender;
 import org.bukkit.permissions.Permission;
+import org.jetbrains.annotations.Nullable;

 import java.util.List;

@@ -153,37 +155,6 @@ public interface MultiverseWorld {
     */
    boolean setPropertyValue(String property, String value) throws PropertyDoesNotExistException;

-    /**
-     * Gets the actual MVConfigProperty from this world.
-     * It will throw a PropertyDoesNotExistException if the property is not found.
-     *
-     * @param property The name of a world property to get.
-     * @param expected The type of the expected property. Use Object.class if this doesn't matter for you.
-     * @param <T> The type of the expected property.
-     *
-     * @return A valid MVWorldProperty.
-     *
-     * @throws PropertyDoesNotExistException Thrown if the property was not found in the world.
-     * @deprecated We don't use {@link com.onarandombox.MultiverseCore.configuration.MVConfigProperty} any longer!
-     */
-    @Deprecated
-    <T> com.onarandombox.MultiverseCore.configuration.MVConfigProperty<T> getProperty(String property, Class<T> expected) throws PropertyDoesNotExistException;
-
-    // old config
-    /**
-     * Adds the property to the given value.
-     * It will throw a PropertyDoesNotExistException if the property is not found.
-     *
-     * @param property The name of a world property to set.
-     * @param value    A value in string representation, it will be parsed to the correct type.
-     * @param sender   The sender who wants this value to be set.
-     * @return True if the value was set, false if not.
-     * @throws PropertyDoesNotExistException Thrown if the property was not found in the world.
-     * @deprecated Use {@link #setPropertyValue(String, String)} instead.
-     */
-    @Deprecated
-    boolean setProperty(String property, String value, CommandSender sender) throws PropertyDoesNotExistException;
-
    /**
     * Adds a value to the given property. The property must be a {@link com.onarandombox.MultiverseCore.enums.AddProperties}.
     *
@@ -504,26 +475,28 @@ public interface MultiverseWorld {
    /**
     * Sets the price for entry to this world.
     * You can think of this like an amount.
-     * The type can be set with {@link #setCurrency(int)}
+     * The type can be set with {@link #setCurrency(Material)}
     *
     * @param price The Amount of money/item to enter the world.
     */
    void setPrice(double price);

    /**
-     * Gets the Type of currency that will be used when users enter this world.
+     * Gets the type of currency that will be used when users enter this world. A value of null indicates a non-item
+     * based currency is used.
     *
-     * @return The Type of currency that will be used when users enter this world.
+     * @return The type of currency that will be used when users enter this world.
     */
-    int getCurrency();
+    @Nullable
+    Material getCurrency();

    /**
     * Sets the type of item that will be required given the price is not 0.
-     * Use -1 to use an AllPay economy, or any valid itemid
+     * Use a value of null to specify a non-item based currency.
     *
     * @param item The Type of currency that will be used when users enter this world.
     */
-    void setCurrency(int item);
+    void setCurrency(@Nullable Material item);

    /**
     * Gets the world players will respawn in if they die in this one.
@@ -17,7 +17,7 @@ public interface WorldPurger {
    void purgeWorlds(List<MultiverseWorld> worlds);

    /**
-     * Convenience method for {@link #purgeWorld(CommandSender, MultiverseWorld, List, boolean, boolean)} that takes the settings from the world-config.
+     * Convenience method for {@link #purgeWorld(MultiverseWorld, java.util.List, boolean, boolean)} that takes the settings from the world-config.
     *
     * @param world The {@link MultiverseWorld}.
     */
@@ -62,7 +62,7 @@ public class CoordCommand extends MultiverseCommand {
            df.setMaximumFractionDigits(2);
            p.sendMessage(ChatColor.AQUA + "Coordinates: " + ChatColor.WHITE + plugin.getLocationManipulation().strCoords(p.getLocation()));
            p.sendMessage(ChatColor.AQUA + "Direction: " + ChatColor.WHITE + plugin.getLocationManipulation().getDirection(p.getLocation()));
-            p.sendMessage(ChatColor.AQUA + "Block: " + ChatColor.WHITE + Material.getMaterial(world.getBlockTypeIdAt(p.getLocation())));
+            p.sendMessage(ChatColor.AQUA + "Block: " + ChatColor.WHITE + world.getBlockAt(p.getLocation()).getType());
        } else {
            sender.sendMessage("This command needs to be used from a Player.");
        }
@@ -33,9 +33,13 @@ public class DeleteCommand extends MultiverseCommand {

    @Override
    public void runCommand(CommandSender sender, List<String> args) {
+        String worldName = args.get(0);
+
        Class<?>[] paramTypes = {String.class};
        List<Object> objectArgs = new ArrayList<Object>(args);
-        this.plugin.getCommandHandler().queueCommand(sender, "mvdelete", "deleteWorld", objectArgs,
-                paramTypes, ChatColor.GREEN + "World Deleted!", ChatColor.RED + "World could NOT be deleted!");
+        this.plugin.getCommandHandler()
+                   .queueCommand(sender, "mvdelete", "deleteWorld", objectArgs,
+                                 paramTypes, ChatColor.GREEN + "World '" + worldName + "' Deleted!",
+                                 ChatColor.RED + "World '" + worldName + "' could NOT be deleted!");
    }
 }
@@ -57,7 +57,7 @@ public class ImportCommand extends MultiverseCommand {
            File[] files = worldFolder.listFiles(new FilenameFilter() {
                @Override
                public boolean accept(File file, String name) {
-                    return name.equalsIgnoreCase("level.dat");
+                    return name.toLowerCase().endsWith(".dat");
                }
            });
            if (files != null && files.length > 0) {
@@ -95,10 +95,15 @@ public class ImportCommand extends MultiverseCommand {
        }
        return worldList;
    }
+    
+    private String trimWorldName(String userInput) {
+        // Removes relative paths.
+        return userInput.replaceAll("^[./\\\\]+", "");
+    }

    @Override
    public void runCommand(CommandSender sender, List<String> args) {
-        String worldName = args.get(0);
+        String worldName = trimWorldName(args.get(0));

        if (worldName.toLowerCase().equals("--list") || worldName.toLowerCase().equals("-l")) {
            String worldList = this.getPotentialWorlds();
@@ -112,7 +117,7 @@ public class ImportCommand extends MultiverseCommand {
        }
        // Since we made an exception for the list, we have to make sure they have at least 2 params:
        // Note the exception is --list, which is covered above.
-        if (args.size() == 1) {
+        if (args.size() == 1 || worldName.length() < 1) {
            this.showHelp(sender);
            return;
        }
@@ -142,21 +147,25 @@ public class ImportCommand extends MultiverseCommand {
            return;
        }

-        if (worldFile.exists() && env != null) {
-            Command.broadcastCommandMessage(sender, String.format("Starting import of world '%s'...", worldName));
-            if (this.worldManager.addWorld(worldName, environment, null, null, null, generator, useSpawnAdjust))
-                Command.broadcastCommandMessage(sender, ChatColor.GREEN + "Complete!");
-            else
-                Command.broadcastCommandMessage(sender, ChatColor.RED + "Failed!");
+        if (!worldFile.exists()) {
+            sender.sendMessage(ChatColor.RED + "FAILED.");
+            String worldList = this.getPotentialWorlds();
+            sender.sendMessage("That world folder does not exist. These look like worlds to me:");
+            sender.sendMessage(worldList);
+        } else if (!checkIfIsWorld(worldFile)) {
+            sender.sendMessage(ChatColor.RED + "FAILED.");
+            sender.sendMessage(String.format("'%s' does not appear to be a world. It is lacking a .dat file.",
+                                             worldName));
        } else if (env == null) {
            sender.sendMessage(ChatColor.RED + "FAILED.");
            sender.sendMessage("That world environment did not exist.");
            sender.sendMessage("For a list of available world types, type: " + ChatColor.AQUA + "/mvenv");
        } else {
-            sender.sendMessage(ChatColor.RED + "FAILED.");
-            String worldList = this.getPotentialWorlds();
-            sender.sendMessage("That world folder does not exist. These look like worlds to me:");
-            sender.sendMessage(worldList);
+            Command.broadcastCommandMessage(sender, String.format("Starting import of world '%s'...", worldName));
+            if (this.worldManager.addWorld(worldName, environment, null, null, null, generator, useSpawnAdjust))
+                Command.broadcastCommandMessage(sender, ChatColor.GREEN + "Complete!");
+            else
+                Command.broadcastCommandMessage(sender, ChatColor.RED + "Failed!");
        }
    }
 }
@@ -181,8 +181,8 @@ public class InfoCommand extends MultiverseCommand {
        message.add(new FancyHeader("Animal Settings", colors));
        message.add(new FancyMessage("Multiverse Setting: ", world.canAnimalsSpawn() + "", colors));
        message.add(new FancyMessage("Bukkit Setting: ", world.getCBWorld().getAllowAnimals() + "", colors));
-        if (world.getMonsterList().size() > 0) {
-            if (world.canMonstersSpawn()) {
+        if (world.getAnimalList().size() > 0) {
+            if (world.canAnimalsSpawn()) {
                message.add(new FancyMessage("Animals that" + ChatColor.RED + " CAN NOT "
                        + ChatColor.GREEN + "spawn: ", toCommaSeperated(world.getAnimalList()), colors));
            } else {
@@ -77,7 +77,7 @@ public class ModifyAddCommand extends MultiverseCommand {

        if (!ModifyCommand.validateAction(Action.Add, property)) {
            sender.sendMessage("Sorry, you can't ADD to " + property);
-            sender.sendMessage("Please visit our Github Wiki for more information: http://goo.gl/4W8cY");
+            sender.sendMessage("Please visit our Github Wiki for more information: https://goo.gl/OMGwzx");
            return;
        }

@@ -72,7 +72,7 @@ public class ModifyClearCommand extends MultiverseCommand {

        if (!ModifyCommand.validateAction(Action.Clear, property)) {
            sender.sendMessage("Sorry, you can't use CLEAR with " + property);
-            sender.sendMessage("Please visit our Github Wiki for more information: http://goo.gl/cgB2B");
+            sender.sendMessage("Please visit our Github Wiki for more information: https://goo.gl/q1h01S");
            return;
        }
        // TODO fix this
@@ -78,7 +78,7 @@ public class ModifyRemoveCommand extends MultiverseCommand {

        if (!ModifyCommand.validateAction(Action.Remove, property)) {
            sender.sendMessage("Sorry, you can't REMOVE anything from" + property);
-            sender.sendMessage("Please visit our Github Wiki for more information: http://goo.gl/4W8cY");
+            sender.sendMessage("Please visit our Github Wiki for more information: https://goo.gl/OMGwzx");
            return;
        }
        // TODO fix this
@@ -18,6 +18,7 @@ import org.bukkit.entity.Player;
 import org.bukkit.permissions.PermissionDefault;

 import java.util.List;
+import org.bukkit.Bukkit;

 /**
 * Sets the spawn for a world.
@@ -27,7 +28,7 @@ public class SetSpawnCommand extends MultiverseCommand {
        super(plugin);
        this.setName("Set World Spawn");
        this.setCommandUsage("/mv setspawn");
-        this.setArgRange(0, 0);
+        this.setArgRange(0, 6);
        this.addKey("mvsetspawn");
        this.addKey("mvss");
        this.addKey("mv set spawn");
@@ -37,43 +38,110 @@ public class SetSpawnCommand extends MultiverseCommand {
        this.setPermission("multiverse.core.spawn.set", "Sets the spawn for the current world.", PermissionDefault.OP);
    }

+    /**
+     * Dispatches the user's command depending on the number of parameters
+     * @param sender The player who executes the command, may be console as well.
+     * @param args Command line parameters
+     */
    @Override
    public void runCommand(CommandSender sender, List<String> args) {
-        setWorldSpawn(sender);
+        if (args.isEmpty()) {
+            setWorldSpawn(sender);
+        } else if (args.size() == 4) {
+            setWorldSpawn(sender, args.get(0), args.get(1), args.get(2), args.get(3));
+        } else if (args.size() == 6) {
+            setWorldSpawn(sender, args.get(0), args.get(1), args.get(2), args.get(3), args.get(4), args.get(5));
+        } else {
+            sender.sendMessage("Use no arguments for your current location, or world/x/y/z, or world/x/y/z/yaw/pitch!");
+        }
    }

    /**
-     * Does the actual spawn-setting-work.
-     *
-     * @param sender The {@link CommandSender} that's setting the spawn.
+     * Set the world spawn when no parameters are given
+     * @param sender The {@link CommandSender} who executes the command. 
+     *               Everything not a {@link Player}, e.g. console, gets rejected, as we can't get coordinates from there.
     */
    protected void setWorldSpawn(CommandSender sender) {
        if (sender instanceof Player) {
            Player p = (Player) sender;
            Location l = p.getLocation();
            World w = p.getWorld();
-            MultiverseWorld foundWorld = this.plugin.getMVWorldManager().getMVWorld(w.getName());
-            if (foundWorld != null) {
-                foundWorld.setSpawnLocation(p.getLocation());
-                BlockSafety bs = this.plugin.getBlockSafety();
-                if (!bs.playerCanSpawnHereSafely(p.getLocation()) && foundWorld.getAdjustSpawn()) {
-                    sender.sendMessage("It looks like that location would normally be unsafe. But I trust you.");
-                    sender.sendMessage("I'm turning off the Safe-T-Teleporter for spawns to this world.");
-                    sender.sendMessage("If you want this turned back on just do:");
-                    sender.sendMessage(ChatColor.AQUA + "/mvm set adjustspawn true " + foundWorld.getAlias());
-                    foundWorld.setAdjustSpawn(false);
-                }
-                sender.sendMessage("Spawn was set to: " + plugin.getLocationManipulation().strCoords(p.getLocation()));
-                if (!plugin.saveWorldConfig()) {
-                    sender.sendMessage(ChatColor.RED + "There was an issue saving worlds.yml!  Your changes will only be temporary!");
-                }
-            } else {
-                w.setSpawnLocation(l.getBlockX(), l.getBlockY(), l.getBlockZ());
-                sender.sendMessage("Multiverse does not know about this world, only X,Y and Z set. Please import it to set the spawn fully (Pitch/Yaws).");
-            }
-
+            setWorldSpawn(sender, w, l);
        } else {
-            sender.sendMessage("You cannot use this command from the console.");
+            sender.sendMessage("You need to give coordinates to use this command from the console!");
+        }
+    }
+    
+    /**
+     * Set the world spawn when 4 parameters are given
+     * @param sender The {@link CommandSender} who executes the command
+     * @param world The world to set the spawn in
+     * @param x X-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     * @param y Y-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     * @param z Z-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     */
+    protected void setWorldSpawn(CommandSender sender, String world, String x, String y, String z) {
+        setWorldSpawn(sender, world, x, y, z, "0", "0");
+    }
+
+    /**
+     * Set the world spawn when 6 parameters are given
+     * @param sender The {@link CommandSender} who executes the command
+     * @param world The world to set the spawn in
+     * @param x X-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     * @param y Y-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     * @param z Z-coordinate to set the spawn to (as a {@link String} as it's from the command line, gets parsed into a double)
+     * @param yaw Yaw a newly spawned player should look at (as a {@link String} as it's from the command line, gets parsed into a float)
+     * @param pitch Pitch a newly spawned player should look at (as a {@link String} as it's from the command line, gets parsed into a float)
+     */
+    protected void setWorldSpawn(CommandSender sender, String world, String x, String y, String z, String yaw, String pitch) {
+        double dx, dy, dz;
+        float fpitch, fyaw;
+        World bukkitWorld = Bukkit.getWorld(world);
+        if (bukkitWorld == null) {
+            sender.sendMessage("World " + world + " is unknown!");
+            return;
+        }
+        try {
+            dx = Double.parseDouble(x);
+            dy = Double.parseDouble(y);
+            dz = Double.parseDouble(z);
+            fpitch = Float.parseFloat(pitch);
+            fyaw = Float.parseFloat(yaw);
+        } catch (NumberFormatException ex) {
+            sender.sendMessage("All coordinates must be numeric");
+            return;
+        }
+        Location l = new Location(bukkitWorld, dx, dy, dz, fyaw, fpitch);
+        setWorldSpawn(sender, bukkitWorld, l);
+    }
+
+    /**
+     * Does the actual spawn-setting-work.
+     *
+     * @param sender The {@link CommandSender} that's setting the spawn.
+     * @param w The {@link World} to set the spawn in
+     * @param l The {@link Location} to set the spawn to
+     */
+    private void setWorldSpawn(CommandSender sender, World w, Location l) {
+        MultiverseWorld foundWorld = this.plugin.getMVWorldManager().getMVWorld(w.getName());
+        if (foundWorld != null) {
+            foundWorld.setSpawnLocation(l);
+            BlockSafety bs = this.plugin.getBlockSafety();
+            if (!bs.playerCanSpawnHereSafely(l) && foundWorld.getAdjustSpawn()) {
+                sender.sendMessage("It looks like that location would normally be unsafe. But I trust you.");
+                sender.sendMessage("I'm turning off the Safe-T-Teleporter for spawns to this world.");
+                sender.sendMessage("If you want this turned back on just do:");
+                sender.sendMessage(ChatColor.AQUA + "/mvm set adjustspawn true " + foundWorld.getAlias());
+                foundWorld.setAdjustSpawn(false);
+            }
+            sender.sendMessage("Spawn was set to: " + plugin.getLocationManipulation().strCoords(l));
+            if (!plugin.saveWorldConfig()) {
+                sender.sendMessage(ChatColor.RED + "There was an issue saving worlds.yml!  Your changes will only be temporary!");
+            }
+        } else {
+            w.setSpawnLocation(l.getBlockX(), l.getBlockY(), l.getBlockZ());
+            sender.sendMessage("Multiverse does not know about this world, only X,Y and Z set. Please import it to set the spawn fully (Pitch/Yaws).");
        }
    }
 }
@@ -1,66 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2011.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-/*
-package com.onarandombox.MultiverseCore.commands;
-
-import com.onarandombox.MultiverseCore.MultiverseCore;
-import org.bukkit.ChatColor;
-import org.bukkit.command.CommandSender;
-import org.bukkit.entity.Player;
-import org.bukkit.permissions.PermissionDefault;
-import org.getspout.spoutapi.gui.GenericButton;
-import org.getspout.spoutapi.gui.GenericPopup;
-import org.getspout.spoutapi.gui.PopupScreen;
-import org.getspout.spoutapi.player.SpoutPlayer;
-
-import java.util.List;
-
-/**
- * Edit a world with spout.
- * /
-public class SpoutCommand extends MultiverseCommand {
-
-    public SpoutCommand(MultiverseCore plugin) {
-        super(plugin);
-        this.setName("Edit World with Spout");
-        this.setCommandUsage("/mv spout");
-        this.setArgRange(0, 0);
-        this.addKey("mv spout");
-        this.setPermission("multiverse.core.spout", "Edit a world with spout.", PermissionDefault.OP);
-        this.addCommandExample("/mv spout");
-    }
-
-    @Override
-    public void runCommand(CommandSender sender, List<String> args) {
-        if (!(sender instanceof Player)) {
-            sender.sendMessage(ChatColor.RED + "This command must be run as a player!");
-            return;
-        }
-        if (plugin.getSpout() == null) {
-            sender.sendMessage(ChatColor.RED + "You need spout installed on this server to use it with Multiverse!");
-            return;
-        }
-        SpoutPlayer p = (SpoutPlayer) sender;
-        if (!p.isSpoutCraftEnabled()) {
-            sender.sendMessage(ChatColor.RED + p.getName() + "You need to be using the Spoutcraft client to run this command!");
-            return;
-        }
-        PopupScreen pop = new GenericPopup();
-        GenericButton button = new GenericButton("Fish");
-        // TO-DO maybe use constants for these
-        // BEGIN CHECKSTYLE-SUPPRESSION: MagicNumberCheck
-        button.setX(50);
-        button.setY(50);
-        button.setWidth(100);
-        button.setHeight(40);
-        // END CHECKSTYLE-SUPPRESSION: MagicNumberCheck
-        pop.attachWidget(this.plugin, button);
-        sender.sendMessage(ChatColor.GREEN + "YAY!");
-        p.getMainScreen().attachPopupScreen(pop);
-    }
-}
-*/
@@ -16,12 +16,20 @@ import com.onarandombox.MultiverseCore.utils.webpaste.PasteService;
 import com.onarandombox.MultiverseCore.utils.webpaste.PasteServiceFactory;
 import com.onarandombox.MultiverseCore.utils.webpaste.PasteServiceType;
 import com.onarandombox.MultiverseCore.utils.webpaste.URLShortener;
+import com.pneumaticraft.commandhandler.CommandHandler;
+import org.apache.commons.lang.StringUtils;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.ConsoleCommandSender;
 import org.bukkit.entity.Player;
 import org.bukkit.permissions.PermissionDefault;
+import org.bukkit.scheduler.BukkitRunnable;
+import org.bukkit.util.StringUtil;

+import java.io.*;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;

 /**
 * Dumps version info to the console.
@@ -32,8 +40,8 @@ public class VersionCommand extends MultiverseCommand {
    public VersionCommand(MultiverseCore plugin) {
        super(plugin);
        this.setName("Multiverse Version");
-        this.setCommandUsage("/mv version " + ChatColor.GOLD + "-[pb]");
-        this.setArgRange(0, 1);
+        this.setCommandUsage("/mv version " + ChatColor.GOLD + "-[bh] [--include-plugin-list]");
+        this.setArgRange(0, 2);
        this.addKey("mv version");
        this.addKey("mvv");
        this.addKey("mvversion");
@@ -41,6 +49,98 @@ public class VersionCommand extends MultiverseCommand {
                "Dumps version info to the console, optionally to pastie.org with -p or pastebin.com with a -b.", PermissionDefault.TRUE);
    }

+    private String getLegacyString() {
+        StringBuilder legacyFile = new StringBuilder();
+        legacyFile.append("[Multiverse-Core] Multiverse-Core Version: ").append(this.plugin.getDescription().getVersion()).append('\n');
+        legacyFile.append("[Multiverse-Core] Bukkit Version: ").append(this.plugin.getServer().getVersion()).append('\n');
+        legacyFile.append("[Multiverse-Core] Loaded Worlds: ").append(this.plugin.getMVWorldManager().getMVWorlds()).append('\n');
+        legacyFile.append("[Multiverse-Core] Multiverse Plugins Loaded: ").append(this.plugin.getPluginCount()).append('\n');
+        legacyFile.append("[Multiverse-Core] Economy being used: ").append(plugin.getEconomist().getEconomyName()).append('\n');
+        legacyFile.append("[Multiverse-Core] Permissions Plugin: ").append(this.plugin.getMVPerms().getType()).append('\n');
+        legacyFile.append("[Multiverse-Core] Dumping Config Values: (version ")
+                .append(this.plugin.getMVConfig().getVersion()).append(")").append('\n');
+        legacyFile.append("[Multiverse-Core]  messagecooldown: ").append(plugin.getMessaging().getCooldown()).append('\n');
+        legacyFile.append("[Multiverse-Core]  teleportcooldown: ").append(plugin.getMVConfig().getTeleportCooldown()).append('\n');
+        legacyFile.append("[Multiverse-Core]  worldnameprefix: ").append(plugin.getMVConfig().getPrefixChat()).append('\n');
+        legacyFile.append("[Multiverse-Core]  worldnameprefixFormat: ").append(plugin.getMVConfig().getPrefixChatFormat()).append('\n');
+        legacyFile.append("[Multiverse-Core]  enforceaccess: ").append(plugin.getMVConfig().getEnforceAccess()).append('\n');
+        legacyFile.append("[Multiverse-Core]  displaypermerrors: ").append(plugin.getMVConfig().getDisplayPermErrors()).append('\n');
+        legacyFile.append("[Multiverse-Core]  teleportintercept: ").append(plugin.getMVConfig().getTeleportIntercept()).append('\n');
+        legacyFile.append("[Multiverse-Core]  firstspawnoverride: ").append(plugin.getMVConfig().getFirstSpawnOverride()).append('\n');
+        legacyFile.append("[Multiverse-Core]  firstspawnworld: ").append(plugin.getMVConfig().getFirstSpawnWorld()).append('\n');
+        legacyFile.append("[Multiverse-Core]  debug: ").append(plugin.getMVConfig().getGlobalDebug()).append('\n');
+        legacyFile.append("[Multiverse-Core] Special Code: FRN002").append('\n');
+        return legacyFile.toString();
+    }
+
+    private String getMarkdownString() {
+        StringBuilder markdownString = new StringBuilder();
+        markdownString.append("# Multiverse-Core\n");
+        markdownString.append("## Overview\n");
+        markdownString.append("| Name | Value |\n");
+        markdownString.append("| --- | --- |\n");
+        markdownString.append("| Multiverse-Core Version | `").append(this.plugin.getDescription().getVersion()).append("` |\n");
+        markdownString.append("| Bukkit Version | `").append(this.plugin.getServer().getVersion()).append("` |\n");
+        //markdownString.append("| Loaded Worlds | `").append(this.plugin.getMVWorldManager().getMVWorlds()).append("` |\n");
+        markdownString.append("| Multiverse Plugins Loaded | `").append(this.plugin.getPluginCount()).append("` |\n");
+        markdownString.append("| Economy being used | `").append(plugin.getEconomist().getEconomyName()).append("` |\n");
+        markdownString.append("| Permissions Plugin | `").append(this.plugin.getMVPerms().getType()).append("` |\n");
+        markdownString.append("## Parsed Config\n");
+        markdownString.append("These are what Multiverse thought the in-memory values of the config were.\n\n");
+        markdownString.append("| Config Key  | Value |\n");
+        markdownString.append("| --- | --- |\n");
+        markdownString.append("| version | `").append(this.plugin.getMVConfig().getVersion()).append("` |\n");
+        markdownString.append("| messagecooldown | `").append(plugin.getMessaging().getCooldown()).append("` |\n");
+        markdownString.append("| teleportcooldown | `").append(plugin.getMVConfig().getTeleportCooldown()).append("` |\n");
+        markdownString.append("| worldnameprefix | `").append(plugin.getMVConfig().getPrefixChat()).append("` |\n");
+        markdownString.append("| worldnameprefixFormat | `").append(plugin.getMVConfig().getPrefixChatFormat()).append("` |\n");
+        markdownString.append("| enforceaccess | `").append(plugin.getMVConfig().getEnforceAccess()).append("` |\n");
+        markdownString.append("| displaypermerrors | `").append(plugin.getMVConfig().getDisplayPermErrors()).append("` |\n");
+        markdownString.append("| teleportintercept | `").append(plugin.getMVConfig().getTeleportIntercept()).append("` |\n");
+        markdownString.append("| firstspawnoverride | `").append(plugin.getMVConfig().getFirstSpawnOverride()).append("` |\n");
+        markdownString.append("| firstspawnworld | `").append(plugin.getMVConfig().getFirstSpawnWorld()).append("` |\n");
+        markdownString.append("| debug | `").append(plugin.getMVConfig().getGlobalDebug()).append("` |\n");
+        return markdownString.toString();
+    }
+
+    private String readFile(final String filename) {
+        String result;
+        try {
+            FileReader reader = new FileReader(filename);
+            BufferedReader bufferedReader = new BufferedReader(reader);
+            String line;
+            result = "";
+            while ((line = bufferedReader.readLine()) != null) {
+                result += line + '\n';
+            }
+        } catch (FileNotFoundException e) {
+            Logging.severe("Unable to find %s. Here's the traceback: %s", filename, e.getMessage());
+            e.printStackTrace();
+            result = String.format("ERROR: Could not load: %s", filename);
+        } catch (IOException e) {
+            Logging.severe("Something bad happend when reading %s. Here's the traceback: %s", filename, e.getMessage());
+            e.printStackTrace();
+            result = String.format("ERROR: Could not load: %s", filename);
+        }
+        return result;
+    }
+
+    private Map<String, String> getVersionFiles() {
+        Map<String, String> files = new HashMap<String, String>();
+
+        // Add the legacy file, but as markdown so it's readable
+        files.put("version.md", this.getMarkdownString());
+
+        // Add the config.yml
+        File configFile = new File(this.plugin.getDataFolder(), "config.yml");
+        files.put(configFile.getName(), this.readFile(configFile.getAbsolutePath()));
+
+        // Add the config.yml
+        File worldConfig = new File(this.plugin.getDataFolder(), "worlds.yml");
+        files.put(worldConfig.getName(), this.readFile(worldConfig.getAbsolutePath()));
+        return files;
+    }
+
    @Override
    public void runCommand(final CommandSender sender, final List<String> args) {
        // Check if the command was sent from a Player.
@@ -48,71 +148,80 @@ public class VersionCommand extends MultiverseCommand {
            sender.sendMessage("Version info dumped to console. Please check your server logs.");
        }

-        StringBuilder buffer = new StringBuilder();
-        buffer.append("[Multiverse-Core] Multiverse-Core Version: ").append(this.plugin.getDescription().getVersion()).append('\n');
-        buffer.append("[Multiverse-Core] Bukkit Version: ").append(this.plugin.getServer().getVersion()).append('\n');
-        buffer.append("[Multiverse-Core] Loaded Worlds: ").append(this.plugin.getMVWorldManager().getMVWorlds()).append('\n');
-        buffer.append("[Multiverse-Core] Multiverse Plugins Loaded: ").append(this.plugin.getPluginCount()).append('\n');
-        buffer.append("[Multiverse-Core] Economy being used: ").append(plugin.getEconomist().getEconomyName()).append('\n');
-        buffer.append("[Multiverse-Core] Permissions Plugin: ").append(this.plugin.getMVPerms().getType()).append('\n');
-        buffer.append("[Multiverse-Core] Dumping Config Values: (version ")
-                .append(this.plugin.getMVConfig().getVersion()).append(")").append('\n');
-        buffer.append("[Multiverse-Core]  messagecooldown: ").append(plugin.getMessaging().getCooldown()).append('\n');
-        buffer.append("[Multiverse-Core]  teleportcooldown: ").append(plugin.getMVConfig().getTeleportCooldown()).append('\n');
-        buffer.append("[Multiverse-Core]  worldnameprefix: ").append(plugin.getMVConfig().getPrefixChat()).append('\n');
-        buffer.append("[Multiverse-Core]  worldnameprefixFormat: ").append(plugin.getMVConfig().getPrefixChatFormat()).append('\n');
-        buffer.append("[Multiverse-Core]  enforceaccess: ").append(plugin.getMVConfig().getEnforceAccess()).append('\n');
-        buffer.append("[Multiverse-Core]  displaypermerrors: ").append(plugin.getMVConfig().getDisplayPermErrors()).append('\n');
-        buffer.append("[Multiverse-Core]  teleportintercept: ").append(plugin.getMVConfig().getTeleportIntercept()).append('\n');
-        buffer.append("[Multiverse-Core]  firstspawnoverride: ").append(plugin.getMVConfig().getFirstSpawnOverride()).append('\n');
-        buffer.append("[Multiverse-Core]  firstspawnworld: ").append(plugin.getMVConfig().getFirstSpawnWorld()).append('\n');
-        buffer.append("[Multiverse-Core]  debug: ").append(plugin.getMVConfig().getGlobalDebug()).append('\n');
-        buffer.append("[Multiverse-Core] Special Code: FRN002").append('\n');
-
-        MVVersionEvent versionEvent = new MVVersionEvent(buffer.toString());
+        MVVersionEvent versionEvent = new MVVersionEvent(this.getLegacyString(), this.getVersionFiles());
+        final Map<String, String> files = this.getVersionFiles();
        this.plugin.getServer().getPluginManager().callEvent(versionEvent);

-        // log to console
-        final String data = versionEvent.getVersionInfo();
-        String[] lines = data.split("\n");
-        for (String line : lines) {
-            Logging.info(line);
+        String versionInfo = versionEvent.getVersionInfo();
+
+        if (CommandHandler.hasFlag("--include-plugin-list", args)) {
+            versionInfo = versionInfo + "\nPlugins: " + getPluginList();
        }

-        this.plugin.getServer().getScheduler().scheduleAsyncDelayedTask(this.plugin, new Runnable() {
+        final String data = versionInfo;
+
+        // log to console
+        String[] lines = data.split("\n");
+        for (String line : lines) {
+            if (!line.isEmpty()) {
+                Logging.info(line);
+            }
+        }
+
+        BukkitRunnable logPoster = new BukkitRunnable() {
            @Override
            public void run() {
-                if (args.size() == 1) {
+                if (args.size() > 0) {
                    String pasteUrl;
-                    if (args.get(0).equalsIgnoreCase("-p")) {
-                        pasteUrl = postToService(PasteServiceType.PASTIE, true, data); // private post to pastie
-                    } else if (args.get(0).equalsIgnoreCase("-b")) {
-                        pasteUrl = postToService(PasteServiceType.PASTEBIN, true, data); // private post to pastie
+                    if (CommandHandler.hasFlag("-b", args)) {
+                        // private post to pastebin
+                        pasteUrl = postToService(PasteServiceType.PASTEBIN, true, data, files);
+                    } else if (CommandHandler.hasFlag("-h", args)) {
+                        // private post to pastebin
+                        pasteUrl = postToService(PasteServiceType.HASTEBIN, true, data, files);
                    } else {
                        return;
                    }

-                    sender.sendMessage("Version info dumped here: " + ChatColor.GREEN + pasteUrl);
+                    if (!(sender instanceof ConsoleCommandSender)) {
+                        sender.sendMessage("Version info dumped here: " + ChatColor.GREEN + pasteUrl);
+                    }
                    Logging.info("Version info dumped here: %s", pasteUrl);
                }
            }
-        });
+        };
+
+        // Run the log posting operation asynchronously, since we don't know how long it will take.
+        logPoster.runTaskAsynchronously(this.plugin);
    }

    /**
     * Send the current contents of this.pasteBinBuffer to a web service.
     *
-     * @param type      Service type to send to
-     * @param isPrivate Should the paste be marked as private.
+     * @param type       Service type to send paste data to.
+     * @param isPrivate  Should the paste be marked as private.
+     * @param pasteData  Legacy string only data to post to a service.
+     * @param pasteFiles Map of filenames/contents of debug info.
     * @return URL of visible paste
     */
-    private static String postToService(PasteServiceType type, boolean isPrivate, String pasteData) {
+    private static String postToService(PasteServiceType type, boolean isPrivate, String pasteData,
+                                        Map<String, String> pasteFiles) {
        PasteService ps = PasteServiceFactory.getService(type, isPrivate);
        try {
-            return SHORTENER.shorten(ps.postData(ps.encodeData(pasteData), ps.getPostURL()));
+            String result;
+            if (ps.supportsMultiFile()) {
+                result = ps.postData(ps.encodeData(pasteFiles), ps.getPostURL());
+            } else {
+                result = ps.postData(ps.encodeData(pasteData), ps.getPostURL());
+            }
+            return SHORTENER.shorten(result);
        } catch (PasteFailedException e) {
            System.out.print(e);
            return "Error posting to service";
        }
    }
+
+    private String getPluginList() {
+        return StringUtils.join(plugin.getServer().getPluginManager().getPlugins(), ", ");
+    }
 }
@@ -2,10 +2,14 @@ package com.onarandombox.MultiverseCore.configuration;

 import java.util.Map;

+import com.onarandombox.MultiverseCore.utils.MaterialConverter;
 import me.main__.util.SerializationConfig.Property;
 import me.main__.util.SerializationConfig.SerializationConfig;

+import me.main__.util.SerializationConfig.Serializor;
+import org.bukkit.Material;
 import org.bukkit.configuration.serialization.SerializableAs;
+import org.jetbrains.annotations.Nullable;

 /**
 * Entryfee-settings.
@@ -14,8 +18,9 @@ import org.bukkit.configuration.serialization.SerializableAs;
 public class EntryFee extends SerializationConfig {
    @Property
    private double amount;
-    @Property
-    private int currency;
+    @Property(serializor = EntryFeeCurrencySerializor.class)
+    @Nullable
+    private Material currency;

    public EntryFee() {
        super();
@@ -31,7 +36,7 @@ public class EntryFee extends SerializationConfig {
    @Override
    protected void setDefaults() {
        amount = 0D;
-        currency = -1;
+        currency = null;
    }

    /**
@@ -44,7 +49,8 @@ public class EntryFee extends SerializationConfig {
    /**
     * @return the currency
     */
-    public int getCurrency() {
+    @Nullable
+    public Material getCurrency() {
        return currency;
    }

@@ -60,7 +66,19 @@ public class EntryFee extends SerializationConfig {
     * Sets the currency.
     * @param currency The new value.
     */
-    public void setCurrency(int currency) {
+    public void setCurrency(@Nullable Material currency) {
        this.currency = currency;
    }
+
+    public static final class EntryFeeCurrencySerializor implements Serializor<Material, Object> {
+        @Override
+        public String serialize(Material material) {
+            return material.toString();
+        }
+
+        @Override
+        public Material deserialize(Object o, Class<Material> aClass) {
+            return MaterialConverter.convertTypeString(o.toString());
+        }
+    }
 }
@@ -1,37 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2012.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-
-package com.onarandombox.MultiverseCore.configuration;
-
-/**
- * An "active" {@link MVConfigProperty} that uses the specified method to be "actually" set.
- * @param <T> The type of the config-property.
- * @deprecated This is deprecated.
- * @see MVConfigProperty
- */
-@Deprecated
-public interface MVActiveConfigProperty<T> extends MVConfigProperty<T> {
-    /**
-     * Gets the method that will be executed.
-     *
-     * @return The name of the method in MVWorld to be called.
-     */
-    String getMethod();
-
-    /**
-     * Sets the method that will be executed.
-     *
-     * @param methodName The name of the method in MVWorld to be called.
-     */
-    void setMethod(String methodName);
-
-    /**
-     * Returns the class of the object we're looking at.
-     * @return the class of the object we're looking at.
-     */
-    Class<?> getPropertyClass();
-}
@@ -1,69 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2011.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-
-package com.onarandombox.MultiverseCore.configuration;
-
-/**
- * A generic config-property.
- *
- * @param <T> The type of the config-property.
- * @deprecated This is deprecated.
- */
-@Deprecated
-public interface MVConfigProperty<T> {
-    /**
-     * Gets the name of this property.
-     *
-     * @return The name of this property.
-     */
-    String getName();
-
-    /**
-     * Gets the value of this property.
-     *
-     * @return The value of this property.
-     */
-    T getValue();
-
-    /**
-     * Gets the string representation of this value.
-     *
-     * @return The value of this property as a string.
-     */
-    String toString();
-
-    /**
-     * Gets the help string for this.
-     *
-     * @return The value of this property as a string.
-     */
-    String getHelp();
-
-    /**
-     * Sets the value of this property.
-     *
-     * @param value The T representation of this value.
-     * @return True the value was successfully set.
-     */
-    boolean setValue(T value);
-
-    /**
-     * This parseValue should be used with strings.
-     *
-     * @param value The string representation of the value to set.
-     *
-     * @return True if the value was set, false if not.
-     */
-    boolean parseValue(String value);
-
-    /**
-     * Gets the name of the config-node that this {@link MVConfigProperty} is saved as.
-     *
-     * @return The name of the config-node that this {@link MVConfigProperty} is saved as.
-     */
-    String getConfigNode();
-}
@@ -21,9 +21,10 @@ import java.util.List;
 * An exact {@link MVDestination}.
 */
 public class ExactDestination implements MVDestination {
-    private final String coordRegex = "(-?[\\d]+\\.?[\\d]*),(-?[\\d]+\\.?[\\d]*),(-?[\\d]+\\.?[\\d]*)";
+    private final String coordRegex = "(-?[\\d]+\\.?[\\d]*|~-?[\\d]+\\.?[\\d]*|~),(-?[\\d]+\\.?[\\d]*|~-?[\\d]+\\.?[\\d]*|~),(-?[\\d]+\\.?[\\d]*|~-?[\\d]+\\.?[\\d]*|~)";
    private boolean isValid;
    private Location location;
+    private boolean relativeX, relativeY, relativeZ;

    /**
     * {@inheritDoc}
@@ -88,7 +89,20 @@ public class ExactDestination implements MVDestination {
     */
    @Override
    public Location getLocation(Entity e) {
-        return this.location;
+        Location loc = this.location.clone();
+        if (relativeX || relativeY || relativeZ) {
+            Location eLoc = e.getLocation();
+            loc.add(relativeX ? eLoc.getX() : 0, relativeY ? eLoc.getY() : 0, relativeZ ? eLoc.getZ() : 0);
+            // Since the location is relative, it makes sense to use the entity's pitch and yaw unless those were
+            // specified in the destination.
+            if (loc.getPitch() == 0) {
+                loc.setPitch(eLoc.getPitch());
+            }
+            if (loc.getYaw() == 0) {
+                loc.setYaw(eLoc.getYaw());
+            }
+        }
+        return loc;
    }

    /**
@@ -134,11 +148,42 @@ public class ExactDestination implements MVDestination {
        double[] coords = new double[3];
        String[] coordString = parsed.get(2).split(",");
        for (int i = 0; i < 3; i++) {
-            try {
-                coords[i] = Double.parseDouble(coordString[i]);
-            } catch (NumberFormatException e) {
-                this.isValid = false;
-                return;
+            String[] relSplit = coordString[i].split("~");
+            boolean relative = false;
+            if (relSplit.length == 0) {
+                // coord is "~" form
+                relative = true;
+                coords[i] = 0;
+            } else if (relSplit.length == 1) {
+                // coord is "123" form
+                try {
+                    coords[i] = Double.parseDouble(relSplit[0]);
+                } catch (NumberFormatException e) {
+                    this.isValid = false;
+                    return;
+                }
+            } else {
+                // coord is "~123" form
+                relative = true;
+                try {
+                    coords[i] = Double.parseDouble(relSplit[1]);
+                } catch (NumberFormatException e) {
+                    this.isValid = false;
+                    return;
+                }
+            }
+            if (relative) {
+                switch (i) {
+                    case 0:
+                        relativeX = true;
+                        break;
+                    case 1:
+                        relativeY = true;
+                        break;
+                    case 2:
+                        relativeZ = true;
+                        break;
+                }
            }
        }
        this.location.setX(coords[0]);
@@ -0,0 +1,43 @@
+package com.onarandombox.MultiverseCore.event;
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+/**
+ * Called when Core's debug level is changed.
+ */
+public class MVDebugModeEvent extends Event {
+
+    private static final HandlerList HANDLERS = new HandlerList();
+
+    private final int level;
+
+    public MVDebugModeEvent(int level) {
+        this.level = level;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public HandlerList getHandlers() {
+        return HANDLERS;
+    }
+
+    /**
+     * Gets the handler list. This is required by the event system.
+     * @return A list of HANDLERS.
+     */
+    public static HandlerList getHandlerList() {
+        return HANDLERS;
+    }
+
+    /**
+     * Returns the current debug level of Core.
+     *
+     * @return the current debug level of Core.
+     */
+    public int getLevel() {
+        return level;
+    }
+}
@@ -3,15 +3,19 @@ package com.onarandombox.MultiverseCore.event;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;

+import java.util.Map;
+
 /**
 * Called when somebody requests version information about Multiverse.
 */
 public class MVVersionEvent extends Event {

    private final StringBuilder versionInfoBuilder;
+    private final Map<String, String> detailedVersionInfo;

-    public MVVersionEvent(String versionInfo) {
-        this.versionInfoBuilder = new StringBuilder(versionInfo);
+    public MVVersionEvent(String legacyVersionInfo, Map<String, String> files) {
+        this.versionInfoBuilder = new StringBuilder(legacyVersionInfo);
+        this.detailedVersionInfo = files;
    }

    private static final HandlerList HANDLERS = new HandlerList();
@@ -40,6 +44,21 @@ public class MVVersionEvent extends Event {
        return this.versionInfoBuilder.toString();
    }

+    /**
+     * Gets the key/value pair of the detailed version info.
+     *
+     * This information is used for advanced paste services that would prefer
+     * to get the information as several files. Examples include config.yml or
+     * portals.yml.
+     *
+     * The keys are filenames, the values are the contents of the files.
+     *
+     * @return The key value mapping of files and the contents of those files.
+     */
+    public Map<String, String> getDetailedVersionInfo() {
+        return this.detailedVersionInfo;
+    }
+
    /**
     * Appends more version-info to the version-info currently saved in this event.
     * @param moreVersionInfo The version-info to add. Should end with '\n'.
@@ -7,6 +7,10 @@

 package com.onarandombox.MultiverseCore.listeners;

+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
+import java.util.logging.Level;
+
 import com.dumptruckman.minecraft.util.Logging;
 import com.onarandombox.MultiverseCore.MultiverseCore;
 import com.onarandombox.MultiverseCore.api.MVWorldManager;
@@ -29,10 +33,6 @@ import org.bukkit.event.player.PlayerQuitEvent;
 import org.bukkit.event.player.PlayerRespawnEvent;
 import org.bukkit.event.player.PlayerTeleportEvent;

-import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
-import java.util.logging.Level;
-
 /**
 * Multiverse's {@link Listener} for players.
 */
@@ -304,8 +304,16 @@ public class MVPlayerListener implements Listener {
                    + "' was allowed to go to '" + event.getTo().getWorld().getName()
                    + "' because enforceaccess is off.");
        }
-        if (!plugin.getMVConfig().isUsingDefaultPortalSearch() && event.getPortalTravelAgent() != null) {
-            event.getPortalTravelAgent().setSearchRadius(plugin.getMVConfig().getPortalSearchRadius());
+        if (!plugin.getMVConfig().isUsingDefaultPortalSearch()) {
+            try {
+                Class.forName("org.bukkit.TravelAgent");
+                if (event.getPortalTravelAgent() != null) {
+                    event.getPortalTravelAgent().setSearchRadius(plugin.getMVConfig().getPortalSearchRadius());
+                }
+            } catch (ClassNotFoundException ignore) {
+                plugin.log(Level.FINE, "TravelAgent not available for PlayerPortalEvent for " + event.getPlayer().getName());
+            }
+
        }
    }

@@ -140,8 +140,17 @@ public class AnchorManager {
            if (ancLoc == null) {
                continue;
            }
-            if (p.hasPermission("multiverse.access." + ancLoc.getWorld().getName())) {
+            String worldPerm = "multiverse.access." + ancLoc.getWorld().getName();
+            // Add to the list if we're not enforcing access
+            // OR
+            // We are enforcing access and the user has the permission.
+            if (!this.plugin.getMVConfig().getEnforceAccess() ||
+                    (this.plugin.getMVConfig().getEnforceAccess() && p.hasPermission(worldPerm))) {
                myAnchors.add(anchor);
+            } else {
+                Logging.finer(String.format("Not adding anchor %s to the list, user %s doesn't have the %s " +
+                        "permission and 'enforceaccess' is enabled!",
+                        anchor, p.getName(), worldPerm));
            }
        }
        return Collections.unmodifiableSet(myAnchors);
@@ -1,281 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2011.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-
-package com.onarandombox.MultiverseCore.utils;
-
-import com.dumptruckman.minecraft.util.Logging;
-import org.bukkit.Location;
-import org.bukkit.Material;
-import org.bukkit.World;
-import org.bukkit.entity.Minecart;
-import org.bukkit.entity.Vehicle;
-
-/**
- * Used to determine block/location-related facts.
- *
- * @deprecated Use instead: {@link com.onarandombox.MultiverseCore.api.BlockSafety} and {@link SimpleBlockSafety}.
- */
-@Deprecated
-public class BlockSafety {
-
-    /**
-     * This function checks whether the block at the given coordinates are above air or not.
-     * @param l The {@link Location} of the block.
-     * @return True if the block at that {@link Location} is above air.
-     */
-    public boolean isBlockAboveAir(Location l) {
-        Location downOne = l.clone();
-        downOne.setY(downOne.getY() - 1);
-        return (downOne.getBlock().getType() == Material.AIR);
-    }
-
-    // TODO maybe remove this?
-    private boolean blockIsNotSafe(World world, double x, double y, double z) {
-        return !playerCanSpawnHereSafely(world, x, y, z);
-    }
-
-    /**
-     * Checks if a player can spawn safely at the given coordinates.
-     * @param world The {@link World}.
-     * @param x The x-coordinate.
-     * @param y The y-coordinate.
-     * @param z The z-coordinate.
-     * @return True if a player can spawn safely at the given coordinates.
-     */
-    public boolean playerCanSpawnHereSafely(World world, double x, double y, double z) {
-        Location l = new Location(world, x, y, z);
-        return playerCanSpawnHereSafely(l);
-    }
-
-    /**
-     * This function checks whether the block at the coordinates given is safe or not by checking for Lava/Fire/Air
-     * etc. This also ensures there is enough space for a player to spawn!
-     *
-     * @param l The {@link Location}
-     * @return Whether the player can spawn safely at the given {@link Location}
-     */
-    public boolean playerCanSpawnHereSafely(Location l) {
-        if (l == null) {
-            // Can't safely spawn at a null location!
-            return false;
-        }
-
-        World world = l.getWorld();
-        Location actual = l.clone();
-        Location upOne = l.clone();
-        Location downOne = l.clone();
-        upOne.setY(upOne.getY() + 1);
-        downOne.setY(downOne.getY() - 1);
-
-        if (this.isSolidBlock(world.getBlockAt(actual).getType())
-                || this.isSolidBlock(upOne.getBlock().getType())) {
-            Logging.finest("Error Here (Actual)? (%s)[%s]", actual.getBlock().getType(),
-                    this.isSolidBlock(actual.getBlock().getType()));
-            Logging.finest("Error Here (upOne)? (%s)[%s]", upOne.getBlock().getType(),
-                    this.isSolidBlock(upOne.getBlock().getType()));
-            return false;
-        }
-
-        if (downOne.getBlock().getType() == Material.LAVA || downOne.getBlock().getType() == Material.STATIONARY_LAVA) {
-            Logging.finest("Error Here (downOne)? (%s)[%s]", downOne.getBlock().getType(),
-                    this.isSolidBlock(downOne.getBlock().getType()));
-            return false;
-        }
-
-        if (downOne.getBlock().getType() == Material.FIRE) {
-            Logging.finest("There's fire below! (%s)[%s]", actual.getBlock().getType(),
-                    this.isSolidBlock(actual.getBlock().getType()));
-            return false;
-        }
-
-        if (isBlockAboveAir(actual)) {
-            Logging.finest("Is block above air [%s]", isBlockAboveAir(actual));
-            Logging.finest("Has 2 blocks of water below [%s]", this.hasTwoBlocksofWaterBelow(actual));
-            return this.hasTwoBlocksofWaterBelow(actual);
-        }
-        return true;
-    }
-
-    /**
-     * Gets the location of the top block at the specified {@link Location}.
-     * @param l Any {@link Location}.
-     * @return The {@link Location} of the top-block.
-     */
-    public Location getTopBlock(Location l) {
-        Location check = l.clone();
-        check.setY(127); // SUPPRESS CHECKSTYLE: MagicNumberCheck
-        while (check.getY() > 0) {
-            if (this.playerCanSpawnHereSafely(check)) {
-                return check;
-            }
-            check.setY(check.getY() - 1);
-        }
-        return null;
-    }
-
-    /**
-     * Gets the location of the top block at the specified {@link Location}.
-     * @param l Any {@link Location}.
-     * @return The {@link Location} of the top-block.
-     */
-    public Location getBottomBlock(Location l) {
-        Location check = l.clone();
-        check.setY(0);
-        while (check.getY() < 127) { // SUPPRESS CHECKSTYLE: MagicNumberCheck
-            if (this.playerCanSpawnHereSafely(check)) {
-                return check;
-            }
-            check.setY(check.getY() + 1);
-        }
-        return null;
-    }
-
-    /*
-     * If someone has a better way of this... Please either tell us, or submit a pull request!
-     */
-    private boolean isSolidBlock(Material type) {
-        switch (type) {
-            case AIR:
-                return false;
-            case SNOW:
-                return false;
-            case TRAP_DOOR:
-                return false;
-            case TORCH:
-                return false;
-            case YELLOW_FLOWER:
-                return false;
-            case RED_ROSE:
-                return false;
-            case RED_MUSHROOM:
-                return false;
-            case BROWN_MUSHROOM:
-                return false;
-            case REDSTONE:
-                return false;
-            case REDSTONE_WIRE:
-                return false;
-            case RAILS:
-                return false;
-            case POWERED_RAIL:
-                return false;
-            case REDSTONE_TORCH_ON:
-                return false;
-            case REDSTONE_TORCH_OFF:
-                return false;
-            case DEAD_BUSH:
-                return false;
-            case SAPLING:
-                return false;
-            case STONE_BUTTON:
-                return false;
-            case LEVER:
-                return false;
-            case LONG_GRASS:
-                return false;
-            case PORTAL:
-                return false;
-            case STONE_PLATE:
-                return false;
-            case WOOD_PLATE:
-                return false;
-            case SEEDS:
-                return false;
-            case SUGAR_CANE_BLOCK:
-                return false;
-            case WALL_SIGN:
-                return false;
-            case SIGN_POST:
-                return false;
-            case WOODEN_DOOR:
-                return false;
-            case STATIONARY_WATER:
-                return false;
-            case WATER:
-                return false;
-            default:
-                return true;
-        }
-    }
-
-    /**
-     * Checks if an entity would be on track at the specified {@link Location}.
-     * @param l The {@link Location}.
-     * @return True if an entity would be on tracks at the specified {@link Location}.
-     */
-    public boolean isEntitiyOnTrack(Location l) {
-        Material currentBlock = l.getBlock().getType();
-        return (currentBlock == Material.POWERED_RAIL || currentBlock == Material.DETECTOR_RAIL || currentBlock == Material.RAILS);
-    }
-
-    // TODO maybe remove this?
-    private void showDangers(Location l) {
-        Location actual = new Location(l.getWorld(), l.getX(), l.getY(), l.getZ());
-        Location upOne = new Location(l.getWorld(), l.getX(), l.getY(), l.getZ());
-        Location downOne = new Location(l.getWorld(), l.getX(), l.getY(), l.getZ());
-        upOne.setY(upOne.getY() + 1);
-        downOne.setY(downOne.getY() - 1);
-
-        System.out.print("Location Up:   " + upOne.getBlock().getType());
-        System.out.print("               " + upOne);
-        System.out.print("Location:      " + actual.getBlock().getType());
-        System.out.print("               " + actual);
-        System.out.print("Location Down: " + downOne.getBlock().getType());
-        System.out.print("               " + downOne);
-    }
-
-    /**
-     * Checks recursively below a {@link Location} for 2 blocks of water.
-     *
-     * @param l The {@link Location}
-     * @return Whether there are 2 blocks of water
-     */
-    private boolean hasTwoBlocksofWaterBelow(Location l) {
-        if (l.getBlockY() < 0) {
-            return false;
-        }
-        Location oneBelow = l.clone();
-        oneBelow.subtract(0, 1, 0);
-        if (oneBelow.getBlock().getType() == Material.WATER || oneBelow.getBlock().getType() == Material.STATIONARY_WATER) {
-            Location twoBelow = oneBelow.clone();
-            twoBelow.subtract(0, 1, 0);
-            return (oneBelow.getBlock().getType() == Material.WATER || oneBelow.getBlock().getType() == Material.STATIONARY_WATER);
-        }
-        if (oneBelow.getBlock().getType() != Material.AIR) {
-            return false;
-        }
-        return hasTwoBlocksofWaterBelow(oneBelow);
-    }
-
-    /**
-     * Checks if the specified {@link Minecart} can spawn safely.
-     * @param cart The {@link Minecart}.
-     * @return True if the minecart can spawn safely.
-     */
-    public boolean canSpawnCartSafely(Minecart cart) {
-        if (this.isBlockAboveAir(cart.getLocation())) {
-            return true;
-        }
-        if (this.isEntitiyOnTrack(LocationManipulation.getNextBlock(cart))) {
-            return true;
-        }
-        return false;
-    }
-
-    /**
-     * Checks if the specified {@link Vehicle} can spawn safely.
-     * @param vehicle The {@link Vehicle}.
-     * @return True if the vehicle can spawn safely.
-     */
-    public boolean canSpawnVehicleSafely(Vehicle vehicle) {
-        if (this.isBlockAboveAir(vehicle.getLocation())) {
-            return true;
-        }
-        return false;
-    }
-
-}
@@ -0,0 +1,110 @@
+package com.onarandombox.MultiverseCore.utils;
+
+import java.util.logging.Level;
+
+import com.onarandombox.MultiverseCore.api.SafeTTeleporter;
+import com.onarandombox.MultiverseCore.destination.CannonDestination;
+import org.bukkit.Location;
+import org.bukkit.TravelAgent;
+import org.bukkit.event.player.PlayerPortalEvent;
+
+public class BukkitTravelAgent implements TravelAgent {
+    private final MVTravelAgent agent;
+
+    public BukkitTravelAgent(MVTravelAgent agent) {
+        this.agent = agent;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public BukkitTravelAgent setSearchRadius(int radius) {
+        return this;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public int getSearchRadius() {
+        return 0;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public BukkitTravelAgent setCreationRadius(int radius) {
+        return this;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public int getCreationRadius() {
+        return 0;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public boolean getCanCreatePortal() {
+        return false;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public void setCanCreatePortal(boolean create) {
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public Location findOrCreate(Location location) {
+        return this.getSafeLocation();
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public Location findPortal(Location location) {
+        return this.getSafeLocation();
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public boolean createPortal(Location location) {
+        return false;
+    }
+
+    private Location getSafeLocation() {
+        // At this time, these can never use the velocity.
+        if (agent.destination instanceof CannonDestination) {
+            agent.core.log(Level.FINE, "Using Stock TP method. This cannon will have 0 velocity");
+        }
+        SafeTTeleporter teleporter = agent.core.getSafeTTeleporter();
+        Location newLoc = agent.destination.getLocation(agent.player);
+        if (agent.destination.useSafeTeleporter()) {
+            newLoc = teleporter.getSafeLocation(agent.player, agent.destination);
+        }
+        if (newLoc == null) {
+            return agent.player.getLocation();
+        }
+        return newLoc;
+
+    }
+
+    public void setPortalEventTravelAgent(PlayerPortalEvent event) {
+        event.setPortalTravelAgent(this);
+        event.useTravelAgent(true);
+    }
+}
@@ -22,9 +22,9 @@ public class FancyMessage implements FancyText {
     * Allows easy creation of an alternating colored list.
     * TODO: Documentation! Why does CheckStyle just ignore this?
     *
-     * @param title
-     * @param message
-     * @param scheme
+     * @param title The title.
+     * @param message The body of the message.
+     * @param scheme The color scheme to use for easy styling.
     */
    public FancyMessage(String title, String message, FancyColorScheme scheme) {
        this.title = title;
@@ -7,11 +7,21 @@

 package com.onarandombox.MultiverseCore.utils;

-import com.dumptruckman.minecraft.util.Logging;
+import static java.nio.file.StandardCopyOption.COPY_ATTRIBUTES;

 import java.io.File;
 import java.io.IOException;
+import java.nio.file.FileVisitResult;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.SimpleFileVisitor;
+import java.nio.file.attribute.BasicFileAttributes;
+import java.util.Comparator;
+import java.util.logging.Level;
 import java.util.logging.Logger;
+import java.util.stream.Stream;
+
+import com.dumptruckman.minecraft.util.Logging;

 /**
 * File-utilities.
@@ -28,8 +38,8 @@ public class FileUtils {
     * @return true if the folder was successfully deleted.
     */
    public static boolean deleteFolder(File file) {
-        try {
-            org.apache.commons.io.FileUtils.deleteDirectory(file);
+        try (Stream<Path> files = Files.walk(file.toPath())) {
+            files.sorted(Comparator.reverseOrder()).map(Path::toFile).forEach(File::delete);
            return true;
        } catch (IOException e) {
            Logging.warning(e.getMessage());
@@ -44,8 +54,11 @@ public class FileUtils {
     * @return true if the contents were successfully deleted
     */
    public static boolean deleteFolderContents(File file) {
-        try {
-            org.apache.commons.io.FileUtils.cleanDirectory(file);
+        try (Stream<Path> files = Files.walk(file.toPath())){
+            files.sorted(Comparator.reverseOrder())
+                    .map(Path::toFile)
+                    .filter(f -> !f.equals(file))
+                    .forEach(File::delete);
            return true;
        } catch (IOException e) {
            Logging.warning(e.getMessage());
@@ -53,8 +66,6 @@ public class FileUtils {
        }
    }

-    private static final int COPY_BLOCK_SIZE = 1024;
-
    /**
     * Helper method to copy the world-folder.
     * @param source Source-File
@@ -64,12 +75,42 @@ public class FileUtils {
     * @return if it had success
     */
    public static boolean copyFolder(File source, File target, Logger log) {
+        Path sourceDir = source.toPath();
+        Path targetDir = target.toPath();
+
        try {
-            org.apache.commons.io.FileUtils.copyDirectory(source, target);
+            Files.walkFileTree(sourceDir, new CopyDirFileVisitor(sourceDir, targetDir));
            return true;
        } catch (IOException e) {
-            log.warning(e.getMessage());
+            log.log(Level.WARNING, "Unable to copy directory", e);
            return false;
        }
    }
+
+    private static class CopyDirFileVisitor extends SimpleFileVisitor<Path> {
+
+        private final Path sourceDir;
+        private final Path targetDir;
+
+        private CopyDirFileVisitor(Path sourceDir, Path targetDir) {
+            this.sourceDir = sourceDir;
+            this.targetDir = targetDir;
+        }
+
+        @Override
+        public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException {
+            Path newDir = targetDir.resolve(sourceDir.relativize(dir));
+            if (!Files.isDirectory(newDir)) {
+                Files.createDirectory(newDir);
+            }
+            return FileVisitResult.CONTINUE;
+        }
+
+        @Override
+        public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
+            Path targetFile = targetDir.resolve(sourceDir.relativize(file));
+            Files.copy(file, targetFile, COPY_ATTRIBUTES);
+            return FileVisitResult.CONTINUE;
+        }
+    }
 }
@@ -1,284 +1,285 @@
-package com.onarandombox.MultiverseCore.utils;
-
-import org.bukkit.ChatColor;
-import org.bukkit.Material;
-import org.bukkit.World;
-import org.bukkit.entity.Player;
-import org.bukkit.inventory.ItemStack;
-import org.bukkit.plugin.Plugin;
-
-import java.util.HashMap;
-
-/**
- * Multiverse's Friendly Economist. This is used to deal with external economies and also item costs for stuff in MV.
- */
-public class MVEconomist {
-
-    private final VaultHandler vaultHandler;
-
-    public MVEconomist(Plugin plugin) {
-        vaultHandler = new VaultHandler(plugin);
-    }
-
-    private boolean isUsingVault(int currency) {
-        return !isItemCurrency(currency) && getVaultHandler().hasEconomy();
-    }
-
-    /**
-     * Checks if an economy plugin is in use.
-     *
-     * @return true if an economy plugin is detected by Vault.
-     */
-    public boolean isUsingEconomyPlugin() {
-        return getVaultHandler().hasEconomy();
-    }
-
-    /**
-     * Formats the amount to a human readable currency string.
-     *
-     * @param amount the amount of currency.
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     * @return the human readable currency string.
-     */
-    public String formatPrice(double amount, int currency) {
-        if (isUsingVault(currency)) {
-            return getVaultHandler().getEconomy().format(amount);
-        } else {
-            return ItemEconomy.getFormattedPrice(amount, currency);
-        }
-    }
-
-    /**
-     * Returns the name of the economy in use.
-     *
-     * @return the name of the economy in use.
-     */
-    public String getEconomyName() {
-        if (getVaultHandler().hasEconomy()) {
-            return getVaultHandler().getEconomy().getName();
-        } else {
-            return ItemEconomy.getName();
-        }
-    }
-
-    /**
-     * Determines if a player has enough of a given currency.
-     * @param player the player to check for currency.
-     * @param amount the amount of currency.
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     * @return true if the player has enough of the given currency or the amount is 0 or less.
-     */
-    public boolean isPlayerWealthyEnough(Player player, double amount, int currency) {
-        if (amount <= 0D) {
-            return true;
-        } else if (isUsingVault(currency)) {
-            return getVaultHandler().getEconomy().has(player, amount);
-        } else {
-            return ItemEconomy.hasEnough(player, amount, currency);
-        }
-    }
-
-    /**
-     * Formats a message for a player indicating they don't have enough currency.
-     *
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     * @param message The more specific message to append to the generic message of not having enough.
-     * @return the formatted insufficient funds message.
-     */
-    public String getNSFMessage(int currency, String message) {
-        return "Sorry, you don't have enough " + (isItemCurrency(currency) ? "items" : "funds") + ". " + message;
-    }
-
-    /**
-     * Deposits a given amount of currency either into the player's economy account or inventory if the currency
-     * represents an item.
-     *
-     * @param player the player to give currency to.
-     * @param amount the amount to give.
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     */
-    public void deposit(Player player, double amount, int currency) {
-        if (isUsingVault(currency)) {
-            getVaultHandler().getEconomy().depositPlayer(player, amount);
-        } else {
-            ItemEconomy.deposit(player, amount, currency);
-        }
-    }
-
-    /**
-     * Withdraws a given amount of currency either from the player's economy account or inventory if the currency
-     * represents an item.
-     *
-     * @param player the player to take currency from.
-     * @param amount the amount to take.
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     */
-    public void withdraw(Player player, double amount, int currency) {
-        if (isUsingVault(currency)) {
-            getVaultHandler().getEconomy().withdrawPlayer(player, amount);
-        } else {
-            ItemEconomy.withdraw(player, amount, currency);
-        }
-    }
-
-    /**
-     * Returns the economy balance of the given player.
-     *
-     * @param player the player to get the balance for.
-     * @return the economy balance of the given player.
-     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
-     */
-    public double getBalance(Player player) throws IllegalStateException {
-        return getBalance(player, null);
-    }
-
-    /**
-     * Returns the economy balance of the given player in the given world. If the economy plugin does not have world
-     * specific balances then the global balance will be returned.
-     *
-     * @param player the player to get the balance for.
-     * @param world the world to get the balance for.
-     * @return the economy balance of the given player in the given world.
-     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
-     */
-    public double getBalance(Player player, World world) throws IllegalStateException {
-        if (!isUsingEconomyPlugin()) {
-            throw new IllegalStateException("getBalance is only available when using an economy plugin with Vault");
-        }
-        if (world != null) {
-            return getVaultHandler().getEconomy().getBalance(player, world.getName());
-        } else {
-            return getVaultHandler().getEconomy().getBalance(player);
-        }
-    }
-
-    /**
-     * Sets the economy balance for the given player.
-     *
-     * @param player the player to set the balance for.
-     * @param amount the amount to set the player's balance to.
-     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
-     */
-    public void setBalance(Player player, double amount) throws IllegalStateException {
-        setBalance(player, null, amount);
-    }
-
-    /**
-     * Sets the economy balance for the given player in the given world. If the economy plugin does not have world
-     * specific balances then the global balance will be set.
-     *
-     * @param player the player to set the balance for.
-     * @param world the world to get the balance for.
-     * @param amount the amount to set the player's balance to.
-     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
-     */
-    public void setBalance(Player player, World world, double amount) throws IllegalStateException {
-        if (!isUsingEconomyPlugin()) {
-            throw new IllegalStateException("getBalance is only available when using an economy plugin with Vault");
-        }
-        if (world != null) {
-            getVaultHandler().getEconomy().withdrawPlayer(player, world.getName(), getBalance(player, world));
-            getVaultHandler().getEconomy().depositPlayer(player, world.getName(), amount);
-        } else {
-            getVaultHandler().getEconomy().withdrawPlayer(player, getBalance(player));
-            getVaultHandler().getEconomy().depositPlayer(player, amount);
-        }
-    }
-
-    /**
-     * This method is public for backwards compatibility.
-     *
-     * @return the old VaultHandler.
-     * @deprecated just use the other methods in this class for economy stuff.
-     */
-    // TODO make private
-    @Deprecated
-    public VaultHandler getVaultHandler() {
-        return vaultHandler;
-    }
-
-    /**
-     * Determines if the currency type given represents an item currency.
-     *
-     * @param currency the type of currency. A value greater than -1 indicates the material type used for currency.
-     * @return true if currency is greater than -1.
-     */
-    public static boolean isItemCurrency(int currency) {
-        return currency >= 0;
-    }
-
-    private static class ItemEconomy {
-
-        private static final String ECONOMY_NAME = "Simple Item Economy";
-
-        private static String getFormattedPrice(double amount, int currency) {
-            if (isItemCurrency(currency)) {
-                Material m = Material.getMaterial(currency);
-                return m != null ? amount + " " + m.toString() : "NO ITEM FOUND";
-            } else {
-                return "";
-            }
-        }
-
-        private static String getName() {
-            return ECONOMY_NAME;
-        }
-
-        private static boolean hasEnough(Player player, double amount, int currency) {
-            if (isItemCurrency(currency)) {
-                return player.getInventory().contains(currency, (int) amount);
-            } else {
-                return true;
-            }
-        }
-
-        private static void deposit(Player player, double amount, int currency) {
-            if (isItemCurrency(currency)) {
-                giveItem(player, amount, currency);
-            }
-        }
-
-        private static void withdraw(Player player, double amount, int currency) {
-            if (isItemCurrency(currency)) {
-                takeItem(player, amount, currency);
-            }
-        }
-
-        private static void giveItem(Player player, double amount, int type) {
-            ItemStack item = new ItemStack(type, (int) amount);
-            player.getInventory().addItem(item);
-            showReceipt(player, (amount * -1), type);
-        }
-
-        private static void takeItem(Player player, double amount, int type) {
-            int removed = 0;
-            HashMap<Integer, ItemStack> items = (HashMap<Integer, ItemStack>) player.getInventory().all(type);
-            for (int i : items.keySet()) {
-                if (removed >= amount) {
-                    break;
-                }
-                int diff = (int) (amount - removed);
-                int amt = player.getInventory().getItem(i).getAmount();
-                if (amt - diff > 0) {
-                    player.getInventory().getItem(i).setAmount(amt - diff);
-                    break;
-                } else {
-                    removed += amt;
-                    player.getInventory().clear(i);
-                }
-            }
-            showReceipt(player, amount, type);
-        }
-
-        private static void showReceipt(Player player, double price, int item) {
-            if (price > 0D) {
-                player.sendMessage(String.format("%s%s%s %s",
-                        ChatColor.WHITE, "You have been charged", ChatColor.GREEN, getFormattedPrice(price, item)));
-            } else if (price < 0D) {
-                player.sendMessage(String.format("%s%s%s %s",
-                        ChatColor.DARK_GREEN, getFormattedPrice((price * -1), item),
-                        ChatColor.WHITE, "has been added to your inventory."));
-            }
-        }
-    }
-}
+package com.onarandombox.MultiverseCore.utils;
+
+import org.bukkit.ChatColor;
+import org.bukkit.Material;
+import org.bukkit.World;
+import org.bukkit.entity.Player;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.plugin.Plugin;
+import org.jetbrains.annotations.Nullable;
+
+import java.util.HashMap;
+
+/**
+ * Multiverse's Friendly Economist. This is used to deal with external economies and also item costs for stuff in MV.
+ */
+public class MVEconomist {
+
+    private final VaultHandler vaultHandler;
+
+    public MVEconomist(Plugin plugin) {
+        vaultHandler = new VaultHandler(plugin);
+    }
+
+    private boolean isUsingVault(Material currency) {
+        return !isItemCurrency(currency) && getVaultHandler().hasEconomy();
+    }
+
+    /**
+     * Checks if an economy plugin is in use.
+     *
+     * @return true if an economy plugin is detected by Vault.
+     */
+    public boolean isUsingEconomyPlugin() {
+        return getVaultHandler().hasEconomy();
+    }
+
+    /**
+     * Formats the amount to a human readable currency string.
+     *
+     * @param amount the amount of currency.
+     * @param currency the type of currency. Null indicates a non-item currency is used.
+     * @return the human readable currency string.
+     */
+    public String formatPrice(double amount, @Nullable Material currency) {
+        if (isUsingVault(currency)) {
+            return getVaultHandler().getEconomy().format(amount);
+        } else {
+            return ItemEconomy.getFormattedPrice(amount, currency);
+        }
+    }
+
+    /**
+     * Returns the name of the economy in use.
+     *
+     * @return the name of the economy in use.
+     */
+    public String getEconomyName() {
+        if (getVaultHandler().hasEconomy()) {
+            return getVaultHandler().getEconomy().getName();
+        } else {
+            return ItemEconomy.getName();
+        }
+    }
+
+    /**
+     * Determines if a player has enough of a given currency.
+     *
+     * @param player the player to check for currency.
+     * @param amount the amount of currency.
+     * @param currency the type of currency. Null indicates non-item currency is used.
+     * @return true if the player has enough of the given currency or the amount is 0 or less.
+     */
+    public boolean isPlayerWealthyEnough(Player player, double amount, Material currency) {
+        if (amount <= 0D) {
+            return true;
+        } else if (isUsingVault(currency)) {
+            return getVaultHandler().getEconomy().has(player, amount);
+        } else {
+            return ItemEconomy.hasEnough(player, amount, currency);
+        }
+    }
+
+    /**
+     * Formats a message for a player indicating they don't have enough currency.
+     *
+     * @param currency the type of currency. Null indicates a non-item currency is used.
+     * @param message The more specific message to append to the generic message of not having enough.
+     * @return the formatted insufficient funds message.
+     */
+    public String getNSFMessage(Material currency, String message) {
+        return "Sorry, you don't have enough " + (isItemCurrency(currency) ? "items" : "funds") + ". " + message;
+    }
+
+    /**
+     * Deposits a given amount of currency either into the player's economy account or inventory if the currency
+     * is not null.
+     *
+     * @param player the player to give currency to.
+     * @param amount the amount to give.
+     * @param currency the type of currency.
+     */
+    public void deposit(Player player, double amount, @Nullable Material currency) {
+        if (isUsingVault(currency)) {
+            getVaultHandler().getEconomy().depositPlayer(player, amount);
+        } else {
+            ItemEconomy.deposit(player, amount, currency);
+        }
+    }
+
+    /**
+     * Withdraws a given amount of currency either from the player's economy account or inventory if the currency
+     * is not null.
+     *
+     * @param player the player to take currency from.
+     * @param amount the amount to take.
+     * @param currency the type of currency.
+     */
+    public void withdraw(Player player, double amount, @Nullable Material currency) {
+        if (isUsingVault(currency)) {
+            getVaultHandler().getEconomy().withdrawPlayer(player, amount);
+        } else {
+            ItemEconomy.withdraw(player, amount, currency);
+        }
+    }
+
+    /**
+     * Returns the economy balance of the given player.
+     *
+     * @param player the player to get the balance for.
+     * @return the economy balance of the given player.
+     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
+     */
+    public double getBalance(Player player) throws IllegalStateException {
+        return getBalance(player, null);
+    }
+
+    /**
+     * Returns the economy balance of the given player in the given world. If the economy plugin does not have world
+     * specific balances then the global balance will be returned.
+     *
+     * @param player the player to get the balance for.
+     * @param world the world to get the balance for.
+     * @return the economy balance of the given player in the given world.
+     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
+     */
+    public double getBalance(Player player, World world) throws IllegalStateException {
+        if (!isUsingEconomyPlugin()) {
+            throw new IllegalStateException("getBalance is only available when using an economy plugin with Vault");
+        }
+        if (world != null) {
+            return getVaultHandler().getEconomy().getBalance(player, world.getName());
+        } else {
+            return getVaultHandler().getEconomy().getBalance(player);
+        }
+    }
+
+    /**
+     * Sets the economy balance for the given player.
+     *
+     * @param player the player to set the balance for.
+     * @param amount the amount to set the player's balance to.
+     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
+     */
+    public void setBalance(Player player, double amount) throws IllegalStateException {
+        setBalance(player, null, amount);
+    }
+
+    /**
+     * Sets the economy balance for the given player in the given world. If the economy plugin does not have world
+     * specific balances then the global balance will be set.
+     *
+     * @param player the player to set the balance for.
+     * @param world the world to get the balance for.
+     * @param amount the amount to set the player's balance to.
+     * @throws IllegalStateException thrown if this is used when no economy plugin is available.
+     */
+    public void setBalance(Player player, World world, double amount) throws IllegalStateException {
+        if (!isUsingEconomyPlugin()) {
+            throw new IllegalStateException("getBalance is only available when using an economy plugin with Vault");
+        }
+        if (world != null) {
+            getVaultHandler().getEconomy().withdrawPlayer(player, world.getName(), getBalance(player, world));
+            getVaultHandler().getEconomy().depositPlayer(player, world.getName(), amount);
+        } else {
+            getVaultHandler().getEconomy().withdrawPlayer(player, getBalance(player));
+            getVaultHandler().getEconomy().depositPlayer(player, amount);
+        }
+    }
+
+    /**
+     * This method is public for backwards compatibility.
+     *
+     * @return the old VaultHandler.
+     * @deprecated just use the other methods in this class for economy stuff.
+     */
+    // TODO make private
+    @Deprecated
+    public VaultHandler getVaultHandler() {
+        return vaultHandler;
+    }
+
+    /**
+     * Determines if the currency type string given represents an item currency.
+     *
+     * @param currency the type of currency.
+     * @return true if currency string matches a valid material.
+     */
+    public static boolean isItemCurrency(Material currency) {
+        return currency != null;
+    }
+
+    private static class ItemEconomy {
+
+        private static final String ECONOMY_NAME = "Simple Item Economy";
+
+        private static String getFormattedPrice(double amount, Material currency) {
+            if (isItemCurrency(currency)) {
+                return amount + " " + currency.toString();
+            } else {
+                return "";
+            }
+        }
+
+        private static String getName() {
+            return ECONOMY_NAME;
+        }
+
+        private static boolean hasEnough(Player player, double amount, Material currency) {
+            if (currency != null) {
+                return player.getInventory().contains(currency, (int) amount);
+            } else {
+                return true;
+            }
+        }
+
+        private static void deposit(Player player, double amount, Material currency) {
+            if (isItemCurrency(currency)) {
+                giveItem(player, amount, currency);
+            }
+        }
+
+        private static void withdraw(Player player, double amount, Material currency) {
+            if (isItemCurrency(currency)) {
+                takeItem(player, amount, currency);
+            }
+        }
+
+        private static void giveItem(Player player, double amount, Material type) {
+            ItemStack item = new ItemStack(type, (int) amount);
+            player.getInventory().addItem(item);
+            showReceipt(player, (amount * -1), type);
+        }
+
+        private static void takeItem(Player player, double amount, Material type) {
+            int removed = 0;
+            HashMap<Integer, ItemStack> items = (HashMap<Integer, ItemStack>) player.getInventory().all(type);
+            for (int i : items.keySet()) {
+                if (removed >= amount) {
+                    break;
+                }
+                int diff = (int) (amount - removed);
+                int amt = player.getInventory().getItem(i).getAmount();
+                if (amt - diff > 0) {
+                    player.getInventory().getItem(i).setAmount(amt - diff);
+                    break;
+                } else {
+                    removed += amt;
+                    player.getInventory().clear(i);
+                }
+            }
+            showReceipt(player, amount, type);
+        }
+
+        private static void showReceipt(Player player, double price, Material item) {
+            if (price > 0D) {
+                player.sendMessage(String.format("%s%s%s %s",
+                        ChatColor.WHITE, "You have been charged", ChatColor.GREEN, getFormattedPrice(price, item)));
+            } else if (price < 0D) {
+                player.sendMessage(String.format("%s%s%s %s",
+                        ChatColor.DARK_GREEN, getFormattedPrice((price * -1), item),
+                        ChatColor.WHITE, "has been added to your inventory."));
+            }
+        }
+    }
+}
@@ -230,7 +230,7 @@ public class MVPermissions implements PermissionsInterface {
     *
     * @param sender       Who is requesting the permission.
     * @param node         The permission node in string format; multiverse.core.list.worlds for example.
-     * @param isOpRequired @Deprecated. This is not used for anything anymore.
+     * @param isOpRequired deprecated This is not used for anything anymore.
     * @return True if they have that permission or any parent.
     */
    @Override
@@ -9,114 +9,19 @@ package com.onarandombox.MultiverseCore.utils;

 import com.onarandombox.MultiverseCore.MultiverseCore;
 import com.onarandombox.MultiverseCore.api.MVDestination;
-import com.onarandombox.MultiverseCore.api.SafeTTeleporter;
-import com.onarandombox.MultiverseCore.destination.CannonDestination;
-import org.bukkit.Location;
-import org.bukkit.TravelAgent;
 import org.bukkit.entity.Player;

-import java.util.logging.Level;
-
 /**
- * The Multiverse-{@link TravelAgent}.
+ * The Multiverse TravelAgent.
 */
-public class MVTravelAgent implements TravelAgent {
-    private MVDestination destination;
-    private MultiverseCore core;
-    private Player player;
+public class MVTravelAgent {
+    protected MVDestination destination;
+    protected MultiverseCore core;
+    protected Player player;

    public MVTravelAgent(MultiverseCore multiverseCore, MVDestination d, Player p) {
        this.destination = d;
        this.core = multiverseCore;
        this.player = p;
    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public TravelAgent setSearchRadius(int radius) {
-        return this;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public int getSearchRadius() {
-        return 0;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public TravelAgent setCreationRadius(int radius) {
-        return this;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public int getCreationRadius() {
-        return 0;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public boolean getCanCreatePortal() {
-        return false;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public void setCanCreatePortal(boolean create) {
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public Location findOrCreate(Location location) {
-        return this.getSafeLocation();
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public Location findPortal(Location location) {
-        return this.getSafeLocation();
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public boolean createPortal(Location location) {
-        return false;
-    }
-
-    private Location getSafeLocation() {
-        // At this time, these can never use the velocity.
-        if (this.destination instanceof CannonDestination) {
-            this.core.log(Level.FINE, "Using Stock TP method. This cannon will have 0 velocity");
-        }
-        SafeTTeleporter teleporter = this.core.getSafeTTeleporter();
-        Location newLoc = this.destination.getLocation(this.player);
-        if (this.destination.useSafeTeleporter()) {
-            newLoc = teleporter.getSafeLocation(this.player, this.destination);
-        }
-        if (newLoc == null) {
-            return this.player.getLocation();
-        }
-        return newLoc;
-
-    }
-
 }
@@ -0,0 +1,42 @@
+package com.onarandombox.MultiverseCore.utils;
+
+import de.themoep.idconverter.IdMappings;
+import org.bukkit.Material;
+import org.bukkit.configuration.ConfigurationSection;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+
+/**
+ * A tool for converting values which may be an old type ID to a Material.
+ */
+public class MaterialConverter {
+
+    /**
+     * Converts the value in the given config at the given path from a numeric id or flattened material name to a
+     * Material.
+     *
+     * @param config The config with the value to convert.
+     * @param path The path of the value in the config.
+     * @return The converted Material type or null if no matching type.
+     */
+    @Nullable
+    public static Material convertConfigType(@NotNull ConfigurationSection config, @NotNull String path) {
+        return convertTypeString(config.getString(path));
+    }
+
+    /**
+     * Converts a string representing a numeric id or flattened material name to a Material.
+     *
+     * @param value The value to convert.
+     * @return The converted Material type or null if no matching type.
+     */
+    @Nullable
+    public static Material convertTypeString(@Nullable String value) {
+        IdMappings.Mapping mapping = IdMappings.getById(value != null ? value : "");
+        if (mapping != null) {
+            return Material.matchMaterial(mapping.getFlatteningType());
+        } else {
+            return Material.matchMaterial(value != null ? value : "");
+        }
+    }
+}
@@ -9,6 +9,7 @@ package com.onarandombox.MultiverseCore.utils;

 import com.onarandombox.MultiverseCore.MultiverseCore;
 import com.onarandombox.MultiverseCore.api.MultiverseWorld;
+import org.bukkit.Material;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.ConsoleCommandSender;
 import org.bukkit.entity.Player;
@@ -145,7 +146,7 @@ public class PermissionTools {
            }

            final MVEconomist economist = plugin.getEconomist();
-            final int currency = toWorld.getCurrency();
+            final Material currency = toWorld.getCurrency();
            final String formattedAmount = economist.formatPrice(price, currency);

            if (economist.isPlayerWealthyEnough(teleporterPlayer, price, currency)) {
@@ -171,7 +172,7 @@ public class PermissionTools {
        return true;
    }

-    private void sendTeleportPaymentMessage (MVEconomist economist, Player teleporterPlayer, Player teleportee, String toWorld, double price, int currency) {
+    private void sendTeleportPaymentMessage (MVEconomist economist, Player teleporterPlayer, Player teleportee, String toWorld, double price, Material currency) {
        price = Math.abs(price);
        if (teleporterPlayer.equals(teleportee)) {
            teleporterPlayer.sendMessage("You were " + (price > 0D ? "charged " : "given ") + economist.formatPrice(price, currency) + " for teleporting to " + toWorld);
@@ -27,7 +27,7 @@ import java.util.logging.Level;
 /**
 * Utility class that removes animals from worlds that don't belong there.
 *
- * @deprecated Use instead: {@link WorldPurger} and {@link SimpleWorldPurger}.
+ * @deprecated Use instead: {@link com.onarandombox.MultiverseCore.api.WorldPurger} and {@link SimpleWorldPurger}.
 */
@Deprecated
 public class PurgeWorlds {
@@ -1,358 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2011.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-
-package com.onarandombox.MultiverseCore.utils;
-
-import com.onarandombox.MultiverseCore.MultiverseCore;
-import com.onarandombox.MultiverseCore.api.BlockSafety;
-import com.onarandombox.MultiverseCore.api.MVDestination;
-import com.onarandombox.MultiverseCore.destination.InvalidDestination;
-import com.onarandombox.MultiverseCore.enums.TeleportResult;
-
-import org.bukkit.Location;
-import org.bukkit.Material;
-import org.bukkit.block.Block;
-import org.bukkit.block.BlockFace;
-import org.bukkit.command.CommandSender;
-import org.bukkit.entity.Entity;
-import org.bukkit.entity.Minecart;
-import org.bukkit.entity.Player;
-import org.bukkit.entity.Vehicle;
-import org.bukkit.util.Vector;
-
-import java.util.logging.Level;
-
-/**
- * The {@link SafeTTeleporter}.
- *
- * @deprecated Use instead: {@link com.onarandombox.MultiverseCore.api.SafeTTeleporter} and {@link SimpleSafeTTeleporter}.
- */
-@Deprecated
-public class SafeTTeleporter {
-
-    private MultiverseCore plugin;
-    private BlockSafety bs;
-
-    public SafeTTeleporter(MultiverseCore plugin) {
-        this.plugin = plugin;
-        this.bs = plugin.getBlockSafety();
-    }
-
-    private static final int DEFAULT_TOLERANCE = 6;
-    private static final int DEFAULT_RADIUS = 9;
-
-    /**
-     * Gets the next safe location around the given location.
-     * @param l A {@link Location}.
-     * @return A safe {@link Location}.
-     */
-    public Location getSafeLocation(Location l) {
-        return this.getSafeLocation(l, DEFAULT_TOLERANCE, DEFAULT_RADIUS);
-    }
-
-    /**
-     * Gets the next safe location around the given location.
-     * @param l A {@link Location}.
-     * @param tolerance The tolerance.
-     * @param radius The radius.
-     * @return A safe {@link Location}.
-     */
-    public Location getSafeLocation(Location l, int tolerance, int radius) {
-        // Check around the player first in a configurable radius:
-        Location safe = checkAboveAndBelowLocation(l, tolerance, radius);
-        if (safe != null) {
-            safe.setX(safe.getBlockX() + .5); // SUPPRESS CHECKSTYLE: MagicNumberCheck
-            safe.setZ(safe.getBlockZ() + .5); // SUPPRESS CHECKSTYLE: MagicNumberCheck
-            this.plugin.log(Level.FINE, "Hey! I found one: " + plugin.getLocationManipulation().strCoordsRaw(safe));
-        } else {
-            this.plugin.log(Level.FINE, "Uh oh! No safe place found!");
-        }
-        return safe;
-    }
-
-    private Location checkAboveAndBelowLocation(Location l, int tolerance, int radius) {
-        // Tolerance must be an even number:
-        if (tolerance % 2 != 0) {
-            tolerance += 1;
-        }
-        // We want half of it, so we can go up and down
-        tolerance /= 2;
-        this.plugin.log(Level.FINER, "Given Location of: " + plugin.getLocationManipulation().strCoordsRaw(l));
-        this.plugin.log(Level.FINER, "Checking +-" + tolerance + " with a radius of " + radius);
-
-        // For now this will just do a straight up block.
-        Location locToCheck = l.clone();
-        // Check the main level
-        Location safe = this.checkAroundLocation(locToCheck, radius);
-        if (safe != null) {
-            return safe;
-        }
-        // We've already checked zero right above this.
-        int currentLevel = 1;
-        while (currentLevel <= tolerance) {
-            // Check above
-            locToCheck = l.clone();
-            locToCheck.add(0, currentLevel, 0);
-            safe = this.checkAroundLocation(locToCheck, radius);
-            if (safe != null) {
-                return safe;
-            }
-
-            // Check below
-            locToCheck = l.clone();
-            locToCheck.subtract(0, currentLevel, 0);
-            safe = this.checkAroundLocation(locToCheck, radius);
-            if (safe != null) {
-                return safe;
-            }
-            currentLevel++;
-        }
-
-        return null;
-    }
-
-    /*
-     * For my crappy algorithm, radius MUST be odd.
-     */
-    private Location checkAroundLocation(Location l, int diameter) {
-        if (diameter % 2 == 0) {
-            diameter += 1;
-        }
-        Location checkLoc = l.clone();
-
-        // Start at 3, the min diameter around a block
-        int loopcounter = 3;
-        while (loopcounter <= diameter) {
-            boolean foundSafeArea = checkAroundSpecificDiameter(checkLoc, loopcounter);
-            // If a safe area was found:
-            if (foundSafeArea) {
-                // Return the checkLoc, it is the safe location.
-                return checkLoc;
-            }
-            // Otherwise, let's reset our location
-            checkLoc = l.clone();
-            // And increment the radius
-            loopcounter += 2;
-        }
-        return null;
-    }
-
-    private boolean checkAroundSpecificDiameter(Location checkLoc, int circle) {
-        // Adjust the circle to get how many blocks to step out.
-        // A radius of 3 makes the block step 1
-        // A radius of 5 makes the block step 2
-        // A radius of 7 makes the block step 3
-        // ...
-        int adjustedCircle = ((circle - 1) / 2);
-        checkLoc.add(adjustedCircle, 0, 0);
-        if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-            return true;
-        }
-        // Now we go to the right that adjustedCircle many
-        for (int i = 0; i < adjustedCircle; i++) {
-            checkLoc.add(0, 0, 1);
-            if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-                return true;
-            }
-        }
-
-        // Then down adjustedCircle *2
-        for (int i = 0; i < adjustedCircle * 2; i++) {
-            checkLoc.add(-1, 0, 0);
-            if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-                return true;
-            }
-        }
-
-        // Then left adjustedCircle *2
-        for (int i = 0; i < adjustedCircle * 2; i++) {
-            checkLoc.add(0, 0, -1);
-            if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-                return true;
-            }
-        }
-
-        // Then up Then left adjustedCircle *2
-        for (int i = 0; i < adjustedCircle * 2; i++) {
-            checkLoc.add(1, 0, 0);
-            if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-                return true;
-            }
-        }
-
-        // Then finish up by doing adjustedCircle - 1
-        for (int i = 0; i < adjustedCircle - 1; i++) {
-            checkLoc.add(0, 0, 1);
-            if (this.bs.playerCanSpawnHereSafely(checkLoc)) {
-                return true;
-            }
-        }
-        return false;
-    }
-
-    /**
-     * Safely teleport the entity to the MVDestination. This will perform checks to see if the place is safe, and if
-     * it's not, will adjust the final destination accordingly.
-     *
-     * @param teleporter Person who performed the teleport command.
-     * @param teleportee Entity to teleport
-     * @param d          Destination to teleport them to
-     * @return true for success, false for failure
-     */
-    public TeleportResult safelyTeleport(CommandSender teleporter, Entity teleportee, MVDestination d) {
-        if (d instanceof InvalidDestination) {
-            this.plugin.log(Level.FINER, "Entity tried to teleport to an invalid destination");
-            return TeleportResult.FAIL_INVALID;
-        }
-        Player teleporteePlayer = null;
-        if (teleportee instanceof Player) {
-            teleporteePlayer = ((Player) teleportee);
-        } else if (teleportee.getPassenger() instanceof Player) {
-            teleporteePlayer = ((Player) teleportee.getPassenger());
-        }
-
-        if (teleporteePlayer == null) {
-            return TeleportResult.FAIL_INVALID;
-        }
-        MultiverseCore.addPlayerToTeleportQueue(teleporter.getName(), teleporteePlayer.getName());
-
-        Location safeLoc = d.getLocation(teleportee);
-        if (d.useSafeTeleporter()) {
-            safeLoc = this.getSafeLocation(teleportee, d);
-        }
-
-        if (safeLoc != null) {
-            if (teleportee.teleport(safeLoc)) {
-                if (!d.getVelocity().equals(new Vector(0, 0, 0))) {
-                    teleportee.setVelocity(d.getVelocity());
-                }
-                return TeleportResult.SUCCESS;
-            }
-            return TeleportResult.FAIL_OTHER;
-        }
-        return TeleportResult.FAIL_UNSAFE;
-    }
-
-    /**
-     * Safely teleport the entity to the Location. This may perform checks to
-     * see if the place is safe, and if
-     * it's not, will adjust the final destination accordingly.
-     *
-     * @param teleporter Person who issued the teleport command.
-     * @param teleportee Entity to teleport.
-     * @param location   Location to teleport them to.
-     * @param safely     Should the destination be checked for safety before teleport?
-     * @return true for success, false for failure.
-     */
-    public TeleportResult safelyTeleport(CommandSender teleporter, Entity teleportee, Location location, boolean safely) {
-        if (safely) {
-            location = this.getSafeLocation(location);
-        }
-
-        if (location != null) {
-            if (teleportee.teleport(location)) {
-                return TeleportResult.SUCCESS;
-            }
-            return TeleportResult.FAIL_OTHER;
-        }
-        return TeleportResult.FAIL_UNSAFE;
-    }
-
-    /**
-     * Returns a safe location for the entity to spawn at.
-     *
-     * @param e The entity to spawn
-     * @param d The MVDestination to take the entity to.
-     * @return A new location to spawn the entity at.
-     */
-    public Location getSafeLocation(Entity e, MVDestination d) {
-        Location l = d.getLocation(e);
-        if (this.bs.playerCanSpawnHereSafely(l)) {
-            plugin.log(Level.FINE, "The first location you gave me was safe.");
-            return l;
-        }
-        if (e instanceof Minecart) {
-            Minecart m = (Minecart) e;
-            if (!this.bs.canSpawnCartSafely(m)) {
-                return null;
-            }
-        } else if (e instanceof Vehicle) {
-            Vehicle v = (Vehicle) e;
-            if (!this.bs.canSpawnVehicleSafely(v)) {
-                return null;
-            }
-        }
-        Location safeLocation = this.getSafeLocation(l);
-        if (safeLocation != null) {
-            // Add offset to account for a vehicle on dry land!
-            if (e instanceof Minecart && !this.bs.isEntitiyOnTrack(safeLocation)) {
-                safeLocation.setY(safeLocation.getBlockY() + .5);
-                this.plugin.log(Level.FINER, "Player was inside a minecart. Offsetting Y location.");
-            }
-            this.plugin.log(Level.FINE, "Had to look for a bit, but I found a safe place for ya!");
-            return safeLocation;
-        }
-        if (e instanceof Player) {
-            Player p = (Player) e;
-            this.plugin.getMessaging().sendMessage(p, "No safe locations found!", false);
-            this.plugin.log(Level.FINER, "No safe location found for " + p.getName());
-        } else if (e.getPassenger() instanceof Player) {
-            Player p = (Player) e.getPassenger();
-            this.plugin.getMessaging().sendMessage(p, "No safe locations found!", false);
-            this.plugin.log(Level.FINER, "No safe location found for " + p.getName());
-        }
-        this.plugin.log(Level.FINE, "Sorry champ, you're basically trying to teleport into a minefield. I should just kill you now.");
-        return null;
-    }
-
-    /**
-     * Finds a portal-block next to the specified {@link Location}.
-     * @param l The {@link Location}
-     * @return The next portal-block's {@link Location}.
-     */
-    public static Location findPortalBlockNextTo(Location l) {
-        Block b = l.getWorld().getBlockAt(l);
-        Location foundLocation = null;
-        if (b.getType() == Material.PORTAL) {
-            return l;
-        }
-        if (b.getRelative(BlockFace.NORTH).getType() == Material.PORTAL) {
-            foundLocation = getCloserBlock(l, b.getRelative(BlockFace.NORTH).getLocation(), foundLocation);
-        }
-        if (b.getRelative(BlockFace.SOUTH).getType() == Material.PORTAL) {
-            foundLocation = getCloserBlock(l, b.getRelative(BlockFace.SOUTH).getLocation(), foundLocation);
-        }
-        if (b.getRelative(BlockFace.EAST).getType() == Material.PORTAL) {
-            foundLocation = getCloserBlock(l, b.getRelative(BlockFace.EAST).getLocation(), foundLocation);
-        }
-        if (b.getRelative(BlockFace.WEST).getType() == Material.PORTAL) {
-            foundLocation = getCloserBlock(l, b.getRelative(BlockFace.WEST).getLocation(), foundLocation);
-        }
-        return foundLocation;
-    }
-
-    private static Location getCloserBlock(Location source, Location blockA, Location blockB) {
-        // If B wasn't given, return a.
-        if (blockB == null) {
-            return blockA;
-        }
-        // Center our calculations
-        blockA.add(.5, 0, .5);
-        blockB.add(.5, 0, .5);
-
-        // Retrieve the distance to the normalized blocks
-        double testA = source.distance(blockA);
-        double testB = source.distance(blockB);
-
-        // Compare and return
-        if (testA <= testB) {
-            return blockA;
-        }
-        return blockB;
-    }
-
-}
@@ -89,7 +89,7 @@ public class SimpleBlockSafety implements BlockSafety {
            return false;
        }

-        if (downOne.getBlock().getType() == Material.LAVA || downOne.getBlock().getType() == Material.STATIONARY_LAVA) {
+        if (downOne.getBlock().getType() == Material.LAVA) {
            Logging.finer("Error Here (downOne)? (%s)[%s]", downOne.getBlock().getType(), isSolidBlock(downOne.getBlock().getType()));
            return false;
        }
@@ -199,69 +199,8 @@ public class SimpleBlockSafety implements BlockSafety {
    /*
     * If someone has a better way of this... Please either tell us, or submit a pull request!
     */
-    private static boolean isSolidBlock(Material type) {
-        switch (type) {
-            case AIR:
-                return false;
-            case SNOW:
-                return false;
-            case TRAP_DOOR:
-                return false;
-            case TORCH:
-                return false;
-            case YELLOW_FLOWER:
-                return false;
-            case RED_ROSE:
-                return false;
-            case RED_MUSHROOM:
-                return false;
-            case BROWN_MUSHROOM:
-                return false;
-            case REDSTONE:
-                return false;
-            case REDSTONE_WIRE:
-                return false;
-            case RAILS:
-                return false;
-            case POWERED_RAIL:
-                return false;
-            case REDSTONE_TORCH_ON:
-                return false;
-            case REDSTONE_TORCH_OFF:
-                return false;
-            case DEAD_BUSH:
-                return false;
-            case SAPLING:
-                return false;
-            case STONE_BUTTON:
-                return false;
-            case LEVER:
-                return false;
-            case LONG_GRASS:
-                return false;
-            case PORTAL:
-                return false;
-            case STONE_PLATE:
-                return false;
-            case WOOD_PLATE:
-                return false;
-            case SEEDS:
-                return false;
-            case SUGAR_CANE_BLOCK:
-                return false;
-            case WALL_SIGN:
-                return false;
-            case SIGN_POST:
-                return false;
-            case WOODEN_DOOR:
-                return false;
-            case STATIONARY_WATER:
-                return false;
-            case WATER:
-                return false;
-            default:
-                return true;
-        }
+    public static boolean isSolidBlock(Material type) {
+        return type.isSolid();
    }

    /**
@@ -285,10 +224,10 @@ public class SimpleBlockSafety implements BlockSafety {
        }
        Location oneBelow = l.clone();
        oneBelow.subtract(0, 1, 0);
-        if (oneBelow.getBlock().getType() == Material.WATER || oneBelow.getBlock().getType() == Material.STATIONARY_WATER) {
+        if (oneBelow.getBlock().getType() == Material.WATER) {
            Location twoBelow = oneBelow.clone();
            twoBelow.subtract(0, 1, 0);
-            return (oneBelow.getBlock().getType() == Material.WATER || oneBelow.getBlock().getType() == Material.STATIONARY_WATER);
+            return (oneBelow.getBlock().getType() == Material.WATER);
        }
        if (oneBelow.getBlock().getType() != Material.AIR) {
            return false;
@@ -1,30 +0,0 @@
-/******************************************************************************
- * Multiverse 2 Copyright (c) the Multiverse Team 2011.                       *
- * Multiverse 2 is licensed under the BSD License.                            *
- * For more information please check the README.md file included              *
- * with this project.                                                         *
- ******************************************************************************/
-/*
-package com.onarandombox.MultiverseCore.utils;
-
-import org.getspout.spoutapi.SpoutManager;
-
-/**
- * A helper-class holding the {@link SpoutManager}.
- * /
-public class SpoutInterface {
-    private SpoutManager spoutManager;
-
-    public SpoutInterface() {
-        this.spoutManager = SpoutManager.getInstance();
-    }
-
-    /**
-     * Gets the {@link SpoutManager}.
-     * @return The {@link SpoutManager}.
-     * /
-    public SpoutManager getManager() {
-        return this.spoutManager;
-    }
-}
-*/
@@ -7,6 +7,31 @@

 package com.onarandombox.MultiverseCore.utils;

+import com.dumptruckman.minecraft.util.Logging;
+import com.onarandombox.MultiverseCore.MVWorld;
+import com.onarandombox.MultiverseCore.MultiverseCore;
+import com.onarandombox.MultiverseCore.MultiverseCoreConfiguration;
+import com.onarandombox.MultiverseCore.WorldProperties;
+import com.onarandombox.MultiverseCore.api.MVWorldManager;
+import com.onarandombox.MultiverseCore.api.MultiverseWorld;
+import com.onarandombox.MultiverseCore.api.SafeTTeleporter;
+import com.onarandombox.MultiverseCore.api.WorldPurger;
+import com.onarandombox.MultiverseCore.event.MVWorldDeleteEvent;
+import org.bukkit.Bukkit;
+import org.bukkit.Location;
+import org.bukkit.World;
+import org.bukkit.World.Environment;
+import org.bukkit.WorldCreator;
+import org.bukkit.WorldType;
+import org.bukkit.configuration.ConfigurationSection;
+import org.bukkit.configuration.file.FileConfiguration;
+import org.bukkit.configuration.file.YamlConfiguration;
+import org.bukkit.entity.Player;
+import org.bukkit.generator.ChunkGenerator;
+import org.bukkit.permissions.Permission;
+import org.bukkit.permissions.PermissionDefault;
+import org.bukkit.plugin.Plugin;
+
 import java.io.File;
 import java.io.FilenameFilter;
 import java.io.IOException;
@@ -22,30 +47,6 @@ import java.util.concurrent.Callable;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.logging.Level;

-import org.bukkit.Location;
-import org.bukkit.World;
-import org.bukkit.World.Environment;
-import org.bukkit.WorldCreator;
-import org.bukkit.WorldType;
-import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.configuration.file.FileConfiguration;
-import org.bukkit.configuration.file.YamlConfiguration;
-import org.bukkit.entity.Player;
-import org.bukkit.generator.ChunkGenerator;
-import org.bukkit.permissions.Permission;
-import org.bukkit.permissions.PermissionDefault;
-import org.bukkit.plugin.Plugin;
-
-import com.dumptruckman.minecraft.util.Logging;
-import com.onarandombox.MultiverseCore.MVWorld;
-import com.onarandombox.MultiverseCore.MultiverseCore;
-import com.onarandombox.MultiverseCore.WorldProperties;
-import com.onarandombox.MultiverseCore.api.MVWorldManager;
-import com.onarandombox.MultiverseCore.api.MultiverseWorld;
-import com.onarandombox.MultiverseCore.api.SafeTTeleporter;
-import com.onarandombox.MultiverseCore.api.WorldPurger;
-import com.onarandombox.MultiverseCore.event.MVWorldDeleteEvent;
-
 /**
 * Public facing API to add/remove Multiverse worlds.
 */
@@ -221,6 +222,9 @@ public class WorldManager implements MVWorldManager {
    @Override
    public boolean addWorld(String name, Environment env, String seedString, WorldType type, Boolean generateStructures,
                            String generator, boolean useSpawnAdjust) {
+        if (name.equalsIgnoreCase("plugins") || name.equalsIgnoreCase("logs")) {
+            return false;
+        }
        Long seed = null;
        WorldCreator c = new WorldCreator(name);
        if (seedString != null && seedString.length() > 0) {
@@ -409,6 +413,12 @@ public class WorldManager implements MVWorldManager {
        this.plugin.log(Level.SEVERE, "https://github.com/Fenixin/Minecraft-Region-Fixer");
    }

+    private void nullWorld(String name) {
+        this.plugin.log(Level.SEVERE, "The world '" + name + "' could NOT be loaded because the server didn't like it!");
+        this.plugin.log(Level.SEVERE, "We don't really know why this is. Contact the developer of your server software!");
+        this.plugin.log(Level.SEVERE, "Server version info: " + Bukkit.getServer().getVersion());
+    }
+
    private boolean doLoad(String name) {
        return doLoad(name, false, null);
    }
@@ -460,8 +470,14 @@ public class WorldManager implements MVWorldManager {
            brokenWorld(worldName);
            return false;
        }
+        if (cbworld == null) {
+            nullWorld(worldName);
+            return false;
+        }
        MVWorld world = new MVWorld(plugin, cbworld, mvworld);
-        this.worldPurger.purgeWorld(world);
+        if (MultiverseCoreConfiguration.getInstance().isAutoPurgeEnabled()) {
+            this.worldPurger.purgeWorld(world);
+        }
        this.worlds.put(worldName, world);
        return true;
    }
@@ -0,0 +1,99 @@
+package com.onarandombox.MultiverseCore.utils.webpaste;
+
+import com.google.gson.JsonObject;
+import com.google.gson.JsonParser;
+import com.google.gson.JsonPrimitive;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.io.InputStreamReader;
+import java.io.OutputStreamWriter;
+import java.net.MalformedURLException;
+import java.net.URL;
+import java.net.URLConnection;
+import java.util.HashMap;
+import java.util.Map;
+
+public class GithubPasteService implements PasteService {
+
+    private final boolean isPrivate;
+
+    public GithubPasteService(boolean isPrivate) {
+        this.isPrivate = isPrivate;
+    }
+
+    @Override
+    public String encodeData(String data) {
+        Map<String, String> mapData = new HashMap<String, String>();
+        mapData.put("multiverse.txt", data);
+        return this.encodeData(mapData);
+    }
+
+    @Override
+    public String encodeData(Map<String, String> files) {
+        JsonObject root = new JsonObject();
+        root.add("description", new JsonPrimitive("Multiverse-Core Debug Info"));
+        root.add("public", new JsonPrimitive(!this.isPrivate));
+        JsonObject fileList = new JsonObject();
+        for (Map.Entry<String, String> entry : files.entrySet())
+        {
+            JsonObject fileObject = new JsonObject();
+            fileObject.add("content", new JsonPrimitive(entry.getValue()));
+            fileList.add(entry.getKey(), fileObject);
+        }
+        root.add("files", fileList);
+        return root.toString();
+    }
+
+    @Override
+    public URL getPostURL() {
+        try {
+            return new URL("https://api.github.com/gists");
+            //return new URL("http://jsonplaceholder.typicode.com/posts");
+        } catch (MalformedURLException e) {
+            return null; // should never hit here
+        }
+    }
+
+    @Override
+    public String postData(String encodedData, URL url) throws PasteFailedException {
+        OutputStreamWriter wr = null;
+        BufferedReader rd = null;
+        try {
+            URLConnection conn = url.openConnection();
+            conn.setDoOutput(true);
+            wr = new OutputStreamWriter(conn.getOutputStream());
+            wr.write(encodedData);
+            wr.flush();
+
+            rd = new BufferedReader(new InputStreamReader(conn.getInputStream()));
+            String line;
+            String pastieUrl = "";
+            //Pattern pastiePattern = this.getURLMatchingPattern();
+            StringBuilder responseString = new StringBuilder();
+
+            while ((line = rd.readLine()) != null) {
+                responseString.append(line);
+            }
+            return new JsonParser().parse(responseString.toString()).getAsJsonObject().get("html_url").getAsString();
+        } catch (Exception e) {
+            throw new PasteFailedException(e);
+        } finally {
+            if (wr != null) {
+                try {
+                    wr.close();
+                } catch (IOException ignore) { }
+            }
+            if (rd != null) {
+                try {
+                    rd.close();
+                } catch (IOException ignore) { }
+            }
+        }
+    }
+
+    @Override
+    public boolean supportsMultiFile() {
+        return true;
+    }
+}
@@ -0,0 +1,81 @@
+package com.onarandombox.MultiverseCore.utils.webpaste;
+
+import com.google.gson.JsonElement;
+import com.google.gson.JsonParser;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.io.InputStreamReader;
+import java.io.OutputStreamWriter;
+import java.net.MalformedURLException;
+import java.net.URL;
+import java.net.URLConnection;
+import java.util.Map;
+
+/**
+ * Pastes to {@code hastebin.com}.
+ */
+public class HastebinPasteService implements PasteService {
+
+    @Override
+    public String encodeData(String data) {
+        return data;
+    }
+
+    @Override
+    public String encodeData(Map<String, String> data) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public URL getPostURL() {
+        try {
+            return new URL("https://hastebin.com/documents");
+        } catch (MalformedURLException e) {
+            return null; // should never hit here
+        }
+    }
+
+    @Override
+    public String postData(String encodedData, URL url) throws PasteFailedException {
+        OutputStreamWriter wr = null;
+        BufferedReader rd = null;
+        try {
+            URLConnection conn = url.openConnection();
+            conn.setDoOutput(true);
+
+            wr = new OutputStreamWriter(conn.getOutputStream());
+            rd = new BufferedReader(new InputStreamReader(conn.getInputStream()));
+
+            wr.write(encodedData);
+            wr.flush();
+
+            String line;
+            StringBuilder responseString = new StringBuilder();
+            while ((line = rd.readLine()) != null) {
+                responseString.append(line);
+            }
+            String key = new JsonParser().parse(responseString.toString()).getAsJsonObject().get("key").getAsString();
+
+            return "https://hastebin.com/" + key;
+        } catch (Exception e) {
+            throw new PasteFailedException(e);
+        } finally {
+            if (wr != null) {
+                try {
+                    wr.close();
+                } catch (IOException ignore) { }
+            }
+            if (rd != null) {
+                try {
+                    rd.close();
+                } catch (IOException ignore) { }
+            }
+        }
+    }
+
+    @Override
+    public boolean supportsMultiFile() {
+        return false;
+    }
+}
@@ -1,6 +1,7 @@
 package com.onarandombox.MultiverseCore.utils.webpaste;

 import java.net.URL;
+import java.util.Map;

 /**
 * An interface to a web-based text-pasting service. Classes implementing this
@@ -12,7 +13,7 @@ import java.net.URL;
 *
 * Services that provide a distinction between "public" and "private" pastes
 * should implement a custom constructor that specifies which kind the PasteService
- * instance is submitting; an example of this is the PastiePasteService class.
+ * instance is submitting; an example of this is the PastebinPasteService class.
 */
 public interface PasteService {

@@ -24,6 +25,14 @@ public interface PasteService {
     */
    String encodeData(String data);

+    /**
+     * Encode the given Map data into a format suitable for transmission in an HTTP request.
+     *
+     * @param data The raw data to encode.
+     * @return A URL-encoded string.
+     */
+    String encodeData(Map<String, String> data);
+
    /**
     * Get the URL to which this paste service sends new pastes.
     *
@@ -42,4 +51,13 @@ public interface PasteService {
     */
    String postData(String encodedData, URL url) throws PasteFailedException;

+    /**
+     * Does this service support uploading multiple files.
+     *
+     * Newer services like gist support multi-file which allows us to upload configs
+     * in addition to the standard logs.
+     *
+     * @return True if this service supports multiple file upload.
+     */
+    boolean supportsMultiFile();
 }
@@ -16,8 +16,10 @@ public class PasteServiceFactory {
        switch(type) {
            case PASTEBIN:
                return new PastebinPasteService(isPrivate);
-            case PASTIE:
-                return new PastiePasteService(isPrivate);
+            case HASTEBIN:
+                return new HastebinPasteService();
+            case GITHUB:
+                return new GithubPasteService(isPrivate);
            default:
                return null;
        }
@@ -12,7 +12,11 @@ public enum PasteServiceType {
     */
    PASTEBIN,
    /**
-     * @see PastiePasteService
+     * @see HastebinPasteService
     */
-    PASTIE
+    HASTEBIN,
+    /**
+     * @see GithubPasteService
+     */
+    GITHUB
 }
@@ -9,6 +9,7 @@ import java.net.MalformedURLException;
 import java.net.URL;
 import java.net.URLConnection;
 import java.net.URLEncoder;
+import java.util.Map;

 /**
 * Pastes to {@code pastebin.com}.
@@ -50,6 +51,11 @@ public class PastebinPasteService implements PasteService {
        }
    }

+    @Override
+    public String encodeData(Map<String, String> data) {
+        return null;
+    }
+
    /**
     * {@inheritDoc}
     */
@@ -86,4 +92,9 @@ public class PastebinPasteService implements PasteService {
            }
        }
    }
+
+    @Override
+    public boolean supportsMultiFile() {
+        return false;
+    }
 }
@@ -1,107 +0,0 @@
-package com.onarandombox.MultiverseCore.utils.webpaste;
-
-import java.io.BufferedReader;
-import java.io.IOException;
-import java.io.InputStreamReader;
-import java.io.OutputStreamWriter;
-import java.io.UnsupportedEncodingException;
-import java.net.MalformedURLException;
-import java.net.URL;
-import java.net.URLConnection;
-import java.net.URLEncoder;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-
-/**
- * Pastes to {@code pastie.org}.
- */
-public class PastiePasteService implements PasteService {
-
-    private boolean isPrivate;
-
-    public PastiePasteService(boolean isPrivate) {
-        this.isPrivate = isPrivate;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public URL getPostURL() {
-        try {
-            return new URL("http://pastie.org/pastes");
-        } catch (MalformedURLException e) {
-            return null; // should never hit here
-        }
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public String encodeData(String data) {
-        try {
-            String encData = URLEncoder.encode("paste[authorization]", "UTF-8") + "=" + URLEncoder.encode("burger", "UTF-8"); // burger is magic
-            encData += "&" + URLEncoder.encode("paste[restricted]", "UTF-8") + "=" + URLEncoder.encode(this.isPrivate ? "1" : "0", "UTF-8");
-            encData += "&" + URLEncoder.encode("paste[parser_id]", "UTF-8") + "=" + URLEncoder.encode("6", "UTF-8"); // 6 is plain text
-            encData += "&" + URLEncoder.encode("paste[body]", "UTF-8") + "=" + URLEncoder.encode(data, "UTF-8");
-            return encData;
-        } catch (UnsupportedEncodingException e) {
-            return ""; // should never hit here
-        }
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    public String postData(String encodedData, URL url) throws PasteFailedException {
-        OutputStreamWriter wr = null;
-        BufferedReader rd = null;
-        try {
-            URLConnection conn = url.openConnection();
-            conn.setDoOutput(true);
-            wr = new OutputStreamWriter(conn.getOutputStream());
-            wr.write(encodedData);
-            wr.flush();
-
-            rd = new BufferedReader(new InputStreamReader(conn.getInputStream()));
-            String line;
-            String pastieUrl = "";
-            Pattern pastiePattern = this.getURLMatchingPattern();
-            while ((line = rd.readLine()) != null) {
-                Matcher m = pastiePattern.matcher(line);
-                if (m.matches()) {
-                    String pastieID = m.group(1);
-                    pastieUrl = this.formatURL(pastieID);
-                }
-            }
-            return pastieUrl;
-        } catch (Exception e) {
-            throw new PasteFailedException(e);
-        } finally {
-            if (wr != null) {
-                try {
-                    wr.close();
-                } catch (IOException ignore) { }
-            }
-            if (rd != null) {
-                try {
-                    rd.close();
-                } catch (IOException ignore) { }
-            }
-        }
-    }
-
-    private Pattern getURLMatchingPattern() {
-        if (this.isPrivate) {
-            return Pattern.compile(".*http://pastie.org/.*key=([0-9a-z]+).*");
-        } else {
-            return Pattern.compile(".*http://pastie.org/([0-9]+).*");
-        }
-    }
-
-    private String formatURL(String pastieID) {
-        return "http://pastie.org/" + (this.isPrivate ? "private/" : "") + pastieID;
-    }
-}
@@ -1,6 +1,7 @@
 name: Multiverse-Core
 main: com.onarandombox.MultiverseCore.MultiverseCore
-authors: ['Rigby', 'fernferret', 'lithium3141', 'main--']
+authors: ['Rigby', 'fernferret', 'lithium3141', 'main--', 'dumptruckman']
+website: 'https://dev.bukkit.org/projects/multiverse-core'
 version: maven-version-number
 commands:
  mv:
@@ -15,10 +15,12 @@ import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -30,7 +32,8 @@ import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;

@RunWith(PowerMockRunner.class)
-@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class })
+@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class, JavaPluginLoader.class})
+@PowerMockIgnore("javax.script.*")
 public class TestDebugMode {
    TestInstanceCreator creator;
    Server mockServer;
@@ -3,8 +3,10 @@ package com.onarandombox.MultiverseCore;
 import com.onarandombox.MultiverseCore.api.MVWorldManager;
 import com.onarandombox.MultiverseCore.api.MultiverseWorld;
 import com.onarandombox.MultiverseCore.listeners.MVEntityListener;
+import com.onarandombox.MultiverseCore.utils.MockWorldFactory;
 import com.onarandombox.MultiverseCore.utils.TestInstanceCreator;
 import org.bukkit.World;
+import org.bukkit.WorldType;
 import org.bukkit.entity.Entity;
 import org.bukkit.entity.EntityType;
 import org.bukkit.entity.LivingEntity;
@@ -13,10 +15,12 @@ import org.bukkit.entity.Zombie;
 import org.bukkit.event.entity.CreatureSpawnEvent;
 import org.bukkit.event.entity.CreatureSpawnEvent.SpawnReason;
 import org.bukkit.plugin.PluginDescriptionFile;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -25,11 +29,13 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;

+import static junit.framework.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 import static org.mockito.Mockito.*;

@RunWith(PowerMockRunner.class)
-@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class })
+@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class, JavaPluginLoader.class})
+@PowerMockIgnore("javax.script.*")
 public class TestEntitySpawnRules {
    TestInstanceCreator creator;
    MultiverseCore core;
@@ -52,7 +58,7 @@ public class TestEntitySpawnRules {
        listener = core.getEntityListener();

        mvWorld = mock(MultiverseWorld.class);
-        cbworld = mock(World.class);
+        cbworld = MockWorldFactory.makeNewMockWorld("world", World.Environment.NORMAL, WorldType.NORMAL);
        when(mvWorld.getCBWorld()).thenReturn(cbworld);

        MVWorldManager worldman = mock(MVWorldManager.class);
@@ -0,0 +1,65 @@
+package com.onarandombox.MultiverseCore;
+
+import com.onarandombox.MultiverseCore.utils.TestInstanceCreator;
+import org.bukkit.Material;
+import org.bukkit.plugin.PluginDescriptionFile;
+import org.bukkit.plugin.java.JavaPluginLoader;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
+import org.powermock.core.classloader.annotations.PrepareForTest;
+import org.powermock.modules.junit4.PowerMockRunner;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.Assert.*;
+
+@RunWith(PowerMockRunner.class)
+@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class, JavaPluginLoader.class})
+@PowerMockIgnore("javax.script.*")
+public class TestEntryFeeConversion {
+
+    private TestInstanceCreator creator;
+    private MultiverseCore core;
+    Map<String, Object> config;
+    Map<String, Object> entryFee;
+
+    @Before
+    public void setUp() {
+        creator = new TestInstanceCreator();
+        assertTrue(creator.setUp());
+        core = creator.getCore();
+
+        config = new HashMap<>();
+        entryFee = new HashMap<>();
+        config.put("entryfee", entryFee);
+        entryFee.put("==", "MVEntryFee");
+    }
+
+    @After
+    public void tearDown() {
+        creator.tearDown();
+    }
+
+    @Test
+    public void testConvertIntegerCurrencyToMaterialCurrency() {
+        entryFee.put("currency", -1);
+        WorldProperties props = new WorldProperties(config);
+        assertNull(props.entryfee.getCurrency());
+
+        entryFee.put("currency", 1);
+        props = new WorldProperties(config);
+        assertEquals(Material.STONE, props.entryfee.getCurrency());
+
+        entryFee.put("currency", "1");
+        props = new WorldProperties(config);
+        assertEquals(Material.STONE, props.entryfee.getCurrency());
+
+        entryFee.put("currency", "stone");
+        props = new WorldProperties(config);
+        assertEquals(Material.STONE, props.entryfee.getCurrency());
+    }
+}
@@ -7,10 +7,12 @@ import org.bukkit.World.Environment;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.plugin.PluginDescriptionFile;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -20,7 +22,8 @@ import static org.mockito.Mockito.when;


@RunWith(PowerMockRunner.class)
-@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class })
+@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class, JavaPluginLoader.class })
+@PowerMockIgnore("javax.script.*")
 public class TestModifyCommand {
    TestInstanceCreator creator;
    Server mockServer;
@@ -11,6 +11,7 @@ import com.onarandombox.MultiverseCore.api.MVWorldManager;
 import com.onarandombox.MultiverseCore.api.MultiverseWorld;
 import com.onarandombox.MultiverseCore.configuration.SpawnLocation;
 import com.onarandombox.MultiverseCore.listeners.MVAsyncPlayerChatListener;
+import com.onarandombox.MultiverseCore.utils.MockWorldFactory;
 import com.onarandombox.MultiverseCore.utils.TestInstanceCreator;
 import com.onarandombox.MultiverseCore.utils.WorldManager;
 import org.bukkit.Bukkit;
@@ -18,7 +19,9 @@ import org.bukkit.ChatColor;
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
 import org.bukkit.Location;
+import org.bukkit.Material;
 import org.bukkit.World;
+import org.bukkit.WorldType;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.configuration.file.FileConfiguration;
@@ -36,12 +39,14 @@ import org.bukkit.event.weather.WeatherChangeEvent;
 import org.bukkit.permissions.Permission;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.PluginManager;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.internal.verification.VerificationModeFactory;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -56,7 +61,8 @@ import static org.mockito.Mockito.*;
@PrepareForTest({ PluginManager.class, MultiverseCore.class, Permission.class, Bukkit.class,
        WeatherChangeEvent.class, ThunderChangeEvent.class, AsyncPlayerChatEvent.class,
        PlayerJoinEvent.class, PlayerRespawnEvent.class, EntityRegainHealthEvent.class,
-        FoodLevelChangeEvent.class, WorldManager.class, PluginDescriptionFile.class })
+        FoodLevelChangeEvent.class, WorldManager.class, PluginDescriptionFile.class, JavaPluginLoader.class })
+@PowerMockIgnore("javax.script.*")
 public class TestWorldProperties {
    private TestInstanceCreator creator;
    private MultiverseCore core;
@@ -85,6 +91,8 @@ public class TestWorldProperties {
        assertTrue(creator.setUp());
        core = creator.getCore();
        mockCommandSender = creator.getCommandSender();
+        MockWorldFactory.createWorldDirectory("world");
+        MockWorldFactory.createWorldDirectory("world_nether");
    }

    @After
@@ -104,6 +112,12 @@ public class TestWorldProperties {
        verify(mockCommandSender).sendMessage("Starting import of world 'world'...");
        verify(mockCommandSender).sendMessage(ChatColor.GREEN + "Complete!");

+        assertEquals(core.getServer().getWorlds().size(), 1);
+        assertEquals(core.getServer().getWorlds().get(0).getName(), "world");
+        assertEquals(core.getServer().getWorlds().get(0).getEnvironment(), World.Environment.NORMAL);
+        assertEquals(core.getServer().getWorlds().get(0).getWorldType(), WorldType.NORMAL);
+
+
        // Import a second world
        String[] netherArgs = new String[] { "import", "world_nether", "nether" };
        core.onCommand(mockCommandSender, mockCommand, "", netherArgs);
@@ -111,6 +125,12 @@ public class TestWorldProperties {
        verify(mockCommandSender, VerificationModeFactory.times(2)).sendMessage(
                ChatColor.GREEN + "Complete!");

+        assertEquals(core.getServer().getWorlds().size(), 2);
+        assertEquals(core.getServer().getWorlds().get(0).getName(), "world");
+        assertEquals(core.getServer().getWorlds().get(1).getName(), "world_nether");
+        assertEquals(core.getServer().getWorlds().get(1).getEnvironment(), World.Environment.NETHER);
+        assertEquals(core.getServer().getWorlds().get(1).getWorldType(), WorldType.NORMAL);
+
        // ////////////////////////////////////////////////
        // let's set some world-properties
        // we can test the API with this, too :D
@@ -137,7 +157,7 @@ public class TestWorldProperties {
        assertEquals(Difficulty.NORMAL, mvWorld.getDifficulty());
        assertTrue(mvWorld.canAnimalsSpawn());
        assertTrue(mvWorld.canMonstersSpawn());
-        assertEquals(-1, mvWorld.getCurrency());
+        assertNull(mvWorld.getCurrency());
        assertEquals(0, mvWorld.getPrice(), 0);
        assertTrue(mvWorld.getHunger());
        assertTrue(mvWorld.getAutoHeal());
@@ -219,8 +239,8 @@ public class TestWorldProperties {
        assertEquals(false, mvWorld.canAnimalsSpawn());
        mvWorld.setAllowMonsterSpawn(false);
        assertEquals(false, mvWorld.canMonstersSpawn());
-        mvWorld.setCurrency(1);
-        assertEquals(1, mvWorld.getCurrency());
+        mvWorld.setCurrency(Material.STONE);
+        assertEquals(Material.STONE, mvWorld.getCurrency());
        mvWorld.setPrice(1D);
        assertEquals(1D, mvWorld.getPrice(), 0);
        mvWorld.setHunger(false);
@@ -318,7 +338,7 @@ public class TestWorldProperties {
        assertEquals(Difficulty.PEACEFUL, mvWorld.getDifficulty());
        assertEquals(false, mvWorld.canAnimalsSpawn());
        assertEquals(false, mvWorld.canMonstersSpawn());
-        assertEquals(1, mvWorld.getCurrency());
+        assertEquals(Material.STONE, mvWorld.getCurrency());
        assertEquals(1D, mvWorld.getPrice(), 0);
        assertEquals(false, mvWorld.getHunger());
        assertEquals(false, mvWorld.getAutoHeal());
@@ -9,10 +9,12 @@ import org.bukkit.entity.EntityType;
 import org.bukkit.entity.Sheep;
 import org.bukkit.entity.Zombie;
 import org.bukkit.plugin.PluginDescriptionFile;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -22,7 +24,8 @@ import static org.junit.Assert.assertTrue;
 import static org.mockito.Mockito.*;

@RunWith(PowerMockRunner.class)
-@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class })
+@PrepareForTest({ MultiverseCore.class, PluginDescriptionFile.class, JavaPluginLoader.class })
+@PowerMockIgnore("javax.script.*")
 public class TestWorldPurger {
    TestInstanceCreator creator;
    MultiverseCore core;
@@ -9,6 +9,7 @@ package com.onarandombox.MultiverseCore;

 import com.onarandombox.MultiverseCore.api.MultiverseWorld;
 import com.onarandombox.MultiverseCore.exceptions.PropertyDoesNotExistException;
+import com.onarandombox.MultiverseCore.utils.MockWorldFactory;
 import com.onarandombox.MultiverseCore.utils.TestInstanceCreator;
 import com.onarandombox.MultiverseCore.utils.WorldCreatorMatcher;
 import com.onarandombox.MultiverseCore.utils.WorldManager;
@@ -23,12 +24,15 @@ import org.bukkit.permissions.Permission;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.PluginManager;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.mockito.ArgumentMatchers;
 import org.mockito.Matchers;
 import org.mockito.internal.verification.VerificationModeFactory;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;

@@ -38,7 +42,9 @@ import static junit.framework.Assert.*;
 import static org.mockito.Mockito.*;

@RunWith(PowerMockRunner.class)
-@PrepareForTest({ PluginManager.class, MultiverseCore.class, Permission.class, Bukkit.class, WorldManager.class, PluginDescriptionFile.class })
+@PrepareForTest({ PluginManager.class, MultiverseCore.class, Permission.class, Bukkit.class, WorldManager.class,
+        PluginDescriptionFile.class, JavaPluginLoader.class })
+@PowerMockIgnore("javax.script.*")
 public class TestWorldStuff {

    private TestInstanceCreator creator;
@@ -94,6 +100,10 @@ public class TestWorldStuff {

    @Test
    public void testWorldImport() {
+        MockWorldFactory.createWorldDirectory("world");
+        MockWorldFactory.createWorldDirectory("world_nether");
+        MockWorldFactory.createWorldDirectory("world_the_end");
+
        // Pull a core instance from the server.
        Plugin plugin = mockServer.getPluginManager().getPlugin("Multiverse-Core");

@@ -168,7 +178,7 @@ public class TestWorldStuff {
        verify(mockCommandSender).sendMessage("Complete!");

        WorldCreatorMatcher matcher = new WorldCreatorMatcher(new WorldCreator("newworld"));
-        verify(mockServer).createWorld(Matchers.argThat(matcher));
+        verify(mockServer).createWorld(ArgumentMatchers.argThat(matcher));
    }

    @Test
@@ -231,7 +241,7 @@ public class TestWorldStuff {
        verify(mockCommandSender).sendMessage("Complete!");

        WorldCreatorMatcher matcher = new WorldCreatorMatcher(new WorldCreator("nullworld"));
-        verify(mockServer).createWorld(Matchers.argThat(matcher));
+        verify(mockServer).createWorld(ArgumentMatchers.argThat(matcher));
    }

    @Test
@@ -277,6 +287,9 @@ public class TestWorldStuff {
    }

    private void createInitialWorlds(Plugin plugin, Command command) {
+        MockWorldFactory.createWorldDirectory("world");
+        MockWorldFactory.createWorldDirectory("world_nether");
+        MockWorldFactory.createWorldDirectory("world_the_end");
        plugin.onCommand(mockCommandSender, command, "", new String[]{ "import", "world", "normal" });
        plugin.onCommand(mockCommandSender, command, "", new String[]{ "import", "world_nether", "nether" });
        plugin.onCommand(mockCommandSender, command, "", new String[]{ "import", "world_the_end", "end" });
@@ -0,0 +1,119 @@
+package com.onarandombox.MultiverseCore.utils;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Path;
+
+import com.dumptruckman.minecraft.util.Logging;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+public class FileUtilsTest {
+
+    private Path tempDir;
+    private Path parentDir;
+    private Path parentDirFile;
+    private Path childDir;
+    private Path childDirFile;
+
+
+    @Before
+    public void setUp() throws Exception {
+        tempDir = Files.createTempDirectory("testingTempDir");
+
+        parentDir = Files.createDirectory(tempDir.resolve("parentDir"));
+        parentDirFile = Files.createFile(parentDir.resolve("parentDirFile.txt"));
+
+        childDir = Files.createDirectory(parentDir.resolve("childDir"));
+        childDirFile = Files.createFile(childDir.resolve("childDirFile.txt"));
+
+        assertTrue(Files.isDirectory(parentDir));
+        assertTrue(Files.isRegularFile(parentDirFile));
+        assertTrue(Files.isDirectory(childDir));
+        assertTrue(Files.isRegularFile(childDirFile));
+    }
+
+    @After
+    public void tearDown() throws Exception {
+        try {
+            org.apache.commons.io.FileUtils.deleteDirectory(tempDir.toFile());
+        } catch (IOException e) {
+            if (Files.exists(tempDir)) {
+                throw e;
+            }
+        }
+    }
+
+    @Test
+    public void deleteFolder() {
+        FileUtils.deleteFolder(parentDir.toFile());
+        assertFalse(Files.isDirectory(parentDir));
+        assertFalse(Files.isRegularFile(parentDirFile));
+        assertFalse(Files.isDirectory(childDir));
+        assertFalse(Files.isRegularFile(childDirFile));
+    }
+
+    @Test
+    public void deleteFolderContents() {
+        FileUtils.deleteFolderContents(parentDir.toFile());
+        assertTrue(Files.isDirectory(parentDir));
+        assertFalse(Files.isRegularFile(parentDirFile));
+        assertFalse(Files.isDirectory(childDir));
+        assertFalse(Files.isRegularFile(childDirFile));
+    }
+
+    @Test
+    public void copyFolder() throws Exception {
+        Path targetDir = tempDir.resolve("target");
+        Path targetFile = targetDir.resolve("parentDirFile.txt");
+        Path targetChildDir = targetDir.resolve("childDir");
+        Path targetChildDirFile = targetChildDir.resolve("childDirFile.txt");
+
+        assertFalse(Files.isDirectory(targetDir));
+        assertFalse(Files.isRegularFile(targetFile));
+        assertFalse(Files.isDirectory(targetChildDir));
+        assertFalse(Files.isRegularFile(targetChildDirFile));
+
+        assertTrue(FileUtils.copyFolder(parentDir.toFile(), targetDir.toFile(), Logging.getLogger()));
+
+        assertTrue(Files.isDirectory(targetDir));
+        assertTrue(Files.isRegularFile(targetFile));
+        assertTrue(Files.isDirectory(targetChildDir));
+        assertTrue(Files.isRegularFile(targetChildDirFile));
+    }
+
+    @Test
+    public void copyFolder_intoExistingFolder() throws Exception {
+        Path targetDir = Files.createDirectory(tempDir.resolve("target"));
+        Path targetFile = targetDir.resolve("parentDirFile.txt");
+        Path targetChildDir = targetDir.resolve("childDir");
+        Path targetChildDirFile = targetChildDir.resolve("childDirFile.txt");
+
+        assertTrue(Files.isDirectory(targetDir));
+        assertFalse(Files.isRegularFile(targetFile));
+        assertFalse(Files.isDirectory(targetChildDir));
+        assertFalse(Files.isRegularFile(targetChildDirFile));
+
+        assertTrue(FileUtils.copyFolder(parentDir.toFile(), targetDir.toFile(), Logging.getLogger()));
+
+        assertTrue(Files.isDirectory(targetDir));
+        assertTrue(Files.isRegularFile(targetFile));
+        assertTrue(Files.isDirectory(targetChildDir));
+        assertTrue(Files.isRegularFile(targetChildDirFile));
+    }
+
+    @Test()
+    public void copyFolder_intoExistingFolder_whereFileExists() throws Exception {
+        Path targetDir = Files.createDirectory(tempDir.resolve("target"));
+        Path targetFile = Files.createFile(targetDir.resolve("parentDirFile.txt"));
+
+        assertTrue(Files.isDirectory(targetDir));
+        assertTrue(Files.isRegularFile(targetFile));
+
+        assertFalse(FileUtils.copyFolder(parentDir.toFile(), targetDir.toFile(), Logging.getLogger()));
+    }
+}
@@ -18,18 +18,20 @@ import org.mockito.invocation.InvocationOnMock;
 import org.mockito.stubbing.Answer;

 import java.io.File;
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.UUID;
 import java.util.WeakHashMap;
+import java.util.LinkedHashMap;

 import static org.mockito.Mockito.*;

 public class MockWorldFactory {

-    private static final Map<String, World> createdWorlds = new HashMap<String, World>();
+    private static final Map<String, World> createdWorlds = new LinkedHashMap<String, World>();
    private static final Map<UUID, World> worldUIDS = new HashMap<UUID, World>();

    private static final Map<World, Boolean> pvpStates = new WeakHashMap<World, Boolean>();
@@ -42,7 +44,17 @@ public class MockWorldFactory {
    private static void registerWorld(World world) {
        createdWorlds.put(world.getName(), world);
        worldUIDS.put(world.getUID(), world);
-        new File(TestInstanceCreator.worldsDirectory, world.getName()).mkdir();
+        createWorldDirectory(world.getName());
+    }
+
+    public static void createWorldDirectory(String worldName) {
+        File worldFolder = new File(TestInstanceCreator.worldsDirectory, worldName);
+        worldFolder.mkdir();
+        try {
+            new File(worldFolder, "level.dat").createNewFile();
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
    }

    private static World basics(String world, World.Environment env, WorldType type) {
@@ -125,7 +137,6 @@ public class MockWorldFactory {
                }

                when(mockBlock.getType()).thenReturn(blockType);
-                when(mockBlock.getTypeId()).thenReturn(blockType.getId());
                when(mockBlock.getWorld()).thenReturn(loc.getWorld());
                when(mockBlock.getX()).thenReturn(loc.getBlockX());
                when(mockBlock.getY()).thenReturn(loc.getBlockY());
@@ -169,7 +180,6 @@ public class MockWorldFactory {
                Material blockType = Material.AIR;

                when(mockBlock.getType()).thenReturn(blockType);
-                when(mockBlock.getTypeId()).thenReturn(blockType.getId());
                when(mockBlock.getWorld()).thenReturn(loc.getWorld());
                when(mockBlock.getX()).thenReturn(loc.getBlockX());
                when(mockBlock.getY()).thenReturn(loc.getBlockY());
@@ -179,6 +189,7 @@ public class MockWorldFactory {
                return mockBlock;
            }
        });
+        when(mockWorld.getUID()).thenReturn(UUID.randomUUID());
        return mockWorld;
    }

@@ -212,13 +223,7 @@ public class MockWorldFactory {
    }

    public static List<World> getWorlds() {
-        // we have to invert the order!
-        ArrayList<World> myList = new ArrayList<World>(createdWorlds.values());
-        List<World> retList = new ArrayList<World>();
-        for (int i = (myList.size() - 1); i >= 0; i--) {
-            retList.add(myList.get(i));
-        }
-        return retList;
+        return new ArrayList<World>(createdWorlds.values());
    }

    public static void clearWorlds() {
@@ -25,12 +25,15 @@ import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.PluginManager;
 import org.bukkit.plugin.java.JavaPlugin;
+import org.bukkit.plugin.java.JavaPluginLoader;
 import org.bukkit.scheduler.BukkitScheduler;
-import org.mockito.Matchers;
+import org.mockito.ArgumentMatchers;
 import org.mockito.invocation.InvocationOnMock;
 import org.mockito.stubbing.Answer;
+import org.powermock.api.easymock.PowerMock;
 import org.powermock.api.mockito.PowerMockito;
 import org.powermock.core.MockGateway;
+import org.powermock.reflect.Whitebox;

 import java.io.File;
 import java.lang.reflect.Field;
@@ -40,11 +43,16 @@ import java.util.UUID;
 import java.util.logging.Level;
 import java.util.logging.Logger;

-import static org.mockito.Matchers.anyString;
-import static org.mockito.Mockito.any;
-import static org.mockito.Mockito.anyBoolean;
-import static org.mockito.Mockito.anyLong;
-import static org.mockito.Mockito.*;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyBoolean;
+import static org.mockito.ArgumentMatchers.anyLong;
+import static org.mockito.ArgumentMatchers.isNull;
+import static org.mockito.Mockito.when;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.doAnswer;

 public class TestInstanceCreator {
    private MultiverseCore core;
@@ -58,14 +66,14 @@ public class TestInstanceCreator {
    public boolean setUp() {
        try {
            pluginDirectory.mkdirs();
-            Assert.assertTrue(pluginDirectory.exists());
+            assertTrue(pluginDirectory.exists());

            MockGateway.MOCK_STANDARD_METHODS = false;

-            TestPluginLoader pluginLoader = new TestPluginLoader();
-
            // Initialize the Mock server.
            mockServer = mock(Server.class);
+            JavaPluginLoader mockPluginLoader = PowerMock.createMock(JavaPluginLoader.class);
+            Whitebox.setInternalState(mockPluginLoader, "server", mockServer);
            when(mockServer.getName()).thenReturn("TestBukkit");
            Logger.getLogger("Minecraft").setParent(Util.logger);
            when(mockServer.getLogger()).thenReturn(Util.logger);
@@ -75,8 +83,7 @@ public class TestInstanceCreator {
            PluginDescriptionFile pdf = PowerMockito.spy(new PluginDescriptionFile("Multiverse-Core", "2.2-Test",
                    "com.onarandombox.MultiverseCore.MultiverseCore"));
            when(pdf.getAuthors()).thenReturn(new ArrayList<String>());
-
-            core = PowerMockito.spy(new MultiverseCore(pluginLoader, mockServer, pdf, pluginDirectory, new File(pluginDirectory, "testPluginFile")));
+            core = PowerMockito.spy(new MultiverseCore(mockPluginLoader, pdf, pluginDirectory, new File(pluginDirectory, "testPluginFile")));
            PowerMockito.doAnswer(new Answer<Void>() {
                @Override
                public Void answer(InvocationOnMock invocation) throws Throwable {
@@ -113,8 +120,6 @@ public class TestInstanceCreator {
            Util.log("Creating world-folder: " + worldSkylandsFile.getAbsolutePath());
            worldSkylandsFile.mkdirs();

-
-
            // Give the server some worlds
            when(mockServer.getWorld(anyString())).thenAnswer(new Answer<World>() {
                @Override
@@ -151,7 +156,7 @@ public class TestInstanceCreator {

            when(mockServer.getPluginManager()).thenReturn(mockPluginManager);

-            when(mockServer.createWorld(Matchers.isA(WorldCreator.class))).thenAnswer(
+            when(mockServer.createWorld(ArgumentMatchers.isA(WorldCreator.class))).thenAnswer(
                    new Answer<World>() {
                        @Override
                        public World answer(InvocationOnMock invocation) throws Throwable {
@@ -251,9 +256,9 @@ public class TestInstanceCreator {
            when(commandSender.getServer()).thenReturn(mockServer);
            when(commandSender.getName()).thenReturn("MockCommandSender");
            when(commandSender.isPermissionSet(anyString())).thenReturn(true);
-            when(commandSender.isPermissionSet(Matchers.isA(Permission.class))).thenReturn(true);
+            when(commandSender.isPermissionSet(ArgumentMatchers.isA(Permission.class))).thenReturn(true);
            when(commandSender.hasPermission(anyString())).thenReturn(true);
-            when(commandSender.hasPermission(Matchers.isA(Permission.class))).thenReturn(true);
+            when(commandSender.hasPermission(ArgumentMatchers.isA(Permission.class))).thenReturn(true);
            when(commandSender.addAttachment(core)).thenReturn(null);
            when(commandSender.isOp()).thenReturn(true);

@@ -1,49 +0,0 @@
-package com.onarandombox.MultiverseCore.utils;
-
-import org.bukkit.event.Event;
-import org.bukkit.event.Listener;
-import org.bukkit.plugin.InvalidDescriptionException;
-import org.bukkit.plugin.InvalidPluginException;
-import org.bukkit.plugin.Plugin;
-import org.bukkit.plugin.PluginDescriptionFile;
-import org.bukkit.plugin.PluginLoader;
-import org.bukkit.plugin.RegisteredListener;
-import org.bukkit.plugin.UnknownDependencyException;
-
-import java.io.File;
-import java.util.Map;
-import java.util.Set;
-import java.util.regex.Pattern;
-
-public class TestPluginLoader implements PluginLoader {
-
-    @Override
-    public Plugin loadPlugin(File file) throws InvalidPluginException, UnknownDependencyException {
-        return null;
-    }
-
-    @Override
-    public PluginDescriptionFile getPluginDescription(File file) throws InvalidDescriptionException {
-        return null;
-    }
-
-    @Override
-    public Pattern[] getPluginFileFilters() {
-        return new Pattern[0];
-    }
-
-    @Override
-    public Map<Class<? extends Event>, Set<RegisteredListener>> createRegisteredListeners(Listener listener, Plugin plugin) {
-        return null;
-    }
-
-    @Override
-    public void enablePlugin(Plugin plugin) {
-
-    }
-
-    @Override
-    public void disablePlugin(Plugin plugin) {
-
-    }
-}
@@ -10,7 +10,7 @@ package com.onarandombox.MultiverseCore.utils;
 import org.bukkit.WorldCreator;
 import org.mockito.ArgumentMatcher;

-public class WorldCreatorMatcher extends ArgumentMatcher<WorldCreator> {
+public class WorldCreatorMatcher implements ArgumentMatcher<WorldCreator> {
    private WorldCreator worldCreator;
    private boolean careAboutSeeds = false;
    private boolean careAboutGenerators = false;
@@ -28,23 +28,23 @@ public class WorldCreatorMatcher extends ArgumentMatcher<WorldCreator> {
        this.careAboutGenerators = doICare;
    }

-    public boolean matches(Object creator) {
+    public boolean matches(WorldCreator creator) {
        Util.log("Checking world creators.");
        if (creator == null) {
            Util.log("The given creator was null, but I was checking: " + this.worldCreator.name());
            return false;
        }
-        Util.log("Checking Names...(" + ((WorldCreator) creator).name() + ") vs (" + this.worldCreator.name() + ")");
-        Util.log("Checking Envs...(" + ((WorldCreator) creator).environment() + ") vs (" + this.worldCreator.environment() + ")");
-        if (!((WorldCreator) creator).name().equals(this.worldCreator.name())) {
+        Util.log("Checking Names...(" + creator.name() + ") vs (" + this.worldCreator.name() + ")");
+        Util.log("Checking Envs...(" + creator.environment() + ") vs (" + this.worldCreator.environment() + ")");
+        if (!creator.name().equals(this.worldCreator.name())) {
            return false;
-        } else if (!((WorldCreator) creator).environment().equals(this.worldCreator.environment())) {
+        } else if (!creator.environment().equals(this.worldCreator.environment())) {
            Util.log("Checking Environments...");
            return false;
-        } else if (careAboutSeeds && ((WorldCreator) creator).seed() != this.worldCreator.seed()) {
+        } else if (careAboutSeeds && creator.seed() != this.worldCreator.seed()) {
            Util.log("Checking Seeds...");
            return false;
-        } else if (careAboutGenerators && !((WorldCreator) creator).generator().equals(this.worldCreator.generator())) {
+        } else if (careAboutGenerators && !creator.generator().equals(this.worldCreator.generator())) {
            Util.log("Checking Gens...");
            return false;
        }
Author	SHA1	Message	Date
Jeremy Wood	464eb8510a	Version 4.1.1-1.8.8-SNAPSHOT!	2019-08-25 00:14:43 -04:00
Jeremy Wood	81145fb96a	Bumped version to 4.1.1-SNAPSHOT for continued development. (cherry picked from commit `1ba693ffd5`)	2019-08-25 00:09:58 -04:00
Jeremy Wood	7a64f0613b	Release version 4.1.0. (cherry picked from commit `6ee0d8a05e`)	2019-08-25 00:09:09 -04:00
Jeremy Wood	02e4833b1a	World cloning should work once again! Resolves #1978 . (cherry picked from commit `d3ff2922fd`)	2019-08-25 00:09:03 -04:00
Jeremy Wood	78b32d3e53	Prevent importing worlds without .dat file. Fixes #1917 . (cherry picked from commit `5e2824abeb`)	2019-08-25 00:08:55 -04:00
Jeremy Wood	ec9a13aa45	Removed super dated toString in plugin main class. (cherry picked from commit `926e23bf19`)	2019-08-25 00:08:45 -04:00
Jeremy Wood	8610063307	Added --include-plugin-list to mv version command. (cherry picked from commit `841598232c`)	2019-08-25 00:08:36 -04:00
Jeremy Wood	464209f972	Now using CommandHandler 11! (cherry picked from commit `540b6abcab`)	2019-08-25 00:08:14 -04:00
Jeremy Wood	f33bd16d44	Added debug mode change event. (cherry picked from commit `38d5917915`)	2019-08-25 00:08:05 -04:00
Jeremy Wood	d21569e376	Bumped version to 4.1.0-SNAPSHOT for incoming features. (cherry picked from commit `da6bd20b42`)	2019-08-25 00:07:42 -04:00
Jeremy Wood	8ed67c5fb1	Bumped core protocol to 24. (cherry picked from commit `8859fa78fd`)	2019-08-25 00:07:34 -04:00
Jeremy Wood	e5d7f33512	Bumped version to 4.0.2-SNAPSHOT for continued development. (cherry picked from commit `4de6b9ce47`)	2019-08-25 00:07:26 -04:00
Jeremy Wood	a690516474	Release version 4.0.1. (cherry picked from commit `f595f4cdb9`)	2019-08-25 00:07:21 -04:00
Jeremy Wood	1790bb0f77	Make TravelAgent warning message a debug message. (cherry picked from commit `1581e1e009`)	2019-08-25 00:07:17 -04:00
Jeremy Wood	8f68d7fbcf	Bump version to 4.0.1 for continued development. (cherry picked from commit `32e72bb355`)	2019-08-25 00:07:10 -04:00
Jeremy Wood	d974ac15bb	Release version 4.0.0. (cherry picked from commit `950402a144`)	2019-08-25 00:07:06 -04:00
Jeremy Wood	bd1caf4c09	Bump protocol version to 23. (cherry picked from commit `20a1fcb9f5`)	2019-08-25 00:01:16 -04:00
Jeremy Wood	4d4a9c101f	Wrap the travel agent in an adapter. (cherry picked from commit `253ede1c9a`)	2019-08-25 00:01:08 -04:00
Jeremy Wood	3c1c54ae3d	Bumped version to 4.0.0-SNAPSHOT for 1.14 updates. (cherry picked from commit `d4454703e4`)	2019-08-25 00:00:57 -04:00
Jeremy Wood	24c47ab3d5	Bump version to 3.0.0 for release. (cherry picked from commit `9737b62813`)	2019-08-25 00:00:53 -04:00
Brokkonaut	4eda237b1b	Fix javadocs: it is now setCurrency(Material) instead of setCurrency(int) (cherry picked from commit `06bd1a838f`)	2019-08-25 00:00:48 -04:00
BaronyCraft	6aac64254c	allow coord arguments to setspawn command (#1975 ) * allow coord arguments to setspawn command * Implement PR changes as requested (cherry picked from commit `4343167240`)	2019-08-24 23:54:19 -04:00
Jeremy Wood	806b6e92ea	Removed commons-io as compile dependency to fix weird pom issues. (cherry picked from commit `7762aca019`)	2019-08-24 23:54:13 -04:00
Jeremy Wood	d34649a93e	Removed unnecessary json dependency. (cherry picked from commit `6ebc1188e8`)	2019-08-24 23:54:07 -04:00
Jeremy Wood	7b63c0a2b5	Don't send mvv url response to console twice when run from console. (cherry picked from commit `8b358b0955`)	2019-08-24 23:51:44 -04:00
Jeremy Wood	2e78b744f8	Added a theoretically working Hastebin option for mvv output. (cherry picked from commit `b201b09719`)	2019-08-24 23:51:36 -04:00
Jeremy Wood	8e3d942863	Disabled github paste service. (cherry picked from commit `4c9d108f08`)	2019-08-24 23:51:29 -04:00
Jeremy Wood	34e1d9b911	Removed pastie paste service. (cherry picked from commit `564a7d3684`)	2019-08-24 23:51:23 -04:00
Jeremy Wood	d5df299969	Cleaned up the pom a bit and fixed some dependency shading issues. (cherry picked from commit `aa574b6236`)	2019-08-24 23:51:15 -04:00
Jeremy Wood	bec76364aa	Fix another NPE in MaterialConverter. (cherry picked from commit `d7894eb847`)	2019-08-24 23:39:57 -04:00
Jeremy Wood	a7ba33fb3e	Fixes NPE in MaterialConverter. (cherry picked from commit `5e067fa72d`)	2019-08-24 23:39:51 -04:00
Jeremy Wood	fa3b99b5a3	Wtf... why was this still here. (cherry picked from commit `b1bf68a0ad`)	2019-08-24 23:39:45 -04:00
Jeremy Wood	f20db72dfd	Welcome to fully semantic versioning with 3.0.0-SNAPSHOT. (cherry picked from commit `89a8f24b14`)	2019-08-24 23:38:57 -04:00
Jeremy Wood	41be717b2c	Add null type annotations to MaterialConverter. (cherry picked from commit `43ecdc913a`)	2019-08-24 23:37:01 -04:00
Jeremy Wood	82457e5b79	Updated logging library to 1.1.1. (cherry picked from commit `9b88d1d491`)	2019-08-24 23:36:35 -04:00
Jeremy Wood	b341a6dd3a	Added MaterialConverter util class for type id conversions. (cherry picked from commit `606a3fa3ce`)	2019-08-24 23:30:51 -04:00
Jeremy Wood	8b241f15dc	Fixed potential NPE. (cherry picked from commit `a7cace0fdc`)	2019-08-24 23:25:15 -04:00
Jeremy Wood	ab9783047f	Removed deprecated MVConfigProperty and MVActiveConfigProperty. (cherry picked from commit `ae314108a3`)	2019-08-24 23:25:07 -04:00
Brokkonaut	fb3fe9c235	Option to disable the automatic purge of entities (#1933 ) (cherry picked from commit `d0180f05d5`)	2019-08-24 23:24:58 -04:00
Jeremy Wood	4fa3a34752	Shade in id converter. (cherry picked from commit `4552f70f7e`)	2019-08-24 23:18:53 -04:00
Jeremy Wood	5f1a13f6c4	Updated testing dependencies and maven plugins. (cherry picked from commit `4bdeacfd05`)	2019-08-24 23:18:45 -04:00
Jeremy Wood	459c37b77c	Updated Multiverse-Core for Bukkit 1.13.2. (But not actually) (cherry picked from commit `01f549f408`)	2019-08-24 23:17:42 -04:00
Jeremy Wood	1cca9580b8	Bumped protocol to 22. (cherry picked from commit `f7bbb5029b`)	2019-08-24 22:45:15 -04:00
Jeremy Wood	77df9488ed	Bumped version to 2.7.0-SNAPSHOT. (cherry picked from commit `658b5219c3`)	2019-08-24 22:45:07 -04:00
Jeremy Wood	e97c4fb238	Bumped version to 2.6.1-SNAPSHOT for continued development. (cherry picked from commit `fb505ab161`)	2019-08-24 22:45:03 -04:00
Jeremy Wood	5ba41d50d8	Version 2.6.0 release. (cherry picked from commit `284b3179ad`)	2019-08-24 22:44:59 -04:00
Jeremy Wood	478960945f	Added relativity options to ExactDestination. Implements #1892 . (cherry picked from commit `532ffacf5a`)	2019-08-24 22:44:53 -04:00
Jeremy Wood	4023b42938	Removed unnecessary for #1836 . (cherry picked from commit `2b4d386eb2`)	2019-08-24 22:44:43 -04:00
Jeremy Wood	4a7244b544	Hotfix for #1836 . (cherry picked from commit `bc20ecbe60`)	2019-08-24 22:42:25 -04:00
Jeremy Wood	b09460502a	Bumped protocol version to 21 for upcoming addition of Kotlin lib. (cherry picked from commit `4878395460`)	2019-08-24 22:42:20 -04:00
Jeremy Wood	65e0fe237d	Bump version to 2.6.0-SNAPSHOT for significant incoming changes. (cherry picked from commit `467ec809b5`)	2019-08-24 22:42:11 -04:00
Jeremy Wood	e3e548faec	Bump version to 2.5.1. (cherry picked from commit `ceb6850670`)	2019-08-24 22:42:07 -04:00
Jeremy Wood	fdee9c7e9a	Config travis and circle for Java 8. Resolves #1834 (cherry picked from commit `63abf33414`)	2019-08-24 22:42:03 -04:00
Jeremy Wood	2792001453	Build for Java 8. Might fix Travis and Circle... (cherry picked from commit `bcd7fcaee7`)	2019-08-24 22:41:59 -04:00
Jeremy Wood	537ddff634	Exclude old Bukkit from SerializationConfig. (cherry picked from commit `d18581828d`)	2019-08-24 22:38:19 -04:00
Jeremy Wood	3882164074	Testing new webhook (cherry picked from commit `54209e1535`)	2019-08-24 22:38:14 -04:00
Jeremy Wood	e79a49f7de	Updated to Buscript 2.0. This reduces the size of the plugin jar by over ONE MEGABYTE!! (cherry picked from commit `2ed59a866f`)	2019-08-24 22:38:01 -04:00
Jeremy Wood	1105aa12c8	Bumped version to 2.5.1-SNAPSHOT for continued development. (cherry picked from commit `42adfa8175`)	2019-08-24 22:35:59 -04:00
Jeremy Wood	51f582c3b5	Updated to 2.5.0 for release. (cherry picked from commit `12bcf71028`)	2019-08-24 22:35:44 -04:00
RezzedUp	67a104193a	Remove relative paths from potentially imported worlds. Before, relative paths that left the server's root directory were accepted. Now, the world name has all relative-path components trimmed off. (cherry picked from commit `da6a62b984`)	2019-08-24 22:34:43 -04:00
Jeremy Wood	d512e82a13	Updated plugin.yml. Can't believe I haven't done this sooner. (cherry picked from commit `806c5909c4`)	2019-08-24 22:34:31 -04:00
Eric Stokes	058ce5b29c	If we're gonna use TravisCI it should at least use JDK7 now. (cherry picked from commit `c00847f173`)	2019-08-24 22:34:24 -04:00
Eric Stokes	07e831f458	Added the autogenerated dependency-reduced-pom.xml to the .gitignore (cherry picked from commit `626e5d3e4b`)	2019-08-24 22:34:20 -04:00
Eric Stokes	900af0e497	Fixed deprecated assertTrue call in TestInstanceCreator (cherry picked from commit `f5e8a9c2e8`)	2019-08-24 22:31:17 -04:00
Eric Stokes	727a064a3b	Fixed deprecated use (since it's going away in 1.11) * MultiverseCore constructor shouldn't be deprecated, it just needed a fix. * Removed TestPluginLoader, it should have been mocked anyway. * New constructor takes a JavaPluginLoader instad of a PluginLoader, so we'll have to mock. You can't win, Java. If you make it `final`, I shall become more powerful than you can possibly imagine. (cherry picked from commit `3364709e53`)	2019-08-24 22:28:46 -04:00
rlf	43309dc841	Fixed failing tests (thanks @rlf) * This is the test fixing portion of PR#1746 * This fixes using a HashMap and relying on the order of the values. (cherry picked from commit `8fdc0ad3da`)	2019-08-24 22:28:31 -04:00
Eric Stokes	ff1448fb5f	Removed redundantThrows checkstyle rule, it's gone in 6.2+ See: checkstyle/checkstyle#473 (cherry picked from commit `67d172fd08`)	2019-08-24 22:26:42 -04:00
Eric Stokes	3116fe3c21	Added additional properties to the test checks for world properties (cherry picked from commit `f3a420d85c`)	2019-08-24 22:26:36 -04:00
rlf	92b6a9a6fc	Misc documentation fixes from @rlf (cherry picked from commit `aba1f05c5b`)	2019-08-24 22:26:28 -04:00
Eric Stokes	04e71d4377	Move to Java 7 (Maybe java 8 soon...) * Also bump CH to v10, which uses Java 7 * Probably don't use this build, I have to test it first... (cherry picked from commit `3db57ee1d9`)	2019-08-24 22:16:42 -04:00
httpdop	7bbd9348c2	Update dependencies and versions (cherry picked from commit `b3841adcc1`)	2019-08-24 22:15:35 -04:00
Liam	c5185c58be	Updated Vault repo New vault repo is here: http://nexus.hc.to/content/repositories/pub_releases (cherry picked from commit `0b0b3c0088`)	2019-08-24 22:14:06 -04:00
Jeremy Wood	c553062c14	Perhaps the id was important? (cherry picked from commit `a64a8d9605`)	2019-08-24 22:13:58 -04:00
Jeremy Wood	38c650c19a	Added snapshot repo to distro management. (cherry picked from commit `0a0d3ba527`)	2019-08-24 22:13:54 -04:00
Jeremy Wood	1be9974982	Moving to new version numbers - 2.5.0-SNAPSHOT (cherry picked from commit `a351fbb345`)	2019-08-24 22:13:43 -04:00
Eric Stokes	478da03d48	Fixes #1678 anchor list doesn't obey enforceaccess (cherry picked from commit `46359cfbfe`)	2019-08-24 22:13:37 -04:00
Eric Stokes	c85c68f816	Removed deprecated use of scheduleAsyncDelayedTask (cherry picked from commit `f52004cb76`)	2019-08-24 22:08:23 -04:00
Eric Stokes	f8c2221fcd	Added support for version command to upload to gist.github.com (cherry picked from commit `0e3848874a`)	2019-08-24 21:59:21 -04:00
Eric Stokes	3e58995fc8	Fixed spacing of spigotmc repo in POM (cherry picked from commit `71ba31a0f4`)	2019-08-24 21:59:09 -04:00
Eric Stokes	233fe409ec	Removed deprecated use of loadConfiguration The new method properly uses the Reader class and resolves the reason for deprecation, the lack of encoding support. (cherry picked from commit `7b1e77340b`)	2019-08-24 21:58:41 -04:00
Eric Stokes	fa34bbfc61	Fixed shortened links pointing to wiki pages that didn't exist anymore (cherry picked from commit `bebd4742b9`)	2019-08-24 21:56:16 -04:00
Eric Stokes	174fcdb1f7	#1672 Fixed typo that made animal lists behave like monster lists (cherry picked from commit `b933027523`)	2019-08-24 21:54:31 -04:00
Eric Stokes	58a489191d	#1671 Fixed lists changing not forcing validation of the elements (cherry picked from commit `bd1d852393`)	2019-08-24 21:54:19 -04:00
Eric Stokes	7cfb90759e	#1671 Fixed incorrect WorldPropertyValidator type (cherry picked from commit `648365f336`)	2019-08-24 21:54:07 -04:00
Jeremy Wood	5905b3132d	Deny import of "plugins" and "logs" folder as a world. Should fix #1655 (cherry picked from commit `d2d243caec`)	2019-08-24 21:52:32 -04:00
Jeremy Wood	0992584943	Don't accept null worlds from the server. Addresses #1649 . (cherry picked from commit `865434d1b1`)	2019-08-24 21:51:42 -04:00
Jeremy Wood	0eac36a65e	Use bukkit and not spigot api.	2019-08-24 21:47:22 -04:00