Multi tools won't reset their modes after server restart

WalshyDev commented 7 months ago

Description

Multi tools reset their mode after server restart. I found it really annoying and so did my players, so I decided to fix that. This pull request also adds a display of current multi tool mode in its lore. Feel free to improve my code. I tried to achieve my goal without changing too much stuff, so it shouldn't break anything.

Captura de pantalla 2023-12-28 231315

Proposed changes

Multi tools keep their current mode after server restart
Current multi tool mode is displayed in its lore (works with already existing multi tools)

Checklist

[x] I have fully tested the proposed changes and promise that they will not break everything into chaos.
[x] I have also tested the proposed changes in combination with various popular addons and can confirm my changes do not break them.
[x] I have made sure that the proposed changes do not break compatibility across the supported Minecraft versions (1.16. - 1.20.).
[x] I followed the existing code standards and didn't mess up the formatting.
[ ] I did my best to add documentation to any public classes or methods I added.
[x] I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values
[ ] I added sufficient Unit Tests to cover my code.

github-actions[bot] commented 7 months ago

Pro Tip! You can help us label your Pull Requests by using the following branch naming convention next time you create a pull request. ❤️ Branch naming convention	Label
`feature/**`	🎈 Feature
`fix/**`	✨ Fix
`chore/**`	🧹 Chores
`api/**`	🔧 API
`performance/**`	💡 Performance Optimization
`compatibility/**`	🤝 Compatibility

If your changes do not fall into any of these categories, don't worry. You can just ignore this message in that case! 👀

sonarcloud[bot] commented 7 months ago

Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

7 New issues
0 Security Hotspots
13.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

github-actions[bot] commented 7 months ago

Slimefun preview build

A Slimefun preview build is available for testing! Commit: e24a51a3

https://preview-builds.walshy.dev/download/Slimefun/4122/e24a51a3

Note: This is not a supported build and is only here for the purposes of testing. Do not run this on a live server and do not report bugs anywhere but this PR!

womzil commented 5 months ago

Sorry if I missed something obvious, but what has to be done in order for this change to be merged with the master branch?

Slimefun / Slimefun4