Azimkin/Bukkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Updating plugin nags to use logger

Browse Source
This commit is contained in:
Dinnerbone 2011-02-20 02:27:16 +00:00
parent 3c1255da0c
commit 414a19e247
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/bukkit/plugin/SimplePluginManager.java
View File

@ -240,7 +240,7 @@ public final class SimplePluginManager implements PluginManager {
*/ */
public void registerEvent(Event.Type type, Listener listener, Priority priority, Plugin plugin) { public void registerEvent(Event.Type type, Listener listener, Priority priority, Plugin plugin) {
if (!plugin.isEnabled()) { if (!plugin.isEnabled()) {
System.out.println("Warning! Plugin '" + plugin.getDescription().getName() + "' (ver " + plugin.getDescription().getVersion() + ") is registering events before it is enabled. It may be misbehaving and the author needs to fix this."); server.getLogger().warning("Plugin '" + plugin.getDescription().getName() + "' (ver " + plugin.getDescription().getVersion() + ") is registering events before it is enabled. It may be misbehaving and the author needs to fix this.");
} }
getEventListeners( type ).add(new RegisteredListener(listener, priority, plugin, type)); getEventListeners( type ).add(new RegisteredListener(listener, priority, plugin, type));
} }

4
src/main/java/org/bukkit/plugin/java/JavaPlugin.java
View File

@ -40,11 +40,11 @@ public abstract class JavaPlugin implements Plugin {
ClassLoader cLoader) { ClassLoader cLoader) {
initialize(pluginLoader, instance, desc, folder, plugin, cLoader); initialize(pluginLoader, instance, desc, folder, plugin, cLoader);
System.out.println("Using the stupidly long constructor " + desc.getMain() + "(PluginLoader, Server, PluginDescriptionFile, File, File, ClassLoader) is no longer recommended. Go nag the plugin author of " + desc.getName() + " to remove it! (Nothing is broken, we just like to keep code clean.)"); server.getLogger().warning("Using the stupidly long constructor " + desc.getMain() + "(PluginLoader, Server, PluginDescriptionFile, File, File, ClassLoader) is no longer recommended. Go nag the plugin author of " + desc.getName() + " to remove it! (Nothing is broken, we just like to keep code clean.)");
ArrayList<String> authors = desc.getAuthors(); ArrayList<String> authors = desc.getAuthors();
if (authors.size() > 0) { if (authors.size() > 0) {
System.out.println("Hint! It's probably someone called '" + authors.get(0) + "'"); server.getLogger().info("Hint! It's probably someone called '" + authors.get(0) + "'");
} }
} }