[MC-92759] Luck and Bad Luck don't work with double chests

The effects Luck and Bad Luck don't work when a double chest is opened, but the effects work when a simple chest is opened.

1: Place the file

[media]

in the folder:
.minecraft\saves\ MAP_NAME \data\loot_tables\test\chests\
2: Use the command:
/give @p minecraft:chest{BlockEntityTag:{LootTable:"test:chests/updated_table_loot"}} 1

3: Place and open the chest with effect Luck or Bad Luck.

4: Place and open an another chest (a double chest) with effect Luck or Bad Luck.

________________________________

:light_bulb_on: Look the screenshots in Attachment for the examples.

_{Sorry for my bad english.}

Code analysis by @unknown can be found in this comment.

Linked issues

relates to 2

MC-93483 Spectators opening Loot Table (double)Chests will generate the items inside Resolved

MC-126184 Luck doesn't work with shulker box Resolved

Attachments

1.8.5.json

table_loot.json

updated_table_loot.json

Comments 12

Tan Chin Cheern 2015-11-15T14:49:34Z

Attachment? Can you also post the table_loot.json folder here?

Fenrius 2015-11-15T14:51:43Z

Oops, sorry.

EDIT : Attachments are added. 😉

Tan Chin Cheern 2015-11-15T15:09:44Z

Confirmed.

Skylinerw 2015-12-04T00:45:01Z

Confirmed for 15w49b & 15w50a.

Skylinerw 2015-12-17T15:57:25Z

Confirmed for 15w51b.

2 more comments

Skylinerw 2016-01-20T16:51:58Z

Confirmed for 16w03a.

Skylinerw 2016-01-28T16:40:10Z

Confirmed for 16w04a.

Skylinerw 2016-02-03T17:53:21Z

Confirmed for 16w05a.

Skylinerw 2016-02-17T22:00:43Z

Confirmed for 1.9 pre-release 1.

marcono1234 2016-05-05T22:42:22Z

Please link to this comment in the description

The following is based on a decompiled version of Minecraft 1.9 using MCP 9.24 beta.

The reason for this is that the method net.minecraft.tileentity.TileEntityChest.getStackInSlot(int) generates the loot when you try to open the chest. This method passes null as player opening the chest. Instead the loot should be created when the method net.minecraft.inventory.InventoryLargeChest.createContainer(InventoryPlayer, EntityPlayer) is called by calling the method net.minecraft.tileentity.TileEntityChest.createContainer(InventoryPlayer, EntityPlayer) for the upper and lower chest.