Skip to content

Commit

Permalink
Merge pull request #144 from MUIT-UMC/develop
Browse files Browse the repository at this point in the history
[merge] 250220 / 37th deploy
  • Loading branch information
yhi9839 authored Feb 19, 2025
2 parents ee09675 + e989290 commit dd3fe67
Show file tree
Hide file tree
Showing 6 changed files with 56 additions and 0 deletions.
10 changes: 10 additions & 0 deletions src/main/java/muit/backend/controller/MemberController.java
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,16 @@ public ApiResponse<MyPageResponseDTO> changePassword(@RequestHeader("Authorizati
return ApiResponse.onSuccess(myPageResponseDTO);
}

@PatchMapping("{memberId}/changeAddress")
@Operation(summary = "회원 정보 수정 - 주소")
public ApiResponse<MyPageResponseDTO> changeAddress(@RequestHeader("Authorization") String authorizationHeader,
@PathVariable("memberId") Long memberId,
@RequestBody AddressChangeRequestDTO dto) {
Member member = memberService.getMemberByToken(authorizationHeader);
MyPageResponseDTO myPageResponseDTO = memberService.changeAddress(member.getId(), memberId, dto);
return ApiResponse.onSuccess(myPageResponseDTO);
}

@PostMapping("{memberId}/checkPassword")
@Operation(summary = "회원 정보 변경 전 + 소극장 등록 전 비밀 번호 확인하는 api")
public ApiResponse<Boolean> checkPassword(@RequestHeader("Authorization") String authorizationHeader, @RequestBody PasswordRequestDTO dto) {
Expand Down
5 changes: 5 additions & 0 deletions src/main/java/muit/backend/domain/entity/member/Member.java
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,11 @@ public void changeEmail(String newEmail) {
this.email = newEmail;
}

public void changeAddress(String newAddress) {
this.address = newAddress;
}


public void updateMember(ManageMemberRequestDTO.UpdateMemberRequestDTO requestDTO) {
if (requestDTO.getUsername() != null) {
this.username = requestDTO.getUsername();
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
package muit.backend.dto.memberDTO;

import lombok.AllArgsConstructor;
import lombok.Builder;
import lombok.Getter;
import lombok.NoArgsConstructor;

@Builder
@AllArgsConstructor
@NoArgsConstructor
@Getter
public class AddressChangeRequestDTO {
private String address;
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,6 @@ public class MyPageResponseDTO {
private String username;
private String email;
private String phone;
private String address;
private ActiveStatus status;
}
1 change: 1 addition & 0 deletions src/main/java/muit/backend/service/MemberService.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ public interface MemberService {
public MyPageResponseDTO changeUsername(Long tokenId, Long memberId, UserNameChangeRequestDTO dto);
public MyPageResponseDTO changeEmail(Long tokenId, Long memberId, EmailVerifyRequestDTO dto);
public MyPageResponseDTO changePassword(Long tokenId, Long memberId, PasswordChangeRequestDTO dto);
public MyPageResponseDTO changeAddress(Long tokenId, Long memberId, AddressChangeRequestDTO dto);


public List<MusicalResponseDTO.MusicalHomeDTO> getLikeMusicals(Member member);
Expand Down
25 changes: 25 additions & 0 deletions src/main/java/muit/backend/service/MemberServiceImpl.java
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,7 @@ public MyPageResponseDTO getMyPage(Long tokenId, Long memberId){
.username(member.getUsername())
.phone(member.getPhone())
.email(member.getEmail())
.address(member.getAddress())
.status(member.getActiveStatus())
.build();
}
Expand All @@ -173,6 +174,7 @@ public MyPageResponseDTO deactivateMember(Long tokenId, Long memberId){
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();

}
Expand All @@ -198,6 +200,7 @@ public MyPageResponseDTO changePhoneNumber(Long tokenId, Long memberId, PhoneCha
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();
}

Expand All @@ -216,6 +219,7 @@ public MyPageResponseDTO changeUsername(Long tokenId, Long memberId, UserNameCha
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();
}
@Transactional
Expand All @@ -233,6 +237,7 @@ public MyPageResponseDTO changeEmail(Long tokenId, Long memberId, EmailVerifyReq
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();
}
@Transactional
Expand All @@ -258,6 +263,26 @@ public MyPageResponseDTO changePassword(Long tokenId, Long memberId, PasswordCha
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();
}

@Transactional
@Override
public MyPageResponseDTO changeAddress(Long tokenId, Long memberId, AddressChangeRequestDTO dto){
Member member = memberRepository.findById(memberId).orElseThrow(() -> new GeneralException(ErrorStatus.MEMBER_NOT_FOUND));
if (!tokenId.equals(memberId)) {
throw new GeneralException(ErrorStatus.MEMBER_NOT_AUTHORIZED);
}

member.changeAddress(dto.getAddress());
return MyPageResponseDTO.builder()
.id(memberId)
.phone(member.getPhone())
.email(member.getEmail())
.name(member.getName())
.username(member.getUsername())
.address(member.getAddress())
.status(member.getActiveStatus()).build();
}

Expand Down

0 comments on commit dd3fe67

Please sign in to comment.