From 1c9d419c27e52347122f2eb43e294a36e0613501 Mon Sep 17 00:00:00 2001
From: Celtic Minstrel <celtic.minstrel.ca@some.place>
Date: Sat, 25 Feb 2012 12:39:10 -0500
Subject: [PATCH] [Bleeding] Add missing methods to Bukkit class, fix
 non-static methods, and add a junit test to ensure both these problems will
 be caught in future.

---
 src/main/java/org/bukkit/Bukkit.java          | 40 +++++++++++++++++--
 .../java/org/bukkit/BukkitMirrorTest.java     | 19 +++++++++
 2 files changed, 55 insertions(+), 4 deletions(-)
 create mode 100644 src/test/java/org/bukkit/BukkitMirrorTest.java

diff --git a/src/main/java/org/bukkit/Bukkit.java b/src/main/java/org/bukkit/Bukkit.java
index 893172c5..43fa781b 100644
--- a/src/main/java/org/bukkit/Bukkit.java
+++ b/src/main/java/org/bukkit/Bukkit.java
@@ -194,19 +194,19 @@ public final class Bukkit {
         return server.addRecipe(recipe);
     }
 
-    public List<Recipe> getRecipesFor(ItemStack result) {
+    public static List<Recipe> getRecipesFor(ItemStack result) {
         return server.getRecipesFor(result);
     }
 
-    public Iterator<Recipe> recipeIterator() {
+    public static Iterator<Recipe> recipeIterator() {
         return server.recipeIterator();
     }
 
-    public void clearRecipes() {
+    public static void clearRecipes() {
         server.clearRecipes();
     }
 
-    public void resetRecipes() {
+    public static void resetRecipes() {
         server.resetRecipes();
     }
 
@@ -289,4 +289,36 @@ public final class Bukkit {
     public static Messenger getMessenger() {
         return server.getMessenger();
     }
+
+    public static boolean getAllowEnd() {
+        return server.getAllowEnd();
+    }
+
+    public static File getUpdateFolderFile() {
+        return server.getUpdateFolderFile();
+    }
+
+    public static int getTicksPerAnimalSpawns() {
+        return server.getTicksPerAnimalSpawns();
+    }
+
+    public static int getTicksPerMonsterSpawns() {
+        return server.getTicksPerMonsterSpawns();
+    }
+
+    public static boolean useExactLoginLocation() {
+        return server.useExactLoginLocation();
+    }
+
+    public static GameMode getDefaultGameMode() {
+        return server.getDefaultGameMode();
+    }
+
+    public static void setDefaultGameMode(GameMode mode) {
+        server.setDefaultGameMode(mode);
+    }
+
+    public static OfflinePlayer[] getOfflinePlayers() {
+        return server.getOfflinePlayers();
+    }
 }
diff --git a/src/test/java/org/bukkit/BukkitMirrorTest.java b/src/test/java/org/bukkit/BukkitMirrorTest.java
new file mode 100644
index 00000000..5728e429
--- /dev/null
+++ b/src/test/java/org/bukkit/BukkitMirrorTest.java
@@ -0,0 +1,19 @@
+package org.bukkit;
+
+import static org.junit.Assert.*;
+
+import java.lang.reflect.Method;
+import java.lang.reflect.Modifier;
+
+import org.junit.Test;
+
+public class BukkitMirrorTest {
+    @Test
+    public final void test() throws NoSuchMethodException {
+        Method[] serverMethods = Server.class.getDeclaredMethods();
+        for(Method method : serverMethods) {
+            Method mirrorMethod = Bukkit.class.getDeclaredMethod(method.getName(), method.getParameterTypes());
+            assertTrue("Bukkit." + method.getName() + " must be static!", Modifier.isStatic(mirrorMethod.getModifiers()));
+        }
+    }
+}