From f4ddd7c2f7c51fb9d4357e8ddcf02c39a3b0b1c4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=AE=B7=E5=8B=87?= <shyinyong@outlook.com>
Date: Wed, 20 Sep 2023 19:29:10 +0800
Subject: [PATCH] save

---
 server/imserver/player.go | 31 ++++++++++++-------------------
 1 file changed, 12 insertions(+), 19 deletions(-)

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)
 				}
 			}
 		})