Merge pull request #2597 from Multiverse/cleanup

Cleanup

src/main/java/com/onarandombox/MultiverseCore/destination/DestinationFactory.java

@@ -27,7 +27,9 @@ import java.util.List;
 import java.util.Map;
 import java.util.regex.Pattern;
 
-/** A factory class that will create destinations from specific strings. */
+/**
+ * A factory class that will create destinations from specific strings.
+ */
 public class DestinationFactory {
 
     private static final Pattern CANNON_PATTERN = Pattern.compile("(?i)cannon-[\\d]+(\\.[\\d]+)?");
@@ -165,7 +167,12 @@ public class DestinationFactory {
         this.teleportCommand.addAdditonalPermission(other);
         return true;
     }
-    
+
+    /**
+     * Gets all the {@link MVDestination} identifiers registered.
+     *
+     * @return A collection of destination identifiers.
+     */
     public Collection<String> getRegisteredIdentifiers() {
         return this.destList.keySet();
     }

src/main/java/com/onarandombox/MultiverseCore/utils/PlayerFinder.java

@@ -135,7 +135,7 @@ public class PlayerFinder {
 
         List<Player> matchedPlayers = getMultiBySelector(playerSelector, sender);
         if (matchedPlayers == null || matchedPlayers.isEmpty()) {
-            Logging.warning("No player found with selector '%s' for %s.", playerSelector, sender.getName());
+            Logging.fine("No player found with selector '%s' for %s.", playerSelector, sender.getName());
             return null;
         }
         if (matchedPlayers.size() > 1) {

src/main/java/com/onarandombox/MultiverseCore/utils/WorldManager.java

@@ -55,7 +55,6 @@ import java.util.stream.Collectors;
  */
 public class WorldManager implements MVWorldManager {
     private final MultiverseCore plugin;
-    private final Pattern worldNamePattern = Pattern.compile("[a-zA-Z0-9/._-]+");
     private final WorldPurger worldPurger;
     private final Map<String, MultiverseWorld> worlds;
     private Map<String, WorldProperties> worldsFromTheConfig;