diff options
author | KN4CK3R <admin@oldschoolhack.me> | 2022-03-22 08:03:22 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-03-22 15:03:22 +0800 |
commit | 80fd25524e13dedbdc3527b32d008de152213a89 (patch) | |
tree | 63b2bfe4ffaf1dce12080cabdc2e845c8731a673 /routers/web/org | |
parent | 5495ba7660979973ba19e304786135da474e0e64 (diff) | |
download | gitea-80fd25524e13dedbdc3527b32d008de152213a89.tar.gz gitea-80fd25524e13dedbdc3527b32d008de152213a89.zip |
Renamed ctx.User to ctx.Doer. (#19161)
Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Diffstat (limited to 'routers/web/org')
-rw-r--r-- | routers/web/org/home.go | 10 | ||||
-rw-r--r-- | routers/web/org/members.go | 12 | ||||
-rw-r--r-- | routers/web/org/org.go | 6 | ||||
-rw-r--r-- | routers/web/org/setting.go | 2 | ||||
-rw-r--r-- | routers/web/org/teams.go | 4 |
5 files changed, 17 insertions, 17 deletions
diff --git a/routers/web/org/home.go b/routers/web/org/home.go index fc81ceb719..1466731b9e 100644 --- a/routers/web/org/home.go +++ b/routers/web/org/home.go @@ -39,7 +39,7 @@ func Home(ctx *context.Context) { org := ctx.Org.Organization - if !models.HasOrgOrUserVisible(org.AsUser(), ctx.User) { + if !models.HasOrgOrUserVisible(org.AsUser(), ctx.Doer) { ctx.NotFound("HasOrgOrUserVisible", nil) return } @@ -113,7 +113,7 @@ func Home(ctx *context.Context) { OwnerID: org.ID, OrderBy: orderBy, Private: ctx.IsSigned, - Actor: ctx.User, + Actor: ctx.Doer, Language: language, IncludeDescription: setting.UI.SearchRepoDescription, }) @@ -128,13 +128,13 @@ func Home(ctx *context.Context) { ListOptions: db.ListOptions{Page: 1, PageSize: 25}, } - if ctx.User != nil { - isMember, err := org.IsOrgMember(ctx.User.ID) + if ctx.Doer != nil { + isMember, err := org.IsOrgMember(ctx.Doer.ID) if err != nil { ctx.Error(http.StatusInternalServerError, "IsOrgMember") return } - opts.PublicOnly = !isMember && !ctx.User.IsAdmin + opts.PublicOnly = !isMember && !ctx.Doer.IsAdmin } members, _, err := models.FindOrgMembers(opts) diff --git a/routers/web/org/members.go b/routers/web/org/members.go index b8e7fa1ff5..9e65e61404 100644 --- a/routers/web/org/members.go +++ b/routers/web/org/members.go @@ -36,13 +36,13 @@ func Members(ctx *context.Context) { PublicOnly: true, } - if ctx.User != nil { - isMember, err := ctx.Org.Organization.IsOrgMember(ctx.User.ID) + if ctx.Doer != nil { + isMember, err := ctx.Org.Organization.IsOrgMember(ctx.Doer.ID) if err != nil { ctx.Error(http.StatusInternalServerError, "IsOrgMember") return } - opts.PublicOnly = !isMember && !ctx.User.IsAdmin + opts.PublicOnly = !isMember && !ctx.Doer.IsAdmin } total, err := models.CountOrgMembers(opts) @@ -80,13 +80,13 @@ func MembersAction(ctx *context.Context) { var err error switch ctx.Params(":action") { case "private": - if ctx.User.ID != uid && !ctx.Org.IsOwner { + if ctx.Doer.ID != uid && !ctx.Org.IsOwner { ctx.Error(http.StatusNotFound) return } err = models.ChangeOrgUserStatus(org.ID, uid, false) case "public": - if ctx.User.ID != uid && !ctx.Org.IsOwner { + if ctx.Doer.ID != uid && !ctx.Org.IsOwner { ctx.Error(http.StatusNotFound) return } @@ -105,7 +105,7 @@ func MembersAction(ctx *context.Context) { return } case "leave": - err = org.RemoveMember(ctx.User.ID) + err = org.RemoveMember(ctx.Doer.ID) if models.IsErrLastOrgOwner(err) { ctx.Flash.Error(ctx.Tr("form.last_org_owner")) ctx.JSON(http.StatusOK, map[string]interface{}{ diff --git a/routers/web/org/org.go b/routers/web/org/org.go index c66a0cd4c8..b641f768b2 100644 --- a/routers/web/org/org.go +++ b/routers/web/org/org.go @@ -29,7 +29,7 @@ const ( func Create(ctx *context.Context) { ctx.Data["Title"] = ctx.Tr("new_org") ctx.Data["DefaultOrgVisibilityMode"] = setting.Service.DefaultOrgVisibilityMode - if !ctx.User.CanCreateOrganization() { + if !ctx.Doer.CanCreateOrganization() { ctx.ServerError("Not allowed", errors.New(ctx.Tr("org.form.create_org_not_allowed"))) return } @@ -41,7 +41,7 @@ func CreatePost(ctx *context.Context) { form := *web.GetForm(ctx).(*forms.CreateOrgForm) ctx.Data["Title"] = ctx.Tr("new_org") - if !ctx.User.CanCreateOrganization() { + if !ctx.Doer.CanCreateOrganization() { ctx.ServerError("Not allowed", errors.New(ctx.Tr("org.form.create_org_not_allowed"))) return } @@ -59,7 +59,7 @@ func CreatePost(ctx *context.Context) { RepoAdminChangeTeamAccess: form.RepoAdminChangeTeamAccess, } - if err := models.CreateOrganization(org, ctx.User); err != nil { + if err := models.CreateOrganization(org, ctx.Doer); err != nil { ctx.Data["Err_OrgName"] = true switch { case user_model.IsErrUserAlreadyExist(err): diff --git a/routers/web/org/setting.go b/routers/web/org/setting.go index 404aac8894..72c25ec25d 100644 --- a/routers/web/org/setting.go +++ b/routers/web/org/setting.go @@ -96,7 +96,7 @@ func SettingsPost(ctx *context.Context) { org.Name = form.Name org.LowerName = strings.ToLower(form.Name) - if ctx.User.IsAdmin { + if ctx.Doer.IsAdmin { org.MaxRepoCreation = form.MaxRepoCreation } diff --git a/routers/web/org/teams.go b/routers/web/org/teams.go index f6e09eb4c8..d13c4e854d 100644 --- a/routers/web/org/teams.go +++ b/routers/web/org/teams.go @@ -69,9 +69,9 @@ func TeamsAction(ctx *context.Context) { ctx.Error(http.StatusNotFound) return } - err = ctx.Org.Team.AddMember(ctx.User.ID) + err = ctx.Org.Team.AddMember(ctx.Doer.ID) case "leave": - err = ctx.Org.Team.RemoveMember(ctx.User.ID) + err = ctx.Org.Team.RemoveMember(ctx.Doer.ID) if err != nil { if models.IsErrLastOrgOwner(err) { ctx.Flash.Error(ctx.Tr("form.last_org_owner")) |