Improve automatic column name mapping using camel case conversion about mybatis HOT 13 CLOSED

Actually, it seems like the change can be easily applied in 
Reflector.findPropertyName()

Acutally, ibator v1.2.1 implement mapping table column to POJO property use 
org.apache.ibatis.ibator.internal.util.JavaBeamUtil.getCamelCaseString method, 
trim the char in "_-$@# ", for example: column "THE_FIRST-FIELD" will be 
convert to property name "theFirstField".

@myweboffice:
ok, that's cool and solves half of the problem. So why not use that method 
inside Reflector.findPropertyName() ?
I think that could work without efffort.

Ideally, I think it would be more flexible if MyBatis could let you inject a 
class of your own that handles custom column-to-field mapping, but the fact 
that the Reflector class uses static methods constraints this.

Would a simple solution like this be appropriate for just the underscore 
mapping of field_name > fieldName?

Reflector.findPropertyName():

  public String findPropertyName(String name) {
      String upperName = name.toUpperCase(Locale.ENGLISH);
      return caseInsensitivePropertyMap.containsKey(upperName) ?
             caseInsensitivePropertyMap.get(upperName) :
             caseInsensitivePropertyMap.get(upperName.replaceAll("_", ""));
  }

I think, implementing this feature should be assigned a higher priority, because

1) Using column names without underscores in databases that convert all column 
names to lower case leads to almost unreadable column names.

2) Hibernate uses the mapping between camelCase fields and underscore separated 
column names by default. So it's very inconvenient to switch between Hibernate 
and MyBatis at the moment (either rename all the columns, or manually add all 
the column name mappings in the MyBatis SQL Map XML files).

Seems an easy change. It could break backwards compatibility (very unlikely) so 
maybe a general property should be added.

Patch attached.

Sorry, that patch has unwanted changes, this is the right one

Another try with less code changes. 

Patch applied in r3895. 

Added a new property named mapUnderscoreToCamelCase that defaults to false.
Please give it a try and post your feedback (here or in users ML).

We will update the manual once the change is released.

I am afraid this last fix is too simple :) Maps do not work properly. 

That should be fixed now in r3896