[SOLVED]1.15.2 Items duplicating in player inventory, erratic behavior when interacting with other vanilla containers

[Turtledove]

I've implemented a basic inventory organization system in survival, all the tabs work and it organizes the items successfully across the different containers, all done server-side. However, I've discovered an annoying bug:

 

When players first pick up items (excuse the fact that the slots aren't correctly positioned over the texture, I'm just focused on getting it working first since none of this is permanent), it 

 

a) Doesn't show up in the player hotbar in normal gameplay, outside the inventory.

 

b) After A), when I open the inventory, there's no duplicates, the items are in their correct spot. But when I then open up a crafting table for instance and then open the inventory, the items duplicate and shows up twice in the inventory, in the hot bar and in the first row of the 3x9 space. The following is after:

1. Picking up a stack of 4 potatoes and 1 wheat

2) Opening a furnace, where it correctly shows the items I picked up, and makes the item show up in my hotbar.

3) Exiting the furnace, and opening the inventory, this causes the duplication to occur (it doesn't seem to be a real duplication, just a ghost item).

4) Dropping 2 potatoes with Q

 

There seems to be some client-server problems that I can't pinpoint, since I've implemented this container identical to the default, using the vanilla item handlers. The containers and inventory are custom.

 

 

EDIT: Just tried using the vanilla container instead of my custom implementation, and it works without issue. So something's not right with my container, even though it's identical to the vanilla one except it doesn't have armor slots or crafting tables.

Edited April 28, 2020 by Turtledove

[ChampionAsh5357]

2 hours ago, Turtledove said:

So something's not right with my container

Probably true, you most likely didn't sync the data from the server to the client so the client remembers the items that were originally there without the update. If you have a container, it should be the same process as anything else to add the container to the gui. Telling us something is wrong without showing a reference is also not helpful since we have to guess on what to fix based on prior posts.

[Turtledove]

22 minutes ago, ChampionAsh5357 said:

Probably true, you most likely didn't sync the data from the server to the client so the client remembers the items that were originally there without the update. If you have a container, it should be the same process as anything else to add the container to the gui. Telling us something is wrong without showing a reference is also not helpful since we have to guess on what to fix based on prior posts.

It's just because there's nothing to reference, the container is almost exactly the vanilla one, and the inventory works perfectly fine when I don't remove the code adding the armor and crafting slots. I've registered a containertype and the screen, and it's called on the server.

 

I guess my brain is just scrambled by the fact that it only works when the armor slot and crafting slots are present, which kind of defeats the purpose of a custom inventory, and I was trying to figure out why that is, moreso than the code itself which again, was just copy/pasted from the vanilla one for testing.

Edited April 28, 2020 by Turtledove

[Turtledove]

I may just move it off somewhere else on the screen and make it inaccessible, feels kind of like a hack job but it's what works.

[Turtledove]

Figured out why it's acting this way, it actually had nothing to do with the way I'd handled the client-server, removing the armor and crafting slots fucks with the slot indices, kind of obvious but I overlooked it.