[ https://issues.apache.org/jira/browse/SLING-3683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14040609#comment-14040609 ]
Konrad Windszus commented on SLING-3683: ---------------------------------------- [~sseif...@pro-vision.de] Thanks for spotting this. There is currently not a real test which tests for an annotation which cannot be used for injection, but it should be easy to derive from the already existing tests at https://github.com/apache/sling/blob/trunk/bundles/extensions/models/impl/src/test/java/org/apache/sling/models/impl/InjectorSpecificAnnotationTest.java. > Sling Models: ModelAdapterFactory mixes up annotationProcessor in > injectFieldOrMethod method > -------------------------------------------------------------------------------------------- > > Key: SLING-3683 > URL: https://issues.apache.org/jira/browse/SLING-3683 > Project: Sling > Issue Type: Bug > Components: Extensions > Affects Versions: Sling Models Implementation 1.0.6 > Reporter: Stefan Seifert > Priority: Minor > Attachments: 140620_SLING-3683_annotationProcessor.patch > > > the method AdapterFactory.injectFieldOrMethod gets an annotationProcessor > from the injectors if they implement the InjectAnnotationProcessorFactory. > currently, if a injector does not implement this interface, the instance > assigned to the annotationProcessor may be re-used from the previous injector > which is not intended. > patch attached. > not tested, is there already a unit test where a test for this can should be > added? -- This message was sent by Atlassian JIRA (v6.2#6252)