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

You can disenchant items such as the reinforced armor and move the enchantments from it onto netherite armor. #4201

Closed
4 tasks done
RadiantUwU opened this issue Jun 1, 2024 · 2 comments
Labels
🐞 Bug Report A bug that needs to be fixed. 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced.

Comments

@RadiantUwU
Copy link

RadiantUwU commented Jun 1, 2024

❗ Checklist

  • I am using the official english version of Slimefun and did not modify the jar.
  • I am using an up to date "DEV" (not "RC") version of Slimefun.
  • I am aware that issues related to Slimefun addons need to be reported on their bug trackers and not here.
  • I searched for similar open issues and could not find an existing bug report on this.

📍 Description

I have seen others use this to get completely broken armor which cannot be fixed without modifying the plugin itself.
This bug involves the disenchanting table and auto enchanter.
I'm a server operator on a public minecraft server and having prot 9 netherite armor shouldn't be acceptable.

📑 Reproduction Steps

Disenchant Reinforced armor
Enchant netherite armor

💡 Expected Behavior

Disallow disenchanting any slimefun items.

📷 Screenshots / Videos

No response

📜 Server Log

No response

📂 /error-reports/ folder

No response

💻 Server Software

Spigot

🎮 Minecraft Version

1.20.x

⭐ Slimefun version

image

🧭 Other plugins

No response

@RadiantUwU RadiantUwU added 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. 🐞 Bug Report A bug that needs to be fixed. labels Jun 1, 2024
@ybw0014
Copy link
Contributor

ybw0014 commented Jun 1, 2024

Auto enchanter doesnt exceed vanilla enchantment limit by default, you have allowed this and then later said they are too broken? Either disable them or alter the config, not a bug.

@RadiantUwU
Copy link
Author

This issue is resolved by disabling the "allow disenchanting" here
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug Report A bug that needs to be fixed. 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced.
Projects
None yet
Development

No branches or pull requests

2 participants