Fix bug for ctx usage (#26763)

lunny · web-flow · commit a1cec4141e9b · 2023-08-28T12:20:31.000+08:00
Fix #26684 Backport #26762
diff --git a/services/pull/review.go b/services/pull/review.go
@@ -334,9 +334,9 @@ func DismissApprovalReviews(ctx context.Context, doer *user_model.User, pull *is
 		return err
 	}
 
-	return db.WithTx(ctx, func(subCtx context.Context) error {
+	return db.WithTx(ctx, func(ctx context.Context) error {
 		for _, review := range reviews {
-			if err := issues_model.DismissReview(subCtx, review, true); err != nil {
+			if err := issues_model.DismissReview(ctx, review, true); err != nil {
 				return err
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -334,9 +334,9 @@ func DismissApprovalReviews(ctx context.Context, doer user_model.User, pull is`
`334`	`334`	`return err`
`335`	`335`	`}`
`336`	`336`
`337`		`- return db.WithTx(ctx, func(subCtx context.Context) error {`
	`337`	`+ return db.WithTx(ctx, func(ctx context.Context) error {`
`338`	`338`	`for _, review := range reviews {`
`339`		`- if err := issues_model.DismissReview(subCtx, review, true); err != nil {`
	`339`	`+ if err := issues_model.DismissReview(ctx, review, true); err != nil {`
`340`	`340`	`return err`
`341`	`341`	`}`
`342`	`342`