feat: port growth ring, hairball, knife, laser wrench, magnet ring

src/main/java/de/ellpeck/actuallyadditions/mod/items/ActuallyItems.java

@@ -147,7 +147,7 @@ public final class ActuallyItems {
     public static final RegistryObject<Item> POTION_RING = ITEMS.register("potion_ring", () -> new ItemPotionRing(false));
     public static final RegistryObject<Item> POTION_RING_ADVANCED = ITEMS.register("potion_ring_advanced", () -> new ItemPotionRing(true));
 
-    public static final RegistryObject<Item> HAIRY_BALL = ITEMS.register("hairy_ball", ItemHairyBall::new);
+    public static final RegistryObject<Item> HAIRY_BALL = ITEMS.register("hairy_ball", ItemHairBall::new);
     public static final RegistryObject<Item> COFFEE_BEANS = ITEMS.register("coffee_beans", ItemCoffeeBean::new);
 
     public static final RegistryObject<Item> RICE_SEED = ITEMS.register("rice_seed", () -> new ItemSeed("seedRice", ActuallyBlocks.RICE.get(), FOOD.get(), TheFoods.RICE.ordinal()));
@@ -242,4 +242,8 @@ public final class ActuallyItems {
     public static Item.Properties defaultProps() {
         return new Item.Properties().group(ActuallyAdditions.GROUP);
     }
+
+    public static Item.Properties defaultNonStacking() {
+        return defaultProps().maxStackSize(1);
+    }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemGrowthRing.java

@@ -13,15 +13,16 @@ package de.ellpeck.actuallyadditions.mod.items;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemEnergy;
 import de.ellpeck.actuallyadditions.mod.util.StackUtil;
 import net.minecraft.block.Block;
-import net.minecraft.block.BlockGrass;
+import net.minecraft.block.BlockState;
+import net.minecraft.block.GrassBlock;
 import net.minecraft.block.IGrowable;
 import net.minecraft.entity.Entity;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.math.MathHelper;
 import net.minecraft.world.World;
+import net.minecraft.world.server.ServerWorld;
 import net.minecraftforge.common.IPlantable;
 
 import java.util.ArrayList;
@@ -30,11 +31,11 @@ import java.util.List;
 public class ItemGrowthRing extends ItemEnergy {
 
     public ItemGrowthRing() {
-        super(1000000, 2000, name);
+        super(1000000, 2000);
     }
 
     @Override
-    public void onUpdate(ItemStack stack, World world, Entity entity, int par4, boolean par5) {
+    public void inventoryTick(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) {
         if (!(entity instanceof PlayerEntity) || world.isRemote || entity.isSneaking()) {
             return;
         }
@@ -47,17 +48,17 @@ public class ItemGrowthRing extends ItemEnergy {
             List<BlockPos> blocks = new ArrayList<>();
 
             //Adding all possible Blocks
-            if (player.world.getTotalWorldTime() % 30 == 0) {
+            if (player.world.getGameTime() % 30 == 0) {
                 int range = 3;
                 for (int x = -range; x < range + 1; x++) {
                     for (int z = -range; z < range + 1; z++) {
                         for (int y = -range; y < range + 1; y++) {
-                            int theX = MathHelper.floor(player.posX + x);
+                            int theX = MathHelper.floor(player.getPosX() + x);
-                            int theY = MathHelper.floor(player.posY + y);
+                            int theY = MathHelper.floor(player.getPosY() + y);
-                            int theZ = MathHelper.floor(player.posZ + z);
+                            int theZ = MathHelper.floor(player.getPosZ() + z);
                             BlockPos posInQuestion = new BlockPos(theX, theY, theZ);
                             Block theBlock = world.getBlockState(posInQuestion).getBlock();
-                            if ((theBlock instanceof IGrowable || theBlock instanceof IPlantable) && !(theBlock instanceof BlockGrass)) {
+                            if ((theBlock instanceof IGrowable || theBlock instanceof IPlantable) && !(theBlock instanceof GrassBlock)) {
                                 blocks.add(posInQuestion);
                             }
                         }
@@ -72,12 +73,11 @@ public class ItemGrowthRing extends ItemEnergy {
 
                             BlockState state = world.getBlockState(pos);
                             Block block = state.getBlock();
-                            int metaBefore = block.getMetaFromState(state);
+                            block.tick(world.getBlockState(pos), (ServerWorld) world, pos, world.rand);
-                            block.updateTick(world, pos, world.getBlockState(pos), world.rand);
 
                             //Show Particles if Metadata changed
                             BlockState newState = world.getBlockState(pos);
-                            if (newState.getBlock().getMetaFromState(newState) != metaBefore) {
+                            if (newState != state) {
                                 world.playEvent(2005, pos, 0);
                             }
 
@@ -92,9 +92,4 @@ public class ItemGrowthRing extends ItemEnergy {
             }
         }
     }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
-    }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemHairBall.java

@@ -15,24 +15,28 @@ import de.ellpeck.actuallyadditions.mod.config.values.ConfigBoolValues;
 import de.ellpeck.actuallyadditions.mod.config.values.ConfigIntValues;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import net.minecraft.entity.item.ItemEntity;
-import net.minecraft.entity.passive.EntityOcelot;
+import net.minecraft.entity.passive.OcelotEntity;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.*;
 import net.minecraft.world.World;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.event.entity.living.LivingEvent;
-import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
+import net.minecraftforge.eventbus.api.SubscribeEvent;
 
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
 import java.util.Random;
 import java.util.UUID;
 
-public class ItemHairyBall extends ItemBase {
+public class ItemHairBall extends ItemBase {
 
     private final UUID KittyVanCatUUID = UUID.fromString("681d4e20-10ef-40c9-a0a5-ba2f1995ef44");
 
-    public ItemHairyBall() {
+    public ItemHairBall() {
         super();
+
+        // TODO: [port] move this.
         MinecraftForge.EVENT_BUS.register(this);
     }
 
@@ -40,30 +44,41 @@ public class ItemHairyBall extends ItemBase {
     public void livingUpdateEvent(LivingEvent.LivingUpdateEvent event) {
         //Ocelots dropping Hair Balls
         if (ConfigBoolValues.DO_CAT_DROPS.isEnabled() && event.getEntityLiving() != null && event.getEntityLiving().world != null && !event.getEntityLiving().world.isRemote) {
-            if (event.getEntityLiving() instanceof EntityOcelot && ((EntityOcelot) event.getEntityLiving()).isTamed() || event.getEntityLiving() instanceof PlayerEntity && event.getEntityLiving().getUniqueID().equals(this.KittyVanCatUUID)) {
+            if (event.getEntityLiving() instanceof OcelotEntity && catIsTamedReflection((OcelotEntity) event.getEntityLiving()) || event.getEntityLiving() instanceof PlayerEntity && event.getEntityLiving().getUniqueID().equals(this.KittyVanCatUUID)) {
                 if (event.getEntityLiving().world.rand.nextInt(ConfigIntValues.FUR_CHANCE.getValue()) == 0) {
-                    ItemEntity item = new ItemEntity(event.getEntityLiving().world, event.getEntityLiving().posX + 0.5, event.getEntityLiving().posY + 0.5, event.getEntityLiving().posZ + 0.5, new ItemStack(ActuallyItems.HAIRY_BALL));
+                    ItemEntity item = new ItemEntity(event.getEntityLiving().world, event.getEntityLiving().getPosX() + 0.5, event.getEntityLiving().getPosY() + 0.5, event.getEntityLiving().getPosZ() + 0.5, new ItemStack(ActuallyItems.HAIRY_BALL.get()));
                     event.getEntityLiving().world.addEntity(item);
                 }
             }
         }
     }
 
+    public static boolean catIsTamedReflection(OcelotEntity entity) {
+        try {
+            Method isTrusting = OcelotEntity.class.getDeclaredMethod("isTrusting");
+            return (boolean) isTrusting.invoke(entity);
+        } catch (NoSuchMethodException | InvocationTargetException | IllegalAccessException e) {
+            e.printStackTrace();
+        }
+
+        return false;
+    }
+
     @Override
     public ActionResult<ItemStack> onItemRightClick(World world, PlayerEntity player, Hand hand) {
         ItemStack stack = player.getHeldItem(hand);
         if (!world.isRemote) {
             ItemStack returnItem = this.getRandomReturnItem(world.rand);
             if (!player.inventory.addItemStackToInventory(returnItem)) {
-                ItemEntity entityItem = new ItemEntity(player.world, player.posX, player.posY, player.posZ, returnItem);
+                ItemEntity entityItem = new ItemEntity(player.world, player.getPosX(), player.getPosY(), player.getPosZ(), returnItem);
                 entityItem.setPickupDelay(0);
                 player.world.addEntity(entityItem);
             }
             stack.shrink(1);
 
-            world.playSound(null, player.posX, player.posY, player.posZ, SoundEvents.ENTITY_ITEM_PICKUP, SoundCategory.PLAYERS, 0.2F, world.rand.nextFloat() * 0.1F + 0.9F);
+            world.playSound(null, player.getPosX(), player.getPosY(), player.getPosZ(), SoundEvents.ENTITY_ITEM_PICKUP, SoundCategory.PLAYERS, 0.2F, world.rand.nextFloat() * 0.1F + 0.9F);
         }
-        return new ActionResult<>(EnumActionResult.SUCCESS, stack);
+        return ActionResult.resultSuccess(stack);
     }
 
     public ItemStack getRandomReturnItem(Random rand) {

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemKnife.java

@@ -12,37 +12,31 @@ package de.ellpeck.actuallyadditions.mod.items;
 
 import com.google.common.collect.Multimap;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
-import net.minecraft.entity.SharedMonsterAttributes;
+import net.minecraft.entity.ai.attributes.Attribute;
 import net.minecraft.entity.ai.attributes.AttributeModifier;
-import net.minecraft.inventory.EntityEquipmentSlot;
+import net.minecraft.entity.ai.attributes.Attributes;
-import net.minecraft.item.EnumRarity;
+import net.minecraft.inventory.EquipmentSlotType;
 import net.minecraft.item.ItemStack;
 
 public class ItemKnife extends ItemBase {
 
     public ItemKnife() {
-        super(name);
+        super(ActuallyItems.defaultNonStacking().defaultMaxDamage(100).setNoRepair());
-        this.setMaxDamage(100);
-        this.setMaxStackSize(1);
-        this.setContainerItem(this);
-        this.setNoRepair();
     }
 
-    @Override
+
-    public boolean getShareTag() {
+    //    @Override
-        return true;
+    //    public boolean getShareTag() {
-    }
+    //        return true;
+    //    }
+
 
     @Override
-    public EnumRarity getRarity(ItemStack stack) {
+    public Multimap<Attribute, AttributeModifier> getAttributeModifiers(EquipmentSlotType slot, ItemStack stack) {
-        return EnumRarity.EPIC;
+        Multimap<Attribute, AttributeModifier> map = super.getAttributeModifiers(slot, stack);
-    }
+        if (slot == EquipmentSlotType.MAINHAND) {
-
+            // TODO: [port] validate
-    @Override
+            map.put(Attributes.ATTACK_DAMAGE, new AttributeModifier("Knife Modifier", 3, AttributeModifier.Operation.ADDITION));
-    public Multimap<String, AttributeModifier> getAttributeModifiers(EntityEquipmentSlot slot, ItemStack stack) {
-        Multimap<String, AttributeModifier> map = super.getAttributeModifiers(slot, stack);
-        if (slot == EntityEquipmentSlot.MAINHAND) {
-            map.put(SharedMonsterAttributes.ATTACK_DAMAGE.getName(), new AttributeModifier(ATTACK_DAMAGE_MODIFIER, "Knife Modifier", 3, 0));
         }
         return map;
     }
@@ -50,7 +44,7 @@ public class ItemKnife extends ItemBase {
     @Override
     public ItemStack getContainerItem(ItemStack stack) {
         ItemStack theStack = stack.copy();
-        theStack.setItemDamage(theStack.getItemDamage() + 1);
+        theStack.setDamage(theStack.getDamage() + 1);
         return theStack;
     }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemLaserWrench.java

@@ -14,18 +14,17 @@ import de.ellpeck.actuallyadditions.api.ActuallyAdditionsAPI;
 import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import de.ellpeck.actuallyadditions.mod.tile.TileEntityLaserRelay;
-import de.ellpeck.actuallyadditions.mod.util.StringUtil;
 import net.minecraft.client.util.ITooltipFlag;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
+import net.minecraft.item.ItemUseContext;
 import net.minecraft.tileentity.TileEntity;
-import net.minecraft.util.Direction;
+import net.minecraft.util.ActionResultType;
-import net.minecraft.util.EnumActionResult;
-import net.minecraft.util.Hand;
 import net.minecraft.util.math.BlockPos;
-import net.minecraft.util.text.TextComponentTranslation;
+import net.minecraft.util.text.ITextComponent;
+import net.minecraft.util.text.StringTextComponent;
 import net.minecraft.util.text.TextFormatting;
+import net.minecraft.util.text.TranslationTextComponent;
 import net.minecraft.world.World;
 
 import java.util.List;
@@ -33,20 +32,23 @@ import java.util.List;
 public class ItemLaserWrench extends ItemBase {
 
     public ItemLaserWrench() {
-        super(name);
+        super(ActuallyItems.defaultNonStacking());
-        this.setMaxStackSize(1);
     }
 
     @Override
-    public EnumActionResult onItemUse(PlayerEntity player, World world, BlockPos pos, Hand hand, Direction par7, float par8, float par9, float par10) {
+    public ActionResultType onItemUse(ItemUseContext context) {
-        ItemStack stack = player.getHeldItem(hand);
+        BlockPos pos = context.getPos();
+        World world = context.getWorld();
+        PlayerEntity player = context.getPlayer();
+
+        ItemStack stack = player.getHeldItem(context.getHand());
         TileEntity tile = world.getTileEntity(pos);
         if (tile instanceof TileEntityLaserRelay) {
             TileEntityLaserRelay relay = (TileEntityLaserRelay) tile;
             if (!world.isRemote) {
                 if (ItemPhantomConnector.getStoredPosition(stack) == null) {
                     ItemPhantomConnector.storeConnection(stack, pos.getX(), pos.getY(), pos.getZ(), world);
-                    player.sendStatusMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".laser.stored.desc"), true);
+                    player.sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".laser.stored.desc"), true);
                 } else {
                     BlockPos savedPos = ItemPhantomConnector.getStoredPosition(stack);
                     if (savedPos != null) {
@@ -57,47 +59,44 @@ public class ItemLaserWrench extends ItemBase {
 
                             int lowestRange = Math.min(relay.getMaxRange(), savedRelay.getMaxRange());
                             int range = lowestRange * lowestRange;
-                            if (ItemPhantomConnector.getStoredWorld(stack) == world && savedRelay.type == relay.type && distanceSq <= range && ActuallyAdditionsAPI.connectionHandler.addConnection(savedPos, pos, relay.type, world, false, true)) {
+                            if (ItemPhantomConnector.getStoredWorld(stack) == world.getDimensionKey() && savedRelay.type == relay.type && distanceSq <= range && ActuallyAdditionsAPI.connectionHandler.addConnection(savedPos, pos, relay.type, world, false, true)) {
                                 ItemPhantomConnector.clearStorage(stack, "XCoordOfTileStored", "YCoordOfTileStored", "ZCoordOfTileStored", "WorldOfTileStored");
 
                                 ((TileEntityLaserRelay) savedTile).sendUpdate();
                                 relay.sendUpdate();
 
-                                player.sendStatusMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".laser.connected.desc"), true);
+                                player.sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".laser.connected.desc"), true);
 
-                                return EnumActionResult.SUCCESS;
+                                return ActionResultType.SUCCESS;
                             }
                         }
 
-                        player.sendMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".laser.cantConnect.desc"));
+                        player.sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".laser.cantConnect.desc"), false);
                         ItemPhantomConnector.clearStorage(stack, "XCoordOfTileStored", "YCoordOfTileStored", "ZCoordOfTileStored", "WorldOfTileStored");
                     }
                 }
             }
-            return EnumActionResult.SUCCESS;
+            return ActionResultType.SUCCESS;
         }
-        return EnumActionResult.FAIL;
+        return ActionResultType.FAIL;
     }
 
-    @Override
+    //    // TODO: [port] ensure this is correct
-    public boolean getShareTag() {
+    //    @Nullable
-        return true;
+    //    @Override
-    }
+    //    public CompoundNBT getShareTag(ItemStack stack) {
+    //        return new CompoundNBT();
+    //    }
 
     @Override
-    public void addInformation(ItemStack stack, World playerIn, List<String> list, ITooltipFlag advanced) {
+    public void addInformation(ItemStack stack, World playerIn, List<ITextComponent> list, ITooltipFlag advanced) {
         BlockPos coords = ItemPhantomConnector.getStoredPosition(stack);
         if (coords != null) {
-            list.add(StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".boundTo.desc") + ":");
+            list.add(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".boundTo.desc").appendString(":"));
-            list.add("X: " + coords.getX());
+            list.add(new StringTextComponent("X: " + coords.getX()));
-            list.add("Y: " + coords.getY());
+            list.add(new StringTextComponent("Y: " + coords.getY()));
-            list.add("Z: " + coords.getZ());
+            list.add(new StringTextComponent("Z: " + coords.getZ()));
-            list.add(TextFormatting.ITALIC + StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc"));
+            list.add(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc").mergeStyle(TextFormatting.ITALIC));
         }
     }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
-    }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemLeafBlower.java

@@ -12,25 +12,20 @@ package de.ellpeck.actuallyadditions.mod.items;
 
 import de.ellpeck.actuallyadditions.api.misc.IDisplayStandItem;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
-import net.minecraft.block.Block;
+import net.minecraft.block.*;
-import net.minecraft.block.BlockBush;
+import net.minecraft.entity.LivingEntity;
-import net.minecraft.block.BlockState;
-import net.minecraft.block.Blocks;
-import net.minecraft.entity.EntityLivingBase;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.init.SoundEvents;
-import net.minecraft.item.EnumAction;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
+import net.minecraft.item.UseAction;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.util.ActionResult;
-import net.minecraft.util.EnumActionResult;
 import net.minecraft.util.Hand;
 import net.minecraft.util.SoundCategory;
+import net.minecraft.util.SoundEvents;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.math.MathHelper;
 import net.minecraft.world.World;
-import net.minecraftforge.common.IShearable;
+import net.minecraftforge.common.IForgeShearable;
 
 import java.util.ArrayList;
 import java.util.Collections;
@@ -47,35 +42,27 @@ public class ItemLeafBlower extends ItemBase implements IDisplayStandItem {
     @Override
     public ActionResult<ItemStack> onItemRightClick(World world, PlayerEntity player, Hand hand) {
         player.setActiveHand(hand);
-        return new ActionResult<>(EnumActionResult.SUCCESS, player.getHeldItem(hand));
+        return ActionResult.resultSuccess(player.getHeldItem(hand));
     }
 
     @Override
-    public EnumAction getItemUseAction(ItemStack stack) {
+    public UseAction getUseAction(ItemStack stack) {
-        return EnumAction.BOW;
+        return UseAction.BOW;
     }
 
     @Override
-    public int getMaxItemUseDuration(ItemStack stack) {
+    public int getUseDuration(ItemStack stack) {
-        //Cuz you won't hold it for that long right-clicking anyways
         return Integer.MAX_VALUE;
     }
 
     @Override
-    public EnumRarity getRarity(ItemStack stack) {
+    public void onUsingTick(ItemStack stack, LivingEntity player, int count) {
-        return this.isAdvanced
+        this.doUpdate(player.world, MathHelper.floor(player.getPosX()), MathHelper.floor(player.getPosY()), MathHelper.floor(player.getPosZ()), count, stack);
-            ? EnumRarity.EPIC
-            : EnumRarity.RARE;
-    }
-
-    @Override
-    public void onUsingTick(ItemStack stack, EntityLivingBase player, int time) {
-        this.doUpdate(player.world, MathHelper.floor(player.posX), MathHelper.floor(player.posY), MathHelper.floor(player.posZ), time, stack);
     }
 
     private boolean doUpdate(World world, int x, int y, int z, int time, ItemStack stack) {
         if (!world.isRemote) {
-            if (time <= this.getMaxItemUseDuration(stack) && (this.isAdvanced || time % 3 == 0)) {
+            if (time <= this.getUseDuration(stack) && (this.isAdvanced || time % 3 == 0)) {
                 //Breaks the Blocks
                 boolean broke = this.breakStuff(world, x, y, z);
                 //Plays a Minecart sounds (It really sounds like a Leaf Blower!)
@@ -110,7 +97,7 @@ public class ItemLeafBlower extends ItemBase implements IDisplayStandItem {
                     BlockPos pos = new BlockPos(x + reachX, y + reachY, z + reachZ);
                     Block block = world.getBlockState(pos).getBlock();
 
-                    if (block != null && (block instanceof BlockBush || block instanceof IShearable) && (this.isAdvanced || !block.isLeaves(world.getBlockState(pos), world, pos))) {
+                    if ((block instanceof BushBlock || block instanceof IForgeShearable) && (this.isAdvanced || block instanceof LeavesBlock)) {
                         breakPositions.add(pos);
                     }
                 }
@@ -123,7 +110,8 @@ public class ItemLeafBlower extends ItemBase implements IDisplayStandItem {
             BlockPos theCoord = breakPositions.get(0);
             BlockState theState = world.getBlockState(theCoord);
 
-            theState.getBlock().dropBlockAsItem(world, theCoord, theState, 0);
+            world.destroyBlock(theCoord, true);
+            //            theState.getBlock().dropBlockAsItem(world, theCoord, theState, 0);
             //Plays the Breaking Sound
             world.playEvent(2001, theCoord, Block.getStateId(theState));
 

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemMagnetRing.java

@@ -15,10 +15,8 @@ import de.ellpeck.actuallyadditions.mod.util.ItemUtil;
 import net.minecraft.entity.Entity;
 import net.minecraft.entity.item.ItemEntity;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.ActionResult;
-import net.minecraft.util.EnumActionResult;
 import net.minecraft.util.Hand;
 import net.minecraft.util.math.AxisAlignedBB;
 import net.minecraft.world.World;
@@ -28,7 +26,7 @@ import java.util.List;
 public class ItemMagnetRing extends ItemEnergy {
 
     public ItemMagnetRing() {
-        super(200000, 1000, name);
+        super(200000, 1000);
     }
 
     @Override
@@ -37,7 +35,7 @@ public class ItemMagnetRing extends ItemEnergy {
     }
 
     @Override
-    public void onUpdate(ItemStack stack, World world, Entity entity, int par4, boolean par5) {
+    public void inventoryTick(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) {
         if (entity instanceof PlayerEntity && !world.isRemote && !ItemUtil.isEnabled(stack)) {
             PlayerEntity player = (PlayerEntity) entity;
             if (player.isCreative() || player.isSpectator()) {
@@ -46,13 +44,14 @@ public class ItemMagnetRing extends ItemEnergy {
             if (!entity.isSneaking()) {
                 //Get all the Items in the area
                 int range = 5;
-                List<ItemEntity> items = world.getEntitiesWithinAABB(ItemEntity.class, new AxisAlignedBB(entity.posX - range, entity.posY - range, entity.posZ - range, entity.posX + range, entity.posY + range, entity.posZ + range));
+                List<ItemEntity> items = world.getEntitiesWithinAABB(ItemEntity.class, new AxisAlignedBB(entity.getPosX() - range, entity.getPosY() - range, entity.getPosZ() - range, entity.getPosX() + range, entity.getPosY() + range, entity.getPosZ() + range));
                 if (!items.isEmpty()) {
                     for (ItemEntity item : items) {
-                        if (item.getEntityData().getBoolean("PreventRemoteMovement")) {
+                        // TODO: [port] check this data is being saved on the time
+                        if (item.getPersistentData().getBoolean("PreventRemoteMovement")) {
                             continue;
                         }
-                        if (!item.isDead && !item.cannotPickup()) {
+                        if (item.isAlive() && !item.cannotPickup()) {
                             int energyForItem = 50 * item.getItem().getCount();
 
                             if (this.getEnergyStored(stack) >= energyForItem) {
@@ -61,7 +60,7 @@ public class ItemMagnetRing extends ItemEnergy {
                                 item.onCollideWithPlayer(player);
 
                                 if (!player.isCreative()) {
-                                    if (item.isDead || !ItemStack.areItemStacksEqual(item.getItem(), oldItem)) {
+                                    if (!item.isAlive() || !ItemStack.areItemStacksEqual(item.getItem(), oldItem)) {
                                         this.extractEnergyInternal(stack, energyForItem, false);
                                     }
                                 }
@@ -77,13 +76,9 @@ public class ItemMagnetRing extends ItemEnergy {
     public ActionResult<ItemStack> onItemRightClick(World worldIn, PlayerEntity player, Hand hand) {
         if (!worldIn.isRemote && player.isSneaking()) {
             ItemUtil.changeEnabled(player, hand);
-            return new ActionResult<>(EnumActionResult.SUCCESS, player.getHeldItem(hand));
+            return ActionResult.resultSuccess(player.getHeldItem(hand));
-        }
-        return super.onItemRightClick(worldIn, player, hand);
         }
 
-    @Override
+        return super.onItemRightClick(worldIn, player, hand);
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
     }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemPhantomConnector.java

@@ -14,21 +14,21 @@ import de.ellpeck.actuallyadditions.api.tile.IPhantomTile;
 import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import de.ellpeck.actuallyadditions.mod.tile.TileEntityBase;
-import de.ellpeck.actuallyadditions.mod.util.StringUtil;
 import net.minecraft.client.util.ITooltipFlag;
-import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
+import net.minecraft.item.ItemUseContext;
 import net.minecraft.nbt.CompoundNBT;
 import net.minecraft.tileentity.TileEntity;
-import net.minecraft.util.Direction;
+import net.minecraft.util.ActionResultType;
-import net.minecraft.util.EnumActionResult;
+import net.minecraft.util.RegistryKey;
-import net.minecraft.util.Hand;
+import net.minecraft.util.ResourceLocation;
 import net.minecraft.util.math.BlockPos;
-import net.minecraft.util.text.TextComponentTranslation;
+import net.minecraft.util.registry.Registry;
+import net.minecraft.util.text.ITextComponent;
+import net.minecraft.util.text.StringTextComponent;
 import net.minecraft.util.text.TextFormatting;
+import net.minecraft.util.text.TranslationTextComponent;
 import net.minecraft.world.World;
-import net.minecraftforge.common.DimensionManager;
 
 import javax.annotation.Nullable;
 import java.util.List;
@@ -36,102 +36,91 @@ import java.util.List;
 public class ItemPhantomConnector extends ItemBase {
 
     public ItemPhantomConnector() {
-        super(name);
+        super(ActuallyItems.defaultNonStacking());
-        this.setMaxStackSize(1);
     }
 
-    public static World getStoredWorld(ItemStack stack) {
+    public static RegistryKey<World> getStoredWorld(ItemStack stack) {
-        CompoundNBT tag = stack.getTagCompound();
+        CompoundNBT tag = stack.getOrCreateTag();
-        if (tag != null) {
+        if (!tag.contains("WorldOfTileStored")) {
-            return DimensionManager.getWorld(tag.getInteger("WorldOfTileStored"));
-        }
             return null;
         }
 
+        return RegistryKey.getOrCreateKey(Registry.WORLD_KEY, new ResourceLocation(tag.getString("WorldOfTileStored")));
+    }
+
     public static BlockPos getStoredPosition(ItemStack stack) {
-        CompoundNBT tag = stack.getTagCompound();
+        CompoundNBT tag = stack.getOrCreateTag();
-        if (tag != null) {
+        int x = tag.getInt("XCoordOfTileStored");
-            int x = tag.getInteger("XCoordOfTileStored");
+        int y = tag.getInt("YCoordOfTileStored");
-            int y = tag.getInteger("YCoordOfTileStored");
+        int z = tag.getInt("ZCoordOfTileStored");
-            int z = tag.getInteger("ZCoordOfTileStored");
         if (!(x == 0 && y == 0 && z == 0)) {
             return new BlockPos(x, y, z);
         }
-        }
+
         return null;
     }
 
     public static void clearStorage(ItemStack stack, String... keys) {
-        if (stack.hasTagCompound()) {
+        CompoundNBT compound = stack.getOrCreateTag();
-            CompoundNBT compound = stack.getTagCompound();
         for (String key : keys) {
-                compound.removeTag(key);
+            compound.remove(key);
-            }
         }
     }
 
     public static void storeConnection(ItemStack stack, int x, int y, int z, World world) {
-        CompoundNBT tag = stack.getTagCompound();
+        CompoundNBT tag = stack.getOrCreateTag();
-        if (tag == null) {
-            tag = new CompoundNBT();
-        }
 
-        tag.setInteger("XCoordOfTileStored", x);
+        tag.putInt("XCoordOfTileStored", x);
-        tag.setInteger("YCoordOfTileStored", y);
+        tag.putInt("YCoordOfTileStored", y);
-        tag.setInteger("ZCoordOfTileStored", z);
+        tag.putInt("ZCoordOfTileStored", z);
-        tag.setInteger("WorldOfTileStored", world.provider.getDimension());
+        tag.putString("WorldOfTileStored", world.getDimensionKey().getLocation().toString());
-
-        stack.setTagCompound(tag);
     }
 
     @Override
-    public EnumActionResult onItemUse(PlayerEntity player, World world, BlockPos pos, Hand hand, Direction par7, float par8, float par9, float par10) {
+    public ActionResultType onItemUse(ItemUseContext context) {
-        ItemStack stack = player.getHeldItem(hand);
+        ItemStack stack = context.getPlayer().getHeldItem(context.getHand());
-        if (!world.isRemote) {
+        if (!context.getWorld().isRemote) {
             //Passing Data to Phantoms
-            TileEntity tile = world.getTileEntity(pos);
+            BlockPos pos = context.getPos();
+            TileEntity tile = context.getWorld().getTileEntity(pos);
             if (tile != null) {
                 //Passing to Phantom
                 if (tile instanceof IPhantomTile) {
                     BlockPos stored = getStoredPosition(stack);
-                    if (stored != null && getStoredWorld(stack) == world) {
+                    if (stored != null && getStoredWorld(stack) == context.getWorld().getDimensionKey()) {
                         ((IPhantomTile) tile).setBoundPosition(stored);
                         if (tile instanceof TileEntityBase) {
                             ((TileEntityBase) tile).sendUpdate();
                         }
                         clearStorage(stack, "XCoordOfTileStored", "YCoordOfTileStored", "ZCoordOfTileStored", "WorldOfTileStored");
-                        player.sendStatusMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".phantom.connected.desc"), true);
+                        context.getPlayer().sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".phantom.connected.desc"), true);
-                        return EnumActionResult.SUCCESS;
+                        return ActionResultType.SUCCESS;
                     }
-                    return EnumActionResult.FAIL;
+                    return ActionResultType.FAIL;
                 }
             }
             //Storing Connections
-            storeConnection(stack, pos.getX(), pos.getY(), pos.getZ(), world);
+            storeConnection(stack, pos.getX(), pos.getY(), pos.getZ(), context.getWorld());
-            player.sendStatusMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".phantom.stored.desc"), true);
+            context.getPlayer().sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".phantom.stored.desc"), true);
         }
-        return EnumActionResult.SUCCESS;
+        return ActionResultType.SUCCESS;
+    }
+
+    @Nullable
+    @Override
+    public CompoundNBT getShareTag(ItemStack stack) {
+        return new CompoundNBT();
     }
 
     @Override
-    public boolean getShareTag() {
+    public void addInformation(ItemStack stack, @Nullable World playerIn, List<ITextComponent> list, ITooltipFlag advanced) {
-        return true;
-    }
-
-    @Override
-    public void addInformation(ItemStack stack, @Nullable World playerIn, List<String> list, ITooltipFlag advanced) {
         BlockPos coords = getStoredPosition(stack);
         if (coords != null) {
-            list.add(StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".boundTo.desc") + ":");
+            list.add(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".boundTo.desc").appendString(":"));
-            list.add("X: " + coords.getX());
+            list.add(new StringTextComponent("X: " + coords.getX()));
-            list.add("Y: " + coords.getY());
+            list.add(new StringTextComponent("Y: " + coords.getY()));
-            list.add("Z: " + coords.getZ());
+            list.add(new StringTextComponent("Z: " + coords.getZ()));
-            list.add(TextFormatting.ITALIC + StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc"));
+            list.add(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc").mergeStyle(TextFormatting.ITALIC));
         }
     }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
-    }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemPlayerProbe.java

@@ -14,19 +14,18 @@ import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import de.ellpeck.actuallyadditions.mod.tile.TileEntityPlayerInterface;
 import de.ellpeck.actuallyadditions.mod.util.StackUtil;
-import de.ellpeck.actuallyadditions.mod.util.StringUtil;
 import net.minecraft.client.util.ITooltipFlag;
 import net.minecraft.entity.Entity;
-import net.minecraft.entity.EntityLivingBase;
+import net.minecraft.entity.LivingEntity;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.item.ItemStack;
+import net.minecraft.item.ItemUseContext;
 import net.minecraft.nbt.CompoundNBT;
 import net.minecraft.tileentity.TileEntity;
-import net.minecraft.util.Direction;
+import net.minecraft.util.ActionResultType;
-import net.minecraft.util.EnumActionResult;
 import net.minecraft.util.Hand;
-import net.minecraft.util.math.BlockPos;
+import net.minecraft.util.text.ITextComponent;
-import net.minecraft.util.text.TextComponentTranslation;
+import net.minecraft.util.text.TranslationTextComponent;
 import net.minecraft.world.World;
 
 import javax.annotation.Nullable;
@@ -39,39 +38,42 @@ public class ItemPlayerProbe extends ItemBase {
         super(ActuallyItems.defaultProps().maxStackSize(1));
     }
 
+    // TODO: [port] might be the wrong event
     @Override
-    public void onUpdate(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) {
+    public void inventoryTick(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) {
         if (!world.isRemote) {
-            if (stack.hasTagCompound()) {
+            CompoundNBT compound = stack.getOrCreateTag();
-                CompoundNBT compound = stack.getTagCompound();
+            if (compound.contains("UUIDMost")) {
-                if (compound.hasKey("UUIDMost")) {
                 UUID id = compound.getUniqueId("UUID");
-                    PlayerEntity player = world.getPlayerEntityByUUID(id);
+                PlayerEntity player = world.getPlayerByUuid(id);
                 if (player != null) {
                     if (player.isSneaking()) {
                         ItemPhantomConnector.clearStorage(stack, "UUIDLeast", "UUIDMost", "Name");
-                            entity.sendMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".playerProbe.disconnect.1"));
+                        ((PlayerEntity) entity).sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".playerProbe.disconnect.1"), false);
-                            player.sendMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".playerProbe.notice"));
+                        player.sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".playerProbe.notice"), false);
                         //TheAchievements.GET_UNPROBED.get(player);
                     }
                 } else {
                     ItemPhantomConnector.clearStorage(stack, "UUIDLeast", "UUIDMost", "Name");
-                        entity.sendMessage(new TextComponentTranslation("tooltip." + ActuallyAdditions.MODID + ".playerProbe.disconnect.2"));
+                    ((PlayerEntity) entity).sendStatusMessage(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".playerProbe.disconnect.2"), false);
-                    }
                 }
             }
         }
     }
 
     @Override
-    public EnumActionResult onItemUse(PlayerEntity player, World world, BlockPos pos, Hand hand, Direction facing, float hitX, float hitY, float hitZ) {
+    public ActionResultType onItemUse(ItemUseContext context) {
-        ItemStack stack = player.getHeldItem(hand);
+        PlayerEntity player = context.getPlayer();
-        TileEntity tile = world.getTileEntity(pos);
+        if (player == null) {
+            return ActionResultType.FAIL;
+        }
+
+        ItemStack stack = player.getHeldItem(context.getHand());
+        TileEntity tile = context.getWorld().getTileEntity(context.getPos());
         if (tile instanceof TileEntityPlayerInterface) {
-            if (stack.hasTagCompound()) {
+            CompoundNBT compound = stack.getOrCreateTag();
-                CompoundNBT compound = stack.getTagCompound();
+            if (compound.contains("UUIDMost")) {
-                if (compound.hasKey("UUIDMost")) {
+                if (!context.getWorld().isRemote) {
-                    if (!world.isRemote) {
                     TileEntityPlayerInterface face = (TileEntityPlayerInterface) tile;
                     face.connectedPlayer = compound.getUniqueId("UUID");
                     face.playerName = compound.getString("Name");
@@ -80,15 +82,14 @@ public class ItemPlayerProbe extends ItemBase {
 
                     ItemPhantomConnector.clearStorage(stack, "UUIDLeast", "UUIDMost", "Name");
                 }
-                    return EnumActionResult.SUCCESS;
+                return ActionResultType.SUCCESS;
             }
         }
-        }
+        return ActionResultType.FAIL;
-        return EnumActionResult.FAIL;
     }
 
     @Override
-    public boolean itemInteractionForEntity(ItemStack aStack, PlayerEntity player, EntityLivingBase entity, Hand hand) {
+    public ActionResultType itemInteractionForEntity(ItemStack aStack, PlayerEntity player, LivingEntity entity, Hand hand) {
         if (!player.world.isRemote) {
             ItemStack stack = player.getHeldItemMainhand();
             if (StackUtil.isValid(stack) && stack.getItem() == this) {
@@ -96,28 +97,22 @@ public class ItemPlayerProbe extends ItemBase {
                     PlayerEntity playerHit = (PlayerEntity) entity;
 
                     if (!playerHit.isSneaking()) {
-                        if (!stack.hasTagCompound()) {
+                        CompoundNBT compound = stack.getOrCreateTag();
-                            stack.setTagCompound(new CompoundNBT());
+                        compound.putString("Name", playerHit.getName().getString());
-                        }
+                        compound.putUniqueId("UUID", playerHit.getUniqueID());
-
+                        return ActionResultType.SUCCESS;
-                        CompoundNBT compound = stack.getTagCompound();
-                        compound.setString("Name", playerHit.getName());
-                        compound.setUniqueId("UUID", playerHit.getUniqueID());
-                        return true;
                     }
                 }
             }
         }
-        return false;
+        return ActionResultType.FAIL;
     }
 
     @Override
-    public void addInformation(ItemStack stack, @Nullable World playerIn, List<String> tooltip, ITooltipFlag advanced) {
+    public void addInformation(ItemStack stack, @Nullable World playerIn, List<ITextComponent> tooltip, ITooltipFlag advanced) {
-        if (stack.hasTagCompound()) {
+        if (stack.getOrCreateTag().contains("Name")) {
-            String name = stack.getTagCompound().getString("Name");
+            String name = stack.getOrCreateTag().getString("Name");
-            if (name != null) {
+            tooltip.add(new TranslationTextComponent("tooltip." + ActuallyAdditions.MODID + ".playerProbe.probing").appendString(": " + name));
-                tooltip.add(StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".playerProbe.probing") + ": " + name);
-            }
         }
     }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemResonantRice.java

@@ -12,17 +12,16 @@ package de.ellpeck.actuallyadditions.mod.items;
 
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.ActionResult;
-import net.minecraft.util.EnumActionResult;
 import net.minecraft.util.Hand;
+import net.minecraft.world.Explosion;
 import net.minecraft.world.World;
 
 public class ItemResonantRice extends ItemBase {
 
     public ItemResonantRice() {
-        super(name);
+        super();
     }
 
     @Override
@@ -30,13 +29,8 @@ public class ItemResonantRice extends ItemBase {
         ItemStack stack = player.getHeldItem(hand);
         if (!world.isRemote) {
             stack.shrink(1);
-            world.createExplosion(null, player.posX, player.posY, player.posZ, 0.5F, true);
+            world.createExplosion(null, player.getPosX(), player.getPosY(), player.getPosZ(), 0.5F, Explosion.Mode.DESTROY);
         }
-        return new ActionResult<>(EnumActionResult.SUCCESS, stack);
+        return ActionResult.resultSuccess(stack);
-    }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
     }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemSolidifiedExperience.java

@@ -13,39 +13,39 @@ package de.ellpeck.actuallyadditions.mod.items;
 import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
 import de.ellpeck.actuallyadditions.mod.config.values.ConfigBoolValues;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
-import net.minecraft.entity.EntityCreature;
+import net.minecraft.entity.CreatureEntity;
-import net.minecraft.entity.item.EntityXPOrb;
+import net.minecraft.entity.item.ExperienceOrbEntity;
 import net.minecraft.entity.item.ItemEntity;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.ActionResult;
-import net.minecraft.util.EnumActionResult;
 import net.minecraft.util.Hand;
+import net.minecraft.world.GameRules;
 import net.minecraft.world.World;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.common.util.FakePlayer;
 import net.minecraftforge.event.entity.living.LivingDropsEvent;
-import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
+import net.minecraftforge.eventbus.api.SubscribeEvent;
 
 public class ItemSolidifiedExperience extends ItemBase {
 
     public static final int SOLID_XP_AMOUNT = 8;
 
     public ItemSolidifiedExperience() {
-        super(name);
+        super();
 
+        // TODO: [port] move this to another place
         MinecraftForge.EVENT_BUS.register(this);
     }
 
     @SubscribeEvent
     public void onEntityDropEvent(LivingDropsEvent event) {
         if (ConfigBoolValues.DO_XP_DROPS.isEnabled()) {
-            if (event.getEntityLiving().world != null && !event.getEntityLiving().world.isRemote && event.getSource().getTrueSource() instanceof PlayerEntity && event.getEntityLiving().world.getGameRules().getBoolean("doMobLoot")) {
+            if (event.getEntityLiving().world != null && !event.getEntityLiving().world.isRemote && event.getSource().getTrueSource() instanceof PlayerEntity && event.getEntityLiving().world.getGameRules().getBoolean(GameRules.DO_MOB_LOOT)) {
                 //Drop Solidified XP
-                if (event.getEntityLiving() instanceof EntityCreature) {
+                if (event.getEntityLiving() instanceof CreatureEntity) {
                     if (event.getEntityLiving().world.rand.nextInt(10) <= event.getLootingLevel() * 2) {
-                        event.getDrops().add(new ItemEntity(event.getEntityLiving().world, event.getEntityLiving().posX, event.getEntityLiving().posY, event.getEntityLiving().posZ, new ItemStack(ActuallyItems.SOLIDIFIED_EXPERIENCE, event.getEntityLiving().world.rand.nextInt(2 + event.getLootingLevel()) + 1)));
+                        event.getDrops().add(new ItemEntity(event.getEntityLiving().world, event.getEntityLiving().getPosX(), event.getEntityLiving().getPosY(), event.getEntityLiving().getPosZ(), new ItemStack(ActuallyItems.SOLIDIFIED_EXPERIENCE.get(), event.getEntityLiving().world.rand.nextInt(2 + event.getLootingLevel()) + 1)));
                     }
                 }
             }
@@ -70,18 +70,13 @@ public class ItemSolidifiedExperience extends ItemBase {
             }
 
             if (ConfigBoolValues.SOLID_XP_ALWAYS_ORBS.currentValue || player instanceof FakePlayer) {
-                EntityXPOrb orb = new EntityXPOrb(world, player.posX + 0.5, player.posY + 0.5, player.posZ + 0.5, amount);
+                ExperienceOrbEntity orb = new ExperienceOrbEntity(world, player.getPosX() + 0.5, player.getPosY() + 0.5, player.getPosZ() + 0.5, amount);
-                orb.getEntityData().putBoolean(ActuallyAdditions.MODID + "FromSolidified", true);
+                orb.getPersistentData().putBoolean(ActuallyAdditions.MODID + "FromSolidified", true);
                 world.addEntity(orb);
             } else {
-                player.addExperience(amount);
+                player.addExperienceLevel(amount);
             }
         }
-        return new ActionResult<>(EnumActionResult.SUCCESS, stack);
+        return ActionResult.resultSuccess(stack);
-    }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.UNCOMMON;
     }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemSpawnerChanger.java

@@ -1,141 +1,142 @@
-/*
+// TODO: [port] REMOVE THIS CLASS, NO longer needed
- * This file ("ItemSpawnerChanger.java") is part of the Actually Additions mod for Minecraft.
+///*
- * It is created and owned by Ellpeck and distributed
+// * This file ("ItemSpawnerChanger.java") is part of the Actually Additions mod for Minecraft.
- * under the Actually Additions License to be found at
+// * It is created and owned by Ellpeck and distributed
- * http://ellpeck.de/actaddlicense
+// * under the Actually Additions License to be found at
- * View the source code at https://github.com/Ellpeck/ActuallyAdditions
+// * http://ellpeck.de/actaddlicense
- *
+// * View the source code at https://github.com/Ellpeck/ActuallyAdditions
- * © 2015-2017 Ellpeck
+// *
- */
+// * © 2015-2017 Ellpeck
-
+// */
-package de.ellpeck.actuallyadditions.mod.items;
+//
-
+//package de.ellpeck.actuallyadditions.mod.items;
-import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
+//
-import de.ellpeck.actuallyadditions.mod.config.values.ConfigStringListValues;
+//import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
-import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
+//import de.ellpeck.actuallyadditions.mod.config.values.ConfigStringListValues;
-import de.ellpeck.actuallyadditions.mod.util.StackUtil;
+//import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
-import de.ellpeck.actuallyadditions.mod.util.StringUtil;
+//import de.ellpeck.actuallyadditions.mod.util.StackUtil;
-import net.minecraft.block.BlockState;
+//import de.ellpeck.actuallyadditions.mod.util.StringUtil;
-import net.minecraft.client.util.ITooltipFlag;
+//import net.minecraft.block.BlockState;
-import net.minecraft.entity.EntityList;
+//import net.minecraft.client.util.ITooltipFlag;
-import net.minecraft.entity.EntityLivingBase;
+//import net.minecraft.entity.EntityList;
-import net.minecraft.entity.player.PlayerEntity;
+//import net.minecraft.entity.EntityLivingBase;
-import net.minecraft.item.ItemStack;
+//import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.nbt.CompoundNBT;
+//import net.minecraft.item.ItemStack;
-import net.minecraft.tileentity.MobSpawnerBaseLogic;
+//import net.minecraft.nbt.CompoundNBT;
-import net.minecraft.tileentity.TileEntity;
+//import net.minecraft.tileentity.MobSpawnerBaseLogic;
-import net.minecraft.tileentity.TileEntityMobSpawner;
+//import net.minecraft.tileentity.TileEntity;
-import net.minecraft.util.Direction;
+//import net.minecraft.tileentity.TileEntityMobSpawner;
-import net.minecraft.util.EnumActionResult;
+//import net.minecraft.util.Direction;
-import net.minecraft.util.Hand;
+//import net.minecraft.util.EnumActionResult;
-import net.minecraft.util.ResourceLocation;
+//import net.minecraft.util.Hand;
-import net.minecraft.util.math.BlockPos;
+//import net.minecraft.util.ResourceLocation;
-import net.minecraft.util.text.TextFormatting;
+//import net.minecraft.util.math.BlockPos;
-import net.minecraft.world.World;
+//import net.minecraft.util.text.TextFormatting;
-
+//import net.minecraft.world.World;
-import java.util.List;
+//
-
+//import java.util.List;
-public class ItemSpawnerChanger extends ItemBase {
+//
-
+//public class ItemSpawnerChanger extends ItemBase {
-    public ItemSpawnerChanger() {
+//
-        super(name);
+//    public ItemSpawnerChanger() {
-        this.setMaxStackSize(1);
+//        super();
-    }
+//        this.setMaxStackSize(1);
-
+//    }
-    @Override
+//
-    public EnumActionResult onItemUse(PlayerEntity player, World world, BlockPos pos, Hand hand, Direction facing, float hitX, float hitY, float hitZ) {
+//    @Override
-        if (!world.isRemote) {
+//    public EnumActionResult onItemUse(PlayerEntity player, World world, BlockPos pos, Hand hand, Direction facing, float hitX, float hitY, float hitZ) {
-            ItemStack stack = player.getHeldItemMainhand();
+//        if (!world.isRemote) {
-            if (player.canPlayerEdit(pos.offset(facing), facing, stack)) {
+//            ItemStack stack = player.getHeldItemMainhand();
-                TileEntity tile = world.getTileEntity(pos);
+//            if (player.canPlayerEdit(pos.offset(facing), facing, stack)) {
-                if (tile instanceof TileEntityMobSpawner) {
+//                TileEntity tile = world.getTileEntity(pos);
-                    String entity = this.getStoredEntity(stack);
+//                if (tile instanceof TileEntityMobSpawner) {
-                    if (entity != null) {
+//                    String entity = this.getStoredEntity(stack);
-                        MobSpawnerBaseLogic logic = ((TileEntityMobSpawner) tile).getSpawnerBaseLogic();
+//                    if (entity != null) {
-
+//                        MobSpawnerBaseLogic logic = ((TileEntityMobSpawner) tile).getSpawnerBaseLogic();
-                        //This is a hacky way to remove the spawn potentials that make the spawner reset from time to time
+//
-                        //Don't judge, there isn't a method for it and it's better than Reflection hackiness
+//                        //This is a hacky way to remove the spawn potentials that make the spawner reset from time to time
-                        CompoundNBT compound = new CompoundNBT();
+//                        //Don't judge, there isn't a method for it and it's better than Reflection hackiness
-                        logic.writeToNBT(compound);
+//                        CompoundNBT compound = new CompoundNBT();
-                        compound.removeTag("SpawnPotentials");
+//                        logic.writeToNBT(compound);
-                        compound.removeTag("SpawnData");
+//                        compound.removeTag("SpawnPotentials");
-                        logic.readFromNBT(compound);
+//                        compound.removeTag("SpawnData");
-
+//                        logic.readFromNBT(compound);
-                        logic.setEntityId(new ResourceLocation(entity));
+//
-
+//                        logic.setEntityId(new ResourceLocation(entity));
-                        tile.markDirty();
+//
-
+//                        tile.markDirty();
-                        BlockState state = world.getBlockState(pos);
+//
-                        world.notifyBlockUpdate(pos, state, state, 3);
+//                        BlockState state = world.getBlockState(pos);
-
+//                        world.notifyBlockUpdate(pos, state, state, 3);
-                        ItemPhantomConnector.clearStorage(stack, "Entity");
+//
-
+//                        ItemPhantomConnector.clearStorage(stack, "Entity");
-                        if (!player.isCreative()) {
+//
-                            player.setHeldItem(hand, StackUtil.shrink(stack, 1));
+//                        if (!player.isCreative()) {
-                        }
+//                            player.setHeldItem(hand, StackUtil.shrink(stack, 1));
-
+//                        }
-                        return EnumActionResult.SUCCESS;
+//
-                    }
+//                        return EnumActionResult.SUCCESS;
-                }
+//                    }
-            }
+//                }
-        }
+//            }
-        return EnumActionResult.FAIL;
+//        }
-    }
+//        return EnumActionResult.FAIL;
-
+//    }
-    @Override
+//
-    public boolean itemInteractionForEntity(ItemStack aStack, PlayerEntity player, EntityLivingBase entity, Hand hand) {
+//    @Override
-        if (!player.world.isRemote) {
+//    public boolean itemInteractionForEntity(ItemStack aStack, PlayerEntity player, EntityLivingBase entity, Hand hand) {
-            ItemStack stack = player.getHeldItemMainhand();
+//        if (!player.world.isRemote) {
-            if (this.getStoredEntity(stack) == null) {
+//            ItemStack stack = player.getHeldItemMainhand();
-                if (this.storeClickedEntity(stack, entity)) {
+//            if (this.getStoredEntity(stack) == null) {
-                    entity.setDead();
+//                if (this.storeClickedEntity(stack, entity)) {
-                }
+//                    entity.setDead();
-            }
+//                }
-            return true;
+//            }
-        }
+//            return true;
-        return false;
+//        }
-    }
+//        return false;
-
+//    }
-    private boolean storeClickedEntity(ItemStack stack, EntityLivingBase entity) {
+//
-        if (!stack.hasTagCompound()) {
+//    private boolean storeClickedEntity(ItemStack stack, EntityLivingBase entity) {
-            stack.setTagCompound(new CompoundNBT());
+//        if (!stack.hasTagCompound()) {
-        }
+//            stack.setTagCompound(new CompoundNBT());
-
+//        }
-        if (!(entity instanceof PlayerEntity) && entity.isNonBoss()) {
+//
-            ResourceLocation entityLoc = EntityList.getKey(entity.getClass());
+//        if (!(entity instanceof PlayerEntity) && entity.isNonBoss()) {
-            if (entityLoc != null) {
+//            ResourceLocation entityLoc = EntityList.getKey(entity.getClass());
-                String entityName = entityLoc.toString();
+//            if (entityLoc != null) {
-                if (entityName != null && !entityName.isEmpty()) {
+//                String entityName = entityLoc.toString();
-                    for (String name : ConfigStringListValues.SPAWNER_CHANGER_BLACKLIST.getValue()) {
+//                if (entityName != null && !entityName.isEmpty()) {
-                        if (entityName.equals(name)) {
+//                    for (String name : ConfigStringListValues.SPAWNER_CHANGER_BLACKLIST.getValue()) {
-                            return false;
+//                        if (entityName.equals(name)) {
-                        }
+//                            return false;
-                    }
+//                        }
-
+//                    }
-                    stack.getTagCompound().setString("Entity", entityName);
+//
-                    return true;
+//                    stack.getTagCompound().setString("Entity", entityName);
-                }
+//                    return true;
-            }
+//                }
-        }
+//            }
-        return false;
+//        }
-    }
+//        return false;
-
+//    }
-    private String getStoredEntity(ItemStack stack) {
+//
-        if (stack.hasTagCompound()) {
+//    private String getStoredEntity(ItemStack stack) {
-            String entity = stack.getTagCompound().getString("Entity");
+//        if (stack.hasTagCompound()) {
-            if (entity != null && !entity.isEmpty()) {
+//            String entity = stack.getTagCompound().getString("Entity");
-                return entity;
+//            if (entity != null && !entity.isEmpty()) {
-            }
+//                return entity;
-        }
+//            }
-        return null;
+//        }
-    }
+//        return null;
-
+//    }
-    @Override
+//
-    public void addInformation(ItemStack stack, World playerIn, List<String> list, ITooltipFlag advanced) {
+//    @Override
-        String entity = this.getStoredEntity(stack);
+//    public void addInformation(ItemStack stack, World playerIn, List<String> list, ITooltipFlag advanced) {
-        if (entity != null) {
+//        String entity = this.getStoredEntity(stack);
-            list.add("Entity: " + entity);
+//        if (entity != null) {
-            list.add(TextFormatting.ITALIC + StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc"));
+//            list.add("Entity: " + entity);
-        }
+//            list.add(TextFormatting.ITALIC + StringUtil.localize("tooltip." + ActuallyAdditions.MODID + ".clearStorage.desc"));
-    }
+//        }
-}
+//    }
+//}

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemWingsOfTheBats.java

@@ -10,27 +10,27 @@
 
 package de.ellpeck.actuallyadditions.mod.items;
 
-import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
 import de.ellpeck.actuallyadditions.mod.config.values.ConfigBoolValues;
 import de.ellpeck.actuallyadditions.mod.data.PlayerData;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
 import de.ellpeck.actuallyadditions.mod.network.PacketHandlerHelper;
+import de.ellpeck.actuallyadditions.mod.proxy.ClientProxy;
 import de.ellpeck.actuallyadditions.mod.util.StackUtil;
 import net.minecraft.block.BlockState;
 import net.minecraft.entity.Entity;
+import net.minecraft.entity.LivingEntity;
 import net.minecraft.entity.item.ItemEntity;
-import net.minecraft.entity.passive.EntityBat;
+import net.minecraft.entity.passive.BatEntity;
 import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.EnumRarity;
 import net.minecraft.item.ItemStack;
-import net.minecraft.item.ItemSword;
+import net.minecraft.item.SwordItem;
 import net.minecraft.util.Direction;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.math.MathHelper;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.event.entity.living.LivingDropsEvent;
 import net.minecraftforge.event.entity.living.LivingEvent;
-import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
+import net.minecraftforge.eventbus.api.SubscribeEvent;
 
 public class ItemWingsOfTheBats extends ItemBase {
 
@@ -38,9 +38,9 @@ public class ItemWingsOfTheBats extends ItemBase {
     public static final int MAX_FLY_TIME = 800;
 
     public ItemWingsOfTheBats() {
-        super(name);
+        super(ActuallyItems.defaultProps().maxStackSize(1));
-        this.setMaxStackSize(1);
 
+        // TODO: Lets move this somewhere global. Don't like event logic in a single place.
         MinecraftForge.EVENT_BUS.register(this);
     }
 
@@ -67,27 +67,23 @@ public class ItemWingsOfTheBats extends ItemBase {
 
     @Override
     public double getDurabilityForDisplay(ItemStack stack) {
-        PlayerEntity player = ActuallyAdditions.PROXY.getCurrentPlayer();
+        PlayerEntity player = ClientProxy.getCurrentPlayer();
         if (player != null) {
             PlayerData.PlayerSave data = PlayerData.getDataFromPlayer(player);
-            if (data != null) {
             double diff = MAX_FLY_TIME - data.batWingsFlyTime;
             return 1 - diff / MAX_FLY_TIME;
         }
-        }
         return super.getDurabilityForDisplay(stack);
     }
 
     @Override
     public int getRGBDurabilityForDisplay(ItemStack stack) {
-        PlayerEntity player = ActuallyAdditions.PROXY.getCurrentPlayer();
+        PlayerEntity player = ClientProxy.getCurrentPlayer();
         if (player != null) {
             PlayerData.PlayerSave data = PlayerData.getDataFromPlayer(player);
-            if (data != null) {
             int curr = data.batWingsFlyTime;
             return MathHelper.hsvToRGB(Math.max(0.0F, 1 - (float) curr / MAX_FLY_TIME) / 3.0F, 1.0F, 1.0F);
         }
-        }
         return super.getRGBDurabilityForDisplay(stack);
     }
 
@@ -97,19 +93,21 @@ public class ItemWingsOfTheBats extends ItemBase {
 
         if (event.getEntityLiving().world != null && !event.getEntityLiving().world.isRemote && source instanceof PlayerEntity) {
             //Drop Wings from Bats
-            if (ConfigBoolValues.DO_BAT_DROPS.isEnabled() && event.getEntityLiving() instanceof EntityBat) {
+            if (ConfigBoolValues.DO_BAT_DROPS.isEnabled() && event.getEntityLiving() instanceof BatEntity) {
                 int looting = event.getLootingLevel();
 
                 Iterable<ItemStack> equip = source.getHeldEquipment();
                 for (ItemStack stack : equip) {
-                    if (StackUtil.isValid(stack) && ItemWingsOfTheBats.THE_BAT_BAT.equalsIgnoreCase(stack.getDisplayName()) && stack.getItem() instanceof ItemSword) {
+                    // Todo: [port] this might not work anymore due to the way things are checked
+                    if (StackUtil.isValid(stack) && ItemWingsOfTheBats.THE_BAT_BAT.equalsIgnoreCase(stack.getDisplayName().getString()) && stack.getItem() instanceof SwordItem) {
                         looting += 3;
                         break;
                     }
                 }
 
                 if (event.getEntityLiving().world.rand.nextInt(15) <= looting * 2) {
-                    event.getDrops().add(new ItemEntity(event.getEntityLiving().world, event.getEntityLiving().posX, event.getEntityLiving().posY, event.getEntityLiving().posZ, new ItemStack(ActuallyItems.BAT_WING.get(), event.getEntityLiving().world.rand.nextInt(2 + looting) + 1)));
+                    LivingEntity entityLiving = event.getEntityLiving();
+                    event.getDrops().add(new ItemEntity(event.getEntityLiving().world, entityLiving.getPosX(), entityLiving.getPosY(), entityLiving.getPosZ(), new ItemStack(ActuallyItems.BAT_WING.get(), event.getEntityLiving().world.rand.nextInt(2 + looting) + 1)));
                 }
             }
         }
@@ -139,12 +137,12 @@ public class ItemWingsOfTheBats extends ItemBase {
                         }
                     } else {
                         if (wingsEquipped && data.batWingsFlyTime < MAX_FLY_TIME) {
-                            player.capabilities.allowFlying = true;
+                            player.abilities.allowFlying = true;
 
-                            if (player.capabilities.isFlying) {
+                            if (player.abilities.isFlying) {
                                 data.batWingsFlyTime++;
 
-                                if (player.world.getTotalWorldTime() % 10 == 0) {
+                                if (player.world.getWorldInfo().getGameTime() % 10 == 0) {
                                     shouldSend = true;
                                 }
                             }
@@ -155,21 +153,21 @@ public class ItemWingsOfTheBats extends ItemBase {
                             data.shouldDisableBatWings = true;
                             shouldSend = true;
 
-                            player.capabilities.allowFlying = false;
+                            player.abilities.allowFlying = false;
-                            player.capabilities.isFlying = false;
+                            player.abilities.isFlying = false;
-                            player.capabilities.disableDamage = false;
+                            player.abilities.disableDamage = false;
                         }
                     }
 
                     if (tryDeduct && data.batWingsFlyTime > 0) {
                         int deductTime = 0;
 
-                        if (!player.capabilities.isFlying) {
+                        if (!player.abilities.isFlying) {
                             deductTime = 2;
                         } else {
-                            BlockPos pos = new BlockPos(player.posX, player.posY + player.height, player.posZ);
+                            BlockPos pos = new BlockPos(player.getPosX(), player.getPosY() + player.getHeight(), player.getPosZ());
                             BlockState state = player.world.getBlockState(pos);
-                            if (state != null && state.isSideSolid(player.world, pos, Direction.DOWN)) {
+                            if (state.isSolidSide(player.world, pos, Direction.DOWN)) {
                                 deductTime = 10;
                             }
                         }
@@ -177,7 +175,7 @@ public class ItemWingsOfTheBats extends ItemBase {
                         if (deductTime > 0) {
                             data.batWingsFlyTime = Math.max(0, data.batWingsFlyTime - deductTime);
 
-                            if (player.world.getTotalWorldTime() % 10 == 0) {
+                            if (player.world.getWorldInfo().getGameTime() % 10 == 0) {
                                 shouldSend = true;
                             }
                         }
@@ -189,21 +187,16 @@ public class ItemWingsOfTheBats extends ItemBase {
                     }
                 } else {
                     if (data.hasBatWings) {
-                        player.capabilities.allowFlying = true;
+                        player.abilities.allowFlying = true;
                     } else if (data.shouldDisableBatWings) { //so that other modded flying won't be disabled
                         data.shouldDisableBatWings = false;
 
-                        player.capabilities.allowFlying = false;
+                        player.abilities.allowFlying = false;
-                        player.capabilities.isFlying = false;
+                        player.abilities.isFlying = false;
-                        player.capabilities.disableDamage = false;
+                        player.abilities.disableDamage = false;
                     }
                 }
             }
         }
     }
-
-    @Override
-    public EnumRarity getRarity(ItemStack stack) {
-        return EnumRarity.EPIC;
-    }
 }

src/main/java/de/ellpeck/actuallyadditions/mod/proxy/ClientProxy.java

@@ -81,7 +81,6 @@ public class ClientProxy {
     //        COLOR_PRODIVIDING_BLOCKS_FOR_REGISTERING.add(block);
     //    }
 
-    @Deprecated
     public static ClientPlayerEntity getCurrentPlayer() {
         return Minecraft.getInstance().player;
     }

src/main/java/de/ellpeck/actuallyadditions/mod/tile/TileEntityBase.java

@@ -174,7 +174,7 @@ public abstract class TileEntityBase extends TileEntity implements ITickable {
 
     //    @Override
     //    public ITextComponent getDisplayName() {
-    //        return new TextComponentTranslation(this.getNameForTranslation());
+    //        return new TranslationTextComponent(this.getNameForTranslation());
     //    }