-
Improvement
-
Resolution: Fixed
-
Neutral
-
2.0.1
-
None
-
None
info.magnolia.module.blossom.template.TemplateDefinitionBuilder#resolveTemplateId throws IllegalStateException when class is missing Template annotation.
- Why no error message? Looking at the code its seems very clear what the problem is, the error should communicate this?
- I'm not sure the IllegalStateException is the best exception? Its description reads "Signals that a method has been invoked at an illegal or inappropriate time." A missing annotation is not really an issue regarding the timing of this call? Without the null check the actual exception would have been a NullPointerException, maybe catch that and re-throw it with an message explaining why it happened (related to the missing error message).
Acceptance criteria
- is related to
-
BLOSSOM-96 Exception thrown when trying to export template without required Template annotation should include the class
- Closed