keith/ForgePatch
1
0
Fork 0
Code Issues Pull requests Projects Releases 2 Wiki Activity

Fix FluidRegsitry.registerFluid

Browse source 
FluidRegistry:
```java
    static BiMap<String, Fluid> fluids = HashBiMap.create();
    static BiMap<Fluid, Integer> fluidIDs = HashBiMap.create();
...
    public static boolean registerFluid(Fluid fluid)
    {
        if (fluidIDs.containsKey(fluid.getName()))
            ^^^^^^^
```
There is definitely should be fluids instead fluidIDs. This mistake broke many mods.
This commit is contained in:
Sergey Shatunov 2015-04-08 17:26:40 +06:00
parent 02ebc9ca21
commit a42cf1ae37
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/java/net/minecraftforge/fluids/FluidRegistry.java
View file

@ -79,7 +79,7 @@ public abstract class FluidRegistry
*/ */
public static boolean registerFluid(Fluid fluid) public static boolean registerFluid(Fluid fluid)
{ {
if (fluidIDs.containsKey(fluid.getName())) if (fluids.containsKey(fluid.getName()))
{ {
FMLLog.bigWarning("Duplicate registration attempt for fluid %s (type %s) has occurred. This is not a problem itself, but subsequent failed FluidStacks might be a result if not handled properly", fluid.getName(), fluid.getClass().getName()); FMLLog.bigWarning("Duplicate registration attempt for fluid %s (type %s) has occurred. This is not a problem itself, but subsequent failed FluidStacks might be a result if not handled properly", fluid.getName(), fluid.getClass().getName());
return false; return false;