Jump to content

Open Vanilla GUI w/ item right click?


GravityWolf

Recommended Posts

Hello! I am trying to get an item to get items to open chests, furnaces, and crafting tables. I can't seem to get it to work. Here is the code I am trying to use:

 

public boolean onItemUse(World par1World, int par2, int par3, int par4, EntityPlayer par5EntityPlayer, int par6, float par7, float par8, float par9)
    {
        if (par1World.isRemote)
        {
            return true;
        }
        else
        {
            par5EntityPlayer.displayGUIWorkbench(par2, par3, par4);
            return true;
        }
        
    }

 

I'm pretty sure it's not working cause this never gets called, but I can't put this in onItemRightClick()...anyone have any ideas?

Link to comment
Share on other sites

Alrighty. I got this far:

 

	@Override
 public boolean onItemUse(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, World par3World, int x, int y, int z, int par7, float par8, float par9, float par10)
        {
  if (!par3World.isRemote)
                {
                        return true;
                }
                else
                {
                        par2EntityPlayer.displayGUIWorkbench(x, y, z);
                        return true;
                }
        }
}

 

It only works whilst right clicking on the ground, though. Any ideas?

Link to comment
Share on other sites

Alrighty. I got this far:

 

	@Override
 public boolean onItemUse(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, World par3World, int x, int y, int z, int par7, float par8, float par9, float par10)
        {
  if (!par3World.isRemote)
                {
                        return true;
                }
                else
                {
                        par2EntityPlayer.displayGUIWorkbench(x, y, z);
                        return true;
                }
        }
}

 

It only works whilst right clicking on the ground, though. Any ideas?

 

Try using onItemRightClick() method... And also, why are you returning true in both statements? I am thinking you might want to make on e false

I am Mew. The Legendary Psychic. I behave oddly and am always playing practical jokes.

 

I have also found that I really love making extremely long and extremely but sometimes not so descriptive variables. Sort of like what I just did there xD

Link to comment
Share on other sites

Alrighty. I got this far:

 

	@Override
 public boolean onItemUse(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, World par3World, int x, int y, int z, int par7, float par8, float par9, float par10)
        {
  if (!par3World.isRemote)
                {
                        return true;
                }
                else
                {
                        par2EntityPlayer.displayGUIWorkbench(x, y, z);
                        return true;
                }
        }
}

 

It only works whilst right clicking on the ground, though. Any ideas?

 

Try using onItemRightClick() method... And also, why are you returning true in both statements? I am thinking you might want to make on e false

 

This doesn't work at all:

 

public ItemStack onItemRightClick(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, World par3World, int x, int y, int z, int par7, float par8, float par9, float par10)
        {
  if (!par3World.isRemote)
                {
                        return null;
                }
                else
                {
                        par2EntityPlayer.displayGUIWorkbench(x, y, z);
                        return null;
                        
                }
        }
}

Link to comment
Share on other sites

No... Just.... No...... Do you even know what a null does???? IT RETURNS A NULL POINTER EXCEPTION!!!!! You should return the par1ItemStack....

I am Mew. The Legendary Psychic. I behave oddly and am always playing practical jokes.

 

I have also found that I really love making extremely long and extremely but sometimes not so descriptive variables. Sort of like what I just did there xD

Link to comment
Share on other sites

try taking off the if(!world.isRemote)

 

I don't think client EntityPlayers can open GUIs

 

No, no they can't.... No wonder I thought there was something wrong with this xD

 

Thanks for clearing my mind :D

I am Mew. The Legendary Psychic. I behave oddly and am always playing practical jokes.

 

I have also found that I really love making extremely long and extremely but sometimes not so descriptive variables. Sort of like what I just did there xD

Link to comment
Share on other sites

Are you using onItemUse or onItemRightClick ? use both just to be safe, if it works, undo one of them to figure which one worked :P

 

onItemUse is just when you right click a block with an item, so that's not really what I want. I am using onItemRightClick, but onItemUse did work when I tried it. Just wasn't right, though.

Link to comment
Share on other sites

navigating through some the minecraft classes I've found out what was wrong.

 

you need to damage the ItemStack, so a method called tryUseItem will return true. This method is in a class called net.minecraft.item.ItemInWorldManager. If it returns false it is as if you failed on using the item. So, you have to set the item maxDamage (using the method setMaxDamage of the Item, maybe in the constructor), which is the maximum number of times you can use the item. If you don't set this, it will be zero, and tryUseItem will return false even earlier.

 

The item will break eventually, but it's just a matter of setting a very very large number if you don't want it to break soon. (Like Integer.MAX_VALUE if you want it to last reeeally long)

 

This is what you have to do:

 

public *nameOfYourConstructor*(*parameter list of your constructor*)
{
     super(*parameter list of superclass constructor*);
     *whatever else your constructor does*
     setMaxDamage(*whatever int you want*);
}

