EssentialsX

EssentialsX

2M Downloads

1.9 or up Enchant is not available thru sign

lifehome opened this issue ยท 10 comments

commented

EssentialsX version (/essentials): 2.0.1-b523

Server software (/version): git-Spitogt-65e8124-963b19c (MC: 1.12.1) (API 1.12.1-R0.1-SNAPSHOT)

EssentialsX config (if applicable): default config

Steps to replicate issue:

  1. Put an enchant sign, on line 3 write Mending
  2. Click Done.
  3. It breaks.
commented

It looks like this:

image

commented

By "it breaks", do you mean the placed sign breaks or it doesn't work? In addition, does Mending work in /enchant?

commented

Does /enchant work when you use Mending? This would help us narrow down the cause of the issue.

commented
commented

I believe you also need to specify a level, for example mending:1 for Mending I (even though there's only one level for Mending).

@SupaHam Should we set a default level for enchant signs, either hardcoded or configurable?

commented

Thanks @md678685, it works as you said. However it seems it cannot stack enchantments to enchanted items, to replicate the bug, simply hold an enchanted item and right click at the enchantment sign.

image

commented

Does the enchantment override the existing enchantment, or does it not get applied at all?

commented

It simply rejects all enchanted items to be enchanted.

commented

@md678685 I think it'll be fine to default to 1. However, we mustn't help discourage reading wiki. After all users must know what the signs are doing to prevent issues.

commented

@SupaHam thanks for the consideration. I suggest letting the plugin to evaluate if the line either does not contain enchantment level, to prompt a related error, or to tell the enchantment does not exists.