SteamWar/FightSystem
Archiviert
13
1

Dragonegg Teleport #356

Zusammengeführt
Lixfel hat 4 Commits von Dragonegg_teleport nach master 2022-07-03 16:10:36 +02:00 zusammengeführt
Mitglied
Keine Beschreibung angegeben.
xSpw hat 2 Commits 2022-07-03 14:51:19 +02:00 hinzugefügt
Added BlockFromToListener
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c1493be6c1
Added Format
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
9774b28d57
Lixfel hat 2022-07-03 14:59:10 +02:00 Änderungen angefragt
@ -0,0 +1,31 @@
package de.steamwar.fightsystem.listener;
Besitzer

Hier fehlt noch der AGPL-License Header.

Hier fehlt noch der AGPL-License Header.
@ -0,0 +11,4 @@
public class BlockFromToListener implements Listener {
public BlockFromToListener(){
new StateDependentListener(ArenaMode.All, FightState.All, this);
Besitzer

Man könnte das ganze statt einzelnen Listener mit in den Permanent-Listener hineinpacken.

Man könnte das ganze statt einzelnen Listener mit in den Permanent-Listener hineinpacken.
@ -0,0 +16,4 @@
@EventHandler
public void onBlockFromTo(BlockFromToEvent event) {
if (Fight.getBlueTeam().getSchemRegion().inRegion(event.getBlock())
Besitzer

Schemregion ist glaube hier etwas unpassend (da sich das wirklich auf die Schematicmaße beschränkt). ExtendRegion wäre da (meiner Einschätzung nach) passender, da dann das Ziel überall innerhalb der Ausfahrmaße sein kann.

Schemregion ist glaube hier etwas unpassend (da sich das wirklich auf die Schematicmaße beschränkt). ExtendRegion wäre da (meiner Einschätzung nach) passender, da dann das Ziel überall innerhalb der Ausfahrmaße sein kann.
@ -0,0 +21,4 @@
return;
}
if (Fight.getRedTeam().getSchemRegion().inRegion(event.getBlock())
Besitzer

Das ist hier Codedopplung, es wäre schöner, wenn du das z.B. mit for(FightTeam team : Fight.teams()) machen würdest.

Das ist hier Codedopplung, es wäre schöner, wenn du das z.B. mit for(FightTeam team : Fight.teams()) machen würdest.
@ -0,0 +26,4 @@
return;
}
event.setCancelled(true);
Besitzer

Ich denke, du solltest auch noch den BlockTyp filtern, weil so kann dann auch kein Wasser etc. mehr herausfließen.

Ich denke, du solltest auch noch den BlockTyp filtern, weil so kann dann auch kein Wasser etc. mehr herausfließen.
Autor
Mitglied

Wie meinst du das?

Wie meinst du das?
Besitzer

Naja, das BlockFromToEvent wird nicht nur bei teleportierenden Dracheneiern aufgerufen, aber wir wollen nur die Dracheneier blockieren... müsste über die Methode getBlock().getType() abrufbar sein, was jetzt eigentlich der auslösende Block ist.

Naja, das BlockFromToEvent wird nicht nur bei teleportierenden Dracheneiern aufgerufen, aber wir wollen nur die Dracheneier blockieren... müsste über die Methode getBlock().getType() abrufbar sein, was jetzt eigentlich der auslösende Block ist.
Autor
Mitglied

Stimmt, daran habe ich nicht gedacht

Stimmt, daran habe ich nicht gedacht
xSpw hat 1 Commit 2022-07-03 16:02:51 +02:00 hinzugefügt
Deleted BlockFromToListener
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c245f4c2ba
Updated DragonTeleport in Permanent class
Lixfel hat 2022-07-03 16:04:06 +02:00 überprüft
@ -150,0 +152,4 @@
@EventHandler
public void onBlockFromTo(BlockFromToEvent event) {
if(event.getBlock().getType().equals(Material.DRAGON_EGG)) return;
Besitzer

Jetzt ist dir hier ein kleiner Invertierungsfehler passiert :)

Jetzt ist dir hier ein kleiner Invertierungsfehler passiert :)
Besitzer

Habe gerade noch nachgeschaut, das Material DRAGON_EGG und das BlockFromToEvent gabs schon in der 1.8. Da muss man ggf. immer etwas aufpassen, da manche Materials von der 1.12 zur 1.13 umbenannt wurden und es nicht immer alle Events schon in der 1.8 gab. Wenn mal der Fall auftritt, werde ich dich in unsere Workarounds dazu einführen (SWMaterial und Core.getVersion())

Habe gerade noch nachgeschaut, das Material DRAGON_EGG und das BlockFromToEvent gabs schon in der 1.8. Da muss man ggf. immer etwas aufpassen, da manche Materials von der 1.12 zur 1.13 umbenannt wurden und es nicht immer alle Events schon in der 1.8 gab. Wenn mal der Fall auftritt, werde ich dich in unsere Workarounds dazu einführen (SWMaterial und Core.getVersion())
xSpw hat 1 Commit 2022-07-03 16:09:21 +02:00 hinzugefügt
Fixed Inversion
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
1dad36c5dc
Lixfel hat die Änderungen 2022-07-03 16:10:13 +02:00 genehmigt
Lixfel hat Commit 49caee0c0b in master 2022-07-03 16:10:36 +02:00 gemerged
Lixfel löschte die Branch Dragonegg_teleport 2022-07-03 16:10:36 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.