Always check conversions

Author: pupnewfster

src/main/java/mekanism/common/integration/energy/fluxnetworks/FNIntegration.java

@@ -23,7 +23,7 @@ public long receiveEnergyL(long maxReceive, boolean simulate) {
         if (toInsert == 0) {
             return 0;
         }
-        if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+        if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
             //Before we can actually execute it we need to simulate to calculate how much we can actually insert
             long simulatedRemainder = handler.insertEnergy(toInsert, Action.SIMULATE);
             if (simulatedRemainder == toInsert) {
@@ -60,7 +60,7 @@ public long extractEnergyL(long maxExtract, boolean simulate) {
         if (toExtract == 0) {
             return 0;
         }
-        if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+        if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
             //Before we can actually execute it we need to simulate to calculate how much we can actually extract in our other units
             long simulatedExtracted = handler.extractEnergy(toExtract, Action.SIMULATE);
             //Convert how much we could extract back to FE so that it gets appropriately clamped so that for example 1.5 FE gets treated

src/main/java/mekanism/common/integration/energy/fluxnetworks/FNStrictEnergyHandler.java

@@ -55,7 +55,7 @@ public long insertEnergy(long amount, Action action) {
             if (toInsert == 0) {
                 return amount;
             }
-            if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+            if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
                 //Before we can actually execute it we need to simulate to calculate how much we can actually insert
                 long simulatedInserted = storage.receiveEnergyL(toInsert, true);
                 if (simulatedInserted == 0) {
@@ -103,7 +103,7 @@ public long extractEnergy(long amount, Action action) {
             if (toExtract == 0) {
                 return 0L;
             }
-            if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+            if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
                 //Before we can actually execute it we need to simulate to calculate how much we can actually extract in our other units
                 long simulatedExtracted = storage.extractEnergyL(toExtract, true);
                 //Convert how much we could extract back to Joules so that it gets appropriately clamped so that for example 1 Joule gets treated

src/main/java/mekanism/common/integration/energy/grandpower/GPIntegration.java

@@ -23,7 +23,7 @@ public long receive(long maxReceive, boolean simulate) {
         if (toInsert == 0) {
             return 0;
         }
-        if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+        if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
             //Before we can actually execute it we need to simulate to calculate how much we can actually insert
             long simulatedRemainder = handler.insertEnergy(toInsert, Action.SIMULATE);
             if (simulatedRemainder == toInsert) {
@@ -60,7 +60,7 @@ public long extract(long maxExtract, boolean simulate) {
         if (toExtract == 0) {
             return 0;
         }
-        if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+        if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
             //Before we can actually execute it we need to simulate to calculate how much we can actually extract in our other units
             long simulatedExtracted = handler.extractEnergy(toExtract, Action.SIMULATE);
             //Convert how much we could extract back to FE so that it gets appropriately clamped so that for example 1.5 FE gets treated

src/main/java/mekanism/common/integration/energy/grandpower/GPStrictEnergyHandler.java

@@ -55,7 +55,7 @@ public long insertEnergy(long amount, Action action) {
             if (toInsert == 0) {
                 return amount;
             }
-            if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+            if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
                 //Before we can actually execute it we need to simulate to calculate how much we can actually insert
                 long simulatedInserted = storage.receive(toInsert, true);
                 if (simulatedInserted == 0) {
@@ -103,7 +103,7 @@ public long extractEnergy(long amount, Action action) {
             if (toExtract == 0) {
                 return 0;
             }
-            if (action.execute() && !EnergyUnit.FORGE_ENERGY.isOneToOne()) {
+            if (!EnergyUnit.FORGE_ENERGY.isOneToOne()) {
                 //Before we can actually execute it we need to simulate to calculate how much we can actually extract in our other units
                 long simulatedExtracted = storage.extract(toExtract, true);
                 //Convert how much we could extract back to Joules so that it gets appropriately clamped so that for example 1 Joule gets treated