SteamWar/BungeeCore
SteamWar/BungeeCore
Archiviert
13
2
Fork 1
Code Issues 14 Pull-Requests 4 Releases Wiki Aktivität

Fixed Fabric not Allowed on Testarena #138

Manuell gemergt
Lixfel hat 1 Commits von fixed-fabric nach master 2020-10-27 15:29:43 +01:00 zusammengeführt
Diskussion 1 Commits 1 Geänderte Dateien 1 +1 -1
1 geänderte Dateien mit 1 neuen und 1 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit bf895903f8 werden angezeigt - Alle Commits anzeigen

2
src/de/steamwar/bungeecore/commands/FightCommand.java
Datei anzeigen

@ -128,7 +128,7 @@ public class FightCommand extends BasicCommand {
if(!(sender instanceof ProxiedPlayer))
return;
if(Fabric.isFabric((ProxiedPlayer) sender)) {
if(Fabric.isFabric((ProxiedPlayer) sender) && !precommand.equals("/bau testarena")) {
Lixfel hat 2020-10-25 08:28:55 +01:00 kommentiert
Review
Link kopieren

Der Precommand kann auch anders sein, wenn die Arena anders gestartet wird. Man kann ja durchaus mit Fabric auf Testarenen, nur nicht, wenn man diese Startet.

Der Precommand kann auch anders sein, wenn die Arena anders gestartet wird. Man kann ja durchaus mit Fabric auf Testarenen, nur nicht, wenn man diese Startet.
Chaoscaot hat 2020-10-25 09:51:53 +01:00 kommentiert
Review
Link kopieren

Da liegt dann das Problem, dass die Testarenen auch über createArena erstellt werden, wo sie dann ggf. bei Fabric abgefangen werden.
Und deshalb kann dann auch nur verhindert werden durch fabric sofern es dann keine Testarena ist.

Da liegt dann das Problem, dass die Testarenen auch über createArena erstellt werden, wo sie dann ggf. bei Fabric abgefangen werden. Und deshalb kann dann auch nur verhindert werden durch fabric sofern es dann keine Testarena ist.
YoyoNow hat 2020-10-25 10:25:40 +01:00 kommentiert
Review
Link kopieren

Also im Moment kann man nicht auf TestArenen (diese starten), jedoch soll dieses möglich sein. Somit hat Chaos einfach eine Abfrage gemacht, ob man selbst eine TestArena startet. Somit sehe ich diese Sache als richtig an.
Der value gegen den geprüft wird ist ja an der Stelle, wo die TestArena gestartet wird gehardcoded "/bau testarena " womit die abfrage eigentlich kein Problem geben sollte.
Somit werde ich diesen Pull-Request genehmigen.

Also im Moment kann man nicht auf TestArenen (diese starten), jedoch soll dieses möglich sein. Somit hat Chaos einfach eine Abfrage gemacht, ob man selbst eine TestArena startet. Somit sehe ich diese Sache als richtig an. Der value gegen den geprüft wird ist ja an der Stelle, wo die TestArena gestartet wird gehardcoded `"/bau testarena "` womit die abfrage eigentlich kein Problem geben sollte. Somit werde ich diesen Pull-Request genehmigen.
BungeeCore.send((ProxiedPlayer) sender, BungeeCore.CHAT_PREFIX + "§cHabe ich dir nicht gesagt, dass man mit Fabric nicht auf die Arenen darf?");
return;
}