Remove direct user adding to organization members

Author: lafriks

options/locale/locale_en-US.ini

@@ -274,7 +274,6 @@ enterred_invalid_password = Please ensure the that password you entered is corre
 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
@@ -1028,7 +1027,6 @@ org_name_helper = Great organization names are short and memorable.
 create_org = Create Organization
 repo_updated = Updated
 people = People
-invite_someone = Invite Someone
 teams = Teams
 lower_members = members
 lower_repositories = repositories

routers/org/members.go

@@ -17,8 +17,6 @@ import (
 const (
 	// tplMembers template for organization members page
 	tplMembers base.TplName = "org/member/members"
-	// tplMemberInvite template for orgnization invite page
-	tplMemberInvite base.TplName = "org/member/invite"
 )
 
 // Members render orgnization users page
@@ -94,41 +92,3 @@ func MembersAction(ctx *context.Context) {
 		ctx.Redirect(setting.AppSubURL + "/")
 	}
 }
-
-// Invitation render organization invitation page
-func Invitation(ctx *context.Context) {
-	org := ctx.Org.Organization
-	ctx.Data["Title"] = org.FullName
-	ctx.Data["PageIsOrgMembers"] = true
-
-	if ctx.Req.Method == "POST" {
-		uname := ctx.Query("uname")
-		u, err := models.GetUserByName(uname)
-		if err != nil {
-			if models.IsErrUserNotExist(err) {
-				ctx.Flash.Error(ctx.Tr("form.user_not_exist"))
-				ctx.Redirect(ctx.Org.OrgLink + "/invitations/new")
-			} else {
-				ctx.Handle(500, " GetUserByName", err)
-			}
-			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
-		}
-
-		log.Trace("New member added(%s): %s", org.Name, u.Name)
-		ctx.Redirect(ctx.Org.OrgLink + "/members")
-		return
-	}
-
-	ctx.HTML(200, tplMemberInvite)
-}

routers/routes/routes.go

@@ -407,8 +407,6 @@ func RegisterRoutes(m *macaron.Macaron) {
 
 				m.Route("/delete", "GET,POST", org.SettingsDelete)
 			})
-
-			m.Route("/invitations/new", "GET,POST", org.Invitation)
 		}, context.OrgAssignment(true, true))
 	}, reqSignIn)
 	// ***** END: Organization *****

templates/org/home.tmpl

@@ -52,11 +52,6 @@
 						{{end}}
 					{{end}}
 				</div>
-				{{if .IsOrganizationOwner}}
-					<div class="ui bottom attached segment">
-						<a class="ui blue small button" href="{{.OrgLink}}/invitations/new">{{.i18n.Tr "org.invite_someone"}}</a>
-					</div>
-				{{end}}
 
 				{{if .IsOrganizationMember}}
 					<div class="ui top attached header">

templates/org/member/invite.tmpl

@@ -3,12 +3,6 @@
 	{{template "org/header" .}}
 	<div class="ui container">
 		{{template "base/alert" .}}
-		{{if .IsOrganizationOwner}}
-			<div class="text right">
-				<a class="ui blue button" href="{{.OrgLink}}/invitations/new"><i class="octicon octicon-repo-create"></i> {{.i18n.Tr "org.invite_someone"}}</a>
-			</div>
-			<div class="ui divider"></div>
-		{{end}}
 
 		<div class="list">
 			{{range .Members}}