-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inventory#all doesn't scan the entire inventory #10695
Comments
I think this is "works as intended" due to the documentation on
and continues
I think that second part includes the |
It looks like previously, Paper has added specific API to cover "all slots". There is a Inventory#removeItemAnySlot method to account for the other Inventory#removeItem only removing from the "storage" contents. I think that is the better solution here, adding an "any slot" variant of all the methods that have "storage-specific" versions. |
Expected behavior
Suggested by the docs of the method, it should look up all slots in the entire inventory
Observed/Actual behavior
It only looks up the storage content
Steps/models to reproduce
Well... it's just an inconsistency... soo...
Plugin and Datapack List
unimportent
Paper version
1.20.6-R0.1-SNAPSHOT
Other
docs:
implementation:
The text was updated successfully, but these errors were encountered: