updated config

Common/build.gradle

@@ -28,8 +28,8 @@ repositories {
 dependencies {
      compileOnly group:'org.spongepowered', name:'mixin', version:'0.8.5'
      implementation group: 'com.google.code.findbugs', name: 'jsr305', version: '3.0.1'
-    implementation("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
-    annotationProcessor("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
+    implementation("com.github.llamalad7.mixinextras:mixinextras-common:${mixinextras_version}")
+    annotationProcessor("com.github.llamalad7.mixinextras:mixinextras-common:${mixinextras_version}")
 }
 
 processResources {

Common/src/main/java/tschipp/carryon/CarryOnCommonClient.java

@@ -14,11 +14,11 @@ public class CarryOnCommonClient
 		Player player = mc.player;
 		if(player != null) {
 			CarryOnData carry = CarryOnDataManager.getCarryData(player);
-			if ((CarryOnKeybinds.carryKey.isUnbound() ? player.isShiftKeyDown() : CarryOnKeybinds.carryKey.isDown()) && !carry.isKeyPressed()) {
+			if ((CarryOnKeybinds.carryKey.isUnbound() ? player.isShiftKeyDown() : (CarryOnKeybinds.carryKey.isDown() || checkMouse())) && !carry.isKeyPressed()) {
 				CarryOnKeybinds.onCarryKey(true);
 				carry.setKeyPressed(true);
 				CarryOnDataManager.setCarryData(player, carry);
-			} else if (!(CarryOnKeybinds.carryKey.isUnbound() ? player.isShiftKeyDown() : CarryOnKeybinds.carryKey.isDown()) && carry.isKeyPressed()) {
+			} else if (!(CarryOnKeybinds.carryKey.isUnbound() ? player.isShiftKeyDown() : (CarryOnKeybinds.carryKey.isDown() || checkMouse()) ) && carry.isKeyPressed()) {
 				CarryOnKeybinds.onCarryKey(false);
 				carry.setKeyPressed(false);
 				CarryOnDataManager.setCarryData(player, carry);
@@ -26,6 +26,12 @@ public class CarryOnCommonClient
 		}
 	}
 
+	private static boolean checkMouse()
+	{
+		Minecraft mc = Minecraft.getInstance();
+		return (CarryOnKeybinds.carryKey.matchesMouse(0) && mc.mouseHandler.isLeftPressed()) || (CarryOnKeybinds.carryKey.matchesMouse(1) && mc.mouseHandler.isRightPressed()) || (CarryOnKeybinds.carryKey.matchesMouse(3) && mc.mouseHandler.isMiddlePressed());
+	}
+
 	public static void onCarryClientTick()
 	{
 		Player player = Minecraft.getInstance().player;

Common/src/main/java/tschipp/carryon/client/keybinds/CarryOnKeybinds.java

@@ -14,7 +14,11 @@ public class CarryOnKeybinds
 
 	public static void registerKeybinds(Consumer<KeyMapping> registrar)
 	{
+		if(Services.PLATFORM.isModLoaded("amecsapi"))
+			carryKey = new ConflictFreeKeyMapping("key.carry.desc", InputConstants.KEY_LSHIFT, "key.carry.category");
+		else
 			carryKey = new ConflictFreeKeyMapping("key.carry.desc", Services.PLATFORM.getPlatformName().equals("Forge") ? InputConstants.KEY_LSHIFT : InputConstants.UNKNOWN.getValue(), "key.carry.category");
+
 		registrar.accept(carryKey);
 	}
 

Common/src/main/java/tschipp/carryon/client/render/CarriedObjectRender.java

@@ -1,13 +1,16 @@
 package tschipp.carryon.client.render;
 
 import com.mojang.blaze3d.systems.RenderSystem;
+import com.mojang.blaze3d.vertex.BufferBuilder;
 import com.mojang.blaze3d.vertex.PoseStack;
 import com.mojang.blaze3d.vertex.Tesselator;
 import com.mojang.math.Vector3f;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.renderer.MultiBufferSource;
 import net.minecraft.client.renderer.MultiBufferSource.BufferSource;
+import net.minecraft.client.renderer.RenderType;
 import net.minecraft.client.renderer.entity.EntityRenderDispatcher;
+import net.minecraft.client.renderer.texture.TextureAtlas;
 import net.minecraft.client.resources.model.BakedModel;
 import net.minecraft.world.entity.Entity;
 import net.minecraft.world.entity.HumanoidArm;
@@ -26,11 +29,20 @@ import tschipp.carryon.common.scripting.CarryOnScript;
 import tschipp.carryon.common.scripting.CarryOnScript.ScriptRender;
 import tschipp.carryon.platform.Services;
 
+import java.util.Map;
 import java.util.Optional;
 
 public class CarriedObjectRender
 {
 
+	private static Map<RenderType, BufferBuilder> builders = Map.of(
+			RenderType.glint(), new BufferBuilder(RenderType.glint().bufferSize()),
+			RenderType.glintDirect(), new BufferBuilder(RenderType.glintDirect().bufferSize()),
+			RenderType.glintTranslucent(), new BufferBuilder(RenderType.glintTranslucent().bufferSize()),
+			RenderType.entityGlint(), new BufferBuilder(RenderType.entityGlint().bufferSize()),
+			RenderType.entityGlintDirect(), new BufferBuilder(RenderType.entityGlintDirect().bufferSize())
+	);
+
 	public static boolean drawFirstPerson(Player player, MultiBufferSource buffer, PoseStack matrix, int light, float partialTicks)
 	{
 		if(Services.PLATFORM.isModLoaded("firstperson") || Services.PLATFORM.isModLoaded("firstpersonmod"))
@@ -133,10 +145,11 @@ public class CarriedObjectRender
 			{
 			}
 			manager.setRenderShadow(true);
+			matrix.popPose();
+
 		}
 
 		// RenderSystem.disableAlphaTest();
-		matrix.popPose();
 	}
 
 	/**
@@ -147,7 +160,6 @@ public class CarriedObjectRender
 	public static void drawThirdPerson(float partialticks, PoseStack matrix) {
 		Minecraft mc = Minecraft.getInstance();
 		Level level = mc.level;
-		BufferSource buffer = MultiBufferSource.immediate(Tesselator.getInstance().getBuilder());
 		int light = 0;
 		int perspective = CarryRenderHelper.getPerspective();
 		EntityRenderDispatcher manager = mc.getEntityRenderDispatcher();
@@ -156,9 +168,12 @@ public class CarriedObjectRender
 		RenderSystem.disableCull();
 		RenderSystem.disableDepthTest();
 
+		BufferSource buffer = MultiBufferSource.immediateWithBuffers(builders, Tesselator.getInstance().getBuilder());
+
 		for (Player player : level.players())
 		{
 			try {
+
 				CarryOnData carry = CarryOnDataManager.getCarryData(player);
 
 				if (perspective == 0 && player == mc.player && !(Services.PLATFORM.isModLoaded("firstperson") || Services.PLATFORM.isModLoaded("firstpersonmod")))
@@ -193,7 +208,6 @@ public class CarriedObjectRender
 					RenderSystem.setShaderColor(1.0F, 1.0F, 1.0F, 1.0F);
 
 					CarryRenderHelper.renderBakedModel(tileItem, copy, buffer, light, model);
-					buffer.endBatch();
 
 					matrix.popPose();
 				} else if (carry.isCarrying(CarryType.ENTITY)) {
@@ -216,19 +230,26 @@ public class CarriedObjectRender
 						RenderSystem.setShaderColor(1.0F, 1.0F, 1.0F, 1.0F);
 
 						manager.render(entity, 0, 0, 0, 0f, 0, matrix, buffer, light);
-						buffer.endBatch();
-
 						matrix.popPose();
 						manager.setRenderShadow(true);
 						matrix.popPose();
 					}
 				}
+
+
 			}
 			catch (Exception e)
 			{
 			}
 
 		}
+		buffer.endLastBatch();
+
+		buffer.endBatch(RenderType.entitySolid(TextureAtlas.LOCATION_BLOCKS));
+		buffer.endBatch(RenderType.entityCutout(TextureAtlas.LOCATION_BLOCKS));
+		buffer.endBatch(RenderType.entityCutoutNoCull(TextureAtlas.LOCATION_BLOCKS));
+		buffer.endBatch(RenderType.entitySmoothCutout(TextureAtlas.LOCATION_BLOCKS));
+
 		RenderSystem.enableDepthTest();
 		RenderSystem.enableCull();
 		RenderSystem.disableBlend();

Common/src/main/java/tschipp/carryon/client/render/CarryRenderHelper.java

@@ -1,12 +1,10 @@
 package tschipp.carryon.client.render;
 
-import com.mojang.authlib.minecraft.client.MinecraftClient;
 import com.mojang.blaze3d.vertex.PoseStack;
 import com.mojang.math.Quaternion;
 import com.mojang.math.Vector3f;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.renderer.MultiBufferSource;
-import net.minecraft.client.renderer.RenderType;
 import net.minecraft.client.renderer.block.model.ItemTransforms.TransformType;
 import net.minecraft.client.renderer.entity.ItemRenderer;
 import net.minecraft.client.renderer.texture.OverlayTexture;

Common/src/main/java/tschipp/carryon/common/carry/CarryOnData.java

@@ -6,12 +6,14 @@ import net.minecraft.nbt.CompoundTag;
 import net.minecraft.nbt.NbtOps;
 import net.minecraft.nbt.NbtUtils;
 import net.minecraft.nbt.Tag;
+import net.minecraft.world.entity.AreaEffectCloud;
 import net.minecraft.world.entity.Entity;
 import net.minecraft.world.entity.EntityType;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.entity.BlockEntity;
 import net.minecraft.world.level.block.state.BlockState;
 import net.minecraft.world.level.block.state.properties.BlockStateProperties;
+import tschipp.carryon.Constants;
 import tschipp.carryon.common.scripting.CarryOnScript;
 
 import javax.annotation.Nullable;
@@ -121,7 +123,13 @@ public class CarryOnData {
         if(this.type != CarryType.ENTITY)
             throw new IllegalStateException("Called getEntity on data that contained " + this.type);
 
-        return EntityType.create(nbt.getCompound("entity"), level).orElseThrow(() -> new IllegalStateException("Called EntityType#create even though no entity data was present. Data: " + nbt.toString()));
+        var optionalEntity = EntityType.create(nbt.getCompound("entity"), level);
+        if(optionalEntity.isPresent())
+            return optionalEntity.get();
+
+        Constants.LOG.error("Called EntityType#create even though no entity data was present. Data: " + nbt.toString());
+        this.clear();
+        return new AreaEffectCloud(level, 0, 0, 0);
     }
 
     public Optional<CarryOnScript> getActiveScript()

Common/src/main/java/tschipp/carryon/common/carry/PickupHandler.java

@@ -137,6 +137,9 @@ public class PickupHandler {
         if (entity.invulnerableTime != 0)
             return false;
 
+        if(entity.isRemoved())
+            return false;
+
         if (entity instanceof TamableAnimal tame)
         {
             UUID owner = tame.getOwnerUUID();

Common/src/main/java/tschipp/carryon/common/carry/PlacementHandler.java

@@ -1,7 +1,6 @@
 package tschipp.carryon.common.carry;
 
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.BlockPos.MutableBlockPos;
 import net.minecraft.core.Direction;
 import net.minecraft.server.level.ServerPlayer;
 import net.minecraft.sounds.SoundEvents;
@@ -29,6 +28,8 @@ import tschipp.carryon.common.config.ListHandler;
 import tschipp.carryon.common.scripting.CarryOnScript.ScriptEffects;
 
 import javax.annotation.Nullable;
+import java.util.ArrayList;
+import java.util.Comparator;
 import java.util.List;
 import java.util.function.BiFunction;
 
@@ -77,8 +78,11 @@ public class PlacementHandler
 		}
 
 		level.setBlockAndUpdate(pos, state);
-		if (blockEntity != null)
+		if (blockEntity != null) {
+			blockEntity.setBlockState(state);
 			level.setBlockEntity(blockEntity);
+		}
+
 		level.updateNeighborsAt(pos.relative(Direction.DOWN), level.getBlockState(pos.relative(Direction.DOWN)).getBlock());
 		carry.clear();
 		CarryOnDataManager.setCarryData(player, carry);
@@ -284,28 +288,32 @@ public class PlacementHandler
 
 	private static BlockPos getDeathPlacementPos(BlockState state, ServerPlayer player)
 	{
-		MutableBlockPos pos = new MutableBlockPos();
 		BlockPos p = player.blockPosition();
 
 		int DISTANCE = 15;
 
+		List<BlockPos> potentialPositions = new ArrayList<>();
+
 		for (int j = 0; j < DISTANCE * 2; j++) {
 			for (int i = 0; i < DISTANCE * 2; i++) {
 				for (int k = 0; k < DISTANCE * 2; k++) {
 					int x = i % 2 == 0 ? i / 2 : -(i / 2);
 					int y = j % 2 == 0 ? j / 2 : -(j / 2);
 					int z = k % 2 == 0 ? k / 2 : -(k / 2);
+					potentialPositions.add(new BlockPos(p.getX() + x, p.getY() + y, p.getZ() + z));
+				}
+			}
+		}
 
-					pos.set(p.getX() + x, p.getY() + y, p.getZ() + z);
+		potentialPositions.sort(Comparator.comparingDouble(posA -> posA.distToCenterSqr(player.position())));
 
-					BlockPlaceContext context = new BlockPlaceContext(player, InteractionHand.MAIN_HAND, ItemStack.EMPTY, BlockHitResult.miss(Vec3.atCenterOf(pos), Direction.DOWN, pos));
-
-					boolean canPlace = state.canSurvive(player.level, pos) && player.level.getBlockState(pos).canBeReplaced(context) && player.level.isUnobstructed(state, pos, CollisionContext.of(player));
+		for(BlockPos potential : potentialPositions)
+		{
+			BlockPlaceContext context = new BlockPlaceContext(player, InteractionHand.MAIN_HAND, ItemStack.EMPTY, BlockHitResult.miss(Vec3.atCenterOf(potential), Direction.DOWN, potential));
+			boolean canPlace = state.canSurvive(player.level, potential) && player.level.getBlockState(potential).canBeReplaced(context) && player.level.isUnobstructed(state, potential, CollisionContext.of(player));
 
 			if (canPlace)
-						return pos;
-				}
-			}
+				return potential;
 		}
 
 		return p;

Common/src/main/java/tschipp/carryon/common/config/CarryConfig.java

@@ -5,10 +5,12 @@ import tschipp.carryon.config.annotations.Category;
 import tschipp.carryon.config.annotations.Config;
 import tschipp.carryon.config.annotations.Property;
 
-public class CarryConfig {
+public class CarryConfig
+{
 
 	@Config("carryon-common")
-    public static class Common {
+	public static class Common
+	{
 		//Settings
 		@Property(
 				type = PropertyType.CATEGORY,
@@ -17,7 +19,8 @@ public class CarryConfig {
 		public Settings settings = new Settings();
 
 		@Category("settings")
-        public static class Settings {
+		public static class Settings
+		{
 			@Property(
 					type = PropertyType.DOUBLE,
 					description = "Maximum distance from where Blocks and Entities can be picked up",
@@ -174,8 +177,10 @@ public class CarryConfig {
 
 		//Whitelist
 		public Whitelist whitelist = new Whitelist();
+
 		@Category("whitelist")
-        public static class Whitelist {
+		public static class Whitelist
+		{
 			@Property(
 					type = PropertyType.STRING_ARRAY,
 					description = "Entities that CAN be picked up (useWhitelistEntities must be true)"
@@ -201,8 +206,10 @@ public class CarryConfig {
 				description = "Blacklist. Read about the format here: https://github.com/Tschipp/CarryOn/wiki/Black---and-Whitelist-Config"
 		)
 		public Blacklist blacklist = new Blacklist();
+
 		@Category("blacklist")
-        public static class Blacklist {
+		public static class Blacklist
+		{
 			@Property(
 					type = PropertyType.STRING_ARRAY,
 					description = "Blocks that cannot be picked up"
@@ -231,7 +238,12 @@ public class CarryConfig {
 					"magneticraft:multiblock_gap", "refinedstorage:*", "mcmultipart:*", "enderstorage:*",
 					"betterstorage:*", "practicallogistics2:*", "wearablebackpacks:*", "rftools:screen",
 					"rftools:creative_screen", "create:*", "magic_doorknob:*", "iceandfire:*", "ftbquests:*",
-                    "waystones:*", "contact:*", "framedblocks:*"
+					"waystones:*", "contact:*", "framedblocks:*", "securitycraft:*", "forgemultipartcbe:*", "integrateddynamics:cable",
+					"mekanismgenerators:wind_generator", "cookingforblockheads:cabinet", "cookingforblockheads:corner", "cookingforblockheads:counter",
+					"cookingforblockheads:oven", "cookingforblockheads:toaster", "cookingforblockheads:milk_jar", "cookingforblockheads:cow_jar",
+					"cookingforblockheads:fruit_basket", "cookingforblockheads:cooking_table", "cookingforblockheads:fridge", "cookingforblockheads:sink",
+					"chipped:*",
+					"powah:*", "advancementtrophies:trophy", "mekanismgenerators:heat_generator", "mna:filler_block", "create_enchantment_industry:*", "graveyard:*", "immersivepetroleum:*", "tardis:interior_door"
 			};
 
 			@Property(
@@ -243,7 +255,8 @@ public class CarryConfig {
 					"minecraft:shulker", "minecraft:leash_knot", "minecraft:armor_stand",
 					"minecraft:item_frame", "minecraft:painting", "minecraft:shulker_bullet",
 					"animania:hamster", "animania:ferret*", "animania:hedgehog*", "animania:cart",
-                    "animania:wagon", "mynko:*", "pixelmon:*", "mocreatures:*", "quark:totem", "vehicle:*"
+					"animania:wagon", "mynko:*", "pixelmon:*", "mocreatures:*", "quark:totem", "vehicle:*",
+					"securitycraft:*", "taterzens:npc", "easy_npc:*", "bodiesbodies:dead_body"
 			};
 
 			@Property(
@@ -261,8 +274,10 @@ public class CarryConfig {
 				description = "Custom Pickup Conditions. Read about the format here: https://github.com/Tschipp/CarryOn/wiki/Custom-Pickup-Condition-Config"
 		)
 		public CustomPickupConditions customPickupConditions = new CustomPickupConditions();
+
 		@Category("customPickupConditions")
-        public static class CustomPickupConditions {
+		public static class CustomPickupConditions
+		{
 			@Property(
 					type = PropertyType.STRING_ARRAY,
 					description = "Custom Pickup Conditions for Blocks"
@@ -278,7 +293,8 @@ public class CarryConfig {
 	}
 
 	@Config("carryon-client")
-    public static class Client {
+	public static class Client
+	{
 
 		@Property(
 				type = PropertyType.BOOLEAN,

Common/src/main/java/tschipp/carryon/common/config/ListHandler.java

@@ -1,6 +1,5 @@
 package tschipp.carryon.common.config;
 
-import com.mojang.datafixers.kinds.Const;
 import net.minecraft.core.Registry;
 import net.minecraft.resources.ResourceLocation;
 import net.minecraft.tags.TagKey;
@@ -30,9 +29,7 @@ public class ListHandler {
     private static List<TagKey<EntityType<?>>> FORBIDDEN_STACKING_TAGS = new ArrayList<>();
     private static List<TagKey<EntityType<?>>> ALLOWED_STACKING_TAGS = new ArrayList<>();
 
-
-    private static Set<Class<?>> PROPERTY_EXCEPTION_CLASSES = new HashSet<>();
-
+    private static Set<Property<?>> PROPERTY_EXCEPTION_CLASSES = new HashSet<>();
 
     public static boolean isPermitted(Block block)
     {
@@ -60,7 +57,7 @@ public class ListHandler {
 
     public static boolean isPropertyException(Property<?> prop)
     {
-        return PROPERTY_EXCEPTION_CLASSES.contains(prop.getValueClass());
+        return PROPERTY_EXCEPTION_CLASSES.contains(prop);
     }
 
     private static boolean doCheck(Block block, Set<String> regular, List<TagKey<Block>> tags)
@@ -138,7 +135,7 @@ public class ListHandler {
             for(String propName : props.split(",")) {
                 for (Property<?> prop : blk.defaultBlockState().getProperties()) {
                     if (prop.getName().equals(propName))
-                        PROPERTY_EXCEPTION_CLASSES.add(prop.getValueClass());
+                        PROPERTY_EXCEPTION_CLASSES.add(prop);
                 }
             }
         }

Common/src/main/java/tschipp/carryon/common/scripting/Matchables.java

@@ -29,9 +29,7 @@ public final class Matchables
 	private static float getValueFromStringOrDefault(String toGetFrom, String key, float defaultVal)
 	{
 		Optional<Float> val = getValueFromString(toGetFrom, key);
-		if(val.isPresent())
-			return val.get();
-		return defaultVal;
+		return val.orElse(defaultVal);
 	}
 
 	private static Optional<Float> getValueFromString(String toGetFrom, String key)
@@ -240,9 +238,9 @@ public final class Matchables
 			int iG = cond.indexOf(">");
 			int iL = cond.indexOf("<");
 
-			if (iG == -1 ? true : iE < iG && iL == -1 ? true : iE < iL && iE != -1)
+			if (iG == -1 || (iE < iG && iL == -1 || iE < iL && iE != -1))
 				numb = cond.substring(iE);
-			else if (iE == -1 ? true : iG < iE && iL == -1 ? true : iG < iL && iG != -1)
+			else if (iE == -1 || (iG < iE && iL == -1 || iG < iL && iG != -1))
 				numb = cond.substring(iG);
 			else
 				numb = cond.substring(iL);
@@ -277,8 +275,8 @@ public final class Matchables
 			if (cond == null || cond.isEmpty())
 				return true;
 
-			BlockPos blockpos = new BlockPos(getValueFromStringOrDefault(cond, "x", 0), getValueFromStringOrDefault(cond, "y", 0), getValueFromStringOrDefault(cond, "z", 0));
-			BlockPos expand = new BlockPos(getValueFromStringOrDefault(cond, "dx", 0), getValueFromStringOrDefault(cond, "dy", 0), getValueFromStringOrDefault(cond, "dz", 0));
+			BlockPos blockpos = new BlockPos((int) getValueFromStringOrDefault(cond, "x", 0), (int) getValueFromStringOrDefault(cond, "y", 0), (int) getValueFromStringOrDefault(cond, "z", 0));
+			BlockPos expand = new BlockPos((int) getValueFromStringOrDefault(cond, "dx", 0), (int) getValueFromStringOrDefault(cond, "dy", 0), (int) getValueFromStringOrDefault(cond, "dz", 0));
 			BlockPos expanded = blockpos.offset(expand);
 			BlockPos pos = elem.blockPosition();
 

Fabric/build.gradle

@@ -16,18 +16,26 @@ if (System.getenv('BUILD_NUMBER') != null) {
 }
 repositories {
     maven { url 'https://jitpack.io' }
+    maven {
+        url "https://maven.siphalor.de/"
+        name "Siphalor's Maven"
+    }
 }
 
 
 dependencies {
     minecraft "com.mojang:minecraft:${minecraft_version}"
-    mappings loom.officialMojangMappings()
+    mappings loom.layered() {
+        officialMojangMappings()
+        parchment("org.parchmentmc.data:parchment-${parchment_mappings_fabric}@zip")
+    }
     modImplementation "net.fabricmc:fabric-loader:${fabric_loader_version}"
     modImplementation "net.fabricmc.fabric-api:fabric-api:${fabric_version}"
     implementation group: 'com.google.code.findbugs', name: 'jsr305', version: '3.0.1'
     implementation project(":Common")
-    include implementation("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
-    annotationProcessor("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
+    modRuntimeOnly  "de.siphalor:amecsapi-1.19:1.5.0+mc22w17a"
+    include implementation("com.github.llamalad7.mixinextras:mixinextras-fabric:${mixinextras_version}")
+    annotationProcessor("com.github.llamalad7.mixinextras:mixinextras-fabric:${mixinextras_version}")
 }
 
 loom {

Fabric/src/main/java/tschipp/carryon/compat/ArchitecturyCompat.java

@@ -0,0 +1,66 @@
+package tschipp.carryon.compat;
+
+import net.fabricmc.fabric.api.event.Event;
+import net.minecraft.core.BlockPos;
+import net.minecraft.world.entity.Entity;
+import net.minecraft.world.entity.player.Player;
+import net.minecraft.world.level.Level;
+import net.minecraft.world.level.block.state.BlockState;
+import tschipp.carryon.Constants;
+import tschipp.carryon.platform.Services;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+
+public class ArchitecturyCompat {
+
+    private static Object INVOKER_INSTANCE;
+    private static Method PLACE_BLOCK;
+    private static Method IS_FALSE;
+
+    private static void setup( ) {
+        try {
+            Class BlockEvent = Class.forName("dev.architectury.event.events.common.BlockEvent");
+            Field PLACE = BlockEvent.getField("PLACE");
+            Method invoker = Class.forName("dev.architectury.event.Event").getMethod("invoker");
+            INVOKER_INSTANCE = invoker.invoke(PLACE.get(BlockEvent));
+            Class PlaceClass = Class.forName("dev.architectury.event.events.common.BlockEvent$Place");
+            PLACE_BLOCK = PlaceClass.getMethod("placeBlock", Level.class, BlockPos.class, BlockState.class, Entity.class);
+            Class EventResult = Class.forName("dev.architectury.event.EventResult");
+            IS_FALSE = EventResult.getMethod("isFalse");
+
+        } catch (Exception e) {
+            Constants.LOG.warn("Error while initializing Architectury Compat: " + e);
+        }
+    }
+
+    public static boolean active() {
+        return Services.PLATFORM.isModLoaded("architectury");
+    }
+
+    public static boolean sendPlaceEvent(Level level, BlockPos pos, BlockState state, Player player) {
+        if(!active())
+            return true;
+
+        if(INVOKER_INSTANCE == null || PLACE_BLOCK == null)
+            setup();
+
+        if(INVOKER_INSTANCE != null && PLACE_BLOCK != null && IS_FALSE != null) {
+            try {
+                Object eventResult = PLACE_BLOCK.invoke(INVOKER_INSTANCE, level, pos, state, player);
+                boolean canceled = (boolean) IS_FALSE.invoke(eventResult);
+
+                return !canceled;
+
+            } catch (InvocationTargetException e) {
+                throw new RuntimeException(e);
+            } catch (IllegalAccessException e) {
+                throw new RuntimeException(e);
+            }
+        }
+
+        return true;
+    }
+
+}

Fabric/src/main/java/tschipp/carryon/config/fabric/ConfigLoaderImpl.java

@@ -31,6 +31,11 @@ public class ConfigLoaderImpl {
                 FileUtils.write(cfgFile, GSON.toJson(entry.getKey()), StandardCharsets.UTF_8);
             } else {
                 JsonObject cfgJson = GSON.fromJson(FileUtils.readFileToString(cfgFile, StandardCharsets.UTF_8), JsonObject.class);
+                if(cfgJson == null)
+                {
+                    cfgPath.toFile().mkdirs();
+                    FileUtils.write(cfgFile, GSON.toJson(entry.getKey()), StandardCharsets.UTF_8);
+                }
                 FileUtils.write(cfgFile, GSON.toJson(loadConfig(entry.getValue(), cfgJson)), StandardCharsets.UTF_8);
             }
         }

Fabric/src/main/java/tschipp/carryon/events/CommonEvents.java

@@ -19,6 +19,7 @@ import tschipp.carryon.common.carry.CarryOnDataManager;
 import tschipp.carryon.common.carry.PickupHandler;
 import tschipp.carryon.common.carry.PlacementHandler;
 import tschipp.carryon.common.scripting.ScriptReloadListener;
+import tschipp.carryon.compat.ArchitecturyCompat;
 import tschipp.carryon.config.ConfigLoader;
 import tschipp.carryon.scripting.IdentifiableScriptReloadListener;
 
@@ -42,7 +43,10 @@ public class CommonEvents {
             CarryOnData carry = CarryOnDataManager.getCarryData(player);
             if(!carry.isCarrying())
             {
-                if (PickupHandler.tryPickUpBlock((ServerPlayer) player, pos, world, null))
+                if (PickupHandler.tryPickUpBlock((ServerPlayer) player, pos, world, (pState, pPos) -> {
+                    boolean success = PlayerBlockBreakEvents.BEFORE.invoker().beforeBlockBreak(world, player, pPos, pState, world.getBlockEntity(pPos));
+                    return success;
+                }))
                     return InteractionResult.SUCCESS;
                 return InteractionResult.PASS;
             }
@@ -50,7 +54,9 @@ public class CommonEvents {
             {
                 if(carry.isCarrying(CarryOnData.CarryType.BLOCK))
                 {
-                    if(PlacementHandler.tryPlaceBlock((ServerPlayer) player, pos, facing, null))
+                    if(PlacementHandler.tryPlaceBlock((ServerPlayer) player, pos, facing, (pState, pPos) -> {
+                        return ArchitecturyCompat.sendPlaceEvent(world, pState, pPos, player);
+                    }))
                         return InteractionResult.SUCCESS;
                 }
                 else

Forge/build.gradle

@@ -30,6 +30,8 @@ if (System.getenv('BUILD_NUMBER') != null) {
     version += "." + System.getenv('BUILD_NUMBER')
 }
 
+jarJar.enable()
+
 archivesBaseName = "${mod_id}-forge-${minecraft_version}"
 
 mixin {
@@ -100,11 +102,6 @@ minecraft {
 
 sourceSets.main.resources.srcDir 'src/generated/resources'
 
-configurations {
-    shade
-    implementation.extendsFrom shade
-}
-
 repositories {
     maven {
         url 'https://maven.blamejared.com'
@@ -121,7 +118,13 @@ dependencies {
     compileOnly project(":Common")
     implementation fg.deobf("net.darkhax.gamestages:GameStages-Forge-1.19.2:11.0.2")
     implementation fg.deobf("net.darkhax.bookshelf:Bookshelf-Forge-1.19.2:16.1.9")
-    minecraftLibrary(annotationProcessor(shade(("com.github.LlamaLad7:MixinExtras:${mixinextras_version}"))))
+    //minecraftLibrary("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
+    //jarJar(group: 'com.github.LlamaLad7', name: 'MixinExtras', version: "[${mixinextras_version},)")
+    //annotationProcessor("com.github.LlamaLad7:MixinExtras:${mixinextras_version}")
+    implementation(annotationProcessor("com.github.llamalad7.mixinextras:mixinextras-common:${mixinextras_version}"))
+    implementation(jarJar("com.github.llamalad7.mixinextras:mixinextras-forge:${mixinextras_version}")) {
+        jarJar.ranged(it, "[${mixinextras_version},)")
+    }
     annotationProcessor 'org.spongepowered:mixin:0.8.4-SNAPSHOT:processor'
 
     fileTree("libs").matching {
@@ -135,17 +138,8 @@ dependencies {
     }
 }
 
-shadowJar {
-    configurations = [project.configurations.shade]
-    relocate 'com.llamalad7.mixinextras', 'tschipp.carryon.shaded.mixinextras'
-    finalizedBy 'reobfShadowJar'
-    archiveClassifier = ""
-}
-
-assemble.dependsOn shadowJar
-
 reobf {
-    shadowJar {}
+    jarJar {}
 }
 
 tasks.withType(JavaCompile) {

Forge/src/main/java/tschipp/carryon/CarryOnForge.java

@@ -1,6 +1,7 @@
 package tschipp.carryon;
 
 import net.minecraft.resources.ResourceLocation;
+import net.minecraftforge.fml.InterModComms;
 import net.minecraftforge.fml.common.Mod;
 import net.minecraftforge.fml.common.Mod.EventBusSubscriber;
 import net.minecraftforge.fml.event.lifecycle.FMLCommonSetupEvent;
@@ -20,7 +21,6 @@ public class CarryOnForge {
         // This method is invoked by the Forge mod loader when it is ready
         // to load your mod. You can access Forge and Common code in this
         // project.
-    
         // Use Forge to bootstrap the Common mod.
         CarryOnCommon.registerConfig();
 

Forge/src/main/java/tschipp/carryon/config/forge/ConfigLoaderImpl.java

@@ -11,10 +11,7 @@ import net.minecraftforge.fml.event.config.ModConfigEvent;
 import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
 import tschipp.carryon.config.*;
 
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
+import java.util.*;
 
 public class ConfigLoaderImpl {
 
@@ -95,7 +92,14 @@ public class ConfigLoaderImpl {
             case BOOLEAN -> builder.define(data.getId(), data.getBoolean());
             case INT -> builder.defineInRange(data.getId(), data.getInt(), annotationData.min(), annotationData.max());
             case DOUBLE -> builder.defineInRange(data.getId(), data.getDouble(), annotationData.minD(), annotationData.maxD());
-            case STRING_ARRAY -> builder.defineList(data.getId(), Arrays.asList(data.getStringArray()), obj -> true);
+            case STRING_ARRAY -> builder.defineListAllowEmpty(List.of(data.getId()), () -> {
+                try {
+                    return Arrays.asList(data.getStringArray());
+                } catch (IllegalAccessException e) {
+                    e.printStackTrace();
+                }
+                return new ArrayList<>();
+            }, obj -> obj instanceof String);
             default -> throw new IllegalAccessException("Unknown property type.");
         }
     }

Forge/src/main/java/tschipp/carryon/events/CommonEvents.java

@@ -8,11 +8,13 @@ import net.minecraft.world.entity.Mob;
 import net.minecraft.world.entity.MobSpawnType;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.level.Level;
+import net.minecraft.world.level.ServerLevelAccessor;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.common.util.BlockSnapshot;
 import net.minecraftforge.event.AddReloadListenerEvent;
 import net.minecraftforge.event.OnDatapackSyncEvent;
 import net.minecraftforge.event.RegisterCommandsEvent;
+import net.minecraftforge.event.TagsUpdatedEvent;
 import net.minecraftforge.event.TickEvent.Phase;
 import net.minecraftforge.event.TickEvent.ServerTickEvent;
 import net.minecraftforge.event.entity.living.LivingAttackEvent;
@@ -37,6 +39,7 @@ import tschipp.carryon.common.carry.CarryOnDataManager;
 import tschipp.carryon.common.carry.PickupHandler;
 import tschipp.carryon.common.carry.PlacementHandler;
 import tschipp.carryon.common.scripting.ScriptReloadListener;
+import tschipp.carryon.config.ConfigLoader;
 
 @Mod.EventBusSubscriber(bus = Mod.EventBusSubscriber.Bus.FORGE, modid = Constants.MOD_ID)
 public class CommonEvents
@@ -114,7 +117,6 @@ public class CommonEvents
 				MinecraftForge.EVENT_BUS.post(pickupEvent);
 				return !pickupEvent.isCanceled();
 			})) {
-				event.setResult(Result.DENY);
 				event.setCancellationResult(InteractionResult.SUCCESS);
 				event.setCanceled(true);
 				return;
@@ -147,6 +149,12 @@ public class CommonEvents
 			ScriptReloadListener.syncScriptsWithClient(player);
 	}
 
+	@SubscribeEvent
+	public static void onTagsUpdate(TagsUpdatedEvent event)
+	{
+		ConfigLoader.onConfigLoaded();
+	}
+
 	@SubscribeEvent
 	public static void onServerTick(ServerTickEvent event)
 	{

Forge/src/main/java/tschipp/carryon/events/ModBusEvents.java

@@ -5,7 +5,8 @@ import net.minecraftforge.eventbus.api.SubscribeEvent;
 import net.minecraftforge.fml.InterModComms;
 import net.minecraftforge.fml.InterModComms.IMCMessage;
 import net.minecraftforge.fml.common.Mod;
-import net.minecraftforge.fml.event.lifecycle.FMLDedicatedServerSetupEvent;
+import net.minecraftforge.fml.event.lifecycle.FMLCommonSetupEvent;
+import net.minecraftforge.fml.event.lifecycle.InterModProcessEvent;
 import tschipp.carryon.Constants;
 import tschipp.carryon.client.modeloverride.ModelOverrideHandler;
 import tschipp.carryon.common.config.ListHandler;
@@ -16,7 +17,7 @@ import java.util.stream.Stream;
 public class ModBusEvents {
 
 	@SubscribeEvent(priority = EventPriority.LOW)
-	public void serverLoad(FMLDedicatedServerSetupEvent event)
+	public static void serverLoad(InterModProcessEvent event)
 	{
 		Stream<IMCMessage> messages = InterModComms.getMessages(Constants.MOD_ID);
 
@@ -28,6 +29,7 @@ public class ModBusEvents {
 			if (!(obj instanceof String str))
 				return;
 
+
 			switch (method) {
 				case "blacklistBlock":
 					ListHandler.addForbiddenTiles(str);

gradle.properties

@@ -1,5 +1,5 @@
 # Project
-version=2.0.4
+version=2.1.2
 group=tschipp.carryon
 
 # Common
@@ -14,8 +14,9 @@ parchment_mappings=2022.11.20-1.19.2
 //forge_ats_enabled=true
 
 # Fabric
+parchment_mappings_fabric=1.19.2:2022.11.27
 fabric_version=0.62.0+1.19.2
-fabric_loader_version=0.14.9
+fabric_loader_version=0.14.11
 
 # Quilt
 quilt_loader_version=0.16.0-beta.7
@@ -30,4 +31,4 @@ mod_id=carryon
 org.gradle.jvmargs=-Xmx3G
 org.gradle.daemon=false
 
-mixinextras_version=0.2.0-beta.1
+mixinextras_version=0.2.0-beta.6