fix: EgovReflectionSupport의 setter/getter 이름 생성이 로케일에 따라 깨져 NPE가 나는 문제 수정

[wantaekchoi]

수정 사유 Reason for modification

• 버그수정 Bug fixes
• 기능개선 Enhancements
• 기능추가 Adding features
• 기타 Others

수정된 소스 내용 Modified source

EgovReflectionSupport는 VO 필드명으로 접근자 이름을 만들 때 첫 글자를 toUpperCase()로 올립니다. 인자 없는 toUpperCase()는 JVM 기본 로케일을 따르므로, 터키어·아제르바이잔어 로케일에서 "id".substring(0, 1).toUpperCase()는 점 있는 İ(U+0130)가 되어 setİd/getİd를 찾게 되고 실제 메서드명 setId/getId와 일치하지 않습니다.

retrieveMethod는 못 찾으면 null을 돌려주고 그 null이 그대로 methodMap에 담깁니다. invokeSetterMethod와 invokeGettterMethod는 맵에서 꺼낸 Method로 바로 invoke를 호출하고 catch도 IllegalAccessException/InvocationTargetException만 잡으므로, i로 시작하는 필드가 하나라도 있으면 NullPointerException으로 죽습니다.

AS-IS (generateSetterMethodMap, generateGetterMethodMap 두 곳)

strMethod = "set" + (names[i].substring(0, 1)).toUpperCase() + names[i].substring(1);
strMethod = "get" + (names[i].substring(0, 1)).toUpperCase() + names[i].substring(1);

TO-BE

strMethod = "set" + (names[i].substring(0, 1)).toUpperCase(Locale.ROOT) + names[i].substring(1);
strMethod = "get" + (names[i].substring(0, 1)).toUpperCase(Locale.ROOT) + names[i].substring(1);

같은 결함 클래스를 Locale.ROOT로 고친 선례가 이 저장소에 있습니다 — DefaultMapUserDetailsMapping의 컬럼명 소문자화(#321).

영향 범위

ASCII 필드명은 기본 로케일이 무엇이든 결과가 같습니다. 바뀌는 것은 터키어 계열 로케일에서 i로 시작하는 필드뿐입니다.

JUnit 테스트 JUnit tests

• JUnit 테스트 JUnit tests
• 수동 테스트 Manual testing

EgovReflectionSupportLocaleTest 2건을 추가했습니다. 기본 로케일을 tr-TR로 바꾼 상태에서 setter/getter 양쪽 경로를 각각 확인하고, 영어 로케일 대조군도 함께 봅니다. 기본 로케일은 finally에서 되돌립니다.

수정 지점만 되돌린 상태(RED)

[ERROR] Tests run: 2, Failures: 1, Errors: 1, Skipped: 0
[ERROR]   EgovReflectionSupportLocaleTest.setterMethodMapResolvesUnderTurkishLocale 터키어 로케일에서도 setId 를 찾아야 한다 ==> expected: not <null>
[ERROR]   EgovReflectionSupportLocaleTest.getterMethodMapResolvesUnderTurkishLocale » NullPointer Cannot invoke "java.lang.reflect.Method.invoke(Object, Object[])" because the return value of "java.util.HashMap.get(Object)" is null

수정 후(GREEN, 모듈 전체)

[INFO] Tests run: 76, Failures: 0, Errors: 0, Skipped: 0
[INFO] BUILD SUCCESS