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 7 neuen und 10 gelöschten Zeilen
Nur Änderungen aus Commit 298aa9d3de werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -334,7 +334,7 @@ public class SQLProvider implements Provider {
String dbLocale = rs.getString("Locale");
Locale locale = null;
if (dbLocale != null) {
locale = new Locale(dbLocale);
locale = Locale.forLanguageTag(dbLocale);
Veraltet
Review

Das funktioniert so nicht. byLanguageTag oder so ähnlich.

Das funktioniert so nicht. byLanguageTag oder so ähnlich.
}
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);
}

Datei anzeigen

@ -42,10 +42,9 @@ public class SteamwarUser {
}
public static void invalidate(int userId) {
if (!byId.containsKey(userId))
SteamwarUser user = byId.remove(userId);
if (user == null)
return;
SteamwarUser user = byId.get(userId);
byId.remove(userId);
byName.remove(user.getUserName());
Lixfel markierte diese Unterhaltung als gelöst
Review

Ein Aufruf zu remove gibt das entfernte Objekt (oder null) direkt zurück, ein containsKey und get-Aufruf ist also unnötig.

Ein Aufruf zu remove gibt das entfernte Objekt (oder null) direkt zurück, ein containsKey und get-Aufruf ist also unnötig.
byUUID.remove(user.getUUID());
}
@ -98,13 +97,11 @@ public class SteamwarUser {
public Locale getLocale() {
Locale locale = this.locale;
if (locale == null) {
if (locale == null)
locale = WorldOfColorWrapper.impl.getLocale(Bukkit.getPlayer(uuid));
}
if (locale == null) {
locale = Locale.getDefault();
}
return locale;
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.
}
public static SteamwarUser get(String userName){