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

Robert Munteanu commented on SLING-11507:
-----------------------------------------

[~angela] - looking at the contributors list ( 
https://github.com/apache/sling-org-apache-sling-models-api/graphs/contributors 
, 
https://github.com/apache/sling-org-apache-sling-models-impl/graphs/contributors
 ) I see that [~sseifert], [~kwin], and [~diru] are the most "prolific" 
contributors.

Others did contribute, but either occasionally or for "technical" issues - 
readmes, parent pom, Jenkins, etc.

> Field injection should not inject static fields
> -----------------------------------------------
>
>                 Key: SLING-11507
>                 URL: https://issues.apache.org/jira/browse/SLING-11507
>             Project: Sling
>          Issue Type: Bug
>          Components: Sling Models
>    Affects Versions: Models Impl 1.4.14
>            Reporter: Joerg Hoh
>            Priority: Critical
>             Fix For: Sling Models Implementation 1.5
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Currently this injection works:
> {noformat}
>         @SlingObject
>         private static ResourceResolver resourceResolver;
> {noformat}
> but it should not. Sling Models are Pojos and injection must never inject 
> into static fields. Instead it should throw an exception and an error message 
> indicating the problem (another option would be to refuse the injection with 
> a proper error message).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to