Update to 1.20.5

Author: senseiwells

build.gradle

@@ -1,7 +1,7 @@
 plugins {
 	id "maven-publish"
-	id "fabric-loom" version "1.5-SNAPSHOT"
-	id 'me.modmuss50.mod-publish-plugin' version '0.3.4'
+	id "fabric-loom" version "1.6-SNAPSHOT"
+	id 'me.modmuss50.mod-publish-plugin' version '0.5.1'
 }
 
 repositories {
@@ -18,7 +18,7 @@ repositories {
 	}
 }
 
-version = minecraft_version + "-" + mod_version
+version = "${mod_version}+mc${minecraft_version}"
 group = maven_group
 
 dependencies {
@@ -27,8 +27,8 @@ dependencies {
 	modImplementation "net.fabricmc:fabric-loader:${loader_version}"
 	modImplementation "net.fabricmc.fabric-api:fabric-api:${fabric_version}"
 
-	modImplementation "com.github.gnembon:fabric-carpet:${carpet_version}"
-	modImplementation("com.github.CaffeineMC:lithium-fabric:mc1.20.4-0.12.1") {
+	modImplementation "com.github.senseiwells:fabric-carpet:${carpet_version}"
+	modImplementation("com.github.CaffeineMC:lithium-fabric:${lithium_version}") {
 		exclude group: 'com.github.2No2Name', module: 'McTester'
 	}
 	include modImplementation("me.lucko:fabric-permissions-api:0.2-SNAPSHOT")
@@ -40,7 +40,7 @@ processResources {
 	filesMatching("fabric.mod.json") {
 		def map = [
 			"version"             : mod_version,
-			"carpet_dependency"   : ">=" + carpet_version,
+//			"carpet_dependency"   : ">=" + carpet_version,
 			"minecraft_dependency": minecraft_dependency
 		]
 		expand map

gradle.properties

@@ -1,14 +1,15 @@
 # Done to increase the memory available to gradle.
 org.gradle.jvmargs=-Xmx6G
 # Fabric Properties
-minecraft_version=1.20.4
-yarn_mappings=1.20.4+build.1
-loader_version=0.15.0
+minecraft_version=1.20.5
+yarn_mappings=1.20.5+build.1
+loader_version=0.15.10
 # Mod Properties
-mod_version = 1.3.4
+mod_version = 1.4.0
 maven_group = essentialaddons
 # Dependencies
-carpet_version=1.4.128
-fabric_version=0.91.1+1.20.4
-mixin_extras_version=0.2.0
+carpet_version=8639fa9d957bc41d205280017ffe325093c924ad
+lithium_version=98a10fbeeaf72a1b1162f5fe6ec8315df3e9caa5
+fabric_version=0.97.6+1.20.5
 minecraft_dependency=1.20.x
+

gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.5-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.6-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

src/main/java/essentialaddons/EssentialAddons.java

@@ -2,16 +2,12 @@
 
 import carpet.CarpetExtension;
 import carpet.CarpetServer;
-import carpet.script.CarpetExpression;
 import com.mojang.brigadier.CommandDispatcher;
 import essentialaddons.commands.*;
 import essentialaddons.feature.GameRuleNetworkHandler;
 import essentialaddons.feature.ReloadFakePlayers;
-import essentialaddons.feature.script.PacketEvent;
-import essentialaddons.feature.script.ScriptPacketHandler;
 import essentialaddons.logging.EssentialAddonsLoggerRegistry;
 import essentialaddons.utils.*;
-import essentialaddons.utils.network.NetworkHandler;
 import net.fabricmc.api.ModInitializer;
 import net.minecraft.command.CommandRegistryAccess;
 import net.minecraft.server.MinecraftServer;
@@ -26,7 +22,6 @@
 public class EssentialAddons implements CarpetExtension, ModInitializer {
     public static final Logger LOGGER;
     public static final Set<Config> CONFIG_SET;
-    public static final Set<NetworkHandler> NETWORK_HANDLERS;
     public static MinecraftServer server;
 
     static {
@@ -37,16 +32,14 @@ public class EssentialAddons implements CarpetExtension, ModInitializer {
             ConfigFakePlayerData.INSTANCE,
             ConfigTeamTeleportBlacklist.INSTANCE
         );
-        NETWORK_HANDLERS = Set.of(
-            GameRuleNetworkHandler.INSTANCE,
-            ScriptPacketHandler.INSTANCE
-        );
     }
 
     @Override
     public void onInitialize() {
         CarpetServer.manageExtension(this);
         ConfigCamera.INSTANCE.readConfig();
+
+        GameRuleNetworkHandler.INSTANCE.registerGameRulePayloads();
     }
 
     @Override
@@ -64,11 +57,6 @@ public void registerLoggers() {
         EssentialAddonsLoggerRegistry.registerLoggers();
     }
 
-    @Override
-    public void scarpetApi(CarpetExpression expression) {
-        ScriptPacketHandler.INSTANCE.addScarpetExpression(expression.getExpr());
-    }
-
     @Override
     public void onServerLoaded(MinecraftServer server) {
         EssentialAddons.server = server;
@@ -82,7 +70,6 @@ public void onServerLoadedWorlds(MinecraftServer server) {
         if (EssentialSettings.reloadFakePlayers) {
             ReloadFakePlayers.loadFakePlayers(server);
         }
-        PacketEvent.noop();
     }
 
     @Override
@@ -93,7 +80,7 @@ public void onServerClosed(MinecraftServer server) {
     }
 
     @Override
-    public void registerCommands(CommandDispatcher<ServerCommandSource> dispatcher, CommandRegistryAccess commandBuildContext) {
+    public void registerCommands(CommandDispatcher<ServerCommandSource> dispatcher, CommandRegistryAccess access) {
         CommandFly.register(dispatcher);
         CommandHat.register(dispatcher);
         CommandRepair.register(dispatcher);
@@ -117,7 +104,7 @@ public void registerCommands(CommandDispatcher<ServerCommandSource> dispatcher,
         CommandTop.register(dispatcher);
         CommandNear.register(dispatcher);
         CommandLagSpike.register(dispatcher);
-        CommandRename.register(dispatcher);
+        CommandRename.register(dispatcher, access);
         CommandMods.register(dispatcher);
         CommandGhostPlayer.register(dispatcher);
         CommandConfig.register(dispatcher);
@@ -126,7 +113,7 @@ public void registerCommands(CommandDispatcher<ServerCommandSource> dispatcher,
 
     @Override
     public void onPlayerLoggedIn(ServerPlayerEntity player) {
-        NETWORK_HANDLERS.forEach(networkHandler -> networkHandler.sayHello(player));
+        GameRuleNetworkHandler.INSTANCE.sayHello(player);
     }
 
     @Override

src/main/java/essentialaddons/EssentialUtils.java

@@ -1,15 +1,12 @@
 package essentialaddons;
 
-import carpet.helpers.InventoryHelper;
 import com.google.gson.Gson;
 import com.google.gson.reflect.TypeToken;
 import essentialaddons.utils.Subscription;
 import me.lucko.fabric.api.permissions.v0.Permissions;
 import net.fabricmc.api.EnvType;
 import net.fabricmc.loader.api.FabricLoader;
-import net.minecraft.block.BlockState;
 import net.minecraft.block.ShulkerBoxBlock;
-import net.minecraft.block.entity.BlockEntity;
 import net.minecraft.entity.Entity;
 import net.minecraft.entity.EntityType;
 import net.minecraft.entity.damage.DamageSource;
@@ -28,7 +25,6 @@
 import net.minecraft.text.Text;
 import net.minecraft.util.ItemScatterer;
 import net.minecraft.util.WorldSavePath;
-import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.GameRules;
 import net.minecraft.world.World;
 import org.apache.commons.io.IOUtils;
@@ -44,8 +40,6 @@
 
 import static carpet.utils.CommandHelper.canUseCommand;
 import static essentialaddons.EssentialAddons.server;
-import static net.minecraft.block.Block.dropStack;
-import static net.minecraft.block.Block.getDroppedStacks;
 
 public class EssentialUtils {
     public static final Random RANDOM = new Random();
@@ -93,24 +87,9 @@ public static void breakVehicleStorage(
         }
     }
 
-    public static void placeItemInInventory(BlockState state, World world, BlockPos pos, BlockEntity blockEntity, ServerPlayerEntity player, ItemStack stack){
-        if (world instanceof ServerWorld serverWorld) {
-            getDroppedStacks(state, serverWorld, pos, blockEntity, player, stack).forEach((itemStack) -> {
-                if (!placeItemInInventory(player, itemStack)) {
-                    dropStack(serverWorld, pos, itemStack);
-                }
-            });
-            state.onStacksDropped(serverWorld, pos, stack, true);
-        }
-    }
-
     public static boolean placeItemInInventory(ServerPlayerEntity player, ItemStack itemStack) {
         Item item = itemStack.getItem();
         int itemAmount = itemStack.getCount();
-        if (EssentialSettings.stackableShulkersInPlayerInventories && !InventoryHelper.shulkerBoxHasItems(itemStack) && isItemShulkerBox(itemStack.getItem())) {
-            itemStack.removeSubNbt("BlockEntityTag");
-            item = itemStack.getItem();
-        }
         if (player.getInventory().insertStack(itemStack)) {
             player.getWorld().playSound(null, player.getX(), player.getY(), player.getZ(), SoundEvents.ENTITY_ITEM_PICKUP, SoundCategory.PLAYERS, 0.2f, (RANDOM.nextFloat() - RANDOM.nextFloat()) * 1.4F + 2.0F);
             player.increaseStat(Stats.PICKED_UP.getOrCreateStat(item), itemAmount);

src/main/java/essentialaddons/commands/CommandCameraMode.java

@@ -70,7 +70,7 @@ private static boolean isInDanger(ServerPlayerEntity player) {
         }
 
         for (StatusEffectInstance effect : player.getStatusEffects()) {
-            if (effect.getEffectType().getCategory() == StatusEffectCategory.HARMFUL) {
+            if (effect.getEffectType().value().getCategory() == StatusEffectCategory.HARMFUL) {
                 EssentialUtils.sendToActionBar(player, "§cYou cannot enter spectator because you have a negative status effect");
                 return true;
             }

src/main/java/essentialaddons/commands/CommandRename.java

@@ -1,11 +1,14 @@
 package essentialaddons.commands;
 
+import com.mojang.brigadier.Command;
 import com.mojang.brigadier.CommandDispatcher;
 import com.mojang.brigadier.arguments.StringArgumentType;
 import com.mojang.brigadier.exceptions.SimpleCommandExceptionType;
 import essentialaddons.EssentialSettings;
 import net.fabricmc.loader.api.FabricLoader;
+import net.minecraft.command.CommandRegistryAccess;
 import net.minecraft.command.argument.TextArgumentType;
+import net.minecraft.component.DataComponentTypes;
 import net.minecraft.item.ItemStack;
 import net.minecraft.server.command.ServerCommandSource;
 import net.minecraft.text.Text;
@@ -17,24 +20,24 @@
 public class CommandRename {
     private static final SimpleCommandExceptionType ITEM_IS_AIR = new SimpleCommandExceptionType(Text.literal("Cannot rename air!"));
 
-    public static void register(CommandDispatcher<ServerCommandSource> dispatcher) {
+    public static void register(CommandDispatcher<ServerCommandSource> dispatcher, CommandRegistryAccess access) {
         // We let them handle /rename command
         if (FabricLoader.getInstance().isModLoaded("itemrename")) {
             return;
         }
 
         dispatcher.register(literal("rename").requires(enabled(() -> EssentialSettings.commandRename, "essentialaddons.command.rename"))
             .then(literal("json")
-                .then(argument("name", TextArgumentType.text())
+                .then(argument("name", TextArgumentType.text(access))
                     .executes(context -> {
                         ItemStack stack = context.getSource().getPlayerOrThrow().getMainHandStack();
                         if (!stack.isEmpty()) {
                             Text text = TextArgumentType.getTextArgument(context, "name");
                             context.getSource().sendFeedback(() -> {
                                 return Text.literal("Item name set to: ").append(text);
                             }, false);
-                            stack.setCustomName(text);
-                            return 0;
+                            stack.set(DataComponentTypes.CUSTOM_NAME, text);
+                            return Command.SINGLE_SUCCESS;
                         }
                         throw ITEM_IS_AIR.create();
                     })
@@ -49,8 +52,8 @@ public static void register(CommandDispatcher<ServerCommandSource> dispatcher) {
                             context.getSource().sendFeedback(() -> {
                                 return Text.literal("Item name set to: ").append(name);
                             }, false);
-                            itemStack.setCustomName(Text.literal(name).styled(s -> s.withItalic(false)));
-                            return 0;
+                            itemStack.set(DataComponentTypes.CUSTOM_NAME, Text.literal(name).styled(s -> s.withItalic(false)));
+                            return Command.SINGLE_SUCCESS;
                         }
                         throw ITEM_IS_AIR.create();
                     })