[SLING-3679] Required fields validation

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Sling Models Implementation 1.0.4
• Fix Version/s: Sling Models Implementation 1.0.6
• Component/s: Extensions
• Labels: models
• Assignee: Unassigned
• Reporter: Krystian Panek
• Created: 18/Jun/14 12:17
• Updated: 11/Jul/14 08:04
• Resolved: 25/Jun/14 08:07

Description

Currently if some field cannot be injected (and it is not annotated with @Optional), model adapter factory returns null. However fact that some field has null and all other are properly injected is acceptable in my context.

Proposal:

• model adapter factory does not return null if not all required fields are injected,
• result of requirement validation is serviced as for example:
  • injecting it to some extra annotated field: @Valid boolean valid; (with default false),
  • passing bool parameter in @PostConstruct callback, for example 'valid' (true if all required field are injected, false otherwise),
• behave current behavior, new available only with extra model class annotation, for example @NotNull .

Issue Links

• Is contained by: SLING-3696 Allow a model class to be annotated so that by default injections are optional