I finally finished converting to 1.10.2, so I thought I'd try to compile my mod. I haven't done it for months. I'm getting some errors that I don't know how to deal with. I spent a while Googling but didn't come up with anything.

 

The most common error is a "cannot find symbol" error for "variable worldObj". Here's an example error message:

 

 

.../com/daeruin/primalcraft/network/packets/PrimalThirstPacket.java:61: error: cannot find symbol

Entity entity = Minecraft.getMinecraft().theWorld.getEntityByID(message.playerId);

                                        ^

  symbol:  variable theWorld

  location: class Minecraft

 

 

 

And here's the relevant class:

 

 

package com.daeruin.primalcraft.network.packets;

import com.daeruin.primalcraft.capabilities.IThirst;
import com.daeruin.primalcraft.capabilities.ThirstProvider;

import io.netty.buffer.ByteBuf;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.Entity;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.util.IThreadListener;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class PrimalThirstPacket implements IMessage
{

private int playerId;
private int thirstLevel;

public PrimalThirstPacket()
{
}

public PrimalThirstPacket(EntityPlayer player, int thirstLevel)
{
	this.playerId = player.getEntityId();
	this.thirstLevel = thirstLevel;
}

@Override
public void fromBytes(ByteBuf buffer)
{
	this.playerId = buffer.readInt();
	this.thirstLevel = buffer.readInt();
}

@Override
public void toBytes(ByteBuf buffer)
{
	buffer.writeInt(playerId);
	buffer.writeInt(thirstLevel);
}

public static class PrimalThirstHandler implements IMessageHandler<PrimalThirstPacket, IMessage>
{

	@Override
	public IMessage onMessage(final PrimalThirstPacket message, final MessageContext ctx)
	{
		IThreadListener mainThread = Minecraft.getMinecraft();
		mainThread.addScheduledTask(new Runnable()
		{
			@Override
			public void run()
			{
				Entity entity = Minecraft.getMinecraft().theWorld.getEntityByID(message.playerId);
				if (entity instanceof EntityPlayer)
				{
					IThirst thirstCapability = entity.getCapability(ThirstProvider.THIRST_CAPABILITY, null);
					thirstCapability.setThirst(message.thirstLevel);
				}
			}
		});
		return null;
	}

}

}

 

 

If I could fix all examples of this error, I'd cut the errors in half (I'm getting about 30 errors!).

A lot of fields and methods were renamed in the last few MCP mappings versions for 1.10.2.

 

Minecraft#thePlayer

and

Minecraft#theWorld

were renamed to

Minecraft#player

and

Minecraft#world

.

Entity#worldObj

was renamed to

Entity#world

.

 

MCPBot can tell you the history of a field/method name. Most renamings are documented on this issue tracker.

 

You can also find the new name of a field/method by looking at the class that contains it and any other code that used it.

Ah. So if I go ahead and rename things to match the latest mapping, I'll need to update my Forge installation, correct? Or I could change which mapping I use in my build.gradle file to an older version that matches my installed Forge version.

Well, I decided to go ahead and install a newer version of Forge, but it won't compile. I updated my build.gradle file like so:

 

minecraft {

    version = "1.10.2-12.18.2.2147"

    mappings = "snapshot_20161117"

}

 

I chose those versions because that snapshot is the latest one listed on the MCPbot site, and that version of forge was released on that day (according to the Forge downloads page). But setupDecompWorkspace gives me an error:

 

[ant:javac] ...Primalcraft/build/tmp/recompileMc/sources/net/minecraft/inventory/ContainerPlayer.java:62: error: local variable player is accessed from within inner class; needs to be declared final

[ant:javac]                        return stack.getItem().isValidArmor(stack, entityequipmentslot, player);

[ant:javac]                                                                                        ^

In general Forge should work with any version of MCP mappings, so use either the Latest or Recommended version of Forge for the version of Minecraft you're using.

 

snapshot_20161117

always causes this compilation error, use

stable_29

instead. The stable version was generated a couple of hours after the snapshot and doesn't cause this error.

Yep, that worked. I actually found an issue report you created about that snapshot mere minutes before you posted your reply. Thank you yet again. After updating, it took a few minutes to rename some fields and tweak my mcmod.info file, and the build worked.

 

It's been over two months since I finally decided to update. It's nice to have a functioning mod again.