Shift-clicking within a cLuaWindow does not resend player inventory if the OnClicked callback calls CloseWindow

[sleirsgoevy]

Client version: 1.8.9
Server OS: Linux
Cuberite Commit id: 4fe144d

Expected behavior

OnClicked callback cancels the shift-click. The item stays in the cLuaWindow inventory, player inventory is unchanged.

Actual behavior

The item remains in the player inventory, but only client-side.

Steps to reproduce the behavior

1. Create a cLuaWindow with a SetOnClicked callback that closes the window server-side and returns true
2. Use SetSlot() to put an item inside the window
3. Use OpenWindow() to open the window on the client
4. On the client, shift-click on the item

Server log

Irrelevant.

[sleirsgoevy]

For plugin writers that come across this: if you want your chest UI to have a close button, arrange for the actual close to happen on the next tick.