Related to https://code.google.com/p/mybatis/issues/detail?id=448.

CacheKey should be created with the same order than the
DefaultParameterHandler handles parameters.

Author: emacarron

src/main/java/org/apache/ibatis/executor/BaseExecutor.java

@@ -173,11 +173,11 @@ public CacheKey createCacheKey(MappedStatement ms, Object parameterObject, RowBo
       } else {
         MetaObject metaObject = configuration.newMetaObject(parameterObject);
         for (ParameterMapping parameterMapping : parameterMappings) {
-          String propertyName = parameterMapping.getProperty();
-          if (metaObject.hasGetter(propertyName)) {
-            cacheKey.update(metaObject.getValue(propertyName));
-          } else if (boundSql.hasAdditionalParameter(propertyName)) {
+          String propertyName = parameterMapping.getProperty();       
+          if (boundSql.hasAdditionalParameter(propertyName)) { // issue #448 ask first for additional params
             cacheKey.update(boundSql.getAdditionalParameter(propertyName));
+          } else if (metaObject.hasGetter(propertyName)) {
+            cacheKey.update(metaObject.getValue(propertyName));
           }
         }
       }

src/main/java/org/apache/ibatis/scripting/defaults/DefaultParameterHandler.java

@@ -56,7 +56,6 @@ public void setParameters(PreparedStatement ps) throws SQLException {
     ErrorContext.instance().activity("setting parameters").object(mappedStatement.getParameterMap().getId());
     List<ParameterMapping> parameterMappings = boundSql.getParameterMappings();
     if (parameterMappings != null) {
-      MetaObject metaObject = parameterObject == null ? null : configuration.newMetaObject(parameterObject);
       for (int i = 0; i < parameterMappings.size(); i++) {
         ParameterMapping parameterMapping = parameterMappings.get(i);
         if (parameterMapping.getMode() != ParameterMode.OUT) {
@@ -69,7 +68,8 @@ public void setParameters(PreparedStatement ps) throws SQLException {
           } else if (typeHandlerRegistry.hasTypeHandler(parameterObject.getClass())) {
             value = parameterObject;
           } else {
-            value = metaObject == null ? null : metaObject.getValue(propertyName);
+            MetaObject metaObject = configuration.newMetaObject(parameterObject);
+            value = metaObject.getValue(propertyName);
           }
           TypeHandler typeHandler = parameterMapping.getTypeHandler();
           JdbcType jdbcType = parameterMapping.getJdbcType();