GH-579 Add repository for language. Remove user settings. (#890)

Author: Rollczi

eternalcore-core/src/main/java/com/eternalcode/core/feature/language/Language.java renamed to eternalcore-api/src/main/java/com/eternalcode/core/feature/language/Language.java

@@ -1,10 +1,11 @@
 package com.eternalcode.core.feature.language;
 
-import java.util.ArrayList;
 import java.util.Collections;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Locale;
 import java.util.Objects;
+import java.util.Set;
 
 public class Language {
 
@@ -13,35 +14,37 @@ public class Language {
     public static final Language DEFAULT = Language.fromLocale(Locale.ROOT);
 
     private final String lang;
-    private final List<String> aliases;
+    private final Set<String> aliases;
 
     public Language(String lang, List<String> aliases) {
         this.lang = lang;
-        this.aliases = new ArrayList<>(aliases);
+        this.aliases = new LinkedHashSet<>(aliases);
     }
 
     public String getLang() {
         return this.lang;
     }
 
-    public List<String> getAliases() {
-        return Collections.unmodifiableList(this.aliases);
+    public Set<String> getAliases() {
+        return Collections.unmodifiableSet(this.aliases);
     }
 
     public boolean isEquals(Language other) {
         if (this.lang.equals(other.lang)) {
             return true;
         }
 
-        for (String alias : this.aliases) {
-            if (alias.equals(other.lang)) {
-                return true;
-            }
+        if (this.lang.startsWith(other.lang) || other.lang.startsWith(this.lang)) {
+            return true;
+        }
 
-            for (String otherAlias : other.aliases) {
-                if (alias.equals(otherAlias)) {
-                    return true;
-                }
+        if (this.aliases.contains(other.lang)) {
+            return true;
+        }
+
+        for (String otherAlias : other.aliases) {
+            if (this.aliases.contains(otherAlias)) {
+                return true;
             }
         }
 
@@ -71,7 +74,7 @@ public static Language fromLocale(Locale locale) {
     }
 
     public Locale toLocale() {
-        return new Locale(this.lang);
+        return Locale.of(this.lang);
     }
 
 }

eternalcore-api/src/main/java/com/eternalcode/core/feature/language/LanguageProvider.java

@@ -0,0 +1,9 @@
+package com.eternalcode.core.feature.language;
+
+import java.util.UUID;
+
+public interface LanguageProvider {
+
+    Language getDefaultLanguage(UUID player);
+
+}

eternalcore-api/src/main/java/com/eternalcode/core/feature/language/LanguageService.java

@@ -1,11 +1,20 @@
 package com.eternalcode.core.feature.language;
 
-import java.util.Locale;
-import org.bukkit.entity.Player;
+import java.util.UUID;
+import java.util.concurrent.CompletableFuture;
 
 public interface LanguageService {
 
-    Locale getPlayerLanguage(Player player);
+    void setDefaultProvider(LanguageProvider defaultProvider);
+
+    LanguageProvider getDefaultProvider();
+
+    CompletableFuture<Language> getLanguage(UUID playerUniqueId);
+
+    Language getLanguageNow(UUID playerUniqueId);
+
+    CompletableFuture<Void> setLanguage(UUID playerUniqueId, Language language);
+
+    CompletableFuture<Void> setDefaultLanguage(UUID playerUniqueId);
 
-    void setPlayerLanguage(Player player, Locale locale);
 }

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/AbstractViewerArgument.java

@@ -1,30 +1,31 @@
 package com.eternalcode.core.bridge.litecommand.argument;
 
+import com.eternalcode.core.feature.language.Language;
 import com.eternalcode.core.translation.Translation;
 import com.eternalcode.core.translation.TranslationManager;
-import com.eternalcode.core.viewer.Viewer;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.argument.resolver.ArgumentResolver;
 import dev.rollczi.litecommands.invocation.Invocation;
 import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
 
 public abstract class AbstractViewerArgument<T> extends ArgumentResolver<CommandSender, T> {
 
-    protected final ViewerService viewerService;
     protected final TranslationManager translationManager;
 
-    protected AbstractViewerArgument(ViewerService viewerService, TranslationManager translationManager) {
-        this.viewerService = viewerService;
+    protected AbstractViewerArgument(TranslationManager translationManager) {
         this.translationManager = translationManager;
     }
 
     @Override
     protected ParseResult<T> parse(Invocation<CommandSender> invocation, Argument<T> context, String argument) {
-        Viewer viewer = this.viewerService.any(invocation.sender());
-        Translation translation = this.translationManager.getMessages(viewer.getLanguage());
+        if (invocation.sender() instanceof Player player) {
+            Translation translation = this.translationManager.getMessages(player.getUniqueId());
+            return this.parse(invocation, argument, translation);
+        }
 
+        Translation translation = this.translationManager.getMessages(Language.DEFAULT);
         return this.parse(invocation, argument, translation);
     }
 

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/EnchantmentArgument.java

@@ -4,7 +4,6 @@
 import com.eternalcode.core.injector.annotations.lite.LiteArgument;
 import com.eternalcode.core.translation.Translation;
 import com.eternalcode.core.translation.TranslationManager;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.invocation.Invocation;
@@ -21,8 +20,8 @@
 class EnchantmentArgument extends AbstractViewerArgument<Enchantment> {
 
     @Inject
-    EnchantmentArgument(ViewerService viewerService, TranslationManager translationManager) {
-        super(viewerService, translationManager);
+    EnchantmentArgument(TranslationManager translationManager) {
+        super(translationManager);
     }
 
     @Override

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/GameModeArgument.java

@@ -5,7 +5,6 @@
 import com.eternalcode.core.injector.annotations.lite.LiteArgument;
 import com.eternalcode.core.translation.Translation;
 import com.eternalcode.core.translation.TranslationManager;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.invocation.Invocation;
@@ -23,8 +22,8 @@ class GameModeArgument extends AbstractViewerArgument<GameMode> {
     private final GameModeArgumentSettings gameModeArgumentSettings;
 
     @Inject
-    GameModeArgument(ViewerService viewerService, TranslationManager translationManager, GameModeArgumentSettings gameModeArgumentSettings) {
-        super(viewerService, translationManager);
+    GameModeArgument(TranslationManager translationManager, GameModeArgumentSettings gameModeArgumentSettings) {
+        super(translationManager);
         this.gameModeArgumentSettings = gameModeArgumentSettings;
     }
 

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/NoticeTypeArgument.java

@@ -5,7 +5,6 @@
 import com.eternalcode.core.notice.NoticeTextType;
 import com.eternalcode.core.translation.Translation;
 import com.eternalcode.core.translation.TranslationManager;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.invocation.Invocation;
@@ -19,8 +18,8 @@
 class NoticeTypeArgument extends AbstractViewerArgument<NoticeTextType> {
 
     @Inject
-    NoticeTypeArgument(ViewerService viewerService, TranslationManager translationManager) {
-        super(viewerService, translationManager);
+    NoticeTypeArgument(TranslationManager translationManager) {
+        super(translationManager);
     }
 
     @Override

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/PlayerArgument.java

@@ -7,7 +7,6 @@
 import com.eternalcode.core.injector.annotations.lite.LiteArgument;
 import com.eternalcode.core.translation.Translation;
 import com.eternalcode.core.translation.TranslationManager;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.invocation.Invocation;
@@ -25,12 +24,11 @@ public class PlayerArgument extends AbstractViewerArgument<Player> {
 
     @Inject
     public PlayerArgument(
-        ViewerService viewerService,
         TranslationManager translationManager,
         Server server,
         VanishService vanishService
     ) {
-        super(viewerService, translationManager);
+        super(translationManager);
         this.server = server;
         this.vanishService = vanishService;
     }

eternalcore-core/src/main/java/com/eternalcode/core/bridge/litecommand/argument/UserArgument.java

@@ -6,7 +6,6 @@
 import com.eternalcode.core.translation.TranslationManager;
 import com.eternalcode.core.user.User;
 import com.eternalcode.core.user.UserManager;
-import com.eternalcode.core.viewer.ViewerService;
 import dev.rollczi.litecommands.argument.Argument;
 import dev.rollczi.litecommands.argument.parser.ParseResult;
 import dev.rollczi.litecommands.invocation.Invocation;
@@ -23,8 +22,8 @@ class UserArgument extends AbstractViewerArgument<User> {
     private final UserManager userManager;
 
     @Inject
-    UserArgument(ViewerService viewerService, TranslationManager translationManager, Server server, UserManager userManager) {
-        super(viewerService, translationManager);
+    UserArgument(TranslationManager translationManager, Server server, UserManager userManager) {
+        super(translationManager);
         this.server = server;
         this.userManager = userManager;
     }

eternalcore-core/src/main/java/com/eternalcode/core/feature/afk/AfkKickController.java

@@ -60,7 +60,7 @@ void onAfkSwitch(AfkSwitchEvent event) {
         }
 
         User user = this.userManager.getOrCreate(playerUUID, player.getName());
-        Translation translation = this.translationManager.getMessages(user);
+        Translation translation = this.translationManager.getMessages(user.getUniqueId());
 
         Component component = this.miniMessage.deserialize(translation.afk().afkKickReason());
         player.kickPlayer(legacySection().serialize(component));