Repository navigation
Conversation
jei007
approved these changes
Oct 30, 2026
jei007
left a comment
Contributor
There was a problem hiding this comment.
표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.
1 of 2 tasks
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.
BndtManageDAO.selectBndtManageListTotCnt가 연결된 관리 목록 엔드포인트에서 호출되지만 매핑이 없습니다. 8개 벤더 매퍼에 개수 statement를 추가하고, 각 selectBndtManageList의 WHERE 절을 데이터베이스별 부분 문자열 매칭까지 그대로 옮겼습니다. 엔드포인트와 public API는 그대로 유지됩니다.
검증: Python 리터럴 DAO ID 추출과 XML 파싱으로 모든 벤더(벤더당 매퍼 파일 153건, statement 1245→1246건)에서 정확히 이 비어 있던 매핑만 채워졌음을 확인했습니다. 8/8 신규 개수 WHERE 절은 기존 목록 WHERE 절을 그대로 옮긴(verbatim) 것입니다.
git diff --check통과. Java와 Maven이 없어 실제 MyBatis 로딩과 SQL 실행은 수행하지 못했습니다. 테스트 파일은 추가하지 않았습니다.upstream main
fd9062b8dae0838375272961234946899c2f74a4기준입니다. #1392 및 #1393 과 독립적이며 브랜치 간·도구 버전 간 의존성이 없습니다. Kno insert/delete 매핑 4건은 연기된 blocker로 이 PR 밖에 둡니다.관련: #1315
추적: leejongyoung#11