Backpack dupe items
ybw0014 opened this issue · 10 comments
❗ Checklist
- I am using the official english version of Slimefun and did not modify the jar.
- I am using an up to date "DEV" (not "RC") version of Slimefun.
- I am aware that issues related to Slimefun addons need to be reported on their bug trackers and not here.
- I searched for similar open issues and could not find an existing bug report on this.
📍 Description
Backpacks can be used to dupe items.
Two players, a backpack, and backpack upgrade materials are needed.
📑 Reproduction Steps
- Player A holds a backpack in main hand, and a portable crafting table in off hand.
- Player A opens a backpack, and put items that need to be duped (e.g. 64x diamonds).
- Player A press F to switch backpack & portable, and switch back. (Not closing backpack)
- Player A drops the backpack, without closing the backpack.
- Player B picks up the backpack from Player A, upgrade it in enhanced crafting table.
- Player B opens the backpack, gets the 64x diamonds.
- Player A gets 64x diamonds, in total, 128x diamonds.
💡 Expected Behavior
The opened backpacks are closed.
📷 Screenshots / Videos
📜 Server Log
N/A
📂 /error-reports/
folder
N/A
💻 Server Software
Paper
🎮 Minecraft Version
1.19.x
⭐ Slimefun version
DEV - 1027
🧭 Other plugins
No
Thanks for the detailed bug report!
The issue should be fixed within the latest commit.
Players should no longer be able to swap the backpack from hand to another while having the backpack open.
Additionally, a safeguard was added to close the backpack inventory before upgrading its size.
bug continues in version DEV - 1033 (git 5455fb0)
dupe is still working I can do it right now if I need to, in the video I show that I'm using version 1033
please fix this, people are abusing it, and if I remove it from my server I will lose a lot of items.
I hope it is fixed now, haven't tested it yet. Let me know if the next build will have this issue fixed.
I suspect the reason the previous fix wasn't working was due to a threading issue.