#549 Changed the default value of useActualParamName to true. False is useful only when the legacy style parameter reference (i.e. #{0}, #{1}, ...) is used.

Author: harawata

src/main/java/org/apache/ibatis/reflection/ParamNameResolver.java

@@ -87,7 +87,8 @@ public ParamNameResolver(Configuration config, Method method) {
         // @Param was not specified.
         if (config.isUseActualParamName()) {
           name = getActualParamName(method, paramIndex);
-        } else {
+        }
+        if (name == null) {
           // use the parameter index as the name ("0", "1", ...)
           // gcode issue #71
           name = String.valueOf(map.size());
@@ -100,7 +101,7 @@ public ParamNameResolver(Configuration config, Method method) {
 
   private String getActualParamName(Method method, int paramIndex) {
     if (GET_PARAMS == null) {
-      throw new ReflectionException("Method#getParameters() method does not exist. Relaunch on Java 8 or set false to useActualParamName.");
+      return null;
     }
     try {
       Object[] params = (Object[]) GET_PARAMS.invoke(method);

src/main/java/org/apache/ibatis/session/Configuration.java

@@ -106,7 +106,7 @@ public class Configuration {
   protected boolean useColumnLabel = true;
   protected boolean cacheEnabled = true;
   protected boolean callSettersOnNulls = false;
-  protected boolean useActualParamName = false;
+  protected boolean useActualParamName = true;
 
   protected String logPrefix;
   protected Class <? extends Log> logImpl;

src/site/es/xdoc/configuration.xml

@@ -445,7 +445,7 @@ SqlSessionFactory factory = sqlSessionFactoryBuilder.build(reader, environment,
                 true | false
               </td>
               <td>
-                false
+                true
               </td>
             </tr>
           </tbody>

src/site/ja/xdoc/configuration.xml

@@ -475,7 +475,7 @@ SqlSessionFactory factory = sqlSessionFactoryBuilder.build(reader, environment,
                 true | false
               </td>
               <td>
-                false
+                true
               </td>
             </tr>
           </tbody>

src/site/ko/xdoc/configuration.xml

@@ -452,7 +452,7 @@ SqlSessionFactory factory = sqlSessionFactoryBuilder.build(reader, environment,
                 true | false
               </td>
               <td>
-                false
+                true
               </td>
             </tr>
           </tbody>

src/site/xdoc/configuration.xml

@@ -536,7 +536,7 @@ SqlSessionFactory factory = sqlSessionFactoryBuilder.build(reader, environment,
                 true | false
               </td>
               <td>
-                false
+                true
               </td>
             </tr>
           </tbody>

src/site/zh/xdoc/configuration.xml

@@ -456,7 +456,7 @@ SqlSessionFactory factory = sqlSessionFactoryBuilder.build(reader, environment,
                 true | false
               </td>
               <td>
-                false
+                true
               </td>
             </tr>
           </tbody>

src/test/java/org/apache/ibatis/binding/BindingTest.java

@@ -69,6 +69,7 @@ public static void setup() throws Exception {
     Environment environment = new Environment("Production", transactionFactory, dataSource);
     Configuration configuration = new Configuration(environment);
     configuration.setLazyLoadingEnabled(true);
+    configuration.setUseActualParamName(false); // to test legacy style reference (#{0} #{1})
     configuration.getTypeAliasRegistry().registerAlias(Blog.class);
     configuration.getTypeAliasRegistry().registerAlias(Post.class);
     configuration.getTypeAliasRegistry().registerAlias(Author.class);