Browse Source

fix bug to deny to add orgnization as a member of an orgnization or team (#1815)

tags/v1.2.0-rc1
Lunny Xiao 7 years ago
parent
commit
3611a3e552
3 changed files with 14 additions and 0 deletions
  1. 2
    0
      options/locale/locale_en-US.ini
  2. 6
    0
      routers/org/members.go
  3. 6
    0
      routers/org/teams.go

+ 2
- 0
options/locale/locale_en-US.ini View File

@@ -259,6 +259,8 @@ enterred_invalid_owner_name = Please ensure that the owner name you entered is c
enterred_invalid_password = Please ensure the that password you entered is correct.
user_not_exist = The user does not exist.
last_org_owner = Removing the last user from the owner team is not allowed because there must always be at least one owner in any given organization.
cannot_add_org_to_team = Organization cannot be added as a team member.
cannot_invite_org_to_org = Organization cannot be invited as an organization member.

invalid_ssh_key = Sorry, we were not able to verify your SSH key: %s
invalid_gpg_key = Sorry, we were not able to verify your GPG key: %s

+ 6
- 0
routers/org/members.go View File

@@ -114,6 +114,12 @@ func Invitation(ctx *context.Context) {
return
}

if u.IsOrganization() {
ctx.Flash.Error(ctx.Tr("form.cannot_invite_org_to_org"))
ctx.Redirect(ctx.Org.OrgLink + "/invitations/new")
return
}

if err = org.AddMember(u.ID); err != nil {
ctx.Handle(500, " AddMember", err)
return

+ 6
- 0
routers/org/teams.go View File

@@ -88,6 +88,12 @@ func TeamsAction(ctx *context.Context) {
return
}

if u.IsOrganization() {
ctx.Flash.Error(ctx.Tr("form.cannot_add_org_to_team"))
ctx.Redirect(ctx.Org.OrgLink + "/teams/" + ctx.Org.Team.LowerName)
return
}

err = ctx.Org.Team.AddMember(u.ID)
page = "team"
}

Loading…
Cancel
Save