12
0

Bungee-SchemGUI #63

Geschlossen
Chaoscaot möchte 16 Commits von bungee-schemgui nach master mergen
Besitzer

Closes: #65

Closes: #65
Lixfel hat 2020-10-02 14:31:02 +02:00 Änderungen angefragt
@ -59,0 +84,4 @@
}
}
});
registerHandler(PacketIdManager.SAVE_SCHEMATIC, byteArrayDataInput -> {
Besitzer

Einsenden, schematicladen und schematicspeichern sollte immer noch direkt das SchematicSystem machen, und die entsprechenden Optionen gar nicht angezeigt werden, wenn der Server das Feature gar nicht unterstützt. Du könntest ja für solche Items spezielle Callbacks, die direkt im SpigotCore gecallt werden, einrichten

Einsenden, schematicladen und schematicspeichern sollte immer noch direkt das SchematicSystem machen, und die entsprechenden Optionen gar nicht angezeigt werden, wenn der Server das Feature gar nicht unterstützt. Du könntest ja für solche Items spezielle Callbacks, die direkt im SpigotCore gecallt werden, einrichten
@ -65,1 +127,3 @@
handlerMap.get(handler).handle(in);
if(handlerMap.containsKey(handler))
handlerMap.get(handler).handle(in);
else if(backupMap.containsKey(handler))
Besitzer

Wozu ist diese ganze Backup-Map da? Das sieht mir ziemlich nach einem Hack aus.

Wozu ist diese ganze Backup-Map da? Das sieht mir ziemlich nach einem Hack aus.
@ -0,0 +8,4 @@
import java.util.UUID;
public class MaterialsReturnPacket extends SpigotPacket{
Besitzer

Was macht das Packet?

Was macht das Packet?
Chaoscaot hat den Titel von WIP: Bungee-SchemGUI zu Bungee-SchemGUI 2020-10-05 11:37:04 +02:00 geändert
Chaoscaot hat den Titel von Bungee-SchemGUI zu WIP: Bungee-SchemGUI 2020-10-06 00:08:45 +02:00 geändert
Chaoscaot hat den Titel von WIP: Bungee-SchemGUI zu Bungee-SchemGUI 2020-10-06 11:19:40 +02:00 geändert
Lixfel hat 2020-10-29 09:35:41 +01:00 Änderungen angefragt
@ -56,12 +58,33 @@ public class BungeeReceiver implements PluginMessageListener {
Player player = Bukkit.getPlayer(SteamwarUser.get(byteArrayDataInput.readInt()).getUUID());
player.closeInventory();
});
BungeeReceiver.registerHandler((byte) 0x26, byteArrayDataInput -> {
Besitzer

0x26 ist nicht im PacketIdManager hinterlegt.

0x26 ist nicht im PacketIdManager hinterlegt.
@ -66,0 +85,4 @@
handlerMap.get(handler).handle(in);
else
throw new SecurityException("Could not find Handler");
Besitzer

Da fliegt doch sowieso eine Exception, daher ist das separate Handling nicht nötig.

Da fliegt doch sowieso eine Exception, daher ist das separate Handling nicht nötig.
@ -64,2 +64,2 @@
SQL.update("INSERT INTO Schematic (SchemName, SchemOwner, Item, SchemType) VALUES (?, ?, ?, ?) ON DUPLICATE KEY UPDATE Item = VALUES(Item), SchemType = VALUES(SchemType)",
schemName, schemOwner, item, schemType.toDB());
SQL.update("INSERT INTO Schematic (SchemName, SchemOwner, Item, SchemType, Rank) VALUES (?, ?, ?, ?, ?) ON DUPLICATE KEY UPDATE Item = VALUES(Item), SchemType = VALUES(SchemType)",
schemName, schemOwner, item, schemType.toDB(), 0);
Besitzer

Sollte nicht nötig sein, da der Rank von der Datenbank automatisch auf 0 gesetzt wird.

Sollte nicht nötig sein, da der Rank von der Datenbank automatisch auf 0 gesetzt wird.
Besitzer

On Hold bis nach der WGL (wir brauchen derzeit kein potentielles Bugfest :) )

On Hold bis nach der WGL (wir brauchen derzeit kein potentielles Bugfest :) )
YoyoNow hat die Änderungen 2020-11-16 22:03:41 +01:00 genehmigt
Lixfel hat die Änderungen 2020-11-18 10:03:13 +01:00 genehmigt
@ -59,6 +62,23 @@ public class BungeeReceiver implements PluginMessageListener {
Player player = Bukkit.getPlayer(SteamwarUser.get(byteArrayDataInput.readInt()).getUUID());
player.closeInventory();
});
BungeeReceiver.registerHandler((byte) PacketIdManager.CHANGE_ITEM_INV, byteArrayDataInput -> {
Besitzer

Cast zu byte überflüssig

Cast zu byte überflüssig
@ -32,6 +32,7 @@ import org.bukkit.plugin.java.JavaPlugin;
public class Core extends JavaPlugin{
private static Core instance;
private static final int version;
public static final String SCHEMATIC_PREFIX = "§eSchematic§8» §7";
Besitzer

Ist (zumindest hier) unused, sollte entfernt werden, hat hier also nix verloren.

Ist (zumindest hier) unused, sollte entfernt werden, hat hier also nix verloren.
Lixfel hat diesen Pull-Request 2020-12-15 16:54:20 +01:00 geschlossen

Pull-Request geschlossen

Anmelden, um an der Diskussion teilzunehmen.
Keine Beschreibung angegeben.