small refactors in code base

neoforge/build.gradle

@@ -48,4 +48,6 @@ dependencies {
     additionalRuntimeClasspath "org.yaml:snakeyaml:2.4"
 }
 
+sourceSets.main.resources { srcDir 'src/generated/resources' }
+
 sourceSets.main.resources { srcDir 'src/generated/resources' }

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/BookshelfInspectorNeoForge.java

@@ -1,11 +1,12 @@
 package com.lukasabbe.bookshelfinspector;
 
+import com.lukasabbe.bookshelfinspector.util.EventHandler;
+import com.lukasabbe.bookshelfinspector.util.NetworkHandler;
 import net.neoforged.bus.api.IEventBus;
 import net.neoforged.fml.common.Mod;
 
 @Mod(value = Constants.MOD_ID)
 public class BookshelfInspectorNeoForge {
-
     public BookshelfInspectorNeoForge(IEventBus eventBus) {
         EventHandler.initServer();
         eventBus.addListener(NetworkHandler::registerPayloads);

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/BookshelfInspectorNeoForgeClient.java

@@ -1,5 +1,7 @@
 package com.lukasabbe.bookshelfinspector;
 
+import com.lukasabbe.bookshelfinspector.config.ClothConfigGenerator;
+import com.lukasabbe.bookshelfinspector.util.EventHandler;
 import net.neoforged.api.distmarker.Dist;
 import net.neoforged.bus.api.IEventBus;
 import net.neoforged.fml.ModLoadingContext;

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/config/ClothConfigGenerator.java

@@ -1,5 +1,6 @@
-package com.lukasabbe.bookshelfinspector;
+package com.lukasabbe.bookshelfinspector.config;
 
+import com.lukasabbe.bookshelfinspector.BookshelfInspectorClient;
 import me.shedaniel.clothconfig2.api.ConfigBuilder;
 import me.shedaniel.clothconfig2.api.ConfigEntryBuilder;
 import net.minecraft.client.gui.screens.Screen;

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/platform/NeoForgeEventHelper.java

@@ -1,6 +1,6 @@
 package com.lukasabbe.bookshelfinspector.platform;
 
-import com.lukasabbe.bookshelfinspector.EventHandler;
+import com.lukasabbe.bookshelfinspector.util.EventHandler;
 import com.lukasabbe.bookshelfinspector.platform.handlers.OnPlayerDisconnectEvent;
 import com.lukasabbe.bookshelfinspector.platform.handlers.OnPlayerJoinEvent;
 import com.lukasabbe.bookshelfinspector.platform.services.IEventHelper;
@@ -12,7 +12,5 @@ public class NeoForgeEventHelper implements IEventHelper {
     }
 
     @Override
-    public void registerOnPlayerDisconnect(OnPlayerDisconnectEvent event) {
-        EventHandler.playerDisconnectEvents.add(event);
-    }
+    public void registerOnPlayerDisconnect(OnPlayerDisconnectEvent event) { EventHandler.playerDisconnectEvents.add(event); }
 }

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/util/EventHandler.java

@@ -1,4 +1,4 @@
-package com.lukasabbe.bookshelfinspector;
+package com.lukasabbe.bookshelfinspector.util;
 
 import com.lukasabbe.bookshelfinspector.platform.handlers.OnPlayerDisconnectEvent;
 import com.lukasabbe.bookshelfinspector.platform.handlers.OnPlayerJoinEvent;
@@ -17,9 +17,7 @@ public class EventHandler {
     public static void initClient(){
         NeoForge.EVENT_BUS.addListener(EventHandler::onDisconnect);
     }
-    public static void initServer(){
-        NeoForge.EVENT_BUS.addListener(EventHandler::onJoin);
-    }
+    public static void initServer(){ NeoForge.EVENT_BUS.addListener(EventHandler::onJoin); }
 
     private static void onJoin(PlayerEvent.PlayerLoggedInEvent event){
         playerJoinEvents.forEach(onPlayerJoinEvent -> onPlayerJoinEvent.onPlayerJoin((ServerPlayer) event.getEntity(),event.getEntity().getServer()));

neoforge/src/main/java/com/lukasabbe/bookshelfinspector/util/NetworkHandler.java

@@ -1,5 +1,7 @@
-package com.lukasabbe.bookshelfinspector;
+package com.lukasabbe.bookshelfinspector.util;
 
+import com.lukasabbe.bookshelfinspector.BookshelfInspector;
+import com.lukasabbe.bookshelfinspector.Constants;
 import com.lukasabbe.bookshelfinspector.network.packets.BookShelfInventoryPayload;
 import com.lukasabbe.bookshelfinspector.network.packets.BookShelfInventoryRequestPayload;
 import com.lukasabbe.bookshelfinspector.network.packets.LecternInventoryRequestPayload;
@@ -7,7 +9,6 @@ import com.lukasabbe.bookshelfinspector.network.packets.ModCheckPayload;
 import net.minecraft.client.player.LocalPlayer;
 import net.minecraft.server.level.ServerPlayer;
 import net.neoforged.bus.api.SubscribeEvent;
-import net.neoforged.neoforge.network.event.RegisterConfigurationTasksEvent;
 import net.neoforged.neoforge.network.event.RegisterPayloadHandlersEvent;
 import net.neoforged.neoforge.network.registration.PayloadRegistrar;
 
@@ -15,7 +16,7 @@ public class NetworkHandler {
 
     @SubscribeEvent
     public static void registerPayloads(final RegisterPayloadHandlersEvent event){
-        final PayloadRegistrar registrar = event.registrar("bookshelfinspector").optional();
+        final PayloadRegistrar registrar = event.registrar(Constants.MOD_ID).optional();
         registrar.playToClient(
                 BookShelfInventoryPayload.ID,
                 BookShelfInventoryPayload.CODEC,

neoforge/src/main/resources/META-INF/neoforge.mods.toml

@@ -10,4 +10,16 @@ credits="${credits}" #optional
 authors = "${mod_author}" #optional
 description = '''${description}''' #mandatory (Supports multiline text)
 [[mixins]]
-config = "${mod_id}.mixins.json"
+config = "${mod_id}.mixins.json"
+[[dependencies.${mod_id}]] #optional
+modId = "neoforge" #mandatory
+type = "required" #mandatory (Can be one of "required", "optional", "incompatible" or "discouraged")
+versionRange = "[${neoforge_version},)" #mandatory
+ordering = "NONE" # The order that this dependency should load in relation to your mod, required to be either 'BEFORE' or 'AFTER' if the dependency is not mandatory
+side = "BOTH" # Side this dependency is applied on - 'BOTH', 'CLIENT' or 'SERVER'
+[[dependencies.${mod_id}]]
+modId = "minecraft"
+type="required" #mandatory (Can be one of "required", "optional", "incompatible" or "discouraged")
+versionRange = "${minecraft_version_range}"
+ordering = "NONE"
+side = "BOTH"