SteamWar/BauSystem
SteamWar/BauSystem
Archiviert
13
0
Fork 0
Code Issues Pull-Requests Releases Wiki Aktivität

RedstoneTester #202

Manuell gemergt
YoyoNow hat 29 Commits von RedstoneTester nach master 2021-04-01 18:21:40 +02:00 zusammengeführt
Diskussion 0 Commits 29 Geänderte Dateien 6 +10 -12
4 geänderte Dateien mit 10 neuen und 12 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 0b32638779 werden angezeigt - Alle Commits anzeigen

7
BauSystem_Main/src/de/steamwar/bausystem/BauSystem.java
Datei anzeigen

@ -23,6 +23,7 @@ import de.steamwar.bausystem.commands.*;
import de.steamwar.bausystem.world.*; import de.steamwar.bausystem.world.*;
import de.steamwar.core.CommandRemover; import de.steamwar.core.CommandRemover;
import de.steamwar.core.Core; import de.steamwar.core.Core;
import de.steamwar.core.VersionedRunnable;
import de.steamwar.scoreboard.SWScoreboard; import de.steamwar.scoreboard.SWScoreboard;
import de.steamwar.sql.SteamwarUser; import de.steamwar.sql.SteamwarUser;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
@ -99,9 +100,7 @@ public class BauSystem extends JavaPlugin implements Listener {
getCommand("script").setExecutor(new CommandScript()); getCommand("script").setExecutor(new CommandScript());
getCommand("simulator").setExecutor(new CommandSimulator()); getCommand("simulator").setExecutor(new CommandSimulator());
getCommand("simulator").setTabCompleter(new CommandSimulatorTabCompleter()); getCommand("simulator").setTabCompleter(new CommandSimulatorTabCompleter());
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-20 12:49:35 +01:00 kommentiert
Veraltet
Review
Link kopieren

Das kann man auch mit der VersionedRunnable machen.
btw: Anstat es nur 1.15 zu machen, könnte man es auch machen, dass man sich den Byte des Aktuellen Blockstates speichert und wenn dieser sich ändert dann wird das der Tester getriggert.

Das kann man auch mit der VersionedRunnable machen. btw: Anstat es nur 1.15 zu machen, könnte man es auch machen, dass man sich den Byte des Aktuellen Blockstates speichert und wenn dieser sich ändert dann wird das der Tester getriggert.
YoyoNow hat 2021-03-20 13:07:35 +01:00 kommentiert
Veraltet
Review
Link kopieren

Ich glaube trotzdem, dass diese Sache nur für die 1.15 unsere main bau version gebraucht wird. Und ich finde dies so schöner, weil man dann nicht noch die doppelte verschaltelung der VersionedRunnable hat, welche nur einen Wert hat. Sind 2 Objekte weniger für die VM.

Ich glaube trotzdem, dass diese Sache nur für die 1.15 unsere main bau version gebraucht wird. Und ich finde dies so schöner, weil man dann nicht noch die doppelte verschaltelung der VersionedRunnable hat, welche nur einen Wert hat. Sind 2 Objekte weniger für die VM.
Chaoscaot hat 2021-03-23 01:05:36 +01:00 kommentiert
Veraltet
Review
Link kopieren

Aber trotzdem sollte man es einheitlich halten, und da auch wenn dann eine VersionedRunnable nutzen.
Sonst wäre das jetzt der einzigste ort, wo das so gemacht werden würde.

Aber trotzdem sollte man es einheitlich halten, und da auch wenn dann eine VersionedRunnable nutzen. Sonst wäre das jetzt der einzigste ort, wo das so gemacht werden würde.
if (Core.getVersion() >= 15) {
getCommand("redstonetester").setExecutor(new CommandRedstoneTester()); getCommand("redstonetester").setExecutor(new CommandRedstoneTester());
}
getCommand("gui").setExecutor(new CommandGUI()); getCommand("gui").setExecutor(new CommandGUI());
Bukkit.getPluginManager().registerEvents(this, this); Bukkit.getPluginManager().registerEvents(this, this);
@ -112,9 +111,7 @@ public class BauSystem extends JavaPlugin implements Listener {
Bukkit.getPluginManager().registerEvents(new TNTSimulatorListener(), this); Bukkit.getPluginManager().registerEvents(new TNTSimulatorListener(), this);
Bukkit.getPluginManager().registerEvents(new CommandGUI(), this); Bukkit.getPluginManager().registerEvents(new CommandGUI(), this);
Bukkit.getPluginManager().registerEvents(new DetonatorListener(), this); Bukkit.getPluginManager().registerEvents(new DetonatorListener(), this);
if (Core.getVersion() >= 15) { VersionedRunnable.call(new VersionedRunnable(() -> Bukkit.getPluginManager().registerEvents(new RedstoneListener(), this), 15));
Bukkit.getPluginManager().registerEvents(new RedstoneListener(), this);
}
new AFKStopper(); new AFKStopper();
autoShutdown = Bukkit.getScheduler().runTaskLater(this, Bukkit::shutdown, 1200); autoShutdown = Bukkit.getScheduler().runTaskLater(this, Bukkit::shutdown, 1200);

4
BauSystem_Main/src/de/steamwar/bausystem/commands/CommandGUI.java
Datei anzeigen

@ -81,11 +81,7 @@ public class CommandGUI implements CommandExecutor, Listener {
if (Welt.noPermission(player, Permission.build)) if (Welt.noPermission(player, Permission.build))
return; return;
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-08 10:03:45 +01:00 kommentiert
Veraltet
Review
Link kopieren

Wenn man auf den Wand klickt, dann sollte man auch den Wand bekommen. Und keine Help message

Wenn man auf den Wand klickt, dann sollte man auch den Wand bekommen. Und keine Help message
player.closeInventory(); player.closeInventory();
if (Core.getVersion() >= 15) {
player.performCommand("redstonetester"); player.performCommand("redstonetester");
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-23 01:06:58 +01:00 kommentiert
Veraltet
Review
Link kopieren

Diesen Check sollte man eher im redstonetester command machen

Diesen Check sollte man eher im redstonetester command machen
YoyoNow hat 2021-03-24 17:30:57 +01:00 kommentiert
Veraltet
Review
Link kopieren

Ok mache ich am Wochenende

Ok mache ich am Wochenende
} else {
player.sendMessage(BauSystem.PREFIX + "Der RedstoneTester ist nicht in der 1.12 verfügbar");
}
}); });
inv.setItem(40, getMaterial("WOODEN_AXE", "WOOD_AXE"), "§eWorldedit Axt", getNoPermsLore(Arrays.asList("§8//§7wand"), player, "§cDu hast keine Worldeditrechte", Permission.worldedit), false, clickType -> { inv.setItem(40, getMaterial("WOODEN_AXE", "WOOD_AXE"), "§eWorldedit Axt", getNoPermsLore(Arrays.asList("§8//§7wand"), player, "§cDu hast keine Worldeditrechte", Permission.worldedit), false, clickType -> {

5
BauSystem_Main/src/de/steamwar/bausystem/commands/CommandRedstoneTester.java
Datei anzeigen

@ -24,6 +24,7 @@ import de.steamwar.bausystem.Permission;
import de.steamwar.bausystem.SWUtils; import de.steamwar.bausystem.SWUtils;
import de.steamwar.bausystem.world.RedstoneListener; import de.steamwar.bausystem.world.RedstoneListener;
import de.steamwar.bausystem.world.Welt; import de.steamwar.bausystem.world.Welt;
import de.steamwar.core.Core;
import org.bukkit.command.Command; import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor; import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender; import org.bukkit.command.CommandSender;
@ -44,6 +45,10 @@ public class CommandRedstoneTester implements CommandExecutor {
if (!(commandSender instanceof Player)) if (!(commandSender instanceof Player))
return false; return false;
Player player = (Player) commandSender; Player player = (Player) commandSender;
if (Core.getVersion() >= 15) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-25 11:23:20 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier kann man auch noch die VersionedCallable nehmen. :D

Hier kann man auch noch die VersionedCallable nehmen. :D
player.sendMessage(BauSystem.PREFIX + "Der RedstoneTester ist nicht in der 1.12 verfügbar");
return false;
}
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-10 17:32:05 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier noch der SW-Prefix

Hier noch der SW-Prefix
if (!permissionCheck(player)) { if (!permissionCheck(player)) {
return false; return false;

2
BauSystem_Main/src/de/steamwar/bausystem/world/RedstoneListener.java
Datei anzeigen

@ -100,7 +100,7 @@ public class RedstoneListener implements Listener {
private boolean validBlock(Player player, BlockData block) { private boolean validBlock(Player player, BlockData block) {
YoyoNow markierte diese Unterhaltung als gelöst
Chaoscaot hat 2021-03-08 10:05:18 +01:00 kommentiert
Review
Link kopieren

Der sollte schon irgendeine Permission haben e.g. Build

Der sollte schon irgendeine Permission haben e.g. Build
YoyoNow hat 2021-03-08 10:15:02 +01:00 kommentiert
Review
Link kopieren

Warum sollte man hierfür irgendeine Berechtigung haben brauchen?

Warum sollte man hierfür irgendeine Berechtigung haben brauchen?
if (block instanceof Powerable) return true; if (block instanceof Powerable) return true;
if (block instanceof Piston) return true; if (block instanceof Piston) return true;
player.sendMessage(BauSystem.PREFIX + "Unbekannte Position"); player.sendMessage(BauSystem.PREFIX + "§cUnbekannte Position");
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-23 01:09:50 +01:00 kommentiert
Veraltet
Review
Link kopieren

§c

§c
return false; return false;
} }
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Chaoscaot hat 2021-03-08 10:02:03 +01:00 kommentiert
Veraltet
Review
Link kopieren

Warum hat der Punkt eine andere Farbe?

Warum hat der Punkt eine andere Farbe?
YoyoNow hat 2021-03-08 10:15:23 +01:00 kommentiert
Veraltet
Review
Link kopieren

Weil Sonderzeichen bei uns immer dunkel grau sind.

Weil Sonderzeichen bei uns immer dunkel grau sind.