Resolved a TODO

harawata · harawata · commit cb9e97388672 · 2025-01-29T06:25:59.000+09:00
diff --git a/src/main/java/org/apache/ibatis/type/TypeHandlerRegistry.java b/src/main/java/org/apache/ibatis/type/TypeHandlerRegistry.java
@@ -342,17 +342,7 @@ private Map<JdbcType, TypeHandler<?>> getJdbcHandlerMap(Type type) {
     }
     if (type instanceof Class) {
       Class<?> clazz = (Class<?>) type;
-      if (Enum.class.isAssignableFrom(clazz)) {
-        // TODO: verify
-        // if (clazz.isAnonymousClass()) {
-        // return getJdbcHandlerMap(clazz.getSuperclass());
-        // }
-        // jdbcHandlerMap = getJdbcHandlerMapForEnumInterfaces(clazz, clazz);
-        // if (jdbcHandlerMap == null) {
-        // register(clazz, getInstance(clazz, defaultEnumTypeHandler));
-        // return typeHandlerMap.get(clazz);
-        // }
-      } else {
+      if (!Enum.class.isAssignableFrom(clazz)) {
         jdbcHandlerMap = getJdbcHandlerMapForSuperclass(clazz);
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -342,17 +342,7 @@ private Map<JdbcType, TypeHandler<?>> getJdbcHandlerMap(Type type) {`
`342`	`342`	`}`
`343`	`343`	`if (type instanceof Class) {`
`344`	`344`	`Class<?> clazz = (Class<?>) type;`
`345`		`- if (Enum.class.isAssignableFrom(clazz)) {`
`346`		`- // TODO: verify`
`347`		`- // if (clazz.isAnonymousClass()) {`
`348`		`- // return getJdbcHandlerMap(clazz.getSuperclass());`
`349`		`- // }`
`350`		`- // jdbcHandlerMap = getJdbcHandlerMapForEnumInterfaces(clazz, clazz);`
`351`		`- // if (jdbcHandlerMap == null) {`
`352`		`- // register(clazz, getInstance(clazz, defaultEnumTypeHandler));`
`353`		`- // return typeHandlerMap.get(clazz);`
`354`		`- // }`
`355`		`- } else {`
	`345`	`+ if (!Enum.class.isAssignableFrom(clazz)) {`
`356`	`346`	`jdbcHandlerMap = getJdbcHandlerMapForSuperclass(clazz);`
`357`	`347`	`}`
`358`	`348`	`}`