You've already forked FrameTour-BE
refactor(member): 更新用户信息方法传递用户ID参数
- 在AppMemberController中从BaseContextHandler获取用户ID并传递给服务层 - 修改AppMemberServiceImpl中的update方法签名以接收用户ID参数 - 更新AppMemberService接口定义以包含用户ID参数 - 移除服务实现中重复的BaseContextHandler调用 - 确保用户信息更新时使用正确的用户上下文
This commit is contained in:
@@ -1,6 +1,7 @@
|
|||||||
package com.ycwl.basic.controller.mobile;
|
package com.ycwl.basic.controller.mobile;
|
||||||
|
|
||||||
import com.ycwl.basic.annotation.IgnoreToken;
|
import com.ycwl.basic.annotation.IgnoreToken;
|
||||||
|
import com.ycwl.basic.constant.BaseContextHandler;
|
||||||
import com.ycwl.basic.model.mobile.weChat.DTO.WeChatUserInfoDTO;
|
import com.ycwl.basic.model.mobile.weChat.DTO.WeChatUserInfoDTO;
|
||||||
import com.ycwl.basic.model.mobile.weChat.DTO.WeChatUserInfoUpdateDTO;
|
import com.ycwl.basic.model.mobile.weChat.DTO.WeChatUserInfoUpdateDTO;
|
||||||
import com.ycwl.basic.model.pc.member.resp.MemberRespVO;
|
import com.ycwl.basic.model.pc.member.resp.MemberRespVO;
|
||||||
@@ -67,7 +68,8 @@ public class AppMemberController {
|
|||||||
// 修改用户信息
|
// 修改用户信息
|
||||||
@PostMapping("/update")
|
@PostMapping("/update")
|
||||||
public ApiResponse<?> update(@RequestBody WeChatUserInfoUpdateDTO userInfoUpdateDTO) {
|
public ApiResponse<?> update(@RequestBody WeChatUserInfoUpdateDTO userInfoUpdateDTO) {
|
||||||
return memberService.update(userInfoUpdateDTO);
|
Long userId = Long.parseLong(BaseContextHandler.getUserId());
|
||||||
|
return memberService.update(userId, userInfoUpdateDTO);
|
||||||
}
|
}
|
||||||
|
|
||||||
// 新增或修改景区服务通知状态
|
// 新增或修改景区服务通知状态
|
||||||
|
|||||||
@@ -42,7 +42,7 @@ public interface AppMemberService {
|
|||||||
* @param userInfoUpdateDTO
|
* @param userInfoUpdateDTO
|
||||||
* @return
|
* @return
|
||||||
*/
|
*/
|
||||||
ApiResponse<?> update(WeChatUserInfoUpdateDTO userInfoUpdateDTO);
|
ApiResponse<?> update(Long memberId, WeChatUserInfoUpdateDTO userInfoUpdateDTO);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* 同意用户协议
|
* 同意用户协议
|
||||||
|
|||||||
@@ -139,9 +139,8 @@ public class AppMemberServiceImpl implements AppMemberService {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public ApiResponse<?> update(WeChatUserInfoUpdateDTO userInfoUpdateDTO) {
|
public ApiResponse<?> update(Long userId, WeChatUserInfoUpdateDTO userInfoUpdateDTO) {
|
||||||
if (StringUtils.isNotBlank(userInfoUpdateDTO.getNickname())) {
|
if (StringUtils.isNotBlank(userInfoUpdateDTO.getNickname())) {
|
||||||
Long userId = Long.parseLong(BaseContextHandler.getUserId());
|
|
||||||
MemberRespVO member = memberMapper.getById(userId);
|
MemberRespVO member = memberMapper.getById(userId);
|
||||||
if (member != null && member.getScenicId() != null) {
|
if (member != null && member.getScenicId() != null) {
|
||||||
MpConfigEntity scenicMpConfig = scenicRepository.getScenicMpConfig(member.getScenicId());
|
MpConfigEntity scenicMpConfig = scenicRepository.getScenicMpConfig(member.getScenicId());
|
||||||
@@ -154,7 +153,7 @@ public class AppMemberServiceImpl implements AppMemberService {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
MemberEntity memberEntity = new MemberEntity();
|
MemberEntity memberEntity = new MemberEntity();
|
||||||
memberEntity.setId(Long.parseLong(BaseContextHandler.getUserId()));
|
memberEntity.setId(userId);
|
||||||
memberEntity.setNickname(userInfoUpdateDTO.getNickname());
|
memberEntity.setNickname(userInfoUpdateDTO.getNickname());
|
||||||
memberEntity.setAvatarUrl(userInfoUpdateDTO.getAvatarUrl());
|
memberEntity.setAvatarUrl(userInfoUpdateDTO.getAvatarUrl());
|
||||||
memberEntity.setAgreement(userInfoUpdateDTO.getAgreement());
|
memberEntity.setAgreement(userInfoUpdateDTO.getAgreement());
|
||||||
|
|||||||
Reference in New Issue
Block a user