Added lombok to remove getters and setters and stuff

Author: jamezrin

Main Plugin/pom.xml

@@ -105,5 +105,11 @@
             <version>4.12</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.projectlombok</groupId>
+            <artifactId>lombok</artifactId>
+            <version>1.16.16</version>
+            <scope>provided</scope>
+        </dependency>
     </dependencies>
 </project>

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/section/SectionManager.java

@@ -92,7 +92,7 @@ public void execute(String sectionName, SectionProps sectionProps, ServerSection
                 if (sectionProps.getProvider() == null) {
                     ServerSection current = section.getParent();
                     if (current != null) {
-                        while (current.getExplicitProvider() == null) {
+                        while (current.getProps().getProvider() == null) {
                             current = current.getParent();
                         }
 

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/section/ServerSection.java

@@ -4,12 +4,14 @@
 import com.jaimemartz.playerbalancer.connection.provider.AbstractProvider;
 import com.jaimemartz.playerbalancer.settings.props.shared.SectionProps;
 import com.jaimemartz.playerbalancer.utils.AlphanumComparator;
+import lombok.Data;
 import net.md_5.bungee.api.config.ServerInfo;
 
 import java.util.Collections;
 import java.util.Set;
 import java.util.TreeSet;
 
+@Data
 public class ServerSection {
     private final String name;
     private final SectionProps props;
@@ -34,104 +36,11 @@ public ServerSection(String name, SectionProps props) {
         ));
     }
 
-    public String getName() {
-        return name;
-    }
-
-    public int getPosition() {
-        return position;
-    }
-
-    public void setPosition(int position) {
-        this.position = position;
-    }
-
-    public SectionProps getProps() {
-        return props;
-    }
-
-    public boolean isInherited() {
-        return inherited;
-    }
-
-    public void setInherited(boolean inherited) {
-        this.inherited = inherited;
-    }
-
-    public ServerSection getParent() {
-        return parent;
-    }
-
-    public void setParent(ServerSection parent) {
-        this.parent = parent;
-    }
-
     public ProviderType getImplicitProvider() {
         if (inherited) {
             return parent.getImplicitProvider();
         } else {
             return props.getProvider();
         }
-
-    }
-
-    public ProviderType getExplicitProvider() {
-        return props.getProvider();
-    }
-
-    public void setExplicitProvider(ProviderType provider) {
-        props.setProvider(provider);
-        inherited = false;
-    }
-
-    public void setExternalProvider(AbstractProvider externalProvider) {
-        this.externalProvider = externalProvider;
-    }
-
-    public AbstractProvider getExternalProvider() {
-        return externalProvider;
-    }
-
-    public ServerInfo getServer() {
-        return server;
-    }
-
-    public void setServer(ServerInfo server) {
-        this.server = server;
-    }
-
-    public SectionCommand getCommand() {
-        return command;
-    }
-
-    public void setCommand(SectionCommand command) {
-        this.command = command;
-    }
-
-    public Set<ServerInfo> getServers() {
-        return servers;
-    }
-
-    public boolean isValid() {
-        return valid;
-    }
-
-    public void setValid(boolean valid) {
-        this.valid = valid;
-    }
-
-    @Override
-    public String toString() {
-        return "ServerSection{" +
-                "name='" + name + '\'' +
-                ", props=" + props +
-                ", inherited=" + inherited +
-                ", parent=" + parent +
-                ", position=" + position +
-                ", server=" + server +
-                ", command=" + command +
-                ", servers=" + servers +
-                ", valid=" + valid +
-                '}';
     }
 }

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/settings/props/FeaturesProps.java

@@ -1,10 +1,12 @@
 package com.jaimemartz.playerbalancer.settings.props;
 
 import com.jaimemartz.playerbalancer.settings.props.features.*;
+import lombok.Data;
 import ninja.leaping.configurate.objectmapping.Setting;
 import ninja.leaping.configurate.objectmapping.serialize.ConfigSerializable;
 
 @ConfigSerializable
+@Data
 public class FeaturesProps {
     @Setting(value = "balancer")
     private BalancerProps balancerProps;
@@ -20,55 +22,4 @@ public class FeaturesProps {
 
     @Setting(value = "server-refresh")
     private ServerRefreshProps serverRefreshProps;
-
-    public BalancerProps getBalancerProps() {
-        return balancerProps;
-    }
-
-    public void setBalancerProps(BalancerProps balancerProps) {
-        this.balancerProps = balancerProps;
-    }
-
-    public FallbackCommandProps getFallbackCommandProps() {
-        return fallbackCommandProps;
-    }
-
-    public void setFallbackCommandProps(FallbackCommandProps fallbackCommandProps) {
-        this.fallbackCommandProps = fallbackCommandProps;
-    }
-
-    public ServerCheckerProps getServerCheckerProps() {
-        return serverCheckerProps;
-    }
-
-    public void setServerCheckerProps(ServerCheckerProps serverCheckerProps) {
-        this.serverCheckerProps = serverCheckerProps;
-    }
-
-    public KickHandlerProps getKickHandlerProps() {
-        return kickHandlerProps;
-    }
-
-    public void setKickHandlerProps(KickHandlerProps kickHandlerProps) {
-        this.kickHandlerProps = kickHandlerProps;
-    }
-
-    public ServerRefreshProps getServerRefreshProps() {
-        return serverRefreshProps;
-    }
-
-    public void setServerRefreshProps(ServerRefreshProps serverRefreshProps) {
-        this.serverRefreshProps = serverRefreshProps;
-    }
-
-    @Override
-    public String toString() {
-        return "FeaturesProps{" +
-                "balancerProps=" + balancerProps +
-                ", fallbackCommandProps=" + fallbackCommandProps +
-                ", serverCheckerProps=" + serverCheckerProps +
-                ", kickHandlerProps=" + kickHandlerProps +
-                ", serverRefreshProps=" + serverRefreshProps +
-                '}';
-    }
 }

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/settings/props/GeneralProps.java

@@ -1,9 +1,11 @@
 package com.jaimemartz.playerbalancer.settings.props;
 
+import lombok.Data;
 import ninja.leaping.configurate.objectmapping.Setting;
 import ninja.leaping.configurate.objectmapping.serialize.ConfigSerializable;
 
 @ConfigSerializable
+@Data
 public class GeneralProps {
     @Setting
     private boolean enabled;
@@ -22,64 +24,4 @@ public class GeneralProps {
 
     @Setting
     private String version;
-
-    public boolean isEnabled() {
-        return enabled;
-    }
-
-    public void setEnabled(boolean enabled) {
-        this.enabled = enabled;
-    }
-
-    public boolean isSilent() {
-        return silent;
-    }
-
-    public void setSilent(boolean silent) {
-        this.silent = silent;
-    }
-
-    public boolean isAutoReload() {
-        return autoReload;
-    }
-
-    public void setAutoReload(boolean autoReload) {
-        this.autoReload = autoReload;
-    }
-
-    public boolean isRedisBungee() {
-        return redisBungee;
-    }
-
-    public void setRedisBungee(boolean redisBungee) {
-        this.redisBungee = redisBungee;
-    }
-
-    public boolean isPluginMessaging() {
-        return pluginMessaging;
-    }
-
-    public void setPluginMessaging(boolean pluginMessaging) {
-        this.pluginMessaging = pluginMessaging;
-    }
-
-    public String getVersion() {
-        return version;
-    }
-
-    public void setVersion(String version) {
-        this.version = version;
-    }
-
-    @Override
-    public String toString() {
-        return "GeneralProps{" +
-                "enabled=" + enabled +
-                ", silent=" + silent +
-                ", autoReload=" + autoReload +
-                ", redisBungee=" + redisBungee +
-                ", pluginMessaging=" + pluginMessaging +
-                ", version='" + version + '\'' +
-                '}';
-    }
 }

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/settings/props/MessagesProps.java

@@ -1,9 +1,11 @@
 package com.jaimemartz.playerbalancer.settings.props;
 
+import lombok.Data;
 import ninja.leaping.configurate.objectmapping.Setting;
 import ninja.leaping.configurate.objectmapping.serialize.ConfigSerializable;
 
 @ConfigSerializable
+@Data
 public class MessagesProps {
     @Setting(value = "connecting-server")
     private String connectingMessage;
@@ -31,91 +33,4 @@ public class MessagesProps {
 
     @Setting(value = "same-section")
     private String sameSectionMessage;
-
-    public String getConnectingMessage() {
-        return connectingMessage;
-    }
-
-    public void setConnectingMessage(String connectingMessage) {
-        this.connectingMessage = connectingMessage;
-    }
-
-    public String getConnectedMessage() {
-        return connectedMessage;
-    }
-
-    public void setConnectedMessage(String connectedMessage) {
-        this.connectedMessage = connectedMessage;
-    }
-
-    public String getFailureMessage() {
-        return failureMessage;
-    }
-
-    public void setFailureMessage(String failureMessage) {
-        this.failureMessage = failureMessage;
-    }
-
-    public String getUnknownSectionMessage() {
-        return unknownSectionMessage;
-    }
-
-    public void setUnknownSectionMessage(String unknownSectionMessage) {
-        this.unknownSectionMessage = unknownSectionMessage;
-    }
-
-    public String getInvalidInputMessage() {
-        return invalidInputMessage;
-    }
-
-    public void setInvalidInputMessage(String invalidInputMessage) {
-        this.invalidInputMessage = invalidInputMessage;
-    }
-
-    public String getUnavailableServerMessage() {
-        return unavailableServerMessage;
-    }
-
-    public void setUnavailableServerMessage(String unavailableServerMessage) {
-        this.unavailableServerMessage = unavailableServerMessage;
-    }
-
-    public String getKickMessage() {
-        return kickMessage;
-    }
-
-    public void setKickMessage(String kickMessage) {
-        this.kickMessage = kickMessage;
-    }
-
-    public String getBypassMessage() {
-        return bypassMessage;
-    }
-
-    public void setBypassMessage(String bypassMessage) {
-        this.bypassMessage = bypassMessage;
-    }
-
-    public String getSameSectionMessage() {
-        return sameSectionMessage;
-    }
-
-    public void setSameSectionMessage(String sameSectionMessage) {
-        this.sameSectionMessage = sameSectionMessage;
-    }
-
-    @Override
-    public String toString() {
-        return "MessagesProps{" +
-                "connectingMessage=" + connectingMessage +
-                ", connectedMessage=" + connectedMessage +
-                ", failureMessage=" + failureMessage +
-                ", unknownSectionMessage=" + unknownSectionMessage +
-                ", invalidInputMessage=" + invalidInputMessage +
-                ", unavailableServerMessage=" + unavailableServerMessage +
-                ", kickMessage=" + kickMessage +
-                ", bypassMessage=" + bypassMessage +
-                ", sameSectionMessage=" + sameSectionMessage +
-                '}';
-    }
 }