Fix client detection (#20)

Author: Kogepan229

src/main/java/ca/wescook/nutrition/events/EventEatFood.java

@@ -6,7 +6,6 @@
 import net.minecraft.item.ItemBucketMilk;
 import net.minecraftforge.event.entity.player.PlayerUseItemEvent;
 
-import ca.wescook.nutrition.Nutrition;
 import ca.wescook.nutrition.data.PlayerDataHandler;
 import ca.wescook.nutrition.effects.EffectsManager;
 import ca.wescook.nutrition.nutrients.Nutrient;
@@ -44,7 +43,7 @@ public class EventEatFood {
 
     @SubscribeEvent
     public void onFoodStatsChanged(FoodEvent.FoodStatsAddition event) {
-        if (Nutrition.proxy.isClient()) {
+        if (event.player.getEntityWorld().isRemote) { // Client
             // only run if hunger value increases, also ignoring saturation
             int hungerValue = event.foodValuesToBeAdded.hunger;
             if (hungerValue <= 0) return;
@@ -64,9 +63,7 @@ public void onFoodEaten(FoodEvent.FoodEaten event) {
         if (!event.player.getEntityWorld().isRemote) { // Server
             PlayerDataHandler.getForPlayer(event.player)
                 .add(foundNutrients, nutritionValue);
-        }
-
-        if (Nutrition.proxy.isClient()) { // Client
+        } else { // Client
             ClientProxy.localNutrition.add(foundNutrients, nutritionValue);
             // set that food has now been eaten
             ClientProxy.popHungerChange();

src/main/java/ca/wescook/nutrition/proxy/ClientProxy.java

@@ -38,11 +38,6 @@ public void init(FMLInitializationEvent event) {
         }
     }
 
-    @Override
-    public boolean isClient() {
-        return true;
-    }
-
     public static void pushHungerChange(int hungerValue) {
         hungerValues.push(hungerValue);
     }

src/main/java/ca/wescook/nutrition/proxy/CommonProxy.java

@@ -18,8 +18,4 @@ public void init(FMLInitializationEvent event) {
     public void postInit(FMLPostInitializationEvent event) {
         ModHelperManager.postInit();
     }
-
-    public boolean isClient() {
-        return false;
-    }
 }