[SLING-4218] Throw exception for ValidationService.getValidationModel(...) if invalid validation model is found

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Validation 1.0.0
• Fix Version/s: Validation 1.0.0
• Component/s: Validation
• Assignee: Konrad Windszus
• Reporter: Konrad Windszus
• Created: 05/Dec/14 08:57
• Updated: 20/Sep/17 13:17
• Resolved: 05/Dec/14 09:47

Description

Currently ValidationService.getValidationModel(...) always returns null either if there is no validation model found for the given resource type or the one being found is invalid.
To be able to distinguish those cases and also to ease fixing of invalid validation models I propose that this method is allowed to throw IllegalStateException's in case the matching validation model is invalid (i.e. if it contains an invalid Validator or does not have at least properties subnode or children subnode being set).
That should of course be explicitly stated in the Javadoc.