From bc562b3f5b1d9d0c471d9c66c1bf3b450ce9d18e Mon Sep 17 00:00:00 2001
From: Bukkit/Spigot <noreply+git-bukkit@papermc.io>
Date: Sat, 15 Feb 2014 11:37:33 -0600
Subject: [PATCH] Provide warnings for spaces in plugin names. Addresses
 BUKKIT-5419

By: Wesley Wolfe <weswolf@aol.com>
---
 .../bukkit/plugin/PluginDescriptionFile.java    |  4 +++-
 .../org/bukkit/plugin/SimplePluginManager.java  | 17 ++++++++++++++++-
 2 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/paper-api/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java b/paper-api/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java
index 18f31bff31..9674914762 100644
--- a/paper-api/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java
+++ b/paper-api/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java
@@ -166,6 +166,7 @@ import com.google.common.collect.ImmutableMap;
  */
 public final class PluginDescriptionFile {
     private static final Yaml yaml = new Yaml(new SafeConstructor());
+    String rawName = null;
     private String name = null;
     private String main = null;
     private String classLoaderOf = null;
@@ -228,6 +229,7 @@ public final class PluginDescriptionFile {
      * <li>Case sensitive.
      * <li>The is the token referenced in {@link #getDepend()}, {@link
      *     #getSoftDepend()}, and {@link #getLoadBefore()}.
+     * <li>Using spaces in the plugin's name is deprecated.
      * </ul>
      * <p>
      * In the plugin.yml, this entry is named <code>name</code>.
@@ -799,7 +801,7 @@ public final class PluginDescriptionFile {
 
     private void loadMap(Map<?, ?> map) throws InvalidDescriptionException {
         try {
-            name = map.get("name").toString();
+            name = rawName = map.get("name").toString();
 
             if (!name.matches("^[A-Za-z0-9 _.-]+$")) {
                 throw new InvalidDescriptionException("name '" + name + "' contains invalid characters.");
diff --git a/paper-api/src/main/java/org/bukkit/plugin/SimplePluginManager.java b/paper-api/src/main/java/org/bukkit/plugin/SimplePluginManager.java
index 7d8a281e5b..d2fe422cad 100644
--- a/paper-api/src/main/java/org/bukkit/plugin/SimplePluginManager.java
+++ b/paper-api/src/main/java/org/bukkit/plugin/SimplePluginManager.java
@@ -135,13 +135,28 @@ public final class SimplePluginManager implements PluginManager {
                 if (name.equalsIgnoreCase("bukkit") || name.equalsIgnoreCase("minecraft") || name.equalsIgnoreCase("mojang")) {
                     server.getLogger().log(Level.SEVERE, "Could not load '" + file.getPath() + "' in folder '" + directory.getPath() + "': Restricted Name");
                     continue;
+                } else if (description.rawName.indexOf(' ') != -1) {
+                    server.getLogger().warning(String.format(
+                        "Plugin `%s' uses the space-character (0x20) in its name `%s' - this is discouraged",
+                        description.getFullName(),
+                        description.rawName
+                        ));
                 }
             } catch (InvalidDescriptionException ex) {
                 server.getLogger().log(Level.SEVERE, "Could not load '" + file.getPath() + "' in folder '" + directory.getPath() + "'", ex);
                 continue;
             }
 
-            plugins.put(description.getName(), file);
+            File replacedFile = plugins.put(description.getName(), file);
+            if (replacedFile != null) {
+                server.getLogger().severe(String.format(
+                    "Ambiguous plugin name `%s' for files `%s' and `%s' in `%s'",
+                    description.getName(),
+                    file.getPath(),
+                    replacedFile.getPath(),
+                    directory.getPath()
+                    ));
+            }
 
             Collection<String> softDependencySet = description.getSoftDepend();
             if (softDependencySet != null && !softDependencySet.isEmpty()) {