Jump to content

Recommended Posts

Posted (edited)

When you subclass AbstractContainerMenu you have to implement quickMoveStack.
I don't understand the int paremeter. I assume it is the index of the slot you clicked, but are there not two inventories, the player and the custom container?
I don't know how to reference an index slot on the custom inventory and move an item to a index slot on the player inventory.
Would someone mind explaining the second parameter for this method.

Thanks.
 

Edited by JCox
Mark as solved
Posted (edited)

It is the slot that was clicked.

You decide how the slots are layed out in your constructor. Though vanilla has a "convention" of sorts.

e.g. see BeanconMenu which puts the payment slot at slot 0 then the player's main inventory in the next 27 slots (1 to 27) and finally the 9 hotbar slots (28 to 37).

You can then see these numbers used in quickMoveStack() where it decides the priority of where things should go based on the slot clicked.

 

If you look at the top of BeaconMenu you will also see these same constants defined.

This is a failure of the deobfuscation where obviously the real minecraft code used those symbolic constants instead of the numbers.

You can do the same to make the code more readable and less error prone.

 

If you want a more complicated example, see the BrewingStandMenu which has different customs slots with mayPlace() tests. 

Edited by warjort

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Posted

Thanks so if you have two slots in your custom inventory and add them at index 0 and index 1. And then you start adding your player inventory slots (starting at index 0)
The first hotbar item slot in the quickMoveStack method will be at index 2?
 

Posted (edited)

If that's the way you do it in your constructor.

Except I think you mean inventory slot not hotbar slot which will be at 27+2=29

Edited by warjort

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

  • JCox changed the title to [Solved] Help with quickMoveStack

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



×
×
  • Create New...

Important Information

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