fix(codegen): Java Config 템플릿 셋이 존재하지 않는 jakarta.sql.DataSource 를 import 하는 문제 수정 - #152
Open
wantaekchoi wants to merge 1 commit into
Open
Conversation
…ort 하던 문제 수정 DataSource 는 Java SE(java.sql 모듈)가 내보내는 javax.sql 패키지의 타입이라 jakarta.sql 패키지는 없다. javac 21 이 package jakarta.sql does not exist 로 멈춘다. f0a4568 직전에는 이 셋도 javax.sql 이었다. 그 커밋이 jakarta.sql 로 바꾼 import 는 저장소 전체에서 이 셋뿐이다. 같은 커밋이 함께 고쳐 쓴 jdbc-java.vm·c3p0-java.vm 에는 "Spring Framework 6.x, Jakarta EE 9 호환" 주석을 새로 넣으면서도 import 는 javax.sql 로 두었다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
수정 사유 Reason for modification
수정된 소스 내용 Modified source
eGovFrameTemplates정본 트리의 Java Config 템플릿 셋이jakarta.sql.DataSource를 import 합니다.DataSource는 Java SE 의java.sql모듈이 내보내는javax.sql패키지의 타입입니다. Jakarta EE 가javax.*를jakarta.*로 옮길 때 이 패키지는 대상이 아니었으므로jakarta.sql이라는 패키지는 없습니다. 그래서 이 셋으로 생성한 설정 클래스는 컴파일되지 않습니다.같은 트리에서
DataSource를 import 하는 나머지 Java Config 템플릿 셋은javax.sql.DataSource를 씁니다.f0a4568(eGovframe Plug-Ins v5.0.0) 직전에는 세 템플릿도javax.sql.DataSource였습니다. 그 커밋이jakarta.sql로 바꾼 import 는 저장소 전체에서 이 셋뿐입니다.그 커밋은
jdbc-java.vm·c3p0-java.vm도 함께 고쳐Spring Framework 6.x, Jakarta EE 9 호환주석을 새로 넣으면서도 그 두 파일의 import 는javax.sql로 두었습니다. Jakarta EE 9 대응에서 이 타입이javax.sql로 남는다고 같은 커밋이 스스로 적어 둔 셈입니다.세 템플릿에는 마법사에서 도달합니다.
wizards.xml이 세 마법사를 등록하고(:22New Datasource Transaction ·:29New Sequence ID Generation ·:30New Table ID Generation), 각 마법사의Configuration Type라디오에서Java를 고르면 해당-java.vm을 씁니다. 세 마법사 XML 이 모두 같은 형태이고 아래는egovframework.dev.imp.codegen.template.templates/eGovFrameTemplates/transaction/transaction.xml:95입니다.AS-IS
TO-BE
영향 범위
Configuration Type을XML로 고르는 경로는transaction.vm·sequenceId.vm·tableId.vm을 쓰므로 변화가 없습니다.egovframework.dev.imp.templates의 사본 여섯은 이미javax.sql이라 이번 변경에 넣지 않았습니다. 수정 후에는 저장소에jakarta.sqlimport 가 남지 않습니다.JUnit 테스트 JUnit tests
번들 매니페스트가 요구하는 JavaSE-21 에서 두 import 를 각각 컴파일했습니다.
정본 트리의 import 도 대조했습니다.
수정 전에는 여섯 중 셋이
jakarta.sql이었고 수정 후에는 여섯 모두javax.sql입니다.테스트 브라우저 Test Browser
이클립스 플러그인의 코드 생성 템플릿이라 브라우저 테스트는 하지 않았습니다.
테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video
화면 변경이 없어 위 출력으로 대신합니다.