View Issue Details
|crossmod-compatibility / submodules / API
|Fixed in Version
|Builds for MC 1.7.10
|0001710: Config file missing option to manually set Potion ID of ic2.potion.radiation.
|Not much a bug itself, but rather lack of very essential feature.
Currently there is no config option for manually setting Potion ID of ic2.potion.radiation effect.
Absence of such feature may lead to inability of manually solving Potion ID conflicts.
|No tags attached.
|It's set right after the end of the vanilla potions (24), all the other mods I've seen have at least a one or two ID gap. I guess theoretically another mod without a config could conflict, but it won't normally crash without a mod forcing it like DragonAPI or AntiIDConflict.
Im sorry, but I dont get your point.
Just because something isn't causing crash, doesnt mean it works fine.
When one of your mods has such ID hardcoded, its not a problem YET, but its halfway to be a problem. Something definitely worth solving before it becomes real mess - which actually happened to me.
The funny thing in whole story is that the second mod which has conflict - causes it because it has implemented some sort of anti-IDconflict procedure, which fail to do its task because of mods loading order. Sadly mod author coded it in a way that causes mod to ignore potion ID settings from its own config. I have already told him about this issue and he will fix this in next update - but thats not the point here.
The point of this story is: Things like that actually happen - so why not spend 2 mins on coding to disarm the mine, solving the problem once and for all?
|It may happen but no one's ever complained about it before. Potion IDs were always an untrodden minefield as mods just didn't add potions.
I actually have to agree with Darkos333.
It may not yet be a problem, but it's better to be prepared.
It won't even take more than 2 minutes.
|Note Added: 0004132
|Note Added: 0004133
|Note Added: 0004134
|Note Added: 0004137
|new => confirmed
|confirmed => resolved
|Fixed in Version
|=> Builds for MC 1.7.10
|open => fixed