Fix the first achievement that's needed for all other achievements no…

…t being called serverside #1317
SlimeKnights · Feb 22, 2015 · dd56e0d · dd56e0d
1 parent f634389
commit dd56e0d
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/main/java/tconstruct/tools/items/Manual.java b/src/main/java/tconstruct/tools/items/Manual.java
@@ -8,6 +8,7 @@
 import mantle.client.gui.GuiManual;
 import mantle.items.abstracts.CraftingItem;
 import net.minecraft.entity.player.EntityPlayer;
+import net.minecraft.entity.player.EntityPlayerMP;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.StatCollector;
 import net.minecraft.world.World;
@@ -29,17 +30,23 @@ public Manual()
     }
 
     @Override
-    @SideOnly(Side.CLIENT)
     public ItemStack onItemRightClick (ItemStack stack, World world, EntityPlayer player)
     {
         TAchievements.triggerAchievement(player, "tconstruct.beginner");
 
-        Side side = FMLCommonHandler.instance().getEffectiveSide();
+        if(world.isRemote) {
+            openBook(stack, world, player);
+        }
+        return stack;
+    }
+
+    @SideOnly(Side.CLIENT)
+    public void openBook(ItemStack stack, World world, EntityPlayer player) {
         player.openGui(TConstruct.instance, mantle.client.MProxyClient.manualGuiID, world, 0, 0, 0);
         FMLClientHandler.instance().displayGuiScreen(player, new GuiManual(stack, getData(stack)));
-        return stack;
     }
 
+
     private BookData getData (ItemStack stack)
     {
         switch (stack.getItemDamage())