SteamWar/FightSystem
SteamWar/FightSystem
Archiviert
13
1
Fork 0
Code Issues 6 Pull-Requests Releases Wiki Aktivität

Add Bow Damge and Fire #261

Manuell gemergt
Lixfel hat 14 Commits von record_extension nach master 2021-04-23 19:52:27 +02:00 zusammengeführt
Diskussion 0 Commits 14 Geänderte Dateien 2 +10 -7
2 geänderte Dateien mit 10 neuen und 7 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit a364f86d1f werden angezeigt - Alle Commits anzeigen

2
FightSystem_API/src/de/steamwar/fightsystem/Config.java
Datei anzeigen

@ -384,6 +384,6 @@ public class Config {
return ArenaMode.Test.contains(mode);
}
public static boolean recording(){
return mode == ArenaMode.EVENT;
return true;//mode == ArenaMode.EVENT;
Chaoscaot markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-04-14 20:28:34 +02:00 kommentiert
Veraltet
Review
Link kopieren

Nope!

Nope!
}
}

15
FightSystem_Main/src/de/steamwar/fightsystem/listener/Recording.java
Datei anzeigen

@ -101,7 +101,7 @@ public class Recording implements Listener {
despawnTNT();
}
}.register();
new StateDependent(Config.recording(), FightState.All) {
new StateDependent(Config.recording(), FightState.Ingame) {
Chaoscaot markierte diese Unterhaltung als gelöst Veraltet
YoyoNow hat 2021-04-14 10:31:44 +02:00 kommentiert
Veraltet
Review
Link kopieren

Warum willst du den Bogen Listener immer haben? Ist das nicht etwas unnütz, weil wann genau im Fight hat man einen Bogen? Ich glaube nur im prepare und im Fight selber. Also kann man das hier doch nocht optimieren?

Warum willst du den Bogen Listener immer haben? Ist das nicht etwas unnütz, weil wann genau im Fight hat man einen Bogen? Ich glaube nur im prepare und im Fight selber. Also kann man das hier doch nocht optimieren?
@Override
public void enable() {
ProtocolLibrary.getProtocolManager().addPacketListener(BOW_PACKET_PROCESSOR);
@ -157,15 +157,14 @@ public class Recording implements Listener {
if(e.getEntityType() != EntityType.PLAYER)
return;
if(e.getCause() == EntityDamageEvent.DamageCause.FIRE_TICK || e.getCause() == EntityDamageEvent.DamageCause.FIRE)
RecordSystem.setOnFire(e.getEntity(), false);
Player p = (Player) e.getEntity();
if(isNotSent(p))
Chaoscaot markierte diese Unterhaltung als gelöst Veraltet
YoyoNow hat 2021-04-14 10:32:32 +02:00 kommentiert
Veraltet
Review
Link kopieren

Warum setzt du das Entity erst onFire und checkest dann erst, ob es zu senden ist. Das sollte man doch nur, wenn es gesendet werden muss!

Warum setzt du das Entity erst onFire und checkest dann erst, ob es zu senden ist. Das sollte man doch nur, wenn es gesendet werden muss!
return;
RecordSystem.damageAnimation(p);
if(e.getCause() == EntityDamageEvent.DamageCause.FIRE_TICK || e.getCause() == EntityDamageEvent.DamageCause.FIRE)
RecordSystem.setOnFire(p, false);
}
@EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
@ -173,7 +172,11 @@ public class Recording implements Listener {
if(e.getEntityType() != EntityType.PLAYER)
return;
RecordSystem.setOnFire(e.getEntity(), false);
Player p = (Player) e.getEntity();
if(isNotSent(p))
Chaoscaot markierte diese Unterhaltung als gelöst Veraltet
YoyoNow hat 2021-04-14 10:33:14 +02:00 kommentiert
Veraltet
Review
Link kopieren

Hier ignorierst du ob der Spieler zu senden ist?

Hier ignorierst du ob der Spieler zu senden ist?
return;
RecordSystem.setOnFire(p, false);
}
@EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)