[1.12.2] Work with inventory

[NoName_]

How do I move an item in inventory from cell X to cell Y?
How do I move an item from Inventory X to Box Y of an open chest?
I'm thinking of using the Robot class, i.e. Simulate clicking the mouse buttons, but then you need to know the coordinates of the cells. I do not know how to get them.

I tried to understand the source code of the Inventory Tweaks mod, but I did not understand how it works. It seems, too, with the help of simulation

[deerangle]

Please clarify: What exactly are you trying to acheive. What do you mean by "move an item in inventory". Are you making a custom inventory, an addon for inventory tweaks or perhaps even something totally different?

[NoName_]

Example:
I have potatoes in the inventory in cell X.
My task is to move the potatoes from cell X to cell Y.
The inventory is standard, you just need to move items in it. From one cell to another.

 

It is important that all actions should take place only on the client side

 

[Cadiboo]

get the stack in slot X with inventory#getStackInSlot(X);

COPY THE STACK

set the stack with inventory#setStackInSlot(Y, copyOfX);

clear the old stack in slot X with inventory#setStackInSlot(X, ItemStack.EMPTY.copy())

 

Make sure to do additional checking king to make sure the itemstack can fit/is allowed to be in that slot etc.

Take a look at Container#transferStackInSlot

[Draco18s]

You don't need to clone the empty stack.

[Cadiboo]

  On 10/1/2018 at 3:59 AM, Draco18s said:

You don't need to clone the empty stack.

Expand  

What if some code modifys that stack? Like merges it with a stack that isn’t empty? Won’t that modify ItemStack.EMPTY then?

[Draco18s]

  On 10/1/2018 at 4:02 AM, Cadiboo said:

What if some code modifys that stack? Like merges it with a stack that isn’t empty? Won’t that modify ItemStack.EMPTY then?

Expand  

You can't. ItemStack.EMPTY is meant to be immutable. That's why its a static reference and named the way its named.

 

Its item is Air (and you can't change a stack's item) and Air is always empty.

[Cadiboo]

  On 10/1/2018 at 4:08 AM, Draco18s said:

You can't. ItemStack.EMPTY is meant to be immutable. That's why its a static reference and named the way its named.

 

Its item is Air (and you can't change a stack's item) and Air is always empty.

Expand  

Isn’t it’s item null? Point taken that you can’t change the item though

[Draco18s]

  On 10/1/2018 at 4:58 AM, Cadiboo said:

Isn’t it’s item null?

Expand  

That it is, but null has always been Minecraft's air. Block.AIR was null at one point too.

[NoName_]

  On 10/1/2018 at 3:40 AM, Cadiboo said:

get the stack in slot X with inventory#getStackInSlot(X);

COPY THE STACK

set the stack with inventory#setStackInSlot(Y, copyOfX);

clear the old stack in slot X with inventory#setStackInSlot(X, ItemStack.EMPTY.copy())

Expand  

 

As I understand it, this will not work?

[Draco18s]

  On 10/1/2018 at 2:23 PM, NoName_ said:

As I understand it, this will not work?

Expand  

And why do you think that?

[NoName_]

I don't know. So it will be?))

Will it  work on the client side?

[NoName_]

    Minecraft.getMinecraft().player.inventory.setInventorySlotContents(0, Minecraft.getMinecraft().player.inventory.getStackInSlot(i).copy());
    Minecraft.getMinecraft().player.inventory.setInventorySlotContents(i, ItemStack.EMPTY);

It does not work on a server. A server does not know that I moved the item. I understand me need to use the exchange of packages?

 

Edited October 1, 2018 by NoName_

[larsgerrits]

  On 10/1/2018 at 7:03 PM, NoName_ said:

    Minecraft.getMinecraft().player.inventory.setInventorySlotContents(0, Minecraft.getMinecraft().player.inventory.getStackInSlot(i).copy());
    Minecraft.getMinecraft().player.inventory.setInventorySlotContents(i, ItemStack.EMPTY);

Expand  

That's because your using client-side only code.

 

  On 10/1/2018 at 7:03 PM, NoName_ said:

need to use the exchange of packages?

Expand  

What is "exchange of packets"?

From a user perspective, when do you want this to happen in the inventory? 

[NoName_]

  On 10/1/2018 at 7:10 PM, larsgerrits said:

What is "exchange of packets"?

Expand  

Packets? I do not know how to properly call it. (NetHandlerPlayClient  and NetHandlerPlayServer)

  On 10/1/2018 at 7:10 PM, larsgerrits said:

From a user perspective, when do you want this to happen in the inventory? 

Expand  

I need a method that changes the place of an item in the inventory.
I do with the example of a hoe. I'm looking for a hoe in the inventory, when I found it, I move it to the first location of the inventory.

 

for (int i = 1; i < 36; i++) {
    if(Minecraft.getMinecraft().player.inventory.getStackInSlot(i).getItem() instanceof ItemHoe){
        Minecraft.getMinecraft().player.inventory.setInventorySlotContents(0, Minecraft.getMinecraft().player.inventory.getStackInSlot(i).copy());
        Minecraft.getMinecraft().player.inventory.setInventorySlotContents(i, ItemStack.EMPTY);
        break;
    }
}

[Cadiboo]

I recommend placing some breakpoints, manually doing this & seeing what methods get called. Some breakpoints in CPackets and the CLIENTSIDE containers (as your making a client side mod) would be a good place to start. Make sure that your not looking at the servers handling when debugging

[larsgerrits]

And with "from a user perspective" I meant: what should the user to do run this action? For example: the user presses a key or clicks a button in a GUI.

[NoName_]

User does not do anything. The mod itself under certain circumstances moves the item in the inventory. 

[Cadiboo]

What are the circumstances

[NoName_]

  On 10/2/2018 at 11:28 PM, Cadiboo said:

What are the circumstances

Expand  

For example, if the tool breaks. 
Why is it important?

[Animefan8888]

  On 10/3/2018 at 2:04 PM, NoName_ said:

For example, if the tool breaks. 
Why is it important?

Expand  

What version are you doing this for? And is this only for the client?

[NoName_]

  On 10/3/2018 at 2:13 PM, Animefan8888 said:

What version are you doing this for? And is this only for the client?

Expand  

1.12.2 

Yes, only on the client, because it is necessary for the mod to work on other servers

[Animefan8888]

  On 10/3/2018 at 2:15 PM, NoName_ said:

1.12.2 

Yes, only on the client, because it is necessary for the mod to work on other servers

Expand  

And why not just use Inventory Tweaks? It has the same feature you are requesting from us.

[NoName_]

  On 10/3/2018 at 2:19 PM, Animefan8888 said:

And why not just use Inventory Tweaks? It has the same feature you are requesting from us.

Expand  

I do not understand how it works. This is not good. Besides, why add the whole mod if I need one function. In addition, it would not be bad to understand this issue.

[Animefan8888]

  On 10/3/2018 at 2:22 PM, NoName_ said:

I do not understand how it works. This is not good. In addition, it would not be bad to understand this issue.

Expand  

What you need to do from your original implementation is let the server know that you changed them.

 

  On 10/3/2018 at 2:22 PM, NoName_ said:

Besides, why add the whole mod if I need one function. 

Expand  

Because that one function is already done and the inventory sorting function is a god send.