Skip to content
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 items lost using anvil GUI #12

Closed
Erlebsyk opened this issue Feb 28, 2020 · 2 comments
Closed

Inventory items lost using anvil GUI #12

Erlebsyk opened this issue Feb 28, 2020 · 2 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@Erlebsyk
Copy link

Description
I've noticed a bug where inventory items are permanently lost if the anvil GUI is not used correctly. If any item is shift-cliked while using the anvil GUI, the item is inserted into the second slot and can no longer be returned to the player. Example video.

Impact
This is not an issue with normal use, however it can be very unfortunate if valuable items are lost if someone accidentally shift-clicks one of their items.

Steps to reproduce
Open GUI, add new waypoint or folder to open anvil GUI, then shift-click an item within the inventory to insert it into the second anvil slot. It is also possible to insert an item into the second anvil slot by repeatedly left-clicking it with a selected item. The inserted item is then lost.

System
Java 1.8
Minecraft version 1.15.2
Paper latest version (for now anyway): Paper-121 (API version 1.15.2-R0.1-SNAPSHOT)
Plugin version: 2.6.1

Error logs
No logs or errors displayed in console.

@Sytm
Copy link
Owner

Sytm commented Feb 28, 2020

You should report that bug to WesJD/AnvilGUI, as it is his library I am using for the anvil GUI, so sorry I cannot help you with that.

But thanks for contributing to properly formatted issues!

@Sytm Sytm added bug Something isn't working wontfix This will not be worked on labels Feb 28, 2020
@Erlebsyk
Copy link
Author

Thanks for the quick reply. This bug just got fixed in WesJD/AnvilGUI #62. See the issue referenced above for more details, where WesJD replied: "I appreciate the detailed issue. This issue will be fixed in #62. Please remind the plugin developer to update the plugin once I merge that PR in a few minutes.".

@Sytm Sytm removed the wontfix This will not be worked on label Feb 28, 2020
@Sytm Sytm closed this as completed in 6ec03b6 Feb 28, 2020
@Sytm Sytm added the good first issue Good for newcomers label Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants