Upstream/Minestom
1
0
Fork 0
mirror of https://github.com/Minestom/Minestom.git synced 2024-12-30 13:08:19 +01:00
Code Issues Projects Releases Wiki Activity

Fix broken cursor with window items packet

Browse Source
This commit is contained in:
Moulberry 2021-09-08 21:50:14 +08:00
parent 025052102d
commit 9e5fadaa0e
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/main/java/net/minestom/server/inventory/Inventory.java
View File

@ -130,7 +130,9 @@ public class Inventory extends AbstractInventory implements Viewable {
*/
@Override
public void update() {
sendPacketToViewers(createNewWindowItemsPacket());
for(Player player : viewers) {
player.getPlayerConnection().sendPacket(createNewWindowItemsPacket(player));
}
}
/**
@ -145,7 +147,7 @@ public class Inventory extends AbstractInventory implements Viewable {
return;
final PlayerConnection playerConnection = player.getPlayerConnection();
playerConnection.sendPacket(createNewWindowItemsPacket());
playerConnection.sendPacket(createNewWindowItemsPacket(player));
}
@NotNull
@ -227,8 +229,8 @@ public class Inventory extends AbstractInventory implements Viewable {
*
* @return a new {@link WindowItemsPacket} packet
*/
private @NotNull WindowItemsPacket createNewWindowItemsPacket() {
return new WindowItemsPacket(getWindowId(), 0, getItemStacks(), ItemStack.AIR);
private @NotNull WindowItemsPacket createNewWindowItemsPacket(Player player) {
return new WindowItemsPacket(getWindowId(), 0, getItemStacks(), cursorPlayersItem.getOrDefault(player, ItemStack.AIR));
}
/**

2
src/main/java/net/minestom/server/network/packet/server/play/SetSlotPacket.java
View File

@ -53,6 +53,6 @@ public class SetSlotPacket implements ServerPacket {
* @return a set slot packet to change a player cursor item
*/
public static @NotNull SetSlotPacket createCursorPacket(@NotNull ItemStack cursorItem) {
return new SetSlotPacket((byte) -1, -1, (short) 0, cursorItem);
return new SetSlotPacket((byte) -1, 0, (short) -1, cursorItem);
}
}