big ass refactor to handle loading guilds and users without spring to make it more futureproof

src/main/java/cc/fascinated/bat/features/base/commands/botadmin/premium/RemoveSubCommand.java

@@ -4,6 +4,7 @@ import cc.fascinated.bat.command.BatSubCommand;
 import cc.fascinated.bat.command.CommandInfo;
 import cc.fascinated.bat.model.BatGuild;
 import cc.fascinated.bat.model.BatUser;
+import cc.fascinated.bat.premium.PremiumProfile;
 import cc.fascinated.bat.service.GuildService;
 import lombok.NonNull;
 import net.dv8tion.jda.api.entities.Member;
@@ -41,14 +42,13 @@ public class RemoveSubCommand extends BatSubCommand {
             interaction.reply("The guild with the id %s does not exist".formatted(guildId)).queue();
             return;
         }
-        BatGuild.Premium premium = batGuild.getPremium();
+        PremiumProfile premium = batGuild.getPremiumProfile();
         if (!premium.hasPremium()) {
             interaction.reply("The guild does not have premium").queue();
             return;
         }
 
         premium.removePremium();
-        guildService.saveGuild(batGuild);
         interaction.reply("The guild **%s** has had its premium removed".formatted(guild.getName())).queue();
     }
 }

src/main/java/cc/fascinated/bat/features/base/commands/botadmin/premium/SetSubCommand.java

@@ -4,6 +4,7 @@ import cc.fascinated.bat.command.BatSubCommand;
 import cc.fascinated.bat.command.CommandInfo;
 import cc.fascinated.bat.model.BatGuild;
 import cc.fascinated.bat.model.BatUser;
+import cc.fascinated.bat.premium.PremiumProfile;
 import cc.fascinated.bat.service.GuildService;
 import lombok.NonNull;
 import net.dv8tion.jda.api.entities.Member;
