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

Fabric Mod Sender Integration. Mod itself is not final #307

Zusammengeführt
Lixfel hat 12 Commits von fabric_mod_sender nach master 2022-03-26 10:01:51 +01:00 zusammengeführt
Diskussion 0 Commits 12 Geänderte Dateien 4 +7 -8
2 geänderte Dateien mit 7 neuen und 8 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 98326fcaa0 werden angezeigt - Alle Commits anzeigen

9
src/de/steamwar/bungeecore/listeners/mods/Fabric.java
Datei anzeigen

@ -102,7 +102,7 @@ public class Fabric extends BasicListener {
return; return;
} }
if(!modsAreAbsent(mods)) { if(!neededModsContained(mods)) {
banPlayer(user, player); banPlayer(user, player);
return; return;
} }
@ -124,20 +124,19 @@ public class Fabric extends BasicListener {
return isSorted; return isSorted;
} }
private boolean modsAreAbsent(List<Mod> mods) { private boolean neededModsContained(List<Mod> mods) {
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-23 19:12:27 +01:00 kommentiert
Veraltet
Review
Link kopieren

Der Methodenname ist irreführend, denn der zurückgegebene Boolean ist ja, dass kein Mod fehlt.

Der Methodenname ist irreführend, denn der zurückgegebene Boolean ist ja, dass kein Mod fehlt.
return mods.stream() return mods.stream()
.map(Mod::getModName) .map(Mod::getModName)
.filter(neededMods::contains) .filter(neededMods::contains)
.distinct()
.count() == neededMods.size(); .count() == neededMods.size();
zOnlyKroks markierte diese Unterhaltung als gelöst
Lixfel hat 2022-03-23 19:12:41 +01:00 kommentiert
Review
Link kopieren

Distinct darf hier nicht nötig sein!

Distinct darf hier nicht nötig sein!
} }
public void banPlayer(SteamwarUser user, ProxiedPlayer player) { public void banPlayer(SteamwarUser user, ProxiedPlayer player) {
user.punish(Punishment.PunishmentType.Ban, user.punish(Punishment.PunishmentType.Ban,
Timestamp.from(Instant.now()), Timestamp.from(Instant.now()),
Message.parse("MODIFICATION_BAN_MESSAGE", player), Message.parse("MODIFICATION_BAN_MESSAGE", player, user.getUserName(), user.getId()),
0, 0,
true); true);
BungeeCore.log(Level.SEVERE, user.getUserName() + " " + user.getId() + Message.parse("MODIFICATION_BAN_LOG", player)); BungeeCore.log(Level.SEVERE, user.getUserName() + " " + user.getId() + Message.parse("MODIFICATION_BAN_LOG", player, user.getUserName()));
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-23 20:04:41 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier den String nicht mehr zusammenaddieren (Message.parse-Ergebnis enthält ja schon alles)

Hier den String nicht mehr zusammenaddieren (Message.parse-Ergebnis enthält ja schon alles)
} }
zOnlyKroks markierte diese Unterhaltung als gelöst
Lixfel hat 2022-03-23 19:09:19 +01:00 kommentiert
Review
Link kopieren

Änderung siehe BungeeCore.properties, Message.parse(..., player, user.getUserName(), user.getId())

Änderung siehe BungeeCore.properties, Message.parse(..., player, user.getUserName(), user.getId())
} }

6
src/de/steamwar/messages/BungeeCore.properties
Datei anzeigen

@ -585,6 +585,6 @@ RANK_EMBLEM=§eEmblem§8: {0}
RANK_NEEDED_FIGHTS_LEFT={0} §8(§7noch §e{1}§7 Kämpfe nötig§8) RANK_NEEDED_FIGHTS_LEFT={0} §8(§7noch §e{1}§7 Kämpfe nötig§8)
#Fabric Mod Sender #Fabric Mod Sender
MODIFICATION_BAN_MESSAGE = "Du hast probiert den FabricModSender zu umgehen / zu modifizieren!" MODIFICATION_BAN_MESSAGE="Du hast probiert den FabricModSender zu umgehen / zu modifizieren!"
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-23 20:05:01 +01:00 kommentiert
Veraltet
Review
Link kopieren

Immer noch: "" entfernen! Und in der Zeile drunter das leerzeichen

Immer noch: "" entfernen! Und in der Zeile drunter das leerzeichen
MODIFICATION_BAN_LOG = " hat probiert den Fabric Mod Sender zu editieren / umzugehen!" MODIFICATION_BAN_LOG= "{0} hat probiert den Fabric Mod Sender zu editieren / umzugehen!"
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-23 19:08:45 +01:00 kommentiert
Veraltet
Review
Link kopieren

Keine Leerzeichen um das =. Keine " um den String herum. Statt dem " hat" solltest du "{0} {1} hat" verwenden (dann übergibst du das noch als Parameter und überlässt die Formatierung dem MessageSystem.

Keine Leerzeichen um das =. Keine " um den String herum. Statt dem " hat" solltest du "{0} {1} hat" verwenden (dann übergibst du das noch als Parameter und überlässt die Formatierung dem MessageSystem.
MODIFICATION_CHECK_SUCCESS = "§eDeine Mods wurden geprüft und du darfst nun auf Arenen joinen!" MODIFICATION_CHECK_SUCCESS="§eDeine Mods wurden geprüft und du darfst nun auf Arenen joinen!"