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

Removing /glow with grindstone produces XP #200

Open
melaniebeedot opened this issue Dec 7, 2024 · 1 comment
Open

Removing /glow with grindstone produces XP #200

melaniebeedot opened this issue Dec 7, 2024 · 1 comment

Comments

@melaniebeedot
Copy link

melaniebeedot commented Dec 7, 2024

We're a Paper 1.21.1 server with EpicRename 3.12

I've had a player report this bug to me, that with the unenchanted armor they've /glowed for shiny effect, they found later they cannot add enchantments to. [Split into issue #201]

They were also unable to use /unglow to remove the glow, which I think is already a bug that's been reported for 1.21

They were, however, able to use a grindstone to remove the /glow, and discovered another bug: it produces a fair amount of XP to the player each time, an infinite XP bug if you will. Screenshots attached show the player at 0 XP lvl, and how many XP lvls gained from removing /glow via grindstone for each piece.

glowXP1
glowXP2
glowXP3
glowXP4
glowXP5

@melaniebeedot melaniebeedot changed the title armor with no enchants and /glow cannot be enchanted unenchanted armor with /glow cannot be enchanted; grindstoning /glow off produces XP Dec 7, 2024
@melaniebeedot melaniebeedot changed the title unenchanted armor with /glow cannot be enchanted; grindstoning /glow off produces XP unenchanted armor with /glow cannot be enchanted; removing /glow with grindstone produces XP Dec 7, 2024
@JustBru00 JustBru00 self-assigned this Dec 9, 2024
@JustBru00
Copy link
Owner

@melaniebeedot Thanks for the report. The issues you are seeing are related to #198. It appears that since 1.20.4 spigot changed the maximum enchantment levels supported. This is causing the issues you are seeing.
I had previously tried working on this update, but I am running into some extremely strange behavior where enchantment glow stays on an item even after enchantments are removed.

I will try to have a fix for you on this in the next 2 weeks.

@JustBru00 JustBru00 changed the title unenchanted armor with /glow cannot be enchanted; removing /glow with grindstone produces XP Removing /glow with grindstone produces XP Dec 9, 2024
@JustBru00 JustBru00 added this to the v3.12.1 milestone Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants