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
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2022-05-21 11:12:03 +02:00 hinzugefügt
Add language via db
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
55fb0b8f47
YoyoNow hat den Titel von Add language via db zu WIP: Add language via db 2022-05-21 11:19:43 +02:00 geändert
YoyoNow hat 1 Commit 2022-05-21 13:21:26 +02:00 hinzugefügt
Add LocaleInvalidationHandler
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
22e691dcb8
YoyoNow hat den Titel von WIP: Add language via db zu Add language via db 2022-05-21 13:21:46 +02:00 geändert
YoyoNow hat 1 Commit 2022-05-21 13:23:47 +02:00 hinzugefügt
Fix stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
6eb72a3aa2
YoyoNow hat 1 Commit 2022-05-21 13:43:47 +02:00 hinzugefügt
Fix stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
f8436026e2
Lixfel hat 2022-05-21 16:24:27 +02:00 Änderungen angefragt
@ -335,0 +334,4 @@
String dbLocale = rs.getString("Locale");
Locale locale = null;
if (dbLocale != null) {
locale = new Locale(dbLocale);
Besitzer

Das funktioniert so nicht. byLanguageTag oder so ähnlich.

Das funktioniert so nicht. byLanguageTag oder so ähnlich.
@ -169,3 +170,3 @@
if(player == null)
return null;
return usersByUUID.computeIfAbsent(player.getUniqueId(), uuid -> new SteamwarUser(userId++, uuid, userName, UserGroup.Member, 0, false));
return usersByUUID.computeIfAbsent(player.getUniqueId(), uuid -> new SteamwarUser(userId++, uuid, userName, UserGroup.Member, 0, false, null));
Besitzer

Locale.getDefault() statt null zurückliefern, ggf. falls online-User aktuelle Locale (genauso im byUUID)

Locale.getDefault() statt null zurückliefern, ggf. falls online-User aktuelle Locale (genauso im byUUID)
Autor
Besitzer

Deswegen giobt es den Spezial wert 'null', damit er dann die locale des users nimmt. Das ist absicht. Deswegen auch oben wenn die dbLocale null ist wird die locale null bleiben womit einfach erstmal die locale des spielers genutzt wird. Ist primär dafür da das wir das im Livebetrieb mergen können.

Deswegen giobt es den Spezial wert 'null', damit er dann die locale des users nimmt. Das ist absicht. Deswegen auch oben wenn die dbLocale null ist wird die locale null bleiben womit einfach erstmal die locale des spielers genutzt wird. Ist primär dafür da das wir das im Livebetrieb mergen können.
@ -46,0 +45,4 @@
if (!byId.containsKey(userId))
return;
SteamwarUser user = byId.get(userId);
byId.remove(userId);
Besitzer

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.
Lixfel markierte diese Unterhaltung als gelöst
@ -90,0 +99,4 @@
public Locale getLocale() {
Locale locale = this.locale;
if (locale == null) {
locale = WorldOfColorWrapper.impl.getLocale(Bukkit.getPlayer(uuid));
Besitzer

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

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

s.o.

s.o.
@ -90,0 +101,4 @@
if (locale == null) {
locale = WorldOfColorWrapper.impl.getLocale(Bukkit.getPlayer(uuid));
}
if (locale == null) {
Besitzer

Bitte stattdessen ein != null return machen.

Bitte stattdessen ein != null return machen.
YoyoNow hat 1 Commit 2022-05-21 16:28:23 +02:00 hinzugefügt
Fix stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
298aa9d3de
Lixfel hat 1 Commit 2022-05-24 09:12:46 +02:00 hinzugefügt
Improve control flow readability
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
7b3162ece3
Lixfel hat die Änderungen 2022-05-24 09:13:10 +02:00 genehmigt
Lixfel hat die Änderungen 2022-05-24 09:14:08 +02:00 genehmigt
Lixfel hat Commit 5289907eb4 in master 2022-05-24 09:20:37 +02:00 gemerged
Lixfel löschte die Branch LanguageConfig 2022-05-24 09:20:38 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.