Jump to content

[SOLVED][1.7.10] Gui only opens via Keybind if it is the held item


p455w0rd

Recommended Posts

Well, the gui Open find via Keybind, but only when it is the currently selected item in hotbar. I need to to be opened whenever the player presses corresponding keybind.

 

Project @ GitHub: https://github.com/p455w0rd/WirelessCraftingTerminal

 

I did try replacing KeyInputEvent with ClientTickEvent and responding only on Phase.END, and it does open the Gui, but it seems only for a single tick before closing. If you want more of the code I can provide it, but there is a lot and I figured this would be enough to figure out where I'm going wrong. The container/gui are fine since they open fine via right-click and via keybind when the item is the current player#getHeldItem. I have tested this using deobf server/client as well as single player with same result.

Link to comment
Share on other sites

I can't find where you actually handle the packet.  You send the PacketOpenWirelessTerm to the server and it gets picked up...where?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

I can't find where you actually handle the packet.  You send the PacketOpenWirelessTerm to the server and it gets picked up...where?

 

Yeah, figured someone would want more info..finally pushed it to github @ https://github.com/p455w0rd/WirelessCraftingTerminal - this code is super messy atm cuz of changes i made mid-way that i still have to clean up..to answer your question, it's all handler in net.p455w0rd.wirelesscraftingterminal.core.sync.network.NetworkHandler which is register in main mod file @ net.p455w0rd.wirelesscraftingterminal.common.WirelessCraftingTerminal

 

thx Draco

Link to comment
Share on other sites

btw, this is AE2's packet handling system..I've just adapted it

 

Jesus mother fucking Christ in a hand basket to Hell this is a complicated network system.

 

And on top of that, a complicated GUI setup.

 

AFAICT your problem lies here somewhere:

https://github.com/p455w0rd/WirelessCraftingTerminal/blob/master/src/java/net/p455w0rd/wirelesscraftingterminal/common/container/ContainerWirelessCraftingTerminal.java#L216-L230

 

But I can't tell because of the giant interconnected web of dependencies.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

btw, this is AE2's packet handling system..I've just adapted it

 

Jesus mother fucking Christ in a hand basket to Hell this is a complicated network system.

 

 

lol you can thank AlgorythmX2

 

I have cleaned the code up quite a bit, tho I still need to do some merging. I'll look into the container. Baffles me that it works perfectly when in-hand. The only refences to the player's hand(getHeldItem) are in RandomUtils#getWirelessTerm, which search the player's inventory for a wireless terminal and returns the first one. The exception is if a player is holding a wireless crafting terminal. That one takes precidence since ppl like Soaryn would make 2 terminals on 2 diff networks just to show a mod is broken xD. I have removed the code the prefers the held item to test and it makes no difference....Thanks for at least trying tho

Link to comment
Share on other sites

THANK YOU SO MUCH! I forgot bout Container#setValidConainter, which is, I assume, what you were thinking..if so, you were correct =D

 

Some bit in or about there, yes.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • I am following a tutorial for modding minecraft 1.20.1, and while running the mod with the minecraft of the idea, no errors was present but, when I switched to real minecraft, it started to want a default constructor "public ZeroToAutomation()" instead of the one recomended by the video "public ZeroToAutomation(FMLJavaModLoadingContext context) {...}", and I can't use the default constructor because I need the FMLJavaModLoadingContext to register the blocks and items.   The problem is at: "net/myself/zerotoautomation/ZeroToAutomation.java" the file contains the default constructor, "Logger"s, and the code for the mod the defaut constructor was added after an error requesting it the Loggers was used to see which methods/constructors was used by the Launcher   things that I tried to fix the issue: remove the default constructor, but an error appeared "net.myself.zerotoautomation.ZeroToAutomation.<init>()", and nothing else on the class runs adding an "public static void init()" method, but it changed nothing   The mod files: https://github.com/Matwaua/Zero_To_Automation/tree/not-finished
    • It is an issue/conflict with distanthorizons
    • I have been trying to combat this issue between oculus and embeddium and to no avail... I have already tried a few online tutorials on how to get this working but it doesn't work at all. https://pastebin.com/geQgifjc
    • wow you sound like fun to play with
    • Instale el mod en mi servidor y este se queda iniciando por horas, en la consola me habla de algo de permisos y trate de seguir la ruta que me da pero no existe.   [13:05:32 INFO]: CustomNPC Permissions available: [13:05:32 INFO]: customnpcs.edit.blocks [13:05:32 INFO]: customnpcs.edit.villager [13:05:32 INFO]: customnpcs.global.bank [13:05:32 INFO]: customnpcs.global.dialog [13:05:32 INFO]: customnpcs.global.faction [13:05:32 INFO]: customnpcs.global.linked [13:05:32 INFO]: customnpcs.global.naturalspawn [13:05:32 INFO]: customnpcs.global.playerdata [13:05:32 INFO]: customnpcs.global.quest [13:05:32 INFO]: customnpcs.global.recipe [13:05:32 INFO]: customnpcs.global.transport [13:05:32 INFO]: customnpcs.npc.advanced [13:05:32 INFO]: customnpcs.npc.ai [13:05:32 INFO]: customnpcs.npc.clone [13:05:32 INFO]: customnpcs.npc.create [13:05:32 INFO]: customnpcs.npc.delete [13:05:32 INFO]: customnpcs.npc.display [13:05:32 INFO]: customnpcs.npc.freeze [13:05:32 INFO]: customnpcs.npc.gui [13:05:32 INFO]: customnpcs.npc.inventory [13:05:32 INFO]: customnpcs.npc.reset [13:05:32 INFO]: customnpcs.npc.stats [13:05:32 INFO]: customnpcs.scenes [13:05:32 INFO]: customnpcs.soulstone.all [13:05:32 INFO]: customnpcs.spawner.create [13:05:32 INFO]: customnpcs.spawner.mob [13:05:32 INFO]: customnpcs.tool.mounter [13:05:32 INFO]: customnpcs.tool.nbtbook [13:05:32 INFO]: customnpcs.tool.pather [13:05:32 INFO]: customnpcs.tool.scripter
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.