From a7108bc2ffa185c13c41959ccddc7fbdde789025 Mon Sep 17 00:00:00 2001
From: Christian <cpw@weeksfamily.ca>
Date: Thu, 28 Mar 2013 08:04:56 -0400
Subject: [PATCH] Fix up some liquid logic

---
 .../minecraftforge/liquids/LiquidDictionary.java    |  4 ++++
 common/net/minecraftforge/liquids/LiquidStack.java  | 13 +++++--------
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/common/net/minecraftforge/liquids/LiquidDictionary.java b/common/net/minecraftforge/liquids/LiquidDictionary.java
index 141d18664..b3077c0c9 100644
--- a/common/net/minecraftforge/liquids/LiquidDictionary.java
+++ b/common/net/minecraftforge/liquids/LiquidDictionary.java
@@ -34,6 +34,10 @@ public abstract class LiquidDictionary
      */
     public static LiquidStack getOrCreateLiquid(String name, LiquidStack liquid)
     {
+        if (liquid == null)
+        {
+            throw new NullPointerException("You cannot register a null LiquidStack");
+        }
         LiquidStack existing = liquids.get(name);
         if(existing != null) {
             return existing.copy();
diff --git a/common/net/minecraftforge/liquids/LiquidStack.java b/common/net/minecraftforge/liquids/LiquidStack.java
index 30f93f679..20de276c1 100644
--- a/common/net/minecraftforge/liquids/LiquidStack.java
+++ b/common/net/minecraftforge/liquids/LiquidStack.java
@@ -38,8 +38,8 @@ public class LiquidStack
 
     public NBTTagCompound writeToNBT(NBTTagCompound nbt)
     {
-        nbt.setShort("Id", (short)itemID);
         nbt.setInteger("Amount", amount);
+        nbt.setShort("Id", (short)itemID);
         nbt.setShort("Meta", (short)itemMeta);
         nbt.setString("LiquidName", LiquidDictionary.findLiquidName(this));
         return nbt;
@@ -50,13 +50,10 @@ public class LiquidStack
         String liquidName = nbt.getString("LiquidName");
         itemID = nbt.getShort("Id");
         itemMeta = nbt.getShort("Meta");
-        if (liquidName != null)
-        {
-            LiquidStack liquid = LiquidDictionary.getCanonicalLiquid(liquidName);
-            if(liquid != null) {
-                itemID = liquid.itemID;
-                itemMeta = liquid.itemMeta;
-            }
+        LiquidStack liquid = LiquidDictionary.getCanonicalLiquid(liquidName);
+        if(liquid != null) {
+            itemID = liquid.itemID;
+            itemMeta = liquid.itemMeta;
         }
         amount = nbt.getInteger("Amount");
     }