setycz / ChickensMod

Just a lot of chickens
MIT License
14 stars 32 forks source link

'Potentially Dangerous alternative prefix' #134

Open LemADEC opened 6 years ago

LemADEC commented 6 years ago

Issue Description:

Warnings in console

Steps to reproduce (important):

  1. boot the game

What happens:

[10:44:05] [Server thread/WARN] [FML]: Potentially Dangerous alternative prefix `minecraft` for name `henhouse`, expected `chickens`. This could be a intended override, but in most cases indicates a broken mod.

What you expected to happen:

No warning and errors, especially from forge.


Affected Versions (Do not use "latest"):

GenDeathrow commented 6 years ago

well I will look at that.. but if thats true its been like that since I took over the mod. If I change it, it will break all henhouses. So.. it may not get changed until a new version of minecraft/forge

LemADEC commented 6 years ago

Forge provides ways to convert your tile entities to address that without loosing content.

GenDeathrow commented 6 years ago

Well, I actually did not know that. Do you have a link or reference to what I'm looking for to change this?

LemADEC commented 6 years ago

Previously, you would use a proxy TileEntity that rebuild itself with the new class/id. Nowadays, there's DataFixers.

GenDeathrow commented 6 years ago

I had always wondered what the datafixers were used exactly for. I'll do some more research into them. I appreciate the help. I welcome new ways to do things. I had done the proxy TileEntity once ,before but it was such a hassle in my opinion.