You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ss...@apache.org on 2014/10/15 15:41:31 UTC
svn commit: r1632023 - in /sling/trunk/bundles/extensions/models:
api/src/main/java/org/apache/sling/models/factory/ModelFactory.java
impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java
Author: sseifert
Date: Wed Oct 15 13:41:31 2014
New Revision: 1632023
URL: http://svn.apache.org/r1632023
Log:
SLING-4057 ModelFactory: Ordering of method arguments inconsistent
Modified:
sling/trunk/bundles/extensions/models/api/src/main/java/org/apache/sling/models/factory/ModelFactory.java
sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java
Modified: sling/trunk/bundles/extensions/models/api/src/main/java/org/apache/sling/models/factory/ModelFactory.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/models/api/src/main/java/org/apache/sling/models/factory/ModelFactory.java?rev=1632023&r1=1632022&r2=1632023&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/models/api/src/main/java/org/apache/sling/models/factory/ModelFactory.java (original)
+++ sling/trunk/bundles/extensions/models/api/src/main/java/org/apache/sling/models/factory/ModelFactory.java Wed Oct 15 13:41:31 2014
@@ -43,7 +43,7 @@ public interface ModelFactory {
* @return false in case the given class can not be adapted from the given adaptable
* @throws InvalidModelException in case the given class does not have a model annotation
*/
- public boolean canCreateFromAdaptable(Class<?> modelClass, Object adaptable) throws InvalidModelException;
+ public boolean canCreateFromAdaptable(Object adaptable, Class<?> modelClass) throws InvalidModelException;
/**
*
Modified: sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java?rev=1632023&r1=1632022&r2=1632023&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java (original)
+++ sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/ModelAdapterFactory.java Wed Oct 15 13:41:31 2014
@@ -179,7 +179,7 @@ public class ModelAdapterFactory impleme
}
@Override
- public boolean canCreateFromAdaptable(Class<?> modelClass, Object adaptable) throws InvalidModelException {
+ public boolean canCreateFromAdaptable(Object adaptable, Class<?> modelClass) throws InvalidModelException {
return innerCanCreateFromAdaptable(modelClass, adaptable);
}
@@ -897,7 +897,7 @@ public class ModelAdapterFactory impleme
private Object adaptIfNecessary(Object value, Class<?> type, Type genericType, Result<?> parentResult) {
if (!isAcceptableType(type, genericType, value)) {
Class<?> declaredType = type;
- if (isModelClass(type) && canCreateFromAdaptable(type, value)) {
+ if (isModelClass(type) && canCreateFromAdaptable(value, type)) {
Result<?> result = internalCreateModel(value, type);
if (result.getModel() == null) {
parentResult.appendFailures(result);