Fix for http://code.google.com/p/mybatis/issues/detail?id=202 - unneeded sort in MapWrapper

Author: jeffgbutler

src/main/java/org/apache/ibatis/reflection/wrapper/MapWrapper.java

@@ -4,21 +4,16 @@
 import org.apache.ibatis.reflection.factory.ObjectFactory;
 import org.apache.ibatis.reflection.property.PropertyTokenizer;
 
-import java.util.Arrays;
-import java.util.Comparator;
 import java.util.HashMap;
 import java.util.Map;
 
 public class MapWrapper extends BaseWrapper {
 
   private Map map;
-  private Object[] keyArray;
-
 
   public MapWrapper(MetaObject metaObject, Map map) {
     super(metaObject);
     this.map = map;
-    updateKeyArray();
   }
 
   public Object get(PropertyTokenizer prop) {
@@ -37,28 +32,9 @@ public void set(PropertyTokenizer prop, Object value) {
     } else {
       map.put(prop.getName(), value);
     }
-    updateKeyArray();
   }
 
-  @SuppressWarnings("unchecked")
   public String findProperty(String name) {
-    updateKeyArray();
-    if (name != null) {
-      Arrays.binarySearch(keyArray, name, new Comparator() {
-        public int compare(Object o1, Object o2) {
-          if (o1 == o2) {
-            return 0;
-          } else if (o1 == null && o2 == null) {
-            return 0;
-          } else if (o1 == null) {
-            return -1;
-          } else if (o2 == null) {
-            return 1;
-          }
-          return ((String) o1).toLowerCase().compareTo(((String) o2).toLowerCase());
-        }
-      });
-    }
     return name;
   }
 
@@ -134,9 +110,4 @@ public MetaObject instantiatePropertyValue(String name, PropertyTokenizer prop,
     return MetaObject.forObject(map, metaObject.getObjectFactory(), metaObject.getObjectWrapperFactory());
   }
 
-  private void updateKeyArray() {
-    keyArray = map.keySet().toArray();
-    Arrays.sort(keyArray);
-  }
-
 }

src/main/java/org/apache/ibatis/reflection/wrapper/ObjectWrapper.java

@@ -16,9 +16,9 @@ public interface ObjectWrapper {
 
   String[] getSetterNames();
 
-  Class getSetterType(String name);
+  Class<?> getSetterType(String name);
 
-  Class getGetterType(String name);
+  Class<?> getGetterType(String name);
 
   boolean hasSetter(String name);