Use correct form field for allowed force push users in branch protection API (go-gitea#35894)

zorrobiwan · GiteaBot · commit 3f136599fdc8 · 2025-11-09T21:54:38.000Z
Test was wrong and preventing update of force push allow users list by the API Resolves go-gitea#35893 Signed-off-by: Alberty Pascal <github@albertyorban.be>
diff --git a/routers/api/v1/repo/branch.go b/routers/api/v1/repo/branch.go
@@ -897,7 +897,7 @@ func EditBranchProtection(ctx *context.APIContext) {
 	} else {
 		whitelistUsers = protectBranch.WhitelistUserIDs
 	}
-	if form.ForcePushAllowlistDeployKeys != nil {
+	if form.ForcePushAllowlistUsernames != nil {
 		forcePushAllowlistUsers, err = user_model.GetUserIDsByNames(ctx, form.ForcePushAllowlistUsernames, false)
 		if err != nil {
 			if user_model.IsErrUserNotExist(err) {

Original file line number	Diff line number	Diff line change
`@@ -897,7 +897,7 @@ func EditBranchProtection(ctx *context.APIContext) {`
`897`	`897`	`} else {`
`898`	`898`	`whitelistUsers = protectBranch.WhitelistUserIDs`
`899`	`899`	`}`
`900`		`- if form.ForcePushAllowlistDeployKeys != nil {`
	`900`	`+ if form.ForcePushAllowlistUsernames != nil {`
`901`	`901`	`forcePushAllowlistUsers, err = user_model.GetUserIDsByNames(ctx, form.ForcePushAllowlistUsernames, false)`
`902`	`902`	`if err != nil {`
`903`	`903`	`if user_model.IsErrUserNotExist(err) {`