Changes to settings structure

Author: jamezrin

pom.xml

@@ -68,6 +68,7 @@
                     </execution>
                 </executions>
             </plugin>
+            <!--
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>license-maven-plugin</artifactId>
@@ -81,6 +82,7 @@
                     </execution>
                 </executions>
             </plugin>
+            -->
         </plugins>
         <resources>
             <resource>
@@ -100,7 +102,7 @@
         <dependency>
             <groupId>ch.jalu</groupId>
             <artifactId>configme</artifactId>
-            <version>0.4</version>
+            <version>0.6-SNAPSHOT</version>
             <scope>compile</scope>
             <exclusions>
                 <!-- This is already shaded in BungeeCord -->
@@ -114,7 +116,7 @@
         <dependency>
             <groupId>ch.jalu</groupId>
             <artifactId>injector</artifactId>
-            <version>0.4</version>
+            <version>0.5-SNAPSHOT</version>
             <scope>compile</scope>
         </dependency>
 

src/main/java/com/jaimemartz/playerbalancer/PlayerBalancer.java

@@ -2,11 +2,8 @@
 
 import ch.jalu.injector.Injector;
 import ch.jalu.injector.InjectorBuilder;
-import com.jaimemartz.playerbalancer.commands.MainCommand;
 import com.jaimemartz.playerbalancer.settings.Settings;
-import com.jaimemartz.playerbalancer.settings.beans.SectionsHandler;
-import com.jaimemartz.playerbalancer.settings.provider.SectionHandlerProvider;
-import com.jaimemartz.playerbalancer.settings.provider.SettingsProvider;
+import com.jaimemartz.playerbalancer.settings.SettingsProvider;
 import lombok.Getter;
 import net.md_5.bungee.api.ProxyServer;
 import net.md_5.bungee.api.plugin.Command;
