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

[BUG] Enchanter: No xp consumed on shift-click #352

Closed
1 task done
dnewood opened this issue Jul 26, 2023 · 3 comments · Fixed by #368
Closed
1 task done

[BUG] Enchanter: No xp consumed on shift-click #352

dnewood opened this issue Jul 26, 2023 · 3 comments · Fixed by #368
Labels
MC-1.20.1 P-1-Medium Medium priority. Potentially deleting items or breaking functionality. Type-Bug There is a problem.

Comments

@dnewood
Copy link

dnewood commented Jul 26, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

If a user shift-click's on the enchanted book, no levels are removed from the player.

Expected Behavior

When grabbing the enchanted book, a users xp should decrease by the required enchant amount.

Steps To Reproduce

  1. Open the enchanter and perform a recipe as usual
  2. When grabbing the final enchanted product, use shift + left-click to grab the enchanted book
  3. No levels should have been consumed from the player

Environment

- Client OS: Windows 11
- Server OS: Ubuntu 22.04.2 LTS
- Client Java version: 8 update 241
- Server Java version: 17.0.7+7
- Minecraft version: 1.20.1
- Forge version: 1.20.1-47.1.3
- Version: 6.0.7-alpha

Anything else?

This was done a multiplayer self-hosted server

@dnewood dnewood added Status-Triage A Triage Member should take a look at this Type-Bug There is a problem. labels Jul 26, 2023
@tyler489 tyler489 added MC-1.20.1 P-2-Low Low priority. Not hurting anyone, just annoying. and removed Status-Triage A Triage Member should take a look at this labels Jul 26, 2023
@tyler489
Copy link
Collaborator

Steps to reproduce: put ingredients into Ender IO Enchanter. Shift-click the book from the output slot. You get the book but the ingredients are not consumed nor is the required xp. Removing and replacing the book and quill allows this to be repeated very quickly

AllTheMods/ATM-9#119

@tyler489
Copy link
Collaborator

tyler489 commented Jul 27, 2023

AllTheMods/ATM-9#163

Bumping priority as the ability to create any registered enchant out of thin air is annoying for server owners.

@tyler489 tyler489 added P-1-Medium Medium priority. Potentially deleting items or breaking functionality. and removed P-2-Low Low priority. Not hurting anyone, just annoying. labels Jul 27, 2023
@Rover656
Copy link
Member

Taking a look at this now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MC-1.20.1 P-1-Medium Medium priority. Potentially deleting items or breaking functionality. Type-Bug There is a problem.
Projects
None yet
3 participants