[jira] [Comment Edited] (SLING-8706) Injections for java.util.Optional<> should be automatic optional

["Evgeny Tugarev (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SLING-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16974320#comment-16974320 ] 

Evgeny Tugarev edited comment on SLING-8706 at 11/14/19 8:03 PM:
-----------------------------------------------------------------

Thanks, [~sseifert] this exactly what I meant.


was (Author: etugarev):
Thanks, [~sseifert] this exactly what I meant. I'll check for other tickets :)

> Injections for java.util.Optional<> should be automatic optional 
> -----------------------------------------------------------------
>
>                 Key: SLING-8706
>                 URL: https://issues.apache.org/jira/browse/SLING-8706
>             Project: Sling
>          Issue Type: Improvement
>          Components: Sling Models
>            Reporter: Jörg Hoh
>            Priority: Major
>
> The current approach to support optional injections requires to annotate the field with {{@Optional}} plus proper handling within the javacode (null checks etc), which can be forgotten.
> So instead of
> {code}
> @Inject @Optional
> String fieldname;
> {code}
> it should also be possible to use this
> {code}
> @Inject
> Optional<String> fieldname;
> {code}
> with the very same semantic. But the developer is forced to deal with the case that the value is not present.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)