SteamWar/LobbySystem2.0
Archiviert
13
0
Fork 0
Code Issues 5 Pull-Requests Projekte Releases Wiki Aktivität

Player Seat #3

Zusammengeführt
Lixfel hat 4 Commits von Seat nach master 2022-03-26 11:26:37 +01:00 zusammengeführt
Diskussion 0 Commits 4 Geänderte Dateien 2 +7 -3
1 geänderte Dateien mit 7 neuen und 3 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 9922f0b595 werden angezeigt - Alle Commits anzeigen

10
src/de/steamwar/lobby/listener/PlayerSeatListener.java
Datei anzeigen

@ -25,6 +25,7 @@ import org.bukkit.block.data.Bisected;
import org.bukkit.block.data.type.Stairs;
import org.bukkit.entity.AbstractArrow;
import org.bukkit.entity.Arrow;
import org.bukkit.entity.Entity;
import org.bukkit.entity.EntityType;
import org.bukkit.event.EventHandler;
import org.bukkit.event.block.Action;
@ -70,7 +71,7 @@ public class PlayerSeatListener extends BasicListener{
if (((Stairs) event.getClickedBlock().getBlockData()).getShape() != Stairs.Shape.STRAIGHT)
return;
if (event.getPlayer().isInsideVehicle())
if (event.getPlayer().isInsideVehicle() && isArrow(event.getPlayer().getVehicle()))
Lixfel hat 2022-03-26 10:03:37 +01:00 kommentiert
Veraltet
Review
Link kopieren

Könnten wir hier vor dem Entfernen noch darauf prüfen, dass es sich um einen Pfeil handelt (wie auch unten?). Ggf. könnte man das dann auch in eine separate Funktion auslagern.

Könnten wir hier vor dem Entfernen noch darauf prüfen, dass es sich um einen Pfeil handelt (wie auch unten?). Ggf. könnte man das dann auch in eine separate Funktion auslagern.
YoyoNow hat 2022-03-26 11:18:09 +01:00 kommentiert
Veraltet
Review
Link kopieren

Sollte jetzt behoben sein.

Sollte jetzt behoben sein.
event.getPlayer().getVehicle().remove();
if (event.getClickedBlock().getRelative(0, 1, 0).getType() != Material.AIR)
@ -93,7 +94,7 @@ public class PlayerSeatListener extends BasicListener{
public void onEntityDismount(EntityDismountEvent event) {
seats.remove(getSeatLocation(event.getDismounted().getLocation()));
if (event.getEntityType() != EntityType.PLAYER && event.getDismounted().getType() != EntityType.ARROW)
if (event.getEntityType() != EntityType.PLAYER && !isArrow(event.getDismounted()))
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-24 14:21:58 +01:00 kommentiert
Veraltet
Review
Link kopieren

Das ist zu allgemein (es sollte darauf geprüft werden, dass es tatsächlich ein Sitz ist), da ggf. später noch Bootsrennen in die Lobby kommen und da die Boote nicht unbedingt despawnen sollten.

Das ist zu allgemein (es sollte darauf geprüft werden, dass es tatsächlich ein Sitz ist), da ggf. später noch Bootsrennen in die Lobby kommen und da die Boote nicht unbedingt despawnen sollten.
return;
event.getDismounted().remove();
@ -101,7 +102,7 @@ public class PlayerSeatListener extends BasicListener{
@EventHandler
zOnlyKroks markierte diese Unterhaltung als gelöst
Lixfel hat 2022-03-24 14:22:05 +01:00 kommentiert
Review
Link kopieren

Siehe oben.

Siehe oben.
public void onPlayerQuit(PlayerQuitEvent event) {
if (event.getPlayer().isInsideVehicle() && event.getPlayer().getVehicle().getType() == EntityType.ARROW)
if (event.getPlayer().isInsideVehicle() && isArrow(event.getPlayer().getVehicle()))
event.getPlayer().getVehicle().remove();
}
@ -109,4 +110,7 @@ public class PlayerSeatListener extends BasicListener{
return new Location(world,location.getBlockX(), location.getBlockY(), location.getBlockZ());
}
private boolean isArrow(Entity entity) {
return entity.getType() == EntityType.ARROW;
}
}