Stefan Seifert created SLING-4056:
-------------------------------------

             Summary: ModelFactory.canCreateFromAdaptable reports false errors 
when using "adapters" on models
                 Key: SLING-4056
                 URL: https://issues.apache.org/jira/browse/SLING-4056
             Project: Sling
          Issue Type: Bug
          Components: Extensions
    Affects Versions: Sling Models Impl 1.2.0
            Reporter: Stefan Seifert
            Priority: Critical
             Fix For: Sling Models Impl 1.2.0
         Attachments: ImplementsExtendsTest.patch

SLING-3709 introduced a new "ModelFactory" with methods for checking if 
adaption is possible and adapt to a model with enhanced exception support.

the current implementation does not support the "adapters" variant of models, 
where the model is adapted to an alternate adapter class (see 
[documentation|http://sling.apache.org/documentation/bundles/models.html#specifying-an-alternate-adapter-class-since-sling-models-110]).

adaption of such models via adaptTo still works, but false errors are reported
{noformat}
ERROR org.apache.sling.models.impl.ModelAdapterFactory - Provided Adapter class 
does not have a Model annotation
{noformat}

if using the factory method ModelFactory.canCreateFromAdaptable an exception is 
thrown - this is wrong.

attached is a patch that extends the unit test to include calls to 
canCreateFromAdaptable - all fail [^ImplementsExtendsTest.patch]

[~kwin]: can you have a look?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to