From 600f225d7aed5de4277cc42e8e13e76c1e366c4d Mon Sep 17 00:00:00 2001 From: luisc Date: Fri, 25 Oct 2024 13:45:12 -0500 Subject: [PATCH] Fixed exception when opening scripter on servers. ConfigScript.canScript() should only ever be called on the server side. --- src/main/java/noppes/npcs/items/ItemNpcScripter.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/main/java/noppes/npcs/items/ItemNpcScripter.java b/src/main/java/noppes/npcs/items/ItemNpcScripter.java index 196880778..cb5e7c3ec 100644 --- a/src/main/java/noppes/npcs/items/ItemNpcScripter.java +++ b/src/main/java/noppes/npcs/items/ItemNpcScripter.java @@ -35,13 +35,13 @@ public class ItemNpcScripter extends Item{ @Override public ItemStack onItemRightClick(ItemStack itemStack, World world, EntityPlayer player){ - if(!world.isRemote) + if (!world.isRemote) { + if(!ConfigScript.canScript(player, CustomNpcsPermissions.TOOL_SCRIPTER)){ + player.addChatMessage(new ChatComponentTranslation("availability.permission")); + } return itemStack; - if(!ConfigScript.canScript(player, CustomNpcsPermissions.TOOL_SCRIPTER)){ - player.addChatMessage(new ChatComponentTranslation("availability.permission")); - } else { - CustomNpcs.proxy.openGui(0, 0, 0, EnumGuiType.ScriptEvent, player); } + CustomNpcs.proxy.openGui(0, 0, 0, EnumGuiType.ScriptEvent, player); return itemStack; }