diff --git a/server/imserver/player.go b/server/imserver/player.go index 146f132d..e5c412f1 100644 --- a/server/imserver/player.go +++ b/server/imserver/player.go @@ -341,10 +341,9 @@ func (p *Player) CMApplyToGuild(hdr *f5.MsgHdr, msg *cs.CMApplyToGuild) { for targetAccountId := range guild.Members { target := playerMgr.GetPlayerByAccountId(targetAccountId) - if target == nil { - continue + if target != nil { + target.SendMsg(rspNotify) } - target.SendMsg(rspNotify) } } }, @@ -365,10 +364,9 @@ func (p *Player) CMApplyList(hdr *f5.MsgHdr, msg *cs.CMApplyList) { Level: q5.ToInt32(GuildMemberLevelDefault), } rspMember := p.FillMFGuildMember(member) - if rspMember == nil { - continue + if rspMember != nil { + rspMsg.Members = append(rspMsg.Members, rspMember) } - rspMsg.Members = append(rspMsg.Members, rspMember) } } p.SendMsg(rspMsg) @@ -399,10 +397,9 @@ func (p *Player) CMApprove(hdr *f5.MsgHdr, msg *cs.CMApprove) { for targetAccountId := range guild.Members { target := playerMgr.GetPlayerByAccountId(targetAccountId) - if target == nil { - continue + if target != nil { + target.SendMsg(rspNotify) } - target.SendMsg(rspNotify) } } }, @@ -449,10 +446,9 @@ func (p *Player) CMLeaveGuild(hdr *f5.MsgHdr, msg *cs.CMLeaveGuild) { for targetAccountId := range guild.Members { target := playerMgr.GetPlayerByAccountId(targetAccountId) - if target == nil { - continue + if target != nil { + target.SendMsg(rspNotify) } - target.SendMsg(rspNotify) } } }, @@ -484,10 +480,9 @@ func (p *Player) CMDismissMember(hdr *f5.MsgHdr, msg *cs.CMDismissMember) { for targetAccountId := range guild.Members { target := playerMgr.GetPlayerByAccountId(targetAccountId) - if target == nil { - continue + if target != nil { + target.SendMsg(rspNotify) } - target.SendMsg(rspNotify) } dismissAccount := playerMgr.GetPlayerByAccountId(msg.GetDismissAccountId()) @@ -527,11 +522,9 @@ func (p *Player) CMSetMemberLevel(hdr *f5.MsgHdr, msg *cs.CMSetMemberLevel) { for targetAccountId := range guild.Members { target := playerMgr.GetPlayerByAccountId(targetAccountId) - if target == nil { - continue + if target != nil { + target.SendMsg(rspNotify) } - - target.SendMsg(rspNotify) } } })