Fixed Fabric not Allowed on Testarena #138

Chaoscaot · 2020-10-24T19:22:52+02:00

Chaoscaot hat

2020-10-24 19:22:52 +02:00

kommentiert

Besitzer

Keine Beschreibung angegeben.

YoyoNow hat

2020-10-24 21:00:06 +02:00

kommentiert

Besitzer

Warum sollten auf TestArenen Fabrik auch nicht erlaubt sein, dort ist der Spieler selbst verantwortlich für die, die joinen!

Lixfel hat 2020-10-25 08:28:58 +01:00 Änderungen angefragt

src/de/steamwar/bungeecore/commands/FightCommand.java

						
				@ -129,3 +129,3 @@

				            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

Besitzer

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

Autor

Besitzer

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

Besitzer

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.