SteamWar/SpigotCore
Archiviert
13
0

Add language via db #194

Zusammengeführt
Lixfel hat 6 Commits von LanguageConfig nach master 2022-05-24 09:20:37 +02:00 zusammengeführt
2 geänderte Dateien mit 5 neuen und 9 gelöschten Zeilen
Nur Änderungen aus Commit 7b3162ece3 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -332,11 +332,7 @@ public class SQLProvider implements Provider {
private SteamwarUser newSteamwarUser(ResultSet rs) throws SQLException {
String dbLocale = rs.getString("Locale");
Locale locale = null;
if (dbLocale != null) {
locale = Locale.forLanguageTag(dbLocale);
}
return new SteamwarUser(rs.getInt("id"), UUID.fromString(rs.getString("UUID")), rs.getString("UserName"), UserGroup.getUsergroup(rs.getString("UserGroup")), rs.getInt("Team"), rs.getBoolean("Bedrock"), locale);
return new SteamwarUser(rs.getInt("id"), UUID.fromString(rs.getString("UUID")), rs.getString("UserName"), UserGroup.getUsergroup(rs.getString("UserGroup")), rs.getInt("Team"), rs.getBoolean("Bedrock"), dbLocale != null ? Locale.forLanguageTag(dbLocale) : null);
}
Veraltet
Review

Das funktioniert so nicht. byLanguageTag oder so ähnlich.

Das funktioniert so nicht. byLanguageTag oder so ähnlich.
private static final Statement insert = new Statement("INSERT INTO Exception (server, message, stacktrace) VALUES (?, ?, ?)");

Datei anzeigen

@ -55,7 +55,7 @@ public class SteamwarUser {
private final UserGroup userGroup;
private final int team;
private final boolean bedrock;
private final Locale locale;
private Locale locale;
public SteamwarUser(int id, UUID uuid, String userName, UserGroup userGroup, int team, boolean bedrock, Locale locale) {
this.id = id;
@ -96,9 +96,9 @@ public class SteamwarUser {
}
public Locale getLocale() {
Locale locale = this.locale;
if (locale == null)
locale = WorldOfColorWrapper.impl.getLocale(Bukkit.getPlayer(uuid));
if(locale != null)
return locale;
locale = WorldOfColorWrapper.impl.getLocale(Bukkit.getPlayer(uuid));
if (locale != null)
Veraltet
Review

Das ist unnötig (Entweder in DB oder gibts gar keine Info)

Das ist unnötig (Entweder in DB oder gibts gar keine Info)
Veraltet
Review

s.o.

s.o.
return locale;
return Locale.getDefault();
Veraltet
Review

Bitte stattdessen ein != null return machen.

Bitte stattdessen ein != null return machen.