Fix CommandScript

BauSystem_Main/src/de/steamwar/bausystem/commands/CommandScript.java

@@ -34,23 +34,32 @@ import java.util.List;
 
 public class CommandScript implements CommandExecutor {
 
-    private static final List<String> PAGES = new ArrayList<>();
+    public static final ItemStack BOOK = new ItemStack(Material.WRITTEN_BOOK, 1);
 
     static {
-        PAGES.add("§6Script System§8\n\n- Commands\n- Kommentare\n- Scriptausführung\n- Sleep\n- Variablen\n- Konstanten\n- Abfragen\n- Schleifen\n- \"echo\"");
+        List<String> pages = new ArrayList<>();
-        PAGES.add("§6Commands§8\n\nEin minecraft Befehl wird im Scriptbuch so hingeschrieben. Dabei kann man ein '/' weglassen. Um Befehle zu trennen kommen diese in neue Zeilen.\n\nStatt\n/tnt -> tnt\n//pos1 -> /pos1");
+        pages.add("§6Script System§8\n\n- Commands\n- Kommentare\n- Scriptausführung\n- Sleep\n- Variablen\n- Konstanten\n- Abfragen\n- Schleifen\n- \"echo\"");
-        PAGES.add("§6Kommentare§8\n\nFür ein Kommentar fängt die Zeile mit einem '#' an. Diese Zeilen werden bei dem Ausführen dann ignoriert.\n\nBeispiel:\n§9# TNT an/aus\ntnt§8");
+        pages.add("§6Commands§8\n\nEin minecraft Befehl wird im Scriptbuch so hingeschrieben. Dabei kann man ein '/' weglassen. Um Befehle zu trennen kommen diese in neue Zeilen.\n\nStatt\n/tnt -> tnt\n//pos1 -> /pos1");
-        PAGES.add("§6Scriptausführung§8\n\nWenn du mit dem Buch in der Hand links klickst wird dieses ausgeführt.");
+        pages.add("§6Kommentare§8\n\nFür ein Kommentar fängt die Zeile mit einem '#' an. Diese Zeilen werden bei dem Ausführen dann ignoriert.\n\nBeispiel:\n§9# TNT an/aus\ntnt§8");
-        PAGES.add("§6Sleep§8\n\nUm Sachen langsamer zu machen kann man ein 'sleep' in sein Script schreiben. Danach kommt eine Zahl mit der Anzahl der GameTicks die zu schlafen sind.\n\nBeispiel:\n§9# 1 Sekunde schlafen\nsleep 20§8");
+        pages.add("§6Scriptausführung§8\n\nWenn du mit dem Buch in der Hand links klickst wird dieses ausgeführt.");
-        PAGES.add("§6Variablen§8\n\nMit Variablen kann man sich Zahlen speichern. Man definiert diese mit 'var <NAME> <VALUE>'.\n\nBeispiel:\n§9# Setze i zu 0\nvar i 0§8\n\nEs gibt einige spezial values. Dazu zählen");
+        pages.add("§6Sleep§8\n\nUm Sachen langsamer zu machen kann man ein 'sleep' in sein Script schreiben. Danach kommt eine Zahl mit der Anzahl der GameTicks die zu schlafen sind.\n\nBeispiel:\n§9# 1 Sekunde schlafen\nsleep 20§8");
-        PAGES.add("§8'true', 'yes', 'false' und 'no', welche für 1, 1, 0 und 0 stehen.\n\nMan kann eine Variable auch um einen erhöhen oder verkleinern. Hierfür schreibt man statt einer Zahl '++', 'inc' oder '--', 'dec'.\n\nBeispiel:\n§9var i ++§8");
+        pages.add("§6Variablen§8\n\nMit Variablen kann man sich Zahlen speichern. Man definiert diese mit 'var <NAME> <VALUE>'.\n\nBeispiel:\n§9# Setze i zu 0\nvar i 0§8\n\nEs gibt einige spezial values. Dazu zählen");
-        PAGES.add("§8Variablen kann man referenzieren\ndurch '$' vor dem Variablennamen. Diese kann man in jedem Befehl verwenden.\n\nBeispiel:\n§9# Stacked um 10\nvar stacks 10\n/stack $stacks§8");
+        pages.add("§8'true', 'yes', 'false' und 'no', welche für 1, 1, 0 und 0 stehen.\n\nMan kann eine Variable auch um einen erhöhen oder verkleinern. Hierfür schreibt man statt einer Zahl '++', 'inc' oder '--', 'dec'.\n\nBeispiel:\n§9var i ++§8");
-        PAGES.add("§6Konstanten§8\n\nNeben den variablen gibt es noch 4 Konstante Werte, welche nicht mit dem 'var' Befehl verändert werden können.\n\nDiese sind:\n- trace\n- tnt\n- freeze\n- fire");
+        pages.add("§8Variablen kann man referenzieren\ndurch '$' vor dem Variablennamen. Diese kann man in jedem Befehl verwenden.\n\nBeispiel:\n§9# Stacked um 10\nvar stacks 10\n/stack $stacks§8");
-        PAGES.add("§6Abfragen§8\n\nMit Abfragen kann man nur Gleichheit von 2 Werten überprüft werden. Hierfür verwendet man\n'if <VAL> <VAL>'.\nNach den zwei Werten kann man ein oder 2 Jump-Points schreiben\n'if [...] <JP> (JP)'.");
+        pages.add("§6Konstanten§8\n\nNeben den variablen gibt es noch 4 Konstante Werte, welche nicht mit dem 'var' Befehl verändert werden können.\n\nDiese sind:\n- trace\n- tnt\n- freeze\n- fire");
-        PAGES.add("§8Ein Jump-Point ist eine Zeile Script, wohin man springen kann. Dieser wird mit einem '.' am Anfang der Zeile beschrieben und direkt danach der Jump-Point Namen ohne Leerzeichen.\n\nBeispiel:\n§9# Jump-Point X\n.X§8");
+        pages.add("§6Abfragen§8\n\nMit Abfragen kann man nur Gleichheit von 2 Werten überprüft werden. Hierfür verwendet man\n'if <VAL> <VAL>'.\nNach den zwei Werten kann man ein oder 2 Jump-Points schreiben\n'if [...] <JP> (JP)'.");
-        PAGES.add("§8Um zu einem Jump-Point ohne Abfrage zu springen kann man den\n'jump <JP>' Befehl verwenden.");
+        pages.add("§8Ein Jump-Point ist eine Zeile Script, wohin man springen kann. Dieser wird mit einem '.' am Anfang der Zeile beschrieben und direkt danach der Jump-Point Namen ohne Leerzeichen.\n\nBeispiel:\n§9# Jump-Point X\n.X§8");
-        PAGES.add("§6Schleifen§8\n\nSchleifen werden mit Jump-Points, if Abfragen und Jumps gebaut.\n\nBeispiel:\n§9var i 0\n.JUMP\nvar i ++\nif i 10 END JUMP\n.END§8");
+        pages.add("§8Um zu einem Jump-Point ohne Abfrage zu springen kann man den\n'jump <JP>' Befehl verwenden.");
-        PAGES.add("§6\"echo\"§8\n\nDer echo Befehl ist gut um Ausgaben zu tätigen. Hier drin kann man sowohl Variablen ausgeben, als auch Farbcodes verwenden. Es wird alles nach dem Befehl ausgegeben.\n\nBeispiel:\n§9echo &eSteam&8war &7war hier!§8");
+        pages.add("§6Schleifen§8\n\nSchleifen werden mit Jump-Points, if Abfragen und Jumps gebaut.\n\nBeispiel:\n§9var i 0\n.JUMP\nvar i ++\nif i 10 END JUMP\n.END§8");
+        pages.add("§6\"echo\"§8\n\nDer echo Befehl ist gut um Ausgaben zu tätigen. Hier drin kann man sowohl Variablen ausgeben, als auch Farbcodes verwenden. Es wird alles nach dem Befehl ausgegeben.\n\nBeispiel:\n§9echo &eSteam&8war &7war hier!§8");
+
+        BookMeta bookMeta = (BookMeta) BOOK.getItemMeta();
+        bookMeta.setGeneration(BookMeta.Generation.ORIGINAL);
+        bookMeta.setAuthor("§eSteam§8war");
+        bookMeta.setTitle("§7Script Buch");
+        bookMeta.setDisplayName("§7Script Buch");
+        bookMeta.setPages(pages);
+        BOOK.setItemMeta(bookMeta);
     }
 
     @Override
@@ -58,18 +67,7 @@ public class CommandScript implements CommandExecutor {
         if (!(sender instanceof Player)) {
             return false;
         }
-        Player player = (Player) sender;
+        ((Player) sender).getInventory().setItemInMainHand(BOOK);
-
-        ItemStack itemStack = new ItemStack(Material.WRITTEN_BOOK, 1);
-        BookMeta bookMeta = (BookMeta) itemStack.getItemMeta();
-        bookMeta.setGeneration(BookMeta.Generation.ORIGINAL);
-        bookMeta.setAuthor("§eSteam§8war");
-        bookMeta.setTitle("§7Script Buch");
-        bookMeta.setDisplayName("§7Script Buch");
-        bookMeta.setPages(PAGES);
-        itemStack.setItemMeta(bookMeta);
-
-        player.getInventory().setItemInMainHand(itemStack);
         return false;
     }
 

BauSystem_Main/src/de/steamwar/bausystem/world/ScriptListener.java

@@ -22,6 +22,7 @@ package de.steamwar.bausystem.world;
 import de.steamwar.bausystem.BauSystem;
 import de.steamwar.bausystem.commands.CommandFire;
 import de.steamwar.bausystem.commands.CommandFreeze;
+import de.steamwar.bausystem.commands.CommandScript;
 import de.steamwar.bausystem.commands.CommandTNT;
 import de.steamwar.bausystem.tracer.recorder.RecordManager;
 import de.steamwar.core.Core;
@@ -34,8 +35,6 @@ import org.bukkit.event.Listener;
 import org.bukkit.event.block.Action;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.event.inventory.InventoryCreativeEvent;
-import org.bukkit.event.inventory.InventoryInteractEvent;
-import org.bukkit.event.inventory.InventoryMoveItemEvent;
 import org.bukkit.event.player.PlayerCommandPreprocessEvent;
 import org.bukkit.event.player.PlayerInteractEvent;
 import org.bukkit.inventory.ItemStack;
@@ -58,7 +57,7 @@ public class ScriptListener implements Listener {
         if(item == null || isNoBook(item) || item.getItemMeta() == null)
             return;
 
-        if (item.getItemMeta().getDisplayName().equals("§8Script Buch")) {
+        if (item.getItemMeta().getDisplayName().equals(CommandScript.BOOK.getItemMeta().getDisplayName())) {
             return;
         }
 
@@ -76,43 +75,6 @@ public class ScriptListener implements Listener {
         new ScriptExecutor((BookMeta) item.getItemMeta(), event.getPlayer());
     }
 
-    @EventHandler(priority = EventPriority.HIGH)
-    public void onInventoryClick(InventoryClickEvent event) {
-        if (!(event.isShiftClick() && event.isLeftClick())) {
-            return;
-        }
-
-        ItemStack item = event.getCurrentItem();
-        if(item == null || isNoBook(item) || item.getItemMeta() == null)
-            return;
-
-        if (item.getItemMeta().getDisplayName().equals("§8Script Buch")) {
-            return;
-        }
-
-        event.setCancelled(true);
-        new ScriptExecutor((BookMeta) item.getItemMeta(), (Player) event.getWhoClicked());
-    }
-
-    @EventHandler(priority = EventPriority.HIGH)
-    public void onInventoryCreative(InventoryCreativeEvent event) {
-        System.out.println(event + " " + event.getCursor() + " " + event.getClick() + " " + event.getCurrentItem());
-        if (!(event.isShiftClick() && event.isLeftClick())) {
-            return;
-        }
-
-        ItemStack item = event.getCurrentItem();
-        if(item == null || isNoBook(item) || item.getItemMeta() == null)
-            return;
-
-        if (item.getItemMeta().getDisplayName().equals("§8Script Buch")) {
-            return;
-        }
-
-        event.setCancelled(true);
-        new ScriptExecutor((BookMeta) item.getItemMeta(), (Player) event.getWhoClicked());
-    }
-
     private boolean isNoBook(ItemStack item){
         switch(Core.getVersion()){
             case 12: