From 9222e191aab9397b6a7c6713d1b0b0b96b421de7 Mon Sep 17 00:00:00 2001
From: Christopher Speller <crspeller@gmail.com>
Date: Tue, 18 Aug 2015 11:53:18 -0400
Subject: Requiring team admin to import and fixing a few style issues

---
 api/slackimport.go         | 8 +++-----
 api/team.go                | 3 +--
 web/react/utils/client.jsx | 3 +--
 3 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/api/slackimport.go b/api/slackimport.go
index c662181cb..ca3fdf3d1 100644
--- a/api/slackimport.go
+++ b/api/slackimport.go
@@ -24,7 +24,7 @@ type SlackChannel struct {
 
 type SlackUser struct {
 	Id       string            `json:"id"`
-	UserName string            `json:"name"`
+	Username string            `json:"name"`
 	Profile  map[string]string `json:"profile"`
 }
 
@@ -85,7 +85,6 @@ func SlackAddUsers(teamId string, slackusers []SlackUser, log *bytes.Buffer) map
 	log.WriteString("\n Users Created\n")
 	log.WriteString("===============\n\n")
 
-	// Add users
 	addedUsers := make(map[string]*model.User)
 	for _, sUser := range slackusers {
 		firstName := ""
@@ -101,7 +100,7 @@ func SlackAddUsers(teamId string, slackusers []SlackUser, log *bytes.Buffer) map
 
 		newUser := model.User{
 			TeamId:    teamId,
-			Username:  sUser.UserName,
+			Username:  sUser.Username,
 			FirstName: firstName,
 			LastName:  lastName,
 			Email:     sUser.Profile["email"],
@@ -112,7 +111,7 @@ func SlackAddUsers(teamId string, slackusers []SlackUser, log *bytes.Buffer) map
 			addedUsers[sUser.Id] = mUser
 			log.WriteString("Email, Password: " + newUser.Email + ", " + password + "\n")
 		} else {
-			log.WriteString("Unable to import user: " + sUser.UserName)
+			log.WriteString("Unable to import user: " + sUser.Username)
 		}
 	}
 
@@ -167,7 +166,6 @@ func SlackAddChannels(teamId string, slackchannels []SlackChannel, posts map[str
 	log.WriteString("\n Channels Added \n")
 	log.WriteString("=================\n\n")
 
-	// Add channels
 	addedChannels := make(map[string]*model.Channel)
 	for _, sChannel := range slackchannels {
 		newChannel := model.Channel{
diff --git a/api/team.go b/api/team.go
index 12d087aea..a331e9e34 100644
--- a/api/team.go
+++ b/api/team.go
@@ -494,8 +494,7 @@ func getMyTeam(c *Context, w http.ResponseWriter, r *http.Request) {
 }
 
 func importTeam(c *Context, w http.ResponseWriter, r *http.Request) {
-
-	if !c.HasPermissionsToTeam(c.Session.TeamId, "import") {
+	if !c.HasPermissionsToTeam(c.Session.TeamId, "import") || !c.IsTeamAdmin(c.Session.UserId) {
 		c.Err = model.NewAppError("importTeam", "Only a team admin can import data.", "userId="+c.Session.UserId)
 		c.Err.StatusCode = http.StatusForbidden
 		return
diff --git a/web/react/utils/client.jsx b/web/react/utils/client.jsx
index b25702589..103292abf 100644
--- a/web/react/utils/client.jsx
+++ b/web/react/utils/client.jsx
@@ -1,4 +1,3 @@
-// Copyright (c) 2015 Spinpunch, Inc. All Rights Reserved.
 // See License.txt for license information.
 
 var BrowserStore = require('../stores/browser_store.jsx');
@@ -859,7 +858,7 @@ module.exports.uploadProfileImage = function(imageData, success, error) {
 
 module.exports.importSlack = function(fileData, success, error) {
     $.ajax({
-		url: '/api/v1/teams/import_team',
+        url: '/api/v1/teams/import_team',
         type: 'POST',
         data: fileData,
         cache: false,
-- 
cgit v1.2.3-1-g7c22