Nameless icon indicating copy to clipboard operation
Nameless copied to clipboard

Catch exceptions in modules tab

Open PikaMug opened this issue 2 years ago • 3 comments

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

  1. Install pr12 with compatible modules
  2. Update to pr13
  3. 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

translation

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."

PikaMug avatar Jun 19 '22 16:06 PikaMug

How is it a bug? It is an enchanment

PadowYT2 avatar Jun 19 '22 16:06 PadowYT2

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

partydragen avatar Jun 19 '22 17:06 partydragen

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.

PikaMug avatar Jun 19 '22 17:06 PikaMug

https://github.com/NamelessMC/Nameless/pull/3143

Note this will only apply to disabled modules, or when installing a new incompatible module

samerton avatar Dec 27 '22 18:12 samerton