parseScope with owner/repo always sets owner to zero (#25987)

earl-warren · web-flow · commit ad4c09b59f16 · 2023-07-19T21:21:51.000+08:00
Refs: https://codeberg.org/forgejo/forgejo/pulls/1001
diff --git a/routers/private/actions.go b/routers/private/actions.go
@@ -77,9 +77,10 @@ func parseScope(ctx *context.PrivateContext, scope string) (ownerID, repoID int6
 	if err != nil {
 		return ownerID, repoID, err
 	}
+	ownerID = u.ID
 
 	if !found {
-		return u.ID, repoID, nil
+		return ownerID, repoID, nil
 	}
 
 	r, err := repo_model.GetRepositoryByName(u.ID, repoName)

Original file line number	Diff line number	Diff line change
`@@ -77,9 +77,10 @@ func parseScope(ctx *context.PrivateContext, scope string) (ownerID, repoID int6`
`77`	`77`	`if err != nil {`
`78`	`78`	`return ownerID, repoID, err`
`79`	`79`	`}`
	`80`	`+ ownerID = u.ID`
`80`	`81`
`81`	`82`	`if !found {`
`82`		`- return u.ID, repoID, nil`
	`83`	`+ return ownerID, repoID, nil`
`83`	`84`	`}`
`84`	`85`
`85`	`86`	`r, err := repo_model.GetRepositoryByName(u.ID, repoName)`