[Solved] Help with quickMoveStack

[JCox]

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 October 26, 2022 by JCox
Mark as solved

[warjort]

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 October 26, 2022 by warjort

[JCox]

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?
 

[warjort]

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 October 26, 2022 by warjort

[JCox]

Yes thank you. That has worked.