Refact/#178 pagination refact (#180)

* [REFACT] Pagination 정렬 누락 수정 및 최신순 정렬 * [REFACT] 사용자의 프로젝트 매칭 요청 리스트 조회에서 Pagination 정렬 누락 수정 및 최신순 정렬
Open-Eye-Im-Developer · Feb 28, 2024 · 5f1605a · 5f1605a
1 parent 75cb83a
commit 5f1605a
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 4 deletions.
diff --git a/...va/io/oeid/mogakgo/domain/profile/infrastructure/ProfileCardLikeRepositoryCustomImpl.java b/...va/io/oeid/mogakgo/domain/profile/infrastructure/ProfileCardLikeRepositoryCustomImpl.java
@@ -58,6 +58,8 @@ public CursorPaginationResult<UserProfileLikeInfoAPIRes> getLikeInfoBySender(
                 cursorIdCondition(pageable.getCursorId()),
                 senderIdEq(senderId)
             )
+            // 최근순
+            .orderBy(profileCardLike.id.desc())
             .limit(pageable.getPageSize() + 1)
             .fetch();
 
@@ -83,6 +85,6 @@ private BooleanExpression receiveridEq(Long receiverId) {
     }
 
     private BooleanExpression cursorIdCondition(Long cursorId) {
-        return cursorId != null ? profileCardLike.id.gt(cursorId) : null;
+        return cursorId != null ? profileCardLike.id.lt(cursorId) : null;
     }
 }
diff --git a/...n/java/io/oeid/mogakgo/domain/profile/infrastructure/ProfileCardRepositoryCustomImpl.java b/...n/java/io/oeid/mogakgo/domain/profile/infrastructure/ProfileCardRepositoryCustomImpl.java
@@ -35,6 +35,8 @@ public CursorPaginationResult<UserPublicApiResponse> findByConditionWithPaginati
                 regionEq(region),
                 deletedProfileCardEq()
             )
+            // 최근순
+            .orderBy(profileCard.id.desc())
             .limit(pageable.getPageSize() + 1)
             .fetch();
 
@@ -69,7 +71,7 @@ private BooleanExpression userIdEq(Long userId) {
     }
 
     private BooleanExpression cursorIdCondition(Long cursorId) {
-        return cursorId != null ? profileCard.id.gt(cursorId) : null;
+        return cursorId != null ? profileCard.id.lt(cursorId) : null;
     }
 
     private BooleanExpression deletedProfileCardEq() {

diff --git a/src/main/java/io/oeid/mogakgo/domain/project/infrastructure/ProjectRepositoryCustomImpl.java b/src/main/java/io/oeid/mogakgo/domain/project/infrastructure/ProjectRepositoryCustomImpl.java
@@ -43,6 +43,8 @@ public CursorPaginationResult<ProjectDetailAPIRes> findByConditionWithPagination
                 projectStatusEq(projectStatus),
                 createdAtEq(today)
             )
+            // 최근순
+            .orderBy(project.id.desc())
             .limit(pageable.getPageSize() + 1)
             .fetch();
 
@@ -90,7 +92,7 @@ public List<Region> getDensityRankProjectsByRegion(int limit) {
     }
 
     private BooleanExpression cursorIdCondition(Long cursorId) {
-        return cursorId != null ? project.id.gt(cursorId) : null;
+        return cursorId != null ? project.id.lt(cursorId) : null;
     }
 
     private BooleanExpression userIdEq(Long userId) {

diff --git a/...ogakgo/domain/project_join_req/infrastructure/ProjectJoinRequestRepositoryCustomImpl.java b/...ogakgo/domain/project_join_req/infrastructure/ProjectJoinRequestRepositoryCustomImpl.java
@@ -84,11 +84,13 @@ public CursorPaginationResult<ProjectJoinRequestDetailAPIRes> getBySenderIdWithP
     ) {
         List<ProjectJoinRequest> entities = jpaQueryFactory.selectFrom(projectJoinRequest)
             .where(
-                cursorIdCondition(pageable.getCursorId()),
+                cursorIdConditionForDesc(pageable.getCursorId()),
                 senderIdEq(senderId),
                 projectIdEq(projectId),
                 requestStatusEq(requestStatus)
             )
+            // 최근순
+            .orderBy(projectJoinRequest.id.desc())
             .limit(pageable.getPageSize() + 1)
             .fetch();
 
@@ -125,4 +127,8 @@ private BooleanExpression cursorIdCondition(Long cursorId) {
         return cursorId != null ? projectJoinRequest.id.gt(cursorId) : null;
     }
 
+    private BooleanExpression cursorIdConditionForDesc(Long cursorId) {
+        return cursorId != null ? projectJoinRequest.id.lt(cursorId) : null;
+    }
+
 }