Nameless
Nameless copied to clipboard
Catch exceptions in modules tab
Describe the issue
When updating Nameless, some or all modules are bound to become outdated. An uncaught exception is thrown.
NamelessMC Version
2.0.0-pr13
To Reproduce
- Install pr12 with compatible modules
- Update to pr13
- Click on 'Modules' in admin control panel
Expected Behaviour
Rather than show exception, catch any in this tab and show a more user-friendly message.
Screenshots
Additional Information
Example of a good message could be like the Discord bot says, "Translation file not found - Ensure that you don't have any modules/templates installed that are not compatible with pr13. You can remove modules by going to the /modules directory and deleting (or moving) any outdated modules. For templates you can delete them in the /custom/templates directory."
How is it a bug? It is an enchanment
Catch exceptions wont work due after that language issue then a another issue will popup with is not able to catch due of a missing debug info function
How is it a bug? It is an enchanment
I agree, feel free to change the label.
Catch exceptions wont work due after that language issue then a another issue will popup with is not able to catch due of a missing debug info function
Surely a link to the exception could still be provided? Or instead of a blanket solution, only catch common exceptions with known solutions such as this.
EDIT: This is covered in https://github.com/NamelessMC/Nameless/issues/2822 but I personally see that as a list of issues in need of a cleaner solution. Just my two cents.
https://github.com/NamelessMC/Nameless/pull/3143
Note this will only apply to disabled modules, or when installing a new incompatible module