Merge pull request #2 from UselessBullets/m-carryall

Changed implementation to work on all blocks

Author: average-bread

src/main/java/cursedbread/carry/mixin/CarryMixin.java

@@ -0,0 +1,34 @@
+package cursedbread.carry.mixin;
+
+import net.minecraft.client.player.controller.PlayerController;
+import net.minecraft.core.block.entity.TileEntity;
+import net.minecraft.core.block.motion.CarriedBlock;
+import net.minecraft.core.entity.player.Player;
+import net.minecraft.core.item.ItemStack;
+import net.minecraft.core.util.helper.Side;
+import net.minecraft.core.world.World;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
+
+@Mixin(value = PlayerController.class, remap = false)
+public class PlayerControllerMixin {
+
+	@Inject(method = "useOrPlaceItemStackOnTile(Lnet/minecraft/core/entity/player/Player;Lnet/minecraft/core/world/World;Lnet/minecraft/core/item/ItemStack;IIILnet/minecraft/core/util/helper/Side;DD)Z", at = @At("HEAD"), cancellable = true)
+	public void pickupAnything(Player player, World world, ItemStack itemstack, int blockX, int blockY, int blockZ, Side side, double xPlaced, double yPlaced, CallbackInfoReturnable<Boolean> cir) {
+		if (player.canInteract()) {
+			int blockId = world.getBlockId(blockX, blockY, blockZ);
+			if (player.getHeldObject() == null && blockId != 0) {
+				if (player.isSneaking() && itemstack == null) {
+					TileEntity tileEntity = world.getTileEntity(blockX, blockY, blockZ);
+					if (tileEntity == null) {
+						player.setHeldObject(new CarriedBlock(player, blockId, world.getBlockMetadata(blockX, blockY, blockZ), null));
+						world.setBlockWithNotify(blockX, blockY, blockZ, 0);
+						cir.setReturnValue(true);
+					}
+				}
+			}
+		}
+	}
+}

src/main/java/cursedbread/carry/mixin/PlayerControllerMixin.java

@@ -0,0 +1,34 @@
+package cursedbread.carry.mixin;
+
+import net.minecraft.core.block.entity.TileEntity;
+import net.minecraft.core.block.motion.CarriedBlock;
+import net.minecraft.core.entity.player.Player;
+import net.minecraft.core.item.ItemStack;
+import net.minecraft.core.util.helper.Side;
+import net.minecraft.core.world.World;
+import net.minecraft.server.world.ServerPlayerController;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
+
+@Mixin(value = ServerPlayerController.class, remap = false)
+public class ServerPlayerControllerMixin {
+
+	@Inject(method = "useOrPlaceItemStackOnTile(Lnet/minecraft/core/entity/player/Player;Lnet/minecraft/core/world/World;Lnet/minecraft/core/item/ItemStack;IIILnet/minecraft/core/util/helper/Side;DD)Z", at = @At("HEAD"), cancellable = true)
+	public void pickupAnything(Player player, World world, ItemStack itemstack, int blockX, int blockY, int blockZ, Side side, double xPlaced, double yPlaced, CallbackInfoReturnable<Boolean> cir) {
+		if (player.canInteract()) {
+			int blockId = world.getBlockId(blockX, blockY, blockZ);
+			if (player.getHeldObject() == null && blockId != 0) {
+				if (player.isSneaking() && itemstack == null) {
+					TileEntity tileEntity = world.getTileEntity(blockX, blockY, blockZ);
+					if (tileEntity == null) {
+						player.setHeldObject(new CarriedBlock(player, blockId, world.getBlockMetadata(blockX, blockY, blockZ), null));
+						world.setBlockWithNotify(blockX, blockY, blockZ, 0);
+						cir.setReturnValue(true);
+					}
+				}
+			}
+		}
+	}
+}

src/main/java/cursedbread/carry/mixin/ServerPlayerControllerMixin.java

@@ -4,11 +4,14 @@
   "package": "cursedbread.carry.mixin",
   "compatibilityLevel": "JAVA_8",
   "mixins": [
-    "CarryMixin"
   ],
   "client": [
+    "PlayerControllerMixin"
   ],
   "injectors": {
     "defaultRequire": 1
-  }
+  },
+  "server": [
+    "ServerPlayerControllerMixin"
+  ]
 }