From 6f8446221557e56524d8141ab7c561136dd306e6 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Wed, 6 Aug 2014 15:19:43 +0200
Subject: [PATCH 1/9] Fix a bug, that when activating a faucet and the draining
 fails, the next attempt to activate it would do nothing.

---
 src/main/java/tconstruct/smeltery/logic/FaucetLogic.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/tconstruct/smeltery/logic/FaucetLogic.java b/src/main/java/tconstruct/smeltery/logic/FaucetLogic.java
index ae0aca88..0f4f7b8c 100644
--- a/src/main/java/tconstruct/smeltery/logic/FaucetLogic.java
+++ b/src/main/java/tconstruct/smeltery/logic/FaucetLogic.java
@@ -202,7 +202,8 @@ public class FaucetLogic extends TileEntity implements IFacingLogic, IActiveLogi
         if (!active)
         {
             active = true;
-            activateFaucet();
+            if(!activateFaucet())
+                active = false;
         }
         else
         {

From d5829b0fed61c3bdbee89036eb0b339ea7e24524 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Wed, 6 Aug 2014 15:35:52 +0200
Subject: [PATCH 2/9] Fire a SmelteryCastEvent when a castingTable tries to
 cast something. Allows to deny the casting.

---
 .../library/event/SmelteryCastEvent.java      | 20 +++++++++++++++++++
 .../smeltery/logic/CastingTableLogic.java     | 10 ++++++++++
 2 files changed, 30 insertions(+)
 create mode 100644 src/main/java/tconstruct/library/event/SmelteryCastEvent.java

diff --git a/src/main/java/tconstruct/library/event/SmelteryCastEvent.java b/src/main/java/tconstruct/library/event/SmelteryCastEvent.java
new file mode 100644
index 00000000..21b6d90f
--- /dev/null
+++ b/src/main/java/tconstruct/library/event/SmelteryCastEvent.java
@@ -0,0 +1,20 @@
+package tconstruct.library.event;
+
+import cpw.mods.fml.common.eventhandler.Event;
+import net.minecraftforge.fluids.FluidStack;
+import tconstruct.library.crafting.CastingRecipe;
+
+/**
+ * Fires when somebody tries to pour a liquid into a metal cast.
+ *
+ * Set result to DENY to prevent casting.
+ */
+public class SmelteryCastEvent extends Event {
+    public final CastingRecipe recipe;
+    public final FluidStack fluid;
+
+    public SmelteryCastEvent(CastingRecipe recipe, FluidStack fluid) {
+        this.recipe = recipe;
+        this.fluid = fluid;
+    }
+}
diff --git a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
index 9dae22eb..5b150b74 100644
--- a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
+++ b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
@@ -1,5 +1,6 @@
 package tconstruct.smeltery.logic;
 
+import cpw.mods.fml.common.eventhandler.Event;
 import mantle.blocks.abstracts.InventoryLogic;
 import net.minecraft.entity.player.InventoryPlayer;
 import net.minecraft.inventory.Container;
@@ -10,6 +11,7 @@ import net.minecraft.network.NetworkManager;
 import net.minecraft.network.Packet;
 import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
 import net.minecraft.world.World;
+import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.common.util.ForgeDirection;
 import net.minecraftforge.fluids.Fluid;
 import net.minecraftforge.fluids.FluidEvent;
@@ -19,6 +21,7 @@ import net.minecraftforge.fluids.IFluidHandler;
 import net.minecraftforge.fluids.IFluidTank;
 import tconstruct.TConstruct;
 import tconstruct.library.crafting.CastingRecipe;
+import tconstruct.library.event.SmelteryCastEvent;
 import tconstruct.library.util.IPattern;
 
 public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFluidHandler, ISidedInventory
@@ -120,6 +123,13 @@ public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFl
             CastingRecipe recipe = TConstruct.tableCasting.getCastingRecipe(resource, inventory[0]);
             if (recipe == null)
                 return 0;
+
+            SmelteryCastEvent event = new SmelteryCastEvent(recipe, resource);
+            MinecraftForge.EVENT_BUS.post(event);
+
+            if(event.getResult() == Event.Result.DENY)
+                return 0;
+
             this.capacity = updateCapacity(recipe.castingMetal.amount);
 
             if (inventory[1] == null)

From 3cab9c0063128c218142b8dd6062c699b8c07d50 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Wed, 6 Aug 2014 15:39:29 +0200
Subject: [PATCH 3/9] Fire a SmelteryCastedEvent when a casting table finished
 casting an item. Allows to change/modify the output and consume the cast.

---
 .../library/event/SmelteryCastedEvent.java    | 23 +++++++++++++++++++
 .../smeltery/logic/CastingTableLogic.java     |  8 +++++--
 2 files changed, 29 insertions(+), 2 deletions(-)
 create mode 100644 src/main/java/tconstruct/library/event/SmelteryCastedEvent.java

diff --git a/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java b/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java
new file mode 100644
index 00000000..bf98d3c9
--- /dev/null
+++ b/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java
@@ -0,0 +1,23 @@
+package tconstruct.library.event;
+
+import cpw.mods.fml.common.eventhandler.Event;
+import net.minecraft.item.ItemStack;
+import tconstruct.library.crafting.CastingRecipe;
+
+/**
+ * Fired when an item is cast in the casting table.
+ * If consumeCast is set to true, the cast will be destroyed.
+ */
+public class SmelteryCastedEvent extends Event {
+    public final CastingRecipe recipe;
+    public ItemStack output;
+    public boolean consumeCast;
+
+    public SmelteryCastedEvent(CastingRecipe recipe, ItemStack output) {
+        this.recipe = recipe;
+        this.consumeCast = recipe.consumeCast;
+        this.output = output;
+    }
+
+
+}
diff --git a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
index 5b150b74..e2954610 100644
--- a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
+++ b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
@@ -22,6 +22,7 @@ import net.minecraftforge.fluids.IFluidTank;
 import tconstruct.TConstruct;
 import tconstruct.library.crafting.CastingRecipe;
 import tconstruct.library.event.SmelteryCastEvent;
+import tconstruct.library.event.SmelteryCastedEvent;
 import tconstruct.library.util.IPattern;
 
 public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFluidHandler, ISidedInventory
@@ -339,8 +340,11 @@ public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFl
         CastingRecipe recipe = TConstruct.tableCasting.getCastingRecipe(liquid, inventory[0]);
         if (recipe != null)
         {
-            inventory[1] = recipe.getResult();
-            if (recipe.consumeCast)
+            SmelteryCastedEvent event = new SmelteryCastedEvent(recipe, recipe.getResult());
+            MinecraftForge.EVENT_BUS.post(event);
+
+            inventory[1] = event.output;
+            if (event.consumeCast)
                 inventory[0] = null;
             liquid = null;
             worldObj.markBlockForUpdate(xCoord, yCoord, zCoord);

From 0821f9f77fe44dc5746c340f3d576e0d7e4045a4 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Wed, 6 Aug 2014 15:57:06 +0200
Subject: [PATCH 4/9] Also fire cast-events for CastingBasin. Add separate
 events for CastingBasin and CastingTable.

---
 .../library/event/SmelteryCastEvent.java       | 14 +++++++++++++-
 .../library/event/SmelteryCastedEvent.java     | 13 ++++++++++++-
 .../smeltery/logic/CastingBasinLogic.java      | 18 ++++++++++++++++--
 .../smeltery/logic/CastingTableLogic.java      |  4 ++--
 .../tconstruct/tools/TinkerToolEvents.java     |  2 ++
 5 files changed, 45 insertions(+), 6 deletions(-)

diff --git a/src/main/java/tconstruct/library/event/SmelteryCastEvent.java b/src/main/java/tconstruct/library/event/SmelteryCastEvent.java
index 21b6d90f..c384bc7c 100644
--- a/src/main/java/tconstruct/library/event/SmelteryCastEvent.java
+++ b/src/main/java/tconstruct/library/event/SmelteryCastEvent.java
@@ -9,7 +9,7 @@ import tconstruct.library.crafting.CastingRecipe;
  *
  * Set result to DENY to prevent casting.
  */
-public class SmelteryCastEvent extends Event {
+public abstract class SmelteryCastEvent extends Event {
     public final CastingRecipe recipe;
     public final FluidStack fluid;
 
@@ -17,4 +17,16 @@ public class SmelteryCastEvent extends Event {
         this.recipe = recipe;
         this.fluid = fluid;
     }
+
+    public static class CastingTable extends SmelteryCastEvent {
+        public CastingTable(CastingRecipe recipe, FluidStack fluid) {
+            super(recipe, fluid);
+        }
+    }
+
+    public static class CastingBasin extends SmelteryCastEvent {
+        public CastingBasin(CastingRecipe recipe, FluidStack fluid) {
+            super(recipe, fluid);
+        }
+    }
 }
diff --git a/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java b/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java
index bf98d3c9..f0c420b1 100644
--- a/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java
+++ b/src/main/java/tconstruct/library/event/SmelteryCastedEvent.java
@@ -8,7 +8,7 @@ import tconstruct.library.crafting.CastingRecipe;
  * Fired when an item is cast in the casting table.
  * If consumeCast is set to true, the cast will be destroyed.
  */
-public class SmelteryCastedEvent extends Event {
+public abstract class SmelteryCastedEvent extends Event {
     public final CastingRecipe recipe;
     public ItemStack output;
     public boolean consumeCast;
@@ -20,4 +20,15 @@ public class SmelteryCastedEvent extends Event {
     }
 
 
+    public static class CastingTable extends SmelteryCastedEvent {
+        public CastingTable(CastingRecipe recipe, ItemStack output) {
+            super(recipe, output);
+        }
+    }
+
+    public static class CastingBasin extends SmelteryCastedEvent {
+        public CastingBasin(CastingRecipe recipe, ItemStack output) {
+            super(recipe, output);
+        }
+    }
 }
diff --git a/src/main/java/tconstruct/smeltery/logic/CastingBasinLogic.java b/src/main/java/tconstruct/smeltery/logic/CastingBasinLogic.java
index 88df7d35..b6521828 100644
--- a/src/main/java/tconstruct/smeltery/logic/CastingBasinLogic.java
+++ b/src/main/java/tconstruct/smeltery/logic/CastingBasinLogic.java
@@ -1,5 +1,6 @@
 package tconstruct.smeltery.logic;
 
+import cpw.mods.fml.common.eventhandler.Event;
 import mantle.blocks.abstracts.InventoryLogic;
 import net.minecraft.entity.player.InventoryPlayer;
 import net.minecraft.inventory.Container;
@@ -10,6 +11,7 @@ import net.minecraft.network.NetworkManager;
 import net.minecraft.network.Packet;
 import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
 import net.minecraft.world.World;
+import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.common.util.ForgeDirection;
 import net.minecraftforge.fluids.Fluid;
 import net.minecraftforge.fluids.FluidEvent;
@@ -19,6 +21,8 @@ import net.minecraftforge.fluids.IFluidHandler;
 import net.minecraftforge.fluids.IFluidTank;
 import tconstruct.TConstruct;
 import tconstruct.library.crafting.CastingRecipe;
+import tconstruct.library.event.SmelteryCastEvent;
+import tconstruct.library.event.SmelteryCastedEvent;
 import tconstruct.library.util.IPattern;
 
 public class CastingBasinLogic extends InventoryLogic implements IFluidTank, IFluidHandler, ISidedInventory
@@ -120,6 +124,13 @@ public class CastingBasinLogic extends InventoryLogic implements IFluidTank, IFl
             CastingRecipe recipe = TConstruct.basinCasting.getCastingRecipe(resource, inventory[0]);
             if (recipe == null)
                 return 0;
+
+            SmelteryCastEvent event = new SmelteryCastEvent.CastingBasin(recipe, resource);
+            MinecraftForge.EVENT_BUS.post(event);
+
+            if(event.getResult() == Event.Result.DENY)
+                return 0;
+
             this.capacity = updateCapacity(recipe.castingMetal.amount);
 
             if (inventory[1] == null)
@@ -328,8 +339,11 @@ public class CastingBasinLogic extends InventoryLogic implements IFluidTank, IFl
         CastingRecipe recipe = TConstruct.basinCasting.getCastingRecipe(liquid, inventory[0]);
         if (recipe != null)
         {
-            inventory[1] = recipe.getResult();
-            if (recipe.consumeCast)
+            SmelteryCastedEvent event = new SmelteryCastedEvent.CastingBasin(recipe, recipe.getResult());
+            MinecraftForge.EVENT_BUS.post(event);
+
+            inventory[1] = event.output;
+            if (event.consumeCast)
                 inventory[0] = null;
             liquid = null;
             worldObj.markBlockForUpdate(xCoord, yCoord, zCoord);
diff --git a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
index e2954610..1bdbac81 100644
--- a/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
+++ b/src/main/java/tconstruct/smeltery/logic/CastingTableLogic.java
@@ -125,7 +125,7 @@ public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFl
             if (recipe == null)
                 return 0;
 
-            SmelteryCastEvent event = new SmelteryCastEvent(recipe, resource);
+            SmelteryCastEvent event = new SmelteryCastEvent.CastingTable(recipe, resource);
             MinecraftForge.EVENT_BUS.post(event);
 
             if(event.getResult() == Event.Result.DENY)
@@ -340,7 +340,7 @@ public class CastingTableLogic extends InventoryLogic implements IFluidTank, IFl
         CastingRecipe recipe = TConstruct.tableCasting.getCastingRecipe(liquid, inventory[0]);
         if (recipe != null)
         {
-            SmelteryCastedEvent event = new SmelteryCastedEvent(recipe, recipe.getResult());
+            SmelteryCastedEvent event = new SmelteryCastedEvent.CastingTable(recipe, recipe.getResult());
             MinecraftForge.EVENT_BUS.post(event);
 
             inventory[1] = event.output;
diff --git a/src/main/java/tconstruct/tools/TinkerToolEvents.java b/src/main/java/tconstruct/tools/TinkerToolEvents.java
index 67196bb1..c32ed42b 100644
--- a/src/main/java/tconstruct/tools/TinkerToolEvents.java
+++ b/src/main/java/tconstruct/tools/TinkerToolEvents.java
@@ -30,6 +30,8 @@ import tconstruct.armor.TinkerArmor;
 import tconstruct.armor.player.TPlayerStats;
 import tconstruct.library.TConstructRegistry;
 import tconstruct.library.event.PartBuilderEvent;
+import tconstruct.library.event.SmelteryCastEvent;
+import tconstruct.library.event.SmelteryCastedEvent;
 import tconstruct.library.event.ToolCraftEvent;
 import tconstruct.library.tools.AbilityHelper;
 import tconstruct.library.tools.ArrowMaterial;

From 9b9fb14098cfb1fe99367a338e47a95bdd09558e Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Wed, 6 Aug 2014 20:17:47 +0200
Subject: [PATCH 5/9] Make pulsar manager public, so addons can hook into it.

---
 src/main/java/tconstruct/TConstruct.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/tconstruct/TConstruct.java b/src/main/java/tconstruct/TConstruct.java
index 0a314032..21ee4baa 100644
--- a/src/main/java/tconstruct/TConstruct.java
+++ b/src/main/java/tconstruct/TConstruct.java
@@ -88,7 +88,7 @@ public class TConstruct
     public static TProxyCommon proxy;
 
     /* Loads modules in a way that doesn't clutter the @Mod list */
-    private PulseManager pulsar = new PulseManager(modID, new ForgeCFG("TinkersModules", "Modules: Disabling these will disable a chunk of the mod"));
+    public static PulseManager pulsar = new PulseManager(modID, new ForgeCFG("TinkersModules", "Modules: Disabling these will disable a chunk of the mod"));
 
     public TConstruct()
     {

From b38c8a0ab9237358bc41e462fe64e05454537d53 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Thu, 7 Aug 2014 12:52:02 +0200
Subject: [PATCH 6/9] Fix warning messages on for the default tools ignoring
 supressMissingToolLogs config. Usually set by Iguana Tweaks. Otherwise they
 spam that everytime NEI queries for tools.

---
 src/main/java/tconstruct/library/tools/ToolCore.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/tconstruct/library/tools/ToolCore.java b/src/main/java/tconstruct/library/tools/ToolCore.java
index 0a8c9c81..a67af822 100644
--- a/src/main/java/tconstruct/library/tools/ToolCore.java
+++ b/src/main/java/tconstruct/library/tools/ToolCore.java
@@ -24,6 +24,7 @@ import tconstruct.library.TConstructRegistry;
 import tconstruct.library.crafting.ToolBuilder;
 import tconstruct.library.modifier.IModifyable;
 import tconstruct.library.modifier.ItemModifier;
+import tconstruct.tools.TinkerTools;
 import tconstruct.tools.entity.FancyEntityItem;
 import cofh.api.energy.IEnergyContainerItem;
 import cpw.mods.fml.relauncher.Side;
@@ -556,8 +557,7 @@ public abstract class ToolCore extends Item implements IEnergyContainerItem, IMo
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, id), accessoryStack, extraStack, name + getToolName());
         if (tool == null)
         {
-            boolean supress = false; //TODO: Find this for iguana tweaks
-            if (!supress)
+            if (TinkerTools.supressMissingToolLogs)
             {
                 TConstructRegistry.logger.error("Creative builder failed tool for " + name + this.getToolName());
                 TConstructRegistry.logger.error("Make sure you do not have item ID conflicts");

From e10c5d2770ebb097b3f9f08a8130c6c6c390142c Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Thu, 7 Aug 2014 21:09:07 +0200
Subject: [PATCH 7/9] Fix Mattock using vanilla damageItem. Caused Mattock to
 vanish on breaking.

---
 src/main/java/tconstruct/library/tools/AbilityHelper.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/tconstruct/library/tools/AbilityHelper.java b/src/main/java/tconstruct/library/tools/AbilityHelper.java
index f547677f..06f02725 100644
--- a/src/main/java/tconstruct/library/tools/AbilityHelper.java
+++ b/src/main/java/tconstruct/library/tools/AbilityHelper.java
@@ -491,7 +491,7 @@ public class AbilityHelper
 
             if (event.getResult() == Result.ALLOW)
             {
-                stack.damageItem(1, player);
+                damageTool(stack, 1, player, false);
                 return true;
             }
 
@@ -509,7 +509,7 @@ public class AbilityHelper
                 else
                 {
                     world.setBlock(x, y, z, block1);
-                    stack.damageItem(1, player);
+                    damageTool(stack, 1, player, false);
                     return true;
                 }
             }

From 92ab16edecc6ffdc940c4c462ea5de9c26fc4bc6 Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Fri, 8 Aug 2014 09:58:30 +0200
Subject: [PATCH 8/9] Arrgh. Correctly supress logs when option is stead,
 instead of only displaying them when it's set.

---
 src/main/java/tconstruct/library/tools/ToolCore.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/tconstruct/library/tools/ToolCore.java b/src/main/java/tconstruct/library/tools/ToolCore.java
index a67af822..699e1305 100644
--- a/src/main/java/tconstruct/library/tools/ToolCore.java
+++ b/src/main/java/tconstruct/library/tools/ToolCore.java
@@ -557,7 +557,7 @@ public abstract class ToolCore extends Item implements IEnergyContainerItem, IMo
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, id), accessoryStack, extraStack, name + getToolName());
         if (tool == null)
         {
-            if (TinkerTools.supressMissingToolLogs)
+            if (!TinkerTools.supressMissingToolLogs)
             {
                 TConstructRegistry.logger.error("Creative builder failed tool for " + name + this.getToolName());
                 TConstructRegistry.logger.error("Make sure you do not have item ID conflicts");

From cf0d0cc0ddd7bcb84e771a1f114449cdd8c5b5db Mon Sep 17 00:00:00 2001
From: Bernhard Bonigl <bernhard.bonigl@gmail.com>
Date: Fri, 8 Aug 2014 10:26:51 +0200
Subject: [PATCH 9/9] Remove all tool-build messages (no more IDs) and its
 config option

---
 .../java/tconstruct/items/tools/Arrow.java    | 30 ++-----------------
 .../java/tconstruct/items/tools/BowBase.java  | 16 +++-------
 .../java/tconstruct/items/tools/Mattock.java  | 10 +------
 .../tconstruct/library/tools/ToolCore.java    | 10 +------
 .../java/tconstruct/tools/TinkerTools.java    |  3 --
 .../tconstruct/util/config/PHConstruct.java   |  1 -
 6 files changed, 9 insertions(+), 61 deletions(-)

diff --git a/src/main/java/tconstruct/items/tools/Arrow.java b/src/main/java/tconstruct/items/tools/Arrow.java
index b1d714f9..4bf1a466 100644
--- a/src/main/java/tconstruct/items/tools/Arrow.java
+++ b/src/main/java/tconstruct/items/tools/Arrow.java
@@ -103,15 +103,7 @@ public class Arrow extends ToolCore
         Item extra = getExtraItem();
         ItemStack extraStack = extra != null ? new ItemStack(extra, 1, 0) : null;
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, 3), new ItemStack(getHandleItem(), 1, 0), accessoryStack, extraStack, "");
-        if (tool == null)
-        {
-            if (!TinkerTools.supressMissingToolLogs)
-            {
-                TConstruct.logger.warn("Creative builder failed tool for Vanilla style" + this.getToolName());
-                TConstruct.logger.warn("Make sure you do not have item ID conflicts");
-            }
-        }
-        else
+        if (tool != null)
         {
             tool.stackSize = 1;
             tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
@@ -126,18 +118,7 @@ public class Arrow extends ToolCore
         tool = ToolBuilder.instance
                 .buildTool(new ItemStack(getHeadItem(), 1, random.nextInt(18)), new ItemStack(getHandleItem(), 1, random.nextInt(18)), accessoryStack, extraStack, StatCollector.translateToLocal("item.tool.randomarrow"));
 
-        if (tool == null)
-        {
-            if (!TinkerTools.supressMissingToolLogs)
-            {
-                if (!TinkerTools.supressMissingToolLogs)
-                {
-                    TConstruct.logger.warn("Creative builder failed tool for Vanilla style" + this.getToolName());
-                    TConstruct.logger.warn("Make sure you do not have item ID conflicts");
-                }
-            }
-        }
-        else
+        if (tool != null)
         {
             tool.stackSize = 1;
             tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
@@ -154,12 +135,7 @@ public class Arrow extends ToolCore
         Item extra = getExtraItem();
         ItemStack extraStack = extra != null ? new ItemStack(getExtraItem(), 1, id) : null;
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, id), accessoryStack, extraStack, name + getToolName());
-        if (tool == null)
-        {
-            TConstruct.logger.warn("Creative builder failed tool for " + name + this.getToolName());
-            TConstruct.logger.warn("Make sure you do not have item ID conflicts");
-        }
-        else
+        if (tool != null)
         {
             tool.stackSize = 1;
             tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
diff --git a/src/main/java/tconstruct/items/tools/BowBase.java b/src/main/java/tconstruct/items/tools/BowBase.java
index 8dcb6f8b..93cf2d0e 100644
--- a/src/main/java/tconstruct/items/tools/BowBase.java
+++ b/src/main/java/tconstruct/items/tools/BowBase.java
@@ -776,17 +776,9 @@ public abstract class BowBase extends ToolCore
         ItemStack extraStack = extra != null ? new ItemStack(getExtraItem(), 1, id) : null;
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, 0), accessoryStack, extraStack, name + getToolName());
         if (tool == null)
-        {
-            if (!TinkerTools.supressMissingToolLogs)
-            {
-                TConstruct.logger.warn("Creative builder failed tool for " + name + this.getToolName());
-                TConstruct.logger.warn("Make sure you do not have item ID conflicts");
-            }
-        }
-        else
-        {
-            tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
-            list.add(tool);
-        }
+            return;
+
+        tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
+        list.add(tool);
     }
 }
diff --git a/src/main/java/tconstruct/items/tools/Mattock.java b/src/main/java/tconstruct/items/tools/Mattock.java
index 78f79f29..5bf70343 100644
--- a/src/main/java/tconstruct/items/tools/Mattock.java
+++ b/src/main/java/tconstruct/items/tools/Mattock.java
@@ -120,15 +120,7 @@ public class Mattock extends DualHarvestTool
             Item extra = getExtraItem();
             ItemStack extraStack = extra != null ? new ItemStack(extra, 1, id) : null;
             ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, id), accessoryStack, extraStack, name + getToolName());
