Merge remote-tracking branch 'origin/master'

internal/logic/task/task.go

@@ -525,15 +525,15 @@ func (s *sTask) GetTaskList(ctx context.Context, in *model.GetTaskListV2In) (out
 				// 存在用户记录，自行判断用户是否完成任务
 				if v.UserTimes-one["user_times"].Int64() >= v.TargetTimes {
 					completeTime := gtime.Now()
-
-					// 用户任务完成修改任务记录完成时间
-					_, err := dao.UserTasks.Ctx(ctx).Where(do.UserTasks{UserId: in.UserId, TaskId: v.TaskID}).Data(do.UserTasks{CompletedAt: completeTime}).Update()
-					if err != nil {
-						return nil, ecode.Fail.Sub("修改用户任务完成时间失败")
-					}
 					userTaskStatus := one["status"].Int64()
 					if userTaskStatus == 1 {
 						if err := dao.UserTasks.Transaction(ctx, func(ctx context.Context, tx gdb.TX) (err error) {
+							// 用户任务完成修改任务记录完成时间
+							_, err = dao.UserTasks.Ctx(ctx).Where(do.UserTasks{UserId: in.UserId, TaskId: v.TaskID}).Data(do.UserTasks{CompletedAt: completeTime}).Update()
+							if err != nil {
+								return ecode.Fail.Sub("修改用户任务完成时间失败")
+							}
+
 							if _, err := dao.UserTasks.Ctx(ctx).WherePri(one["id"].Int64()).Where(do.UserTasks{Status: 1}).Data(do.UserTasks{Status: 3}).Update(); err != nil {
 								return ecode.Fail.Sub("修改用户任务状态失败")
 							}