Skip to content
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

[Fix] 모임 일정 전체보기 (익명) #158

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

f1v3-dev
Copy link
Member

@f1v3-dev f1v3-dev commented Dec 3, 2024

#️⃣ 연관된 이슈

resolves #157

📝 작업 내용

모임이 익명(isAnonymous)인 경우에도 사용자의 이름이 보이는 오류를 수정하였습니다.

MeetingRepositoryImpl.java

        // 3. 익명 모임이 아닌 경우, 일정을 할당한 사용자의 닉네임 조회 후 추가
        Boolean isAnonymous = from(meeting)
                .where(meeting.meetingUuid.eq(uuid))
                .select(meeting.isAnonymous)
                .fetchOne();

        if (Boolean.FALSE.equals(isAnonymous)) {
            for (MeetingTime meetingTime : meetingTimeList) {
                List<String> nicknames = from(dateOfSchedule)
                        .join(dateOfSchedule.schedule, schedule)
                        .join(schedule.meeting, meeting)
                        .where(meeting.meetingUuid.eq(uuid)
                                .and(dateOfSchedule.dateOfScheduleStart.eq(meetingTime.getStartTime()))
                                .and(dateOfSchedule.dateOfScheduleEnd.eq(meetingTime.getEndTime()))
                                .and(schedule.assignedAt.loe(requestTime)))
                        .select(schedule.scheduleNickname)
                        .fetch();

                meetingTime.addMemberNames(nicknames);
            }
        }

익명이 아닌 경우에만 닉네임을 조회하여 입력한 사람의 정보를 가져옵니다.

@f1v3-dev f1v3-dev requested a review from RTUnu12 December 3, 2024 03:42
@f1v3-dev f1v3-dev self-assigned this Dec 3, 2024
@f1v3-dev f1v3-dev merged commit 38378ee into develop Dec 3, 2024
@f1v3-dev f1v3-dev deleted the fix/#157-meeting branch December 11, 2024 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

전체 일정 보기 - 익명
1 participant