@Override
public ItemStack onItemRightClick(ItemStack par1ItemStack, World world, EntityPlayer player)
{
     if(world.isRemote)
     {
          player.displayGUIWorkbench((int)player.posX, (int)player.posY, (int)player.posZ);
          setDamage(par1ItemStack, par1ItemStack.getItemDamage() + 1);
     }
     return par1ItemStack;
}

Link to comment
Share on other sites

navigating through some the minecraft classes I've found out what was wrong.

 

you need to damage the ItemStack, so a method called tryUseItem will return true. This method is in a class called net.minecraft.item.ItemInWorldManager. If it returns false it is as if you failed on using the item. So, you have to set the item maxDamage (using the method setMaxDamage of the Item, maybe in the constructor), which is the maximum number of times you can use the item. If you don't set this, it will be zero, and tryUseItem will return false even earlier.

 

The item will break eventually, but it's just a matter of setting a very very large number if you don't want it to break soon. (Like Integer.MAX_VALUE if you want it to last reeeally long)

 

This is what you have to do:

 

public *nameOfYourConstructor*(*parameter list of your constructor*)
{
     super(*parameter list of superclass constructor*);
     *whatever else your constructor does*
     setMaxDamage(*whatever int you want*);
}

@Override
public ItemStack onItemRightClick(ItemStack par1ItemStack, World world, EntityPlayer player)
{
     if(world.isRemote)
     {
          player.displayGUIWorkbench((int)player.posX, (int)player.posY, (int)player.posZ);
          setDamage(par1ItemStack, par1ItemStack.getItemDamage() + 1);
     }
     return par1ItemStack;
}

 

Thanks, it brings up the GUI, but now I have another problem =/