@@ -20,7 +17,6 @@ public class PlayerBalancer extends Plugin {
     //Private instances
     private Injector injector;
     private Settings settings;
-    private SectionsHandler handler;
 
     private Command fallbackCommand, mainCommand, manageCommand;
     private Listener connectListener, kickListener, messageListener, reloadListener;
@@ -35,29 +31,28 @@ public void onEnable() {
         injector.register(ProxyServer.class, this.getProxy());
 
         injector.registerProvider(Settings.class, SettingsProvider.class);
-        injector.registerProvider(SectionsHandler.class, SectionHandlerProvider.class);
 
         settings = injector.getSingleton(Settings.class);
-        handler = injector.getSingleton(SectionsHandler.class);
 
         Metrics metrics = new Metrics(this);
+
         if (this.enable()) {
-            //metrics.addCustomChart(new SingleLineChart("configured_sections", () -> handler.getSections().size()));
+            //metrics.addCustomChart(new Metrics.SingleLineChart("configured_sections", () -> s.size()));
         }
     }
 
     private boolean enable() {
+        /*
         mainCommand = new MainCommand(this);
         getProxy().getPluginManager().registerCommand(this, mainCommand);
 
-        /*
         if (settings.getProperty(GeneralProperties.ENABLED)) {
             if (settings.getProperty(GeneralProperties.SILENT)) {
                 getLogger().setLevel(Level.WARNING);
             }
 
             if (settings.getProperty(GeneralProperties.AUTO_RELOAD)) {
-                reloadListener = new ProxyReloadListener(this);
+                reloadListener = injector.getSingleton(ProxyReloadListener.class);
                 getProxy().getPluginManager().registerListener(this, reloadListener);
             }
 
@@ -82,23 +77,23 @@ private boolean enable() {
                     getProxy().getPluginManager().registerCommand(this, fallbackCommand);
                 }
 
-                connectListener = new ServerConnectListener(this);
+                connectListener = injector.getSingleton(ServerConnectListener.class);
                 getProxy().getPluginManager().registerListener(this, connectListener);
 
-                messageListener = new PluginMessageListener(this);
+                messageListener = injector.getSingleton(PluginMessageListener.class);
                 getProxy().getPluginManager().registerListener(this, messageListener);
 
-                manageCommand = new ManageCommand(this);
+                manageCommand = injector.getSingleton(ManageCommand.class);
                 getProxy().getPluginManager().registerCommand(this, manageCommand);
 
-                getProxy().getPluginManager().registerListener(this, new PlayerDisconnectListener(this));
+                getProxy().getPluginManager().registerListener(this, injector.getSingleton(PlayerDisconnectListener.class));
 
                 getProxy().registerChannel("PlayerBalancer");
 
                 Stream.of(PasteHelper.values()).forEach(a -> a.setUrl(null));
 
                 if (settings.getProperty(ReconnectorProperties.ENABLED)) {
-                    kickListener = new ServerKickListener(this);
+                    kickListener = injector.getSingleton(ServerKickListener.class);
                     getProxy().getPluginManager().registerListener(this, kickListener);
                 }
 
@@ -125,10 +120,10 @@ public void onDisable() {
     }
 
     private void disable() {
+        /*
         getProxy().getPluginManager().unregisterCommand(mainCommand);
         mainCommand = null;
 
-        /*
         if (settings.getProperty(GeneralProperties.ENABLED)) {
             //Do not try to do anything if the plugin has not loaded correctly
             if (isFailed()) return;

src/main/java/com/jaimemartz/playerbalancer/commands/FallbackCommand.java

@@ -1,6 +1,5 @@
 package com.jaimemartz.playerbalancer.commands;
 
-import com.google.common.collect.Iterables;
 import com.jaimemartz.playerbalancer.settings.Settings;
 import com.jaimemartz.playerbalancer.settings.types.CommandProperties;
 import com.jaimemartz.playerbalancer.settings.types.SectionsHolder;
@@ -21,7 +20,7 @@ public FallbackCommand(Settings settings) {
         super(
                 settings.getProperty(CommandProperties.COMMAND).getName(),
                 settings.getProperty(CommandProperties.COMMAND).getPermission(),
-                Iterables.toArray(settings.getProperty(CommandProperties.COMMAND).getAliases(), String.class)
+                settings.getProperty(CommandProperties.COMMAND).getAliases()
         );
     }
 

src/main/java/com/jaimemartz/playerbalancer/commands/ManageCommand.java

@@ -3,7 +3,6 @@
 import com.jaimemartz.playerbalancer.PlayerBalancer;
 import com.jaimemartz.playerbalancer.ping.StatusManager;
 import com.jaimemartz.playerbalancer.settings.Settings;
-import com.jaimemartz.playerbalancer.settings.beans.SectionsHandler;
 import net.md_5.bungee.api.CommandSender;
 import net.md_5.bungee.api.plugin.Command;
 
@@ -13,9 +12,6 @@ public class ManageCommand extends Command {
     @Inject
     private PlayerBalancer plugin;
 
-    @Inject
-    private SectionsHandler holder;
-
     @Inject
     private StatusManager checker;
 

src/main/java/com/jaimemartz/playerbalancer/connection/ProviderType.java

@@ -1,7 +1,15 @@
 package com.jaimemartz.playerbalancer.connection;
 
+import com.jaimemartz.playerbalancer.PlayerBalancer;
+import com.jaimemartz.playerbalancer.manager.NetworkManager;
+import com.jaimemartz.playerbalancer.section.ServerSection;
+import net.md_5.bungee.api.config.ServerInfo;
+import net.md_5.bungee.api.connection.ProxiedPlayer;
+
+import java.util.List;
+import java.util.concurrent.ThreadLocalRandom;
+
 public enum ProviderType {
-    /*
     NONE {
         @Override
         public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, List<ServerInfo> list, ProxiedPlayer player) {
@@ -38,6 +46,7 @@ public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, Li
     PROGRESSIVE {
         @Override
         public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, List<ServerInfo> list, ProxiedPlayer player) {
+            /*
             for (ServerInfo server : list) {
                 ServerStatus status = plugin.getStatusManager().getStatus(server);
                 if (NetworkManager.getPlayers(server).size() < status.getMaximum()) {
@@ -46,12 +55,15 @@ public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, Li
             }
 
             return list.get(ThreadLocalRandom.current().nextInt(list.size()));
+            */
+            return null;
         }
     },
 
     FILLER {
         @Override
         public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, List<ServerInfo> list, ProxiedPlayer player) {
+            /*
             int max = Integer.MIN_VALUE;
             ServerInfo target = null;
 
@@ -66,9 +78,10 @@ public ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, Li
             }
 
             return target;
+            */
+            return null;
         }
     };
 
     public abstract ServerInfo requestTarget(PlayerBalancer plugin, ServerSection section, List<ServerInfo> list, ProxiedPlayer player);
