diff --git a/modules/hero/api_buy.go b/modules/hero/api_buy.go
index 322db29bc..f680a2533 100644
--- a/modules/hero/api_buy.go
+++ b/modules/hero/api_buy.go
@@ -115,62 +115,12 @@ func (this *apiComp) Buy(session comm.IUserSession, req *pb.HeroBuyReq) (code pb
 		}
 	}
 
-	for _, v := range need {
-		if v.A == comm.AttrType && (v.T == comm.ResGold || v.T == comm.ResDiamond) {
-			// this.module.ModuleRtask.SendToRtask(session, comm.Rtype67, v.N, utils.ToInt32(v.T))
-			switch v.T {
-			case comm.ResGold:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 1))
-				break
-			case comm.ResExp:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 3))
-				break
-			case comm.ResDiamond:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.VipExp:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.ResTaskActive:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.ResFriend:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 4))
-				break
-			case comm.StarCoin:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 9))
-				break
-			case comm.SociatyCoin:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 8))
-				break
-			case comm.ArenaCoin:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 11))
-				break
-			case comm.ResPs:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 12))
-				break
-			case comm.Moongold:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 14))
-				break
-			case comm.Talent1:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.Talent2:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.Talent3:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.Talent4:
-				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
-				break
-			case comm.Merchantmoney:
-				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 15))
-				break
-			}
-
-		}
-	}
+	// for _, v := range need {
+	// 	if v.A == comm.AttrType && (v.T == comm.ResGold || v.T == comm.ResDiamond) {
+	// 		// this.module.ModuleRtask.SendToRtask(session, comm.Rtype67, v.N, utils.ToInt32(v.T))
+	// 		tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, utils.ToInt32(v.T)))
+	// 	}
+	// }
 	go this.module.ModuleRtask.TriggerTask(session.GetUserId(), tasks...)
 	session.SendMsg(string(this.module.GetType()), "buy", &pb.HeroBuyResp{
 		IsSucc: true,
diff --git a/modules/practice/api_getgymbuff.go b/modules/practice/api_getgymbuff.go
index 801a64f41..8bdb29197 100644
--- a/modules/practice/api_getgymbuff.go
+++ b/modules/practice/api_getgymbuff.go
@@ -28,6 +28,7 @@ func (this *apiComp) GetGymBuff(session comm.IUserSession, req *pb.PracticeGetGy
 		code = pb.ErrorCode_ConfigNoFound
 		return
 	}
+	go this.module.ModuleRtask.TriggerTask(session.GetUserId(), comm.GettaskParam(comm.Rtype185, 1))
 	session.SendMsg(string(this.module.GetType()), "getgymbuff", &pb.PracticeGetGymBuffResp{Buffid: buff.Id})
 	return
 }
diff --git a/modules/practice/api_gymrefresh.go b/modules/practice/api_gymrefresh.go
index de43dfffc..0774edd2d 100644
--- a/modules/practice/api_gymrefresh.go
+++ b/modules/practice/api_gymrefresh.go
@@ -67,7 +67,7 @@ func (this *apiComp) GymRefresh(session comm.IUserSession, req *pb.PracticeGymRe
 		"gymrefresh":  room.Gymrefresh,
 		"lastrefresh": room.Lastrefresh,
 	})
-	go this.module.ModuleRtask.TriggerTask(session.GetUserId(), comm.GettaskParam(comm.Rtype185, 1))
+
 	session.SendMsg(string(this.module.GetType()), "gymrefresh", &pb.PracticeGymRefreshResp{Lastaction: room.Gymaction, Refreshnum: room.Gymrefresh})
 	return
 }
diff --git a/modules/shop/api_buy.go b/modules/shop/api_buy.go
index 9d473f9ba..cccd6b15d 100644
--- a/modules/shop/api_buy.go
+++ b/modules/shop/api_buy.go
@@ -127,8 +127,58 @@ func (this *apiComp) Buy(session comm.IUserSession, req *pb.ShopBuyReq) (code pb
 	tasks = append(tasks, comm.GettaskParam(comm.Rtype66, 1, int32(req.ShopType)))
 	// this.module.ModuleRtask.SendToRtask(session, comm.Rtype66, 1, int32(req.ShopType))
 	for _, v := range need {
-		if v.A == comm.AttrType && (v.T == comm.ResGold || v.T == comm.ResDiamond) {
-			tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, utils.ToInt32(v.T)))
+		if v.A == comm.AttrType {
+			// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, utils.ToInt32(v.T)))
+			switch v.T {
+			case comm.ResGold:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 1))
+				break
+			case comm.ResExp:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 3))
+				break
+			case comm.ResDiamond:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.VipExp:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.ResTaskActive:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.ResFriend:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 4))
+				break
+			case comm.StarCoin:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 9))
+				break
+			case comm.SociatyCoin:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 8))
+				break
+			case comm.ArenaCoin:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 11))
+				break
+			case comm.ResPs:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 12))
+				break
+			case comm.Moongold:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 14))
+				break
+			case comm.Talent1:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.Talent2:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.Talent3:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.Talent4:
+				// tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 2))
+				break
+			case comm.Merchantmoney:
+				tasks = append(tasks, comm.GettaskParam(comm.Rtype67, v.N, 15))
+				break
+			}
 			// this.module.ModuleRtask.SendToRtask(session, comm.Rtype67, v.N, utils.ToInt32(v.T))
 		}
 	}