Modules: Preserve order of modules for dependency requirements

Author: camnwalter

src/main/java/com/chattriggers/ctjs/internal/mixins/SystemDetailsMixin.java

@@ -10,6 +10,7 @@
 import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
+import java.util.ArrayList;
 import java.util.Comparator;
 import java.util.List;
 import java.util.function.Supplier;
@@ -22,7 +23,7 @@ public abstract class SystemDetailsMixin {
     @Inject(method = "<init>", at = @At("RETURN"))
     private void addModules(CallbackInfo ci) {
         addSection("ChatTriggers Modules", () -> {
-            List<Module> modules = ModuleManager.INSTANCE.getCachedModules();
+            List<Module> modules = new ArrayList<>(ModuleManager.INSTANCE.getCachedModules());
             modules.sort(Comparator.comparing(Module::getName));
 
             StringBuilder sb = new StringBuilder();

src/main/kotlin/com/chattriggers/ctjs/internal/engine/module/ModuleUpdater.kt

@@ -112,9 +112,6 @@ object ModuleUpdater : Initializer {
         }
 
         cachedModules.add(module)
-        cachedModules.sortWith { a, b ->
-            a.name.compareTo(b.name)
-        }
         return listOf(module) + (module.metadata.requires?.map {
             importModule(it, module.name)
         }?.flatten() ?: emptyList())

src/main/kotlin/com/chattriggers/ctjs/internal/engine/module/ModulesGui.kt

@@ -47,7 +47,7 @@ object ModulesGui : UScreen(unlocalizedName = "Modules") {
         window.exit.draw((middle + width / 2f - 17) / 2f, (window.scroll + 99f) / 2f)
 
         window.height = 125f
-        ModuleManager.cachedModules.forEach {
+        ModuleManager.cachedModules.sortedBy { it.name }.forEach {
             window.height += it.draw(middle - width / 2f, window.scroll + window.height, width)
         }