Fix Hologram Projector Shift-Use opening both GUIs (#46)

wlhlm · web-flow · commit 7382db1bbd6f · 2025-09-14T19:00:59.000+02:00
diff --git a/src/main/java/com/gtnewhorizon/structurelib/item/ItemConstructableTrigger.java b/src/main/java/com/gtnewhorizon/structurelib/item/ItemConstructableTrigger.java
@@ -29,9 +29,10 @@ public ItemConstructableTrigger() {
     @Override
     public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player) {
         if (getMovingObjectPositionFromPlayer(world, player, true) == null) {
-            if (!world.isRemote) {
+            // channel gui is opened from server and config gui from client
+            if (!world.isRemote && !player.isSneaking()) {
                 player.openGui(StructureLib.instance(), 0, world, player.inventory.currentItem, 0, 0);
-            } else if (player.isSneaking()) {
+            } else if (world.isRemote && player.isSneaking()) {
                 StructureLib.instance().proxy().displayConfigGUI("registries");
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -29,9 +29,10 @@ public ItemConstructableTrigger() {`
`29`	`29`	`@Override`
`30`	`30`	`public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player) {`
`31`	`31`	`if (getMovingObjectPositionFromPlayer(world, player, true) == null) {`
`32`		`- if (!world.isRemote) {`
	`32`	`+ // channel gui is opened from server and config gui from client`
	`33`	`+ if (!world.isRemote && !player.isSneaking()) {`
`33`	`34`	`player.openGui(StructureLib.instance(), 0, world, player.inventory.currentItem, 0, 0);`
`34`		`- } else if (player.isSneaking()) {`
	`35`	`+ } else if (world.isRemote && player.isSneaking()) {`
`35`	`36`	`StructureLib.instance().proxy().displayConfigGUI("registries");`
`36`	`37`	`}`
`37`	`38`	`}`