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

Ban System Rework #150

Manuell gemergt
Lixfel hat 38 Commits von ban-rework nach master 2021-01-30 08:42:35 +01:00 zusammengeführt
Diskussion 8 Commits 38 Geänderte Dateien 14 +11 -17
3 geänderte Dateien mit 11 neuen und 17 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 4137ebb61b werden angezeigt - Alle Commits anzeigen

2
src/de/steamwar/bungeecore/commands/BanCommand.java
Datei anzeigen

@ -56,7 +56,7 @@ public class BanCommand extends BasicCommand {
}
String msg = banReason.toString();
target.ban(banTime, msg, SteamwarUser.get(sender.getName()).getId(), args[1].equalsIgnoreCase("perma"));
Lixfel hat 2020-11-15 07:22:10 +01:00 kommentiert
Veraltet
Review
Link kopieren

Wenn du das ganze Team benachrichtigst, brauchst du nicht mehr den Spieler persönlich zu benachrichtigen.

Wenn du das ganze Team benachrichtigst, brauchst du nicht mehr den Spieler persönlich zu benachrichtigen.
ServerTeamchatCommand.sendToTeam(BungeeCore.CHAT_PREFIX + "§c" + target.getUserName() + " wurde von " + sender.getName() + " " + (args[1].equalsIgnoreCase("perma")?"Permanent":"bis zum " + banTime.toLocalDateTime().format(BungeeCore.DATE_FORMAT)) + " gebannt. §f§lGrund: §f" + banReason);
ServerTeamchatCommand.sendToTeam(BungeeCore.CHAT_PREFIX + target.getUserGroup().getColorCode() + target.getUserName() + "§c wurde von " + SteamwarUser.get(sender.getName()).getUserGroup().getColorCode() + sender.getName() + " §c" + (args[1].equalsIgnoreCase("perma")?"Permanent":"bis zum " + banTime.toLocalDateTime().format(BungeeCore.DATE_FORMAT)) + " gebannt. §f§lGrund: §f" + banReason);
Lixfel hat 2020-11-18 10:21:29 +01:00 kommentiert
Veraltet
Review
Link kopieren

In internen Messages musst du nicht die Person colorcoden.

In internen Messages musst du nicht die Person colorcoden.
Lixfel hat 2021-01-16 09:26:41 +01:00 kommentiert
Veraltet
Review
Link kopieren

Ok, das gefällt mir doch nicht so ganz mit dem IntArray etc. Evtl. wäre es eleganter, man macht da etwas a la new Message(...equals() ? "BAN_PERMA": "BAN_UNTIL") und wenn das object eine Message ist, (welche sich einfach nur zu merken Braucht, welchen String sie hat, ggf. auch noch parameter) und dann diese Message parst und den String entsprechend einfügt.

Ok, das gefällt mir doch nicht so ganz mit dem IntArray etc. Evtl. wäre es eleganter, man macht da etwas a la new Message(...equals() ? "BAN_PERMA": "BAN_UNTIL") und wenn das object eine Message ist, (welche sich einfach nur zu merken Braucht, welchen String sie hat, ggf. auch noch parameter) und dann diese Message parst und den String entsprechend einfügt.
}
Lixfel hat 2020-11-15 07:21:15 +01:00 kommentiert
Veraltet
Review
Link kopieren

Bitte erst die Aktion ausführen, dann die Aktion verkünden (im Fehlerfall kein Fehlannouncement).

Bitte erst die Aktion ausführen, dann die Aktion verkünden (im Fehlerfall kein Fehlannouncement).
public static Timestamp parseTime(CommandSender sender, String arg){

2
src/de/steamwar/bungeecore/commands/MuteCommand.java
Datei anzeigen

@ -52,6 +52,6 @@ public class MuteCommand extends BasicCommand {
}
String msg = muteReason.toString();
target.mute(muteTime, msg, SteamwarUser.get(sender.getName()).getId(), args[1].equalsIgnoreCase("perma"));
ServerTeamchatCommand.sendToTeam(BungeeCore.CHAT_PREFIX + "§c" + target.getUserName() + " wurde von " + sender.getName() + " " + (args[1].equalsIgnoreCase("perma")?"Permanent":"bis zum " + muteTime.toLocalDateTime().format(BungeeCore.DATE_FORMAT)) + " gemuted. §f§lGrund: §f" + muteReason);
ServerTeamchatCommand.sendToTeam(BungeeCore.CHAT_PREFIX + target.getUserGroup().getColorCode() + target.getUserName() + "§c wurde von " + SteamwarUser.get(sender.getName()).getUserGroup().getColorCode() + sender.getName() + " §c" + (args[1].equalsIgnoreCase("perma")?"Permanent":"bis zum " + muteTime.toLocalDateTime().format(BungeeCore.DATE_FORMAT)) + " gemuted. §f§lGrund: §f" + muteReason);
Lixfel hat 2020-11-15 07:22:18 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier genauso.

Hier genauso.
}
Lixfel hat 2020-11-15 07:21:27 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier genauso.

Hier genauso.
}

24
src/de/steamwar/bungeecore/sql/Punishment.java
Datei anzeigen

@ -119,20 +119,14 @@ public class Punishment {
public void updateEndTime(int from, String newreason, Timestamp newUpdate, boolean perma) {
StringBuilder newReason = new StringBuilder(this.reason);
if(newUpdate.before(endTime) || this.perma)
newReason.append("verkürtz vo" + (this.perma?"n":"m") + " ")
.append(getBantime())
.append(" von ")
.append(SteamwarUser.get(from).getUserName())
.append(" Grund: ")
.append(newreason);
else
newReason.append("verlängert vo" + (this.perma?"n":"m") + " ")
.append(getBantime())
.append(" von ")
.append(SteamwarUser.get(from).getUserName())
.append(" Grund: ")
.append(newreason);
newReason.append("verändert von ")
.append(SteamwarUser.get(from).getUserName())
.append(" von ")
.append(getBantime(endTime, this.perma))
.append(" auf ")
.append(getBantime(newUpdate, perma))
.append(" wegen ")
.append(newreason);
Lixfel hat 2021-01-16 09:27:58 +01:00 kommentiert
Veraltet
Review
Link kopieren

Offenes TODO?

Offenes TODO?
SQL.update("UPDATE Punishments SET EndTime = ?, Reason = ?, Perma = ? WHERE PunishmentId = ?", newUpdate, newReason.toString(), perma, id);
this.reason = newReason.toString();
@ -140,7 +134,7 @@ public class Punishment {
this.endTime = newUpdate;
Lixfel hat 2020-11-15 07:31:46 +01:00 kommentiert
Veraltet
Review
Link kopieren

Schreib doch nur name(), dann kannst du beim Type-Getten auch einfach valueOf() machen.

Schreib doch nur name(), dann kannst du beim Type-Getten auch einfach valueOf() machen.
}
public String getBantime() {
public String getBantime(Timestamp endTime, boolean perma) {
if(perma)
Lixfel hat 2020-11-15 07:31:58 +01:00 kommentiert
Veraltet
Review
Link kopieren

Siehe toDb()

Siehe toDb()
return "Permanent";
else