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

Add some QOL to EventCommand and EventRescheduleCommand #520

Offen
YoyoNow möchte 4 Commits von QOLEventCommands nach master mergen
pullen von: QOLEventCommands
Ziel: SteamWar:master
Diskussion 0 Commits 4 Geänderte Dateien 3 +3 -3
2 geänderte Dateien mit 3 neuen und 3 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit f4b5be22ac werden angezeigt - Alle Commits anzeigen

2
src/de/steamwar/velocitycore/commands/EventCommand.java
Datei anzeigen

@ -145,7 +145,7 @@ public class EventCommand extends SWCommand {
}; };
} }
@ClassMapper(Team.class) @ClassMapper(value = Team.class, local = true)
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2024-08-03 15:21:13 +02:00 kommentiert
Veraltet
Review
Link kopieren

Teams wird massenhaft Duplikate enthalten, ggf. ein Set verwenden, wenn das Backend das nicht ordentlich reduziert.

Teams wird massenhaft Duplikate enthalten, ggf. ein Set verwenden, wenn das Backend das nicht ordentlich reduziert.
public TypeMapper<Team> eventTeams() { public TypeMapper<Team> eventTeams() {
YoyoNow markierte diese Unterhaltung als gelöst
Lixfel hat 2024-08-04 12:10:09 +02:00 kommentiert
Review
Link kopieren

Müsste der nicht local sein oder geht das auch ohne?

Müsste der nicht local sein oder geht das auch ohne?
YoyoNow hat 2024-08-04 12:40:58 +02:00 kommentiert
Review
Link kopieren

Guter Punkt ändere ich

Guter Punkt ändere ich
return eventTeam(eventFight -> { return eventTeam(eventFight -> {
if (eventFight.getStartTime().after(new Timestamp(System.currentTimeMillis()))) return Collections.emptyList(); if (eventFight.getStartTime().after(new Timestamp(System.currentTimeMillis()))) return Collections.emptyList();
YoyoNow markierte diese Unterhaltung als gelöst
Lixfel hat 2024-08-04 12:11:35 +02:00 kommentiert
Review
Link kopieren

Funktioniert das so mit dem Timestamp?

Ansonsten über Instant.now().toTimestamp() konstruieren.

Funktioniert das so mit dem Timestamp? Ansonsten über Instant.now().toTimestamp() konstruieren.
YoyoNow hat 2024-08-04 12:40:46 +02:00 kommentiert
Review
Link kopieren

So hast du es schon beim EventStarter gemacht, deswegen so übernommen

So hast du es schon beim EventStarter gemacht, deswegen so übernommen

4
src/de/steamwar/velocitycore/commands/EventRescheduleCommand.java
Datei anzeigen

@ -65,12 +65,12 @@ public class EventRescheduleCommand extends SWCommand {
sender.system("EVENTRESCHEDULE_NO_FIGHT"); sender.system("EVENTRESCHEDULE_NO_FIGHT");
} }
@Mapper("BLUE_TEAM") @Mapper(value = "BLUE_TEAM", local = true)
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2024-08-03 15:22:43 +02:00 kommentiert
Veraltet
Review
Link kopieren

Starke Codeduplication, bitte mit EventCommand generalisieren.

Starke Codeduplication, bitte mit EventCommand generalisieren.
YoyoNow hat 2024-08-03 15:27:03 +02:00 kommentiert
Veraltet
Review
Link kopieren

Ähm wie genau stellst du dir das vor? Tabcompleted beides nicht das gleiche, also klar das map könnte ich irgendwie generalisieren, aber den Rest?

Ähm wie genau stellst du dir das vor? Tabcompleted beides nicht das gleiche, also klar das map könnte ich irgendwie generalisieren, aber den Rest?
Lixfel hat 2024-08-03 15:28:26 +02:00 kommentiert
Veraltet
Review
Link kopieren

Du hast halt einmal eine generische Condition (war das ein Predicate?) und einmal einen BiConsumer oder ähnlich?

Du hast halt einmal eine generische Condition (war das ein Predicate<?>?) und einmal einen BiConsumer oder ähnlich?
YoyoNow hat 2024-08-03 15:39:44 +02:00 kommentiert
Veraltet
Review
Link kopieren

Ich schaue es mir mal an ok

Ich schaue es mir mal an ok
public TypeMapper<Team> eventBlueTeams() { public TypeMapper<Team> eventBlueTeams() {
return EventCommand.eventTeam(eventFight -> List.of(eventFight.getTeamBlue())); return EventCommand.eventTeam(eventFight -> List.of(eventFight.getTeamBlue()));
} }
@Mapper("RED_TEAM") @Mapper(value = "RED_TEAM", local = true)
public TypeMapper<Team> eventRedTeams() { public TypeMapper<Team> eventRedTeams() {
return EventCommand.eventTeam(eventFight -> List.of(eventFight.getTeamRed())); return EventCommand.eventTeam(eventFight -> List.of(eventFight.getTeamRed()));
} }