-
Notifications
You must be signed in to change notification settings - Fork 61
org.egovframe.rte.ptl.mvc 모듈 컴파일 오류 수정 #190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
org.egovframe.rte.ptl.mvc 모듈 컴파일 오류 수정 #190
Conversation
- Updated spring-modules-validation dependency to use external org.springmodules:spring-modules-validation:0.8a artifact - Fixed spring-modules-validation internal module POM structure to properly inherit from root POM - Resolved missing org.springmodules.validation.commons.FieldChecks class error - Verified compilation and tests pass successfully Fixes compilation failure in RteFieldChecks.java caused by missing spring-modules-validation dependency. The RteFieldChecks class extends FieldChecks from spring-modules-validation library for custom validation rules.
|
org.springmodules 보다는, 같은 서브 모듈에 존재하는 spring-modules-validation을 참조하면서 컴파일 오류가 발생하지 않도록 수정하는게 맞지 않을까 싶습니다. |
|
오늘 확인해보니, org.egovframe.rte.ptl.mvc 뿐만 아니라 org.egovframe.rte.bat.core 쪽에서도 spring-modules-validation 참조 문제로 EgovParallelJobFunctionalTests 테스트가 정상적으로 동작하지 않는 문제가 있는 걸 확인했습니다. 다행인 점은, Maven Repository에 PavLab 리포지토리 쪽으로 동작과 호환성 문제가 없는 라이브러리가 존재하는 것 같습니다. spring-module-validation을 해당 라이브러리로 교체하고, 헷갈리지 않도록 Presentation에 존재하는 spring-module-validation 서브 모듈을 제거하는 것이 어떨지 건의 드려봅니다. |
|
스프링 측에서는 spring modules validation을 유지보수할 의사가 없는 것으로 보이는데 spring-projects/spring-framework#21648 |
현재 egov 패키지 두 개에서 참조하고 있기 때문에 지원중단 하려면 import한 코드를 전부 마이그레이션이 필수고, spring validation 라이브러리는 별도의 리포지토리가 존재하지 않아 현재는 소스 코드를 찾을 수 없어서 직접 포크는 불가능합니다. |
https://github.com/astubbs/spring-modules/tree/release-0_8/projects/validation 에서 0.8 버전의 소스코드 포크를 찾긴 했지만 아무래도 20년 가까이 이전의 프로젝트이다보니 이걸 egov에서 직접 유지보수하기는 어려울 듯 합니다. |
수정 내용 Summary
👉 RteFieldChecks.java 에서 FieldChecks 클래스를 상속할 수 없어 발생한 컴파일 오류를 해결했습니다.
수정 사유 Reason for modification
소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)
수정된 소스 내용 Modified source
검토자를 위해 수정된 소스 내용을 설명해 주세요. Please describe the modified source for reviewers.
JUnit 테스트 JUnit tests
테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.
테스트 브라우저 Test Browser
테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.
테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video
테스트 전과 후의 스크린샷 또는 캡처 영상을 이곳에 첨부해 주세요. Please attach screenshots or video captures of your before and after tests here.