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

1.18 Post Merge Irregular Behaviour #23

Open
qruet opened this issue Dec 13, 2021 · 2 comments
Open

1.18 Post Merge Irregular Behaviour #23

qruet opened this issue Dec 13, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@qruet
Copy link
Owner

qruet commented Dec 13, 2021

Server Version
1.18-R0.1-SNAPSHOT

Plugin Version
3.6-b71

Describe the bug

  • Experience Cost bar does not properly update when actual result cost changes. (see media below)
  • The Enchantment Cost does not properly update either, forcing an inventory update (e.g. clicking the inventory) is necessary to update the displayed Repair Cost.
  • After a repair, clicking the repaired item disappears and then reappears when another inventory slot is clicked. I believe this is from the cursor item slot not being updated (missing packet).

To Reproduce

  • I used a diamond sword with an enchantment book with Sharpness V. Simply by going through the process with and without experience will be able to reproduce these issues.

Media
image

@qruet qruet added the bug Something isn't working label Dec 13, 2021
@qruet qruet pinned this issue Dec 13, 2021
@Gaming32
Copy link
Contributor

Are there any errors in the console?

@qruet
Copy link
Owner Author

qruet commented Dec 13, 2021

No stacktrace to report. I believe all of these issues are simply a result from missing packets or packets with the wrong data being sent. I would look into PacketPlayOutSetSlot which is constructed in IContainerAnvilLot#sendSlotUpdate and make sure the parameters haven't change order or something weird - perhaps the wrong data is being passed. Then also check that the updated repair cost for ContainerAnvil is being sent to the player as well, since the TooExpensiveBar is reading the correct cost but the GUI seems to be having trouble updating the repair cost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants