SteamWar/BungeeCore
Archiviert
13
2

More multilingual transfer #146

Manuell gemergt
Lixfel hat 6 Commits von moreMultiLingual nach master 2020-11-28 09:23:21 +01:00 zusammengeführt
7 geänderte Dateien mit 40 neuen und 20 gelöschten Zeilen
Nur Änderungen aus Commit 89d57e3f48 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -20,6 +20,7 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import net.md_5.bungee.api.ChatColor;
import net.md_5.bungee.api.CommandSender;
@ -32,7 +33,7 @@ public class AlertCommand extends BasicCommand {
@Override
public void execute(CommandSender sender, String[] args) {
if(args.length == 0){
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "/alert [Nachricht]");
Message.send("USAGE_ALERT", sender);
return;
}
@ -41,8 +42,6 @@ public class AlertCommand extends BasicCommand {
for (String arg : args){
msgBuilder.append(arg).append(" ");
}
Review

Ein 'String.join(" ", args)' würde es auch hier tun

Ein 'String.join(" ", args)' würde es auch hier tun
String msg = msgBuilder.toString();
msg = ChatColor.translateAlternateColorCodes('&', msg);
BungeeCore.broadcast(msg);
Message.broadcast("ALERT", ChatColor.translateAlternateColorCodes('&', msgBuilder.toString()));
}
}

Datei anzeigen

@ -19,7 +19,7 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import de.steamwar.bungeecore.sql.SteamwarUser;
import net.md_5.bungee.api.CommandSender;
@ -38,7 +38,7 @@ public class BanCommand extends BasicCommand {
@Override
public void execute(CommandSender sender, String[] args) {
if(args.length < 3){
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "/ban [Spieler] [dd.mm.yyyy oder dd.mm.yyyy_hh:mm oder perma] [Grund]");
Message.send("USAGE_BAN", sender);
return;
}
@ -55,7 +55,7 @@ public class BanCommand extends BasicCommand {
banReason.append(args[i]).append(" ");
}
String msg = banReason.toString();
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "Du hast " + target.getUserName() + " gebannt. Grund: §c" + msg);
Message.send("BAN_YOU_BANNED", sender, target.getUserGroup(), msg);
target.ban(banTime, msg);
}
@ -73,7 +73,7 @@ public class BanCommand extends BasicCommand {
Date parsedDate = dateFormat.parse(arg.split("_")[0]);
return new java.sql.Timestamp(parsedDate.getTime());
}catch(ParseException exception){
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "§cUngültige Zeitangabe.");
Message.send("BAN_INVALID_TIME", sender);
return null;
}
}

Datei anzeigen

@ -20,6 +20,7 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import de.steamwar.bungeecore.sql.SteamwarUser;
import net.md_5.bungee.api.CommandSender;
import net.md_5.bungee.api.ProxyServer;
@ -55,7 +56,7 @@ abstract class BasicCommand extends Command implements TabExecutor {
protected SteamwarUser user(CommandSender sender, String arg){
SteamwarUser target = SteamwarUser.get(arg);
if(target == null)
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "§cDer Spieler existiert nicht.");
Message.send("UNKNWON_PLAYER", sender);
return target;
}
}

Datei anzeigen

@ -19,7 +19,7 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import de.steamwar.bungeecore.sql.SWException;
import de.steamwar.bungeecore.sql.SteamwarUser;
import net.md_5.bungee.api.CommandSender;
@ -40,6 +40,6 @@ public class BugCommand extends BasicCommand {
String message = String.join(" ", args);
SteamwarUser user = SteamwarUser.get(player.getUniqueId());
SWException.log(server, message, player.getName() + " " + user.getId());
BungeeCore.send(player, BungeeCore.CHAT_PREFIX + "§7Dein Bugreport wurde gemeldet.");
Message.send("BUG_SAVED", player);
}
}

Datei anzeigen

@ -19,9 +19,8 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import net.md_5.bungee.api.CommandSender;
import net.md_5.bungee.api.chat.TextComponent;
import net.md_5.bungee.api.connection.ProxiedPlayer;
public class DenyCommand extends BasicCommand {
@ -34,7 +33,7 @@ public class DenyCommand extends BasicCommand {
public void execute(CommandSender sender, String[] args) {
if(sender instanceof ProxiedPlayer){
ProxiedPlayer player = (ProxiedPlayer) sender;
player.sendMessage(new TextComponent(BungeeCore.CHAT_PREFIX + "§cUnbekannter Befehl."));
Message.send("UNKNOWN_COMMAND", player);
}
}
}

Datei anzeigen

@ -19,7 +19,7 @@
package de.steamwar.bungeecore.commands;
import de.steamwar.bungeecore.BungeeCore;
import de.steamwar.bungeecore.Message;
import de.steamwar.bungeecore.sql.IgnoreSystem;
import de.steamwar.bungeecore.sql.SteamwarUser;
import net.md_5.bungee.api.CommandSender;
@ -34,7 +34,7 @@ public class IgnoreCommand extends BasicCommand {
@Override
public void execute(CommandSender sender, String[] args) {
if(args.length < 1) {
BungeeCore.send(sender, BungeeCore.CHAT_PREFIX + "§8/§eignore §8[§7Spieler§8]");
Message.send("USAGE_IGNORE", sender);
return;
}
@ -45,17 +45,17 @@ public class IgnoreCommand extends BasicCommand {
SteamwarUser target = SteamwarUser.get(args[0]);
if(target == null){
BungeeCore.send(p, BungeeCore.CHAT_PREFIX + "§cDiesen Spieler gibt es nicht.");
Message.send("UNKNWON_PLAYER", p);
return;
}else if(target.equals(user)){
Review

Warum sind das hier 'else if' Statements könnte man das nicht einfach alles als einfache 'if' Statements machen. Wo dann mit 'return' immer abgebrochen wird? Ich denke dann kann man das einfacher lesen. Weil es einfach immer ein 'if' dann der Abbruch weil das was sein soll nicht so ist.

Warum sind das hier 'else if' Statements könnte man das nicht einfach alles als einfache 'if' Statements machen. Wo dann mit 'return' immer abgebrochen wird? Ich denke dann kann man das einfacher lesen. Weil es einfach immer ein 'if' dann der Abbruch weil das was sein soll nicht so ist.
BungeeCore.send(p, BungeeCore.CHAT_PREFIX + "§cWie willst du dich selber ignorieren?");
Message.send("IGNORE_YOURSELF", p);
return;
}else if(IgnoreSystem.isIgnored(user, target)){
Review

s.o.

s.o.
BungeeCore.send(p, BungeeCore.CHAT_PREFIX + "§cDu ignorierst diesen Spieler bereits.");
Message.send("IGNORE_ALREADY", p);
return;
}
IgnoreSystem.ignore(user, target);
BungeeCore.send(p, BungeeCore.CHAT_PREFIX + "§7Du ignorierst nun §e" + target.getUserName() + "§8.");
Message.send("IGNORE_IGNORE", p, target.getUserName());
}
}

Datei anzeigen

@ -1,5 +1,9 @@
PREFIX=§eSteam§8War»
UNKNOWN_COMMAND=§cUnbekannter Befehl.
Review

Ich würde noch persönlich für alle properties (Keys) also 'UNKNOWN_COMMAND' und so, mehrere Enums machen, sodass es einfacher ist Sachen zu ändern. Sonst muss man immer genau wissen, wo alles dieser Key verwendet wurde um es zu ändern.

Ich würde noch persönlich für alle properties (Keys) also 'UNKNOWN_COMMAND' und so, mehrere Enums machen, sodass es einfacher ist Sachen zu ändern. Sonst muss man immer genau wissen, wo alles dieser Key verwendet wurde um es zu ändern.
Review

Hierbei würde ich für jede Gruppe an Messages ein ENUM machen, also 'Unknown', 'Help', 'Usage', 'Other', 'Ignore' und dann sowas wie 'Ban', 'Mute' oder 'Punishments'

Hierbei würde ich für jede Gruppe an Messages ein ENUM machen, also 'Unknown', 'Help', 'Usage', 'Other', 'Ignore' und dann sowas wie 'Ban', 'Mute' oder 'Punishments'
Review

Nein, IntelliJ leitet einen nämlich beim Klicken auf z.B: "UNKNOWN_COMMAND"-String direkt in die Config-Datei weiter, wenn da ein Enum extra eingesetzt wird, welches umständlich gewartet werden muss, geht dieser vorteil verloren.

Nein, IntelliJ leitet einen nämlich beim Klicken auf z.B: "UNKNOWN_COMMAND"-String direkt in die Config-Datei weiter, wenn da ein Enum extra eingesetzt wird, welches umständlich gewartet werden muss, geht dieser vorteil verloren.
Review

ok Dann will ich dazu nichts gesagt haben

ok Dann will ich dazu nichts gesagt haben
UNKNWON_PLAYER=§cDiesen Spieler gibt es nicht.
#Help command
HELP_LOBBY=§7Kehre von überall mit §8/§el §7zur Lobby zurück!
HELP_LOBBY_HOVER=§eZurück zur Lobby
HELP_BAU=§7Komme mit §8/§ebau §7auf den Bauserver!
@ -53,3 +57,20 @@ HELP_BAU_TESTARENA=§8/§ebau testarena §8- §7Starte eine Testarena
HELP_BAU_TESTARENA_HOVER=§eTestarena starten
HELP_BAU_BAU=§8/§ehelp bau §8- §7Hilfe zu nützlichen Werkzeugen
HELP_BAU_BAU_HOVER=§eNützliche Zusatzfunktionen
#Usage description of various commands
USAGE_ALERT=§8/§7alert §8[§eNachricht§8]
USAGE_BAN=§8/§7ban §8[§eSpieler§8] [§edd§8.§emm§8.§eyyyy §7oder §edd§8.§emm§8.§eyyyy§8_§ehh§8:§emm §7oder §eperma§8] [§eGrund§8]
USAGE_IGNORE=§8/§7ignore §8[§eSpieler§8]
#Various commands
ALERT=§f{0}
BAN_YOU_BANNED=§7Du hast §e{0} §7gebannt§8. §7Grund§8: §c{1}
Review

Der Name ist finde ich etwas irreführend

Der Name ist finde ich etwas irreführend
Review

Vllt besser 'BAN_MESSAGE_YOU'

Vllt besser 'BAN_MESSAGE_YOU'
BAN_INVALID_TIME=§cUngültige Zeitangabe.
Review

Diese ungültige Zeitangabe gibt es auch beim Muten, womit einfach der Key 'INVALID_TIME' genauso gut wäre. Weil die Nachricht soll doch sicher nicht anders sein, wenn ich jemanden Mute im Gegensatz zu Bannen?

Diese ungültige Zeitangabe gibt es auch beim Muten, womit einfach der Key 'INVALID_TIME' genauso gut wäre. Weil die Nachricht soll doch sicher nicht anders sein, wenn ich jemanden Mute im Gegensatz zu Bannen?
BUG_SAVED=§7Dein Bugreport wurde gespeichert.
Review

Wenn wir die Ban Message und die Ignore Message so machen wie oben und unten beschrieben sollten wir das hier zu 'BUG_MESSAGE' ändern. Auch wenn die Nachricht das speichern meint sollten wir es dann einheitlich machen. Fände ich auch schöner. Weil dann nicht jedes System sein eigenen Message Namen bekommt für eine einfache Nachricht, die zurück gegeben wird.

Wenn wir die Ban Message und die Ignore Message so machen wie oben und unten beschrieben sollten wir das hier zu 'BUG_MESSAGE' ändern. Auch wenn die Nachricht das speichern meint sollten wir es dann einheitlich machen. Fände ich auch schöner. Weil dann nicht jedes System sein eigenen Message Namen bekommt für eine einfache Nachricht, die zurück gegeben wird.
IGNORE_YOURSELF=§cWie willst du dich selber ignorieren?
IGNORE_ALREADY=§cDu ignorierst diesen Spieler bereits.
IGNORE_IGNORE=§7Du ignorierst nun §e{0}§8.
Review

Auch diesen Namen finde ich etwas irreführend

Auch diesen Namen finde ich etwas irreführend
Review

Vllt besser 'IGNORE_MESSAGE'

Vllt besser 'IGNORE_MESSAGE'