blackteay/open-im-server
1
0
Fork 0
mirror of https://github.com/openimsdk/open-im-server.git synced 2025-08-07 19:50:07 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix GetUserInGroupMembers nil panic (#1324)

Browse Source
This commit is contained in:
a3d21 2023-11-06 12:21:31 +08:00 committed by GitHub
parent 3d3781f8d8
commit 1aef30dac4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 37 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

64
internal/rpc/group/group.go
View File

@ -475,11 +475,13 @@ func (s *groupServer) GetGroupAllMember(ctx context.Context, req *pbgroup.GetGro
return nil, err return nil, err
} }
resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo { resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo {
if e.Nickname == "" { if userInfo, ok := publicUserInfoMap[e.UserID]; ok {
e.Nickname = publicUserInfoMap[e.UserID].Nickname if e.Nickname == "" {
} e.Nickname = userInfo.Nickname
if e.FaceURL == "" { }
e.FaceURL = publicUserInfoMap[e.UserID].FaceURL if e.FaceURL == "" {
e.FaceURL = userInfo.FaceURL
}
} }
return convert.Db2PbGroupMember(e) return convert.Db2PbGroupMember(e)
}) })
@ -626,11 +628,13 @@ func (s *groupServer) GetGroupMembersInfo(ctx context.Context, req *pbgroup.GetG
return nil, err return nil, err
} }
resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo { resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo {
if e.Nickname == "" { if userInfo, ok := publicUserInfoMap[e.UserID]; ok {
e.Nickname = publicUserInfoMap[e.UserID].Nickname if e.Nickname == "" {
} e.Nickname = userInfo.Nickname
if e.FaceURL == "" { }
e.FaceURL = publicUserInfoMap[e.UserID].FaceURL if e.FaceURL == "" {
e.FaceURL = userInfo.FaceURL
}
} }
return convert.Db2PbGroupMember(e) return convert.Db2PbGroupMember(e)
}) })
@ -1073,18 +1077,20 @@ func (s *groupServer) GetGroupMembersCMS(ctx context.Context, req *pbgroup.GetGr
return nil, err return nil, err
} }
resp.Total = total resp.Total = total
nameMap, err := s.GetPublicUserInfoMap(ctx, utils.Filter(members, func(e *relationtb.GroupMemberModel) (string, bool) { publicUserInfoMap, err := s.GetPublicUserInfoMap(ctx, utils.Filter(members, func(e *relationtb.GroupMemberModel) (string, bool) {
return e.UserID, e.Nickname == "" || e.FaceURL == "" return e.UserID, e.Nickname == "" || e.FaceURL == ""
}), true) }), true)
if err != nil { if err != nil {
return nil, err return nil, err
} }
resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo { resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo {
if e.Nickname == "" { if userInfo, ok := publicUserInfoMap[e.UserID]; ok {
e.Nickname = nameMap[e.UserID].Nickname if e.Nickname == "" {
} e.Nickname = userInfo.Nickname
if e.FaceURL == "" { }
e.FaceURL = nameMap[e.UserID].FaceURL if e.FaceURL == "" {
e.FaceURL = userInfo.FaceURL
}
} }
return convert.Db2PbGroupMember(e) return convert.Db2PbGroupMember(e)
}) })
@ -1471,17 +1477,19 @@ func (s *groupServer) GetUserInGroupMembers(ctx context.Context, req *pbgroup.Ge
return nil, err return nil, err
} }
publicUserInfoMap, err := s.GetPublicUserInfoMap(ctx, utils.Filter(members, func(e *relationtb.GroupMemberModel) (string, bool) { publicUserInfoMap, err := s.GetPublicUserInfoMap(ctx, utils.Filter(members, func(e *relationtb.GroupMemberModel) (string, bool) {
return e.UserID, e.Nickname == "" return e.UserID, e.Nickname == "" || e.FaceURL == ""
}), true) }), true)
if err != nil { if err != nil {
return nil, err return nil, err
} }
resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo { resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo {
if e.Nickname == "" { if userInfo, ok := publicUserInfoMap[e.UserID]; ok {
e.Nickname = publicUserInfoMap[e.UserID].Nickname if e.Nickname == "" {
} e.Nickname = userInfo.Nickname
if e.FaceURL == "" { }
e.FaceURL = publicUserInfoMap[e.UserID].FaceURL if e.FaceURL == "" {
e.FaceURL = userInfo.FaceURL
}
} }
return convert.Db2PbGroupMember(e) return convert.Db2PbGroupMember(e)
}) })
@ -1513,11 +1521,13 @@ func (s *groupServer) GetGroupMemberRoleLevel(ctx context.Context, req *pbgroup.
return nil, err return nil, err
} }
resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo { resp.Members = utils.Slice(members, func(e *relationtb.GroupMemberModel) *sdkws.GroupMemberFullInfo {
if e.Nickname == "" { if userInfo, ok := publicUserInfoMap[e.UserID]; ok {
e.Nickname = publicUserInfoMap[e.UserID].Nickname if e.Nickname == "" {
} e.Nickname = userInfo.Nickname
if e.FaceURL == "" { }
e.FaceURL = publicUserInfoMap[e.UserID].FaceURL if e.FaceURL == "" {
e.FaceURL = userInfo.FaceURL
}
} }
return convert.Db2PbGroupMember(e) return convert.Db2PbGroupMember(e)
}) })