-            if (tool == null)
-            {
-                if (!TinkerTools.supressMissingToolLogs)
-                {
-                    TConstruct.logger.warn("Creative builder failed tool for " + name + this.getToolName());
-                    TConstruct.logger.warn("Make sure you do not have item ID conflicts");
-                }
-            }
-            else
+            if (tool != null)
             {
                 tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
                 list.add(tool);
diff --git a/src/main/java/tconstruct/library/tools/ToolCore.java b/src/main/java/tconstruct/library/tools/ToolCore.java
index 699e1305..a3d1e287 100644
--- a/src/main/java/tconstruct/library/tools/ToolCore.java
+++ b/src/main/java/tconstruct/library/tools/ToolCore.java
@@ -555,15 +555,7 @@ public abstract class ToolCore extends Item implements IEnergyContainerItem, IMo
         Item extra = getExtraItem();
         ItemStack extraStack = extra != null ? new ItemStack(extra, 1, id) : null;
         ItemStack tool = ToolBuilder.instance.buildTool(new ItemStack(getHeadItem(), 1, id), new ItemStack(getHandleItem(), 1, id), accessoryStack, extraStack, name + getToolName());
-        if (tool == null)
-        {
-            if (!TinkerTools.supressMissingToolLogs)
-            {
-                TConstructRegistry.logger.error("Creative builder failed tool for " + name + this.getToolName());
-                TConstructRegistry.logger.error("Make sure you do not have item ID conflicts");
-            }
-        }
-        else
+        if (tool != null)
         {
             tool.getTagCompound().getCompoundTag("InfiTool").setBoolean("Built", true);
             list.add(tool);
diff --git a/src/main/java/tconstruct/tools/TinkerTools.java b/src/main/java/tconstruct/tools/TinkerTools.java
index 9c06bf6d..acda1012 100644
--- a/src/main/java/tconstruct/tools/TinkerTools.java
+++ b/src/main/java/tconstruct/tools/TinkerTools.java
@@ -143,9 +143,6 @@ public class TinkerTools
     public static Item knifeBlade;
     public static Item wideGuard;
 
-    // Supresses console spam when iguana's tweaks remove stuff
-    public static boolean supressMissingToolLogs = false;
-
     // Patterns and other materials
     public static Item blankPattern;
     public static Item materials; //TODO: Untwine this item
diff --git a/src/main/java/tconstruct/util/config/PHConstruct.java b/src/main/java/tconstruct/util/config/PHConstruct.java
index 6377c03e..0148080a 100644
--- a/src/main/java/tconstruct/util/config/PHConstruct.java
+++ b/src/main/java/tconstruct/util/config/PHConstruct.java
@@ -42,7 +42,6 @@ public class PHConstruct
         //config.load(); /* Load happens in the constructor */
 
         superfunWorld = config.get("Superfun", "All the world is Superfun", false).getBoolean(false);
-        TinkerTools.supressMissingToolLogs = config.get("Logging", "Disable tool build messages", false).getBoolean(false);
 
         keepHunger = config.get("Difficulty Changes", "Keep hunger on death", true).getBoolean(true);
         keepLevels = config.get("Difficulty Changes", "Keep levels on death", true).getBoolean(true);