-    */
 }

…/settings/provider/SettingsProvider.java …rbalancer/settings/SettingsProvider.javasrc/main/java/com/jaimemartz/playerbalancer/settings/provider/SettingsProvider.java renamed to src/main/java/com/jaimemartz/playerbalancer/settings/SettingsProvider.java src/main/java/com/jaimemartz/playerbalancer/settings/provider/SettingsProvider.java renamed to src/main/java/com/jaimemartz/playerbalancer/settings/SettingsProvider.java

@@ -1,12 +1,11 @@
-package com.jaimemartz.playerbalancer.settings.provider;
+package com.jaimemartz.playerbalancer.settings;
 
 import ch.jalu.configme.configurationdata.ConfigurationData;
 import ch.jalu.configme.configurationdata.ConfigurationDataBuilder;
 import ch.jalu.configme.migration.PlainMigrationService;
 import ch.jalu.configme.resource.PropertyResource;
 import ch.jalu.configme.resource.YamlFileResource;
 import com.jaimemartz.playerbalancer.PlayerBalancer;
-import com.jaimemartz.playerbalancer.settings.Settings;
 import com.jaimemartz.playerbalancer.settings.types.*;
 
 import javax.inject.Inject;

src/main/java/com/jaimemartz/playerbalancer/settings/beans/CommandBean.java

@@ -0,0 +1,41 @@
+package com.jaimemartz.playerbalancer.settings.beans;
+
+public final class CommandData {
+    private String name;
+    private String permission;
+    private String[] aliases;
+
+    public CommandData() {
+        
+    }
+
+    public CommandData(String name, String permission, String... aliases) {
+        this.name = name;
+        this.permission = permission;
+        this.aliases = aliases;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public String getPermission() {
+        return permission;
+    }
+
+    public void setPermission(String permission) {
+        this.permission = permission;
+    }
+
+    public String[] getAliases() {
+        return aliases;
+    }
+
+    public void setAliases(String[] aliases) {
+        this.aliases = aliases;
+    }
+}

src/main/java/com/jaimemartz/playerbalancer/settings/beans/CommandData.java

@@ -0,0 +1,71 @@
+package com.jaimemartz.playerbalancer.settings.beans;
+
+import com.jaimemartz.playerbalancer.connection.ProviderType;
+
+import java.util.List;
+
+public class SectionData {
+    private Boolean principal = true;
+    private Boolean dummy;
+    private String parent;
+    private List<String> servers;
+    private ProviderType provider;
+    private CommandData command;
+    private String server;
+
+    public Boolean isPrincipal() {
+        return principal;
+    }
+
+    public void setPrincipal(boolean principal) {
+        this.principal = principal;
+    }
+
+    public Boolean isDummy() {
+        return dummy;
+    }
+
+    public void setDummy(boolean dummy) {
+        this.dummy = dummy;
+    }
+
+    public String getParent() {
+        return parent;
+    }
+
+    public void setParent(String parent) {
+        this.parent = parent;
+    }
+
+    public List<String> getServers() {
+        return servers;
+    }
+
+    public void setServers(List<String> servers) {
+        this.servers = servers;
+    }
+
+    public ProviderType getProvider() {
+        return provider;
+    }
+
+    public void setProvider(ProviderType provider) {
+        this.provider = provider;
+    }
+
+    public CommandData getCommand() {
+        return command;
+    }
+
+    public void setCommand(CommandData command) {
+        this.command = command;
+    }
+
+    public String getServer() {
+        return server;
+    }
+
+    public void setServer(String server) {
+        this.server = server;
+    }
+}