[
https://issues.apache.org/jira/browse/SLING-4155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14209804#comment-14209804
]
Stefan Seifert commented on SLING-4155:
---------------------------------------
such a change on the annotations will indeed break both runtime and
compile-time backward compatibility - we cannot do this unless we raise the
version to v2.0.0 to comply to semantic versioning. i'm not sure if it's worth
only for this issue, although i agree the current functionality is broken when
injector-specific annotations are used.
for a first step we should document this issue (SLING-4156), and perhaps create
a new version in JIRA to sling models 2.0.0 and collect there tickets like this
so we do not forget it when time is ready for v2.
> DefaultInjectionStrategy not considered for injector-specific annotations
> -------------------------------------------------------------------------
>
> Key: SLING-4155
> URL: https://issues.apache.org/jira/browse/SLING-4155
> Project: Sling
> Issue Type: Bug
> Components: Extensions
> Affects Versions: Sling Models Implementation 1.0.6
> Reporter: Konrad Windszus
>
> The default injection strategy (being implemented in SLING-3696) is only
> considered, in case there is no injector-specific annotation being used.
> Otherwise it is just ignored.
> The logic should be like this:
> if annotationProcessor.isOptional() returns null
> -> the default injection strategy should be used
> in any other case the boolean value should be used.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
