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.

Sign up for GitHub

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

Jump to bottom

Users can add more enchants than the "max_enchants" config value #38

Closed
chrismwiggs opened this issue Mar 3, 2021 · 0 comments
Closed

Users can add more enchants than the "max_enchants" config value #38

chrismwiggs opened this issue Mar 3, 2021 · 0 comments
Assignees
@Geolykt
Labels
bug:confirmed The bug was confirmed and a fix is being worked on bug:resolved

Comments

@chrismwiggs
Copy link

Turns out the max_enchants value is not being respected; I have players adding like 10 enchants on a tool despite the config value being set to 2. When they stack up more than a couple on one tool and start going at a mob grinder, the server really grinds down and we see a big tps loss.

This is on paper 1.16.5, EnchantmentsPlus version 2.2.2
@Geolykt Geolykt self-assigned this Mar 3, 2021
@Geolykt Geolykt added the bug:confirmed The bug was confirmed and a fix is being worked on label Mar 3, 2021
Geolykt added a commit that referenced this issue Mar 3, 2021
@Geolykt
Fixed issues with the enchantment limit not being respected
f72bd8d 
Resolves #38
@Geolykt Geolykt closed this as completed in 87aee0b Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Geolykt Geolykt

Labels
bug:confirmed The bug was confirmed and a fix is being worked on bug:resolved
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chrismwiggs @Geolykt