Fix some SonarLint Issues and Modernize Java #517
Keine Reviewer
Lixfel
Label
Kein Label
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: SteamWar/BungeeCore#517
Laden…
In neuem Issue referenzieren
Einen Benutzer sperren
Keine Beschreibung angegeben.
Branch "modernize" löschen
Das Löschen eines Branches ist permanent. Obwohl der Branch für eine kurze Zeit weiter existieren könnte, kann diese Aktion in den meisten Fällen NICHT rückgängig gemacht werden. Fortfahren?
Würde an den Stellen nicht auch das .formatted funktionieren?
@ -51,3 +51,3 @@TOKENS.put(code, user);VelocityCore.getLogger().log(Level.INFO, "Created Discord Auth-Token: " + code + " for: " + user.getAsTag());VelocityCore.getLogger().log(Level.INFO, "Created Discord Auth-Token: {0} for: {1}", new Object[] {code, user.getAsTag()});Das new Object[] Zeugs halte ich für so viel unschöneren Code... bitte umarbeiten.
@ -59,3 +59,3 @@SteamwarUser user = SteamwarUser.get(player.getUniqueId());PunishmentCommand.ban(user, Punishment.PERMA_TIME, "Versuchte Exploit-Ausnutzung", SteamwarUser.get(-1), true);VelocityCore.getLogger().log(Level.SEVERE, user.getUserName() + " " + user.getId() + " wurde automatisch wegen jndi:ldap gebannt.");VelocityCore.getLogger().log(Level.SEVERE, "{0} {1} wurde automatisch wegen jndi:ldap gebannt.", new Object[]{user.getUserName(), user.getId()});.
@ -55,3 +55,3 @@SteamwarUser user = sender.user();playerModMap.put(uuid,new ArrayList<>(mods));VelocityCore.getLogger().log(Level.INFO, user.getUserName() + " declares mods: " + mods.stream().map(mod -> mod.getPlatform() + ":" + mod.getModName()).collect(Collectors.joining(" ")));VelocityCore.getLogger().log(Level.INFO, "{0} declares mods: {1}", new Object[]{user.getUserName(), mods.stream().map(mod -> mod.getPlatform() + ":" + mod.getModName()).collect(Collectors.joining(" "))});.
@ -84,3 +84,3 @@if(max == ModType.RED) {PunishmentCommand.ban(user, Timestamp.from(Instant.now().plus(7, ChronoUnit.DAYS)), message, SteamwarUser.get(-1), false);VelocityCore.getLogger().log(Level.SEVERE, user.getUserName() + " " + user.getId() + " wurde automatisch wegen der Mods " + modList + " gebannt.");VelocityCore.getLogger().log(Level.SEVERE, "{0} {1} wurde automatisch wegen der Mods {2} gebannt.", new Object[]{user.getUserName(), user.getId(), modList});.
@ -31,6 +31,7 @@ import lombok.RequiredArgsConstructor;import net.kyori.adventure.text.Component;import net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer;import net.kyori.adventure.title.Title;import org.jetbrains.annotations.NotNull;Sr, was mir hier gerade wieder eingefallen ist: NotNull und Nullable sind ganz böse (müssen weg). In IntelliJ hast du das Jetbrains-Package, der Buildserver eher weniger... Deswegen bitte rausentfernen.
War hier die einzige Stelle.
Ist durch JDA auch im Build-Path, sollte also auch in der CI-Bauen, kanns aber entfernen.