Skip to content

fix: 대학교 중복 오류 수정 #510

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.

Sign up for GitHub

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

Merged
lsy1307 merged 4 commits into from
Oct 31, 2025
Merged

fix: 대학교 중복 오류 수정 #510

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7b1fd90
fix : 대학교 중복 오류 수정
lsy1307 Sep 22, 2025
04c8e38
fix : keyword에서 NPE이 발생하지 않도록 검증로직 추가
lsy1307 Sep 25, 2025
fe4a11e
refactor : 메서드명 수정
lsy1307 Sep 28, 2025
746c585
refactor: 테스트 코드 수정
lsy1307 Oct 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/main/java/com/example/solidconnection/application/service/ApplicationQueryService.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public ApplicationsResponse getApplicants(long siteUserId, String regionCode, St
// 1. 대학 지원 정보 필터링 (regionCode, keyword)
SiteUser siteUser = siteUserRepository.findById(siteUserId)
.orElseThrow(() -> new CustomException(USER_NOT_FOUND));
List<UnivApplyInfo> univApplyInfos = universityFilterRepository.findAllByRegionCodeAndKeywords(regionCode, List.of(keyword));
List<UnivApplyInfo> univApplyInfos = universityFilterRepository.findAllByRegionCodeAndKeywords(regionCode, List.of(keyword), term);
if (univApplyInfos.isEmpty()) {
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
return new ApplicationsResponse(List.of(), List.of(), List.of());
}
Expand Down
2 changes: 1 addition & 1 deletion ...m/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepository.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

public interface UnivApplyInfoFilterRepository {

List<UnivApplyInfo> findAllByRegionCodeAndKeywords(String regionCode, List<String> keywords);
List<UnivApplyInfo> findAllByRegionCodeAndKeywords(String regionCode, List<String> keywords, String term);

@Gyuhyeok99 Gyuhyeok99 Sep 26, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

함수명도 바꿔주시면 좋을 거 같습니다~

@lsy1307 lsy1307 Sep 28, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

반영했습니다!


List<UnivApplyInfo> findAllByFilter(LanguageTestType testType, String testScore, String term, List<String> countryKoreanNames);

Expand Down
3 changes: 2 additions & 1 deletion ...ample/solidconnection/university/repository/custom/UnivApplyInfoFilterRepositoryImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public UnivApplyInfoFilterRepositoryImpl(EntityManager em) {
}

@Override
public List<UnivApplyInfo> findAllByRegionCodeAndKeywords(String regionCode, List<String> keywords) {
public List<UnivApplyInfo> findAllByRegionCodeAndKeywords(String regionCode, List<String> keywords, String term) {
QUnivApplyInfo univApplyInfo = QUnivApplyInfo.univApplyInfo;
QUniversity university = QUniversity.university;
QCountry country = QCountry.country;
Expand All @@ -45,6 +45,7 @@ public List<UnivApplyInfo> findAllByRegionCodeAndKeywords(String regionCode, Lis
.where(
regionCodeEq(country, regionCode)
.and(countryOrUniversityContainsKeyword(country, university, keywords))
.and(univApplyInfo.term.eq(term))
)
Comment on lines 46 to 49

@coderabbitai coderabbitai Bot Sep 28, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

term null 처리 누락으로 즉시 예외 위험!
term 값이 null이면 Querydsl이 "eq(null) is not allowed" 예외를 곧장 던져서 /application 조회가 바로 깨집니다. 기존 인터페이스에는 term이 없었기 때문에 호출부에서 관성적으로 null을 넣을 여지가 있고, 설정 누락 시에도 즉시 장애가 나니 사전에 가드가 필요해요. 간단하게 null을 검증한 뒤 쿼리를 빌드하도록 바꿔 주세요.

@@
-        return queryFactory
+        if (term == null) {
+            throw new IllegalArgumentException("term must not be null");
+        }
+
+        return queryFactory
                 .selectFrom(univApplyInfo)
@@
-                                .and(univApplyInfo.term.eq(term))
+                                .and(univApplyInfo.term.eq(term))
🤖 Prompt for AI Agents 
In
src/main/java/com/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepositoryImpl.java
around lines 46 to 49, the code calls univApplyInfo.term.eq(term) without
guarding term which causes Querydsl to throw "eq(null) is not allowed" when term
is null; change the query construction to only add the term predicate when term
is non-null (e.g., create a termEq(term) helper that returns the predicate when
term!=null or null otherwise, or conditionally chain the .and(...) only if term
!= null) so the query builder never calls eq(null).

.distinct()
.fetch();
Expand Down
Loading