When I click on the item, it moves right back to the slot it was on previously. Any ideas?

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

    • version used : minecraft forge 1.20.1 Logs : https://bytebin.lucko.me/rtG4EA0vfb hi! so when i try to create a new minecraft world in single player, it just crashes?? does anyone has an idea what could be causing this ?
    • Hey today I was trying to play in 1.21 modpack that I made on fabric (my version of fabric is in 0.16.0) with this list of mods : - Entity Model Features (by traben_0) - Entity Texture Features (by traben_0) - Advanced Chat (by Wesley8081) - Ambient Environment (by Jaredlll08) - Ambient Souds 6 (by CreativeMD) - Animatica (by FoundationGames) - Armor Statues (by Fuzs) - Auto HUD (by Crendgrim) - Bad Packets (by badasintended) - Balm (by BlayTheNinth) - Better Beds (by Motschen) - Better Chunk Loading (by someaddon) - Better Compatibility Checker (by Gaz-) - Better FPS [Render Distance] (by someaddon) - Better Ping Display (by Quintinity) - Better Statistics Screen (by TheCSDev) - Better Armor HUD (by efekos) - Better F3 ( by cominixo) - Blur + (by Motschen) - Bobby (by Johni0702) - Cat Jam (by shmove) - Catalogue (by MrCrayfish) - Chat Heads (by dzwdz) - Cherished Worlds (by TheIllusiveC4) - Chunk Sending (by someaddon) - Chunks Fade In (by kerudion) - Chunky (by pop4959) - Cloth Config (by shedaniel) - Clumps (by Jaredlll08) - Cobweb (CrystalSpider) - Collective (by Serilum) - Configured (by MrCrayfish) - Connectivity (by someaddon) - Continuity (by Pepper_Bell) - Creative Core (by CreativeMD) - Cupboard (by someaddon) - Cut Through (by Fuzs) - Dark Loading Screen (by Neecko5b84) - Debugify (by XanderIsDev) - Double Doors (by Serilum) - Dynamic FPS (by juliand665) - Dynamic Lights (by Tschipcraft) - Easy Anvils (by Fuzs) - Easy Magic (by Fuzs) - Easy Shulker Boxes (by Fuzs) - Eating Animation (by theone-ss) - Enhanced Attack Indicator (by Mineash) - Enhanced Block Entities (by FoundationGames) - Enhanced Visuals (by CreativeMD) - Entity Culling (by tr7zw) - Environmental Water (by AgentMindStorm) - Essential (by SparkUniverse_) - Extra Thicc Packets (by JaselUmena) - Fabric (by modmuss50) - Fabric Language Kotlin (by modmuss50) - Falling Leaves (by RandomMcSomethin) - Fancy Block Particles (by Hantonik) - Fancy Health Bar (by efekos) - Farsight (by someaddon) - Fast IP Ping (by Fallen_Breath) - Ferrite Core (by  malte0811) - Fish On The Line (by Serilum) - Fix GPU Memory Leak  (by someaddon) - Forge Config API Port (by Fuzs) - FPS Reducer (by bre2el) - Freecam (by hashalite) - Fzzy Config (by fzzyhmstrs) - Get It Together (by bl4ckscor3) - Global Packs (by JTK222) - Harvest With Ease (by CrystalSpider) - Ichuntil Util (by iChun) - Immediately Fast (by RaphilMC) - Immersive UI (by SSKirillSS) - Indium (by comp500) - Inventory HUD+ (by dmitrylovin) - Iris Shaders (by coderbot) - Item Physic Full (by CreativeMD) - Jade (by Snownee) - Journey Map (by techbrew) - Just Enough Breeding (by Christofmeg) - Just Enough Items (by mezz) - Just Enough Professions (by Mrbysco) - Language Reload (by Jerozgen) - Leaves Be Gone (by Fuzs) - Let Me Despawn (by frikinjay) - Let Sleeping Dogs Lie (by iChun) - Lithium (by JellySquid) - Loading Backgrounds (by Lupin) - Mineblock's Repeated Utilities (by itsmineblock11) - Menulogue (by MrCrayfish) - Mob Plaques (by Fuzs) - Mod Menu (by Prospector) - Model Gap Fix (by MehVahdJukaar) - Mouse Tweaks (by YaLTeR) - Nether Portal Fix (by BlayTheNinth) - Nicer Skies (by ztereohype) - Noisium (by Steveplays28) - Not Enough Animations (by tr7zw) - Opti GUI (by opekope2) - Packet Fixer (by TonimatasDEV) - Particle Core (by fzzyhmstrs) - Passive Shields (by Serilum) - Pick Up Notifier (by Fuzs) - Ping Wheel (by LukenSkyne) - Puzzle (by Motschen) - Puzzles LIB (by Fuzs) - Reese's Sodium Options (by FlashyReese) - Resource Pack Overrides (by Fuzs) - Server Country Flags (by khajiitos) - Server Performance (by someaddon) - Skin Layer 3D (by tr7zw) - Sodium (by JellySquid) - Sodium Extra (by FlashyReese) - Sodium Shadowy Path Blocks (by Rynnavinx) - Sound Physics Remastered (by henkelmax) - Sounds (by itsmineblock11) - Spark (by lucko) - Stack Refill (by Serilum) - Structure Essentials (by someaddon) - TCD Commons API (by TheCSDev) - Text Placeholder API (by Patbox) - Villager Names (by Serilum) - Visual Overhaul (by Motschen) - Visual Workbench (by Fuzs) - Visuality (by PinkGoosik) - Wakes (by Goby56) - Wavey Capes (by tr7zw) - YDM'S Weapon Master  (by YourDailyModderx) - Yet Another Config LIB (by XanderIsDev) - Zoomify (by XanderIsDev) And When I load it it work, I can create a world and even let him open forever at least I didn't move cause if so the game just crash and tell me this message :   Encountered an unexpected exception net.minecraft.class_148: Ticking entity at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:1025) at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:912) at net.minecraft.class_1132.method_3748(class_1132.java:114) at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:697) at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:281) at java.base/java.lang.Thread.run(Thread.java:1583) Caused by: java.lang.NullPointerException: Cannot invoke "net.minecraft.class_4706.isEmpty()" because "tickets" is null at net.minecraft.class_3204.canNoneExpire(class_3204.java:1034) at net.minecraft.class_3204.handler$dln000$lithium$unregisterExpiringTicket(class_3204.java:1075) at net.minecraft.class_3204.method_17645(class_3204.java:172) at net.minecraft.class_3204.method_17292(class_3204.java:194) at net.minecraft.class_3215.method_17300(class_3215.java:443) at com.betterchunkloading.chunk.PlayerChunkData.checkPrediction(PlayerChunkData.java:406) at com.betterchunkloading.chunk.PlayerChunkData.checkDirection(PlayerChunkData.java:353) at com.betterchunkloading.chunk.PlayerChunkData.trackPlayerMovement(PlayerChunkData.java:177) at com.betterchunkloading.chunk.PlayerChunkData.onChunkChanged(PlayerChunkData.java:130) at com.betterchunkloading.event.EventHandler.onPlayerTick(EventHandler.java:200) at net.minecraft.class_3222.handler$zed000$betterchunkloading$onPlayerTick(class_3222.java:2618) at net.minecraft.class_3222.method_5773(class_3222.java:564) at net.minecraft.class_3218.mixinextras$bridge$method_5773$202(class_3218.java) at net.minecraft.class_3218.wrapOperation$eei000$puzzleslib$tickNonPassenger(class_3218.java:7797) at net.minecraft.class_3218.method_18762(class_3218.java:770) at net.minecraft.class_1937.method_18472(class_1937.java:498) at net.minecraft.class_3218.method_31420(class_3218.java:408) at net.minecraft.class_5574.method_31791(class_5574.java:54) at net.minecraft.class_3218.method_18765(class_3218.java:372) at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:1021) ... 5 more Sorry guys if it obvious but actually I'm an old player of Minecraft but I'm new in modding so yup. But Thanks everyone by advance to help me with this.
    • These forums are for Minecraft Forge, not Fabric. You would probably be better served posting this question wherever fabric support is given, most likely their discord server.
    • Add the new latest.log (logs-folder) with sites like https://mclo.gs/ and paste the link to it here
    • Update: I tried to reproduce the bug and found out that if I place a flare on a block and then dismantle it with the pickarang (Quark), the flare gets no NBT Tag after getting assigned an empty one through the offhand.
  • Topics

×
×
  • Create New...

Important Information

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