@@ -23,7 +24,7 @@ public class SetSubCommand extends BatSubCommand {
     private final GuildService guildService;
 
     @Autowired
-    public SetSubCommand(GuildService guildService) {
+    public SetSubCommand(@NonNull GuildService guildService) {
         this.guildService = guildService;
         super.addOption(OptionType.STRING, "guild", "The guild id to set as premium", true);
         super.addOption(OptionType.BOOLEAN, "infinite", "Whether the premium length should be infinite", true);
@@ -49,13 +50,12 @@ public class SetSubCommand extends BatSubCommand {
             interaction.reply("The guild with the id %s does not exist".formatted(guildId)).queue();
             return;
         }
-        BatGuild.Premium premium = batGuild.getPremium();
+        PremiumProfile premium = batGuild.getPremiumProfile();
         if (!infinite) {
             premium.addTime();
         } else {
             premium.addInfiniteTime();
         }
-        guildService.saveGuild(batGuild);
         if (!infinite) {
             interaction.reply("The guild **%s** has been set as premium until <t:%s>".formatted(guild.getName(), premium.getExpiresAt().toInstant().toEpochMilli() / 1000)).queue();
         } else {

src/main/java/cc/fascinated/bat/features/base/commands/server/PremiumCommand.java

@@ -5,6 +5,7 @@ import cc.fascinated.bat.command.CommandInfo;
 import cc.fascinated.bat.common.EmbedUtils;
 import cc.fascinated.bat.model.BatGuild;
 import cc.fascinated.bat.model.BatUser;
+import cc.fascinated.bat.premium.PremiumProfile;
 import lombok.NonNull;
 import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.Permission;
@@ -21,7 +22,7 @@ import org.springframework.stereotype.Component;
 public class PremiumCommand extends BatCommand {
     @Override
     public void execute(BatGuild guild, @NonNull BatUser user, @NonNull MessageChannel channel, Member member, @NonNull SlashCommandInteraction interaction) {
-        BatGuild.Premium premium = guild.getPremium();
+        PremiumProfile premium = guild.getPremiumProfile();
         EmbedBuilder embed = EmbedUtils.genericEmbed().setAuthor("Premium Information");
         if (premium.hasPremium()) {
             embed.addField("Premium", premium.hasPremium() ? "Yes" : "No", true);

src/main/java/cc/fascinated/bat/features/base/commands/server/feature/DisableSubCommand.java

@@ -8,7 +8,6 @@ import cc.fascinated.bat.features.base.profile.FeatureProfile;
 import cc.fascinated.bat.model.BatGuild;
 import cc.fascinated.bat.model.BatUser;
 import cc.fascinated.bat.service.FeatureService;
-import cc.fascinated.bat.service.GuildService;
 import lombok.NonNull;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.channel.middleman.MessageChannel;
@@ -24,11 +23,8 @@ import org.springframework.stereotype.Component;
 @Component("feature:disable.sub")
 @CommandInfo(name = "disable", description = "Disables a feature")
 public class DisableSubCommand extends BatSubCommand {
-    private final GuildService guildService;
-
     @Autowired
-    public DisableSubCommand(@NonNull GuildService guildService) {
-        this.guildService = guildService;
+    public DisableSubCommand() {
         super.addOption(OptionType.STRING, "feature", "The feature to disable", true);
     }
 
@@ -52,15 +48,14 @@ public class DisableSubCommand extends BatSubCommand {
         Feature feature = FeatureService.INSTANCE.getFeature(featureName);
         if (featureProfile.isFeatureDisabled(feature)) {
             interaction.replyEmbeds(EmbedUtils.errorEmbed()
-                    .setDescription("The feature `%s` is already enabled".formatted(feature.getName()))
+                    .setDescription("The feature `%s` is already disabled".formatted(feature.getName()))
                     .build()).queue();
             return;
         }
 
         featureProfile.disableFeature(feature);
-        guildService.saveGuild(guild);
         interaction.replyEmbeds(EmbedUtils.successEmbed()
-                .setDescription("Successfully enabled the `%s` feature".formatted(feature.getName()))
+                .setDescription("Successfully disabled the `%s` feature".formatted(feature.getName()))
                 .build()).queue();
     }
 }

src/main/java/cc/fascinated/bat/features/base/commands/server/feature/EnableSubCommand.java

@@ -8,7 +8,6 @@ import cc.fascinated.bat.features.base.profile.FeatureProfile;
 import cc.fascinated.bat.model.BatGuild;
 import cc.fascinated.bat.model.BatUser;
 import cc.fascinated.bat.service.FeatureService;
-import cc.fascinated.bat.service.GuildService;
 import lombok.NonNull;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.channel.middleman.MessageChannel;
@@ -24,11 +23,8 @@ import org.springframework.stereotype.Component;
 @Component("feature:enable.sub")
 @CommandInfo(name = "enable", description = "Enables a feature")
 public class EnableSubCommand extends BatSubCommand {
-    private final GuildService guildService;
-
     @Autowired
-    public EnableSubCommand(@NonNull GuildService guildService) {
-        this.guildService = guildService;
+    public EnableSubCommand() {
         super.addOption(OptionType.STRING, "feature", "The feature to enable", true);
     }
 
@@ -58,7 +54,6 @@ public class EnableSubCommand extends BatSubCommand {
         }
 
         featureProfile.enableFeature(feature);
-        guildService.saveGuild(guild);
         interaction.replyEmbeds(EmbedUtils.successEmbed()
                 .setDescription("Successfully enabled the `%s` feature".formatted(feature.getName()))
                 .build()).queue();

src/main/java/cc/fascinated/bat/features/base/profile/FeatureProfile.java

@@ -1,9 +1,12 @@
 package cc.fascinated.bat.features.base.profile;
 
-import cc.fascinated.bat.common.Profile;
+import cc.fascinated.bat.common.Serializable;
 import cc.fascinated.bat.features.Feature;
+import com.google.gson.Gson;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.NoArgsConstructor;
+import org.bson.Document;
 
 import java.util.HashMap;
 import java.util.Map;
@@ -11,7 +14,8 @@ import java.util.Map;
 /**
  * @author Fascinated (fascinated7)
  */
-public class FeatureProfile extends Profile {
+@NoArgsConstructor
+public class FeatureProfile extends Serializable {
     private static final FeatureState DEFAULT_STATE = FeatureState.ENABLED;
 
     /**
@@ -19,19 +23,12 @@ public class FeatureProfile extends Profile {
      */
     private Map<String, FeatureState> featureStates;
 
-    public FeatureProfile() {
-        super("feature");
-    }
-
     /**
      * Gets the feature states
      *
      * @return the feature states
      */
     public FeatureState getFeatureState(Feature feature) {
-        if (this.featureStates == null) {
-            this.featureStates = new HashMap<>();
-        }
         if (feature == null) {
             return DEFAULT_STATE;
         }
@@ -85,9 +82,6 @@ public class FeatureProfile extends Profile {
      * @param state   the state to set
      */
     public void setFeatureState(Feature feature, FeatureState state) {
-        if (this.featureStates == null) {
-            this.featureStates = new HashMap<>();
-        }
         this.featureStates.put(feature.getName().toUpperCase(), state);
     }
 
@@ -96,6 +90,23 @@ public class FeatureProfile extends Profile {
         this.featureStates = null;
     }
 
+    @Override
+    public void load(Document document, Gson gson) {
+        this.featureStates = new HashMap<>();
+        for (String key : document.keySet()) {
+            this.featureStates.put(key, FeatureState.valueOf(document.getString(key)));
+        }
+    }
+
+    @Override
+    public Document serialize(Gson gson) {
+        Document document = new Document();
+        for (String key : this.featureStates.keySet()) {
+            document.put(key, this.featureStates.get(key).name());
+        }
+        return document;
+    }
+
     @AllArgsConstructor @Getter
     public enum FeatureState {
         ENABLED(":white_check_mark:"),
@@ -104,6 +115,6 @@ public class FeatureProfile extends Profile {
         /**
          * The emoji for the feature state
          */
-        private String emoji;
+        private final String emoji;
     }
 }