#1692 APIs: Users Followers

- User profile un/follow
- List user's followers/following

6 files changed, 290 insertions, 76 deletions

diff --git a/routers/api/v1/api.go b/routers/api/v1/api.go
index cc169f708d..e1a23f0d40 100644
--- a/routers/api/v1/api.go
+++ b/routers/api/v1/api.go
@@ -135,19 +135,32 @@ func RegisterRoutes(m *macaron.Macaron) {
 		m.Group("/users", func() {
 			m.Group("/:username", func() {
 				m.Get("/keys", user.ListPublicKeys)
+
+				m.Get("/followers", user.ListFollowers)
+				m.Group("/following", func() {
+					m.Get("", user.ListFollowing)
+					m.Get("/:target", user.CheckFollowing)
+				})
 			})
 		}, ReqToken())
 
 		m.Group("/user", func() {
+			m.Combo("/emails").Get(user.ListEmails).
+				Post(bind(api.CreateEmailOption{}), user.AddEmail).
+				Delete(bind(api.CreateEmailOption{}), user.DeleteEmail)
+
+			m.Get("/followers", user.ListMyFollowers)
+			m.Group("/following", func() {
+				m.Get("", user.ListMyFollowing)
+				m.Combo("/:username").Get(user.CheckMyFollowing).Put(user.Follow).Delete(user.Unfollow)
+			})
+
 			m.Group("/keys", func() {
 				m.Combo("").Get(user.ListMyPublicKeys).
 					Post(bind(api.CreateKeyOption{}), user.CreatePublicKey)
 				m.Combo("/:id").Get(user.GetPublicKey).
 					Delete(user.DeletePublicKey)
 			})
-			m.Combo("/emails").Get(user.ListEmails).
-				Post(bind(api.CreateEmailOption{}), user.AddEmail).
-				Delete(bind(api.CreateEmailOption{}), user.DeleteEmail)
 		}, ReqToken())
 
 		// Repositories
diff --git a/routers/api/v1/user/followers.go b/routers/api/v1/user/followers.go
new file mode 100644
index 0000000000..0a28979647
--- /dev/null
+++ b/routers/api/v1/user/followers.go
@@ -0,0 +1,121 @@
+// Copyright 2015 The Gogs Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package user
+
+import (
+	api "github.com/gogits/go-gogs-client"
+
+	"github.com/gogits/gogs/models"
+	"github.com/gogits/gogs/modules/middleware"
+	"github.com/gogits/gogs/routers/api/v1/convert"
+)
+
+func responseApiUsers(ctx *middleware.Context, users []*models.User) {
+	apiUsers := make([]*api.User, len(users))
+	for i := range users {
+		apiUsers[i] = convert.ToApiUser(users[i])
+	}
+	ctx.JSON(200, &apiUsers)
+}
+
+func listUserFollowers(ctx *middleware.Context, u *models.User) {
+	users, err := u.GetFollowers(ctx.QueryInt("page"))
+	if err != nil {
+		ctx.APIError(500, "GetUserFollowers", err)
+		return
+	}
+	responseApiUsers(ctx, users)
+}
+
+func ListMyFollowers(ctx *middleware.Context) {
+	listUserFollowers(ctx, ctx.User)
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#list-followers-of-a-user
+func ListFollowers(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	listUserFollowers(ctx, u)
+}
+
+func listUserFollowing(ctx *middleware.Context, u *models.User) {
+	users, err := u.GetFollowing(ctx.QueryInt("page"))
+	if err != nil {
+		ctx.APIError(500, "GetFollowing", err)
+		return
+	}
+	responseApiUsers(ctx, users)
+}
+
+func ListMyFollowing(ctx *middleware.Context) {
+	listUserFollowing(ctx, ctx.User)
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#list-users-followed-by-another-user
+func ListFollowing(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	listUserFollowing(ctx, u)
+}
+
+func checkUserFollowing(ctx *middleware.Context, u *models.User, followID int64) {
+	if u.IsFollowing(followID) {
+		ctx.Status(204)
+	} else {
+		ctx.Error(404)
+	}
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#check-if-you-are-following-a-user
+func CheckMyFollowing(ctx *middleware.Context) {
+	target := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	checkUserFollowing(ctx, ctx.User, target.Id)
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#check-if-one-user-follows-another
+func CheckFollowing(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	target := GetUserByParamsName(ctx, ":target")
+	if ctx.Written() {
+		return
+	}
+	checkUserFollowing(ctx, u, target.Id)
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#follow-a-user
+func Follow(ctx *middleware.Context) {
+	target := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	if err := models.FollowUser(ctx.User.Id, target.Id); err != nil {
+		ctx.APIError(500, "FollowUser", err)
+		return
+	}
+	ctx.Status(204)
+}
+
+// https://github.com/gogits/go-gogs-client/wiki/Users-Followers#unfollow-a-user
+func Unfollow(ctx *middleware.Context) {
+	target := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	if err := models.UnfollowUser(ctx.User.Id, target.Id); err != nil {
+		ctx.APIError(500, "UnfollowUser", err)
+		return
+	}
+	ctx.Status(204)
+}
diff --git a/routers/repo/repo.go b/routers/repo/repo.go
index c58d256a72..2879599744 100644
--- a/routers/repo/repo.go
+++ b/routers/repo/repo.go
@@ -244,11 +244,7 @@ func Action(ctx *middleware.Context) {
 	}
 
 	if err != nil {
-		log.Error(4, "Action(%s): %v", ctx.Params(":action"), err)
-		ctx.JSON(200, map[string]interface{}{
-			"ok":  false,
-			"err": err.Error(),
-		})
+		ctx.Handle(500, fmt.Sprintf("Action (%s)", ctx.Params(":action")), err)
 		return
 	}
 
diff --git a/routers/repo/view.go b/routers/repo/view.go
index 191f123454..25e312b1c1 100644
--- a/routers/repo/view.go
+++ b/routers/repo/view.go
@@ -217,7 +217,7 @@ func Home(ctx *middleware.Context) {
 	ctx.HTML(200, HOME)
 }
 
-func renderItems(ctx *middleware.Context, total int, getter func(page int) ([]*models.User, error)) {
+func RenderUserCards(ctx *middleware.Context, total int, getter func(page int) ([]*models.User, error), tpl base.TplName) {
 	page := ctx.QueryInt("page")
 	if page <= 0 {
 		page = 1
@@ -230,21 +230,23 @@ func renderItems(ctx *middleware.Context, total int, getter func(page int) ([]*m
 		ctx.Handle(500, "getter", err)
 		return
 	}
-	ctx.Data["Watchers"] = items
+	ctx.Data["Cards"] = items
 
-	ctx.HTML(200, WATCHERS)
+	ctx.HTML(200, tpl)
 }
 
 func Watchers(ctx *middleware.Context) {
 	ctx.Data["Title"] = ctx.Tr("repo.watchers")
+	ctx.Data["CardsTitle"] = ctx.Tr("repo.watchers")
 	ctx.Data["PageIsWatchers"] = true
-	renderItems(ctx, ctx.Repo.Repository.NumWatches, ctx.Repo.Repository.GetWatchers)
+	RenderUserCards(ctx, ctx.Repo.Repository.NumWatches, ctx.Repo.Repository.GetWatchers, WATCHERS)
 }
 
 func Stars(ctx *middleware.Context) {
 	ctx.Data["Title"] = ctx.Tr("repo.stargazers")
+	ctx.Data["CardsTitle"] = ctx.Tr("repo.stargazers")
 	ctx.Data["PageIsStargazers"] = true
-	renderItems(ctx, ctx.Repo.Repository.NumStars, ctx.Repo.Repository.GetStargazers)
+	RenderUserCards(ctx, ctx.Repo.Repository.NumStars, ctx.Repo.Repository.GetStargazers, WATCHERS)
 }
 
 func Forks(ctx *middleware.Context) {
diff --git a/routers/user/home.go b/routers/user/home.go
index bf97c92398..b5df5e3679 100644
--- a/routers/user/home.go
+++ b/routers/user/home.go
@@ -7,7 +7,6 @@ package user
 import (
 	"bytes"
 	"fmt"
-	"strings"
 
 	"github.com/Unknwon/com"
 	"github.com/Unknwon/paginater"
@@ -21,7 +20,6 @@ import (
 const (
 	DASHBOARD base.TplName = "user/dashboard/dashboard"
 	ISSUES    base.TplName = "user/dashboard/issues"
-	STARS     base.TplName = "user/stars"
 	PROFILE   base.TplName = "user/profile"
 	ORG_HOME  base.TplName = "org/home"
 )
@@ -338,67 +336,6 @@ func showOrgProfile(ctx *middleware.Context) {
 	ctx.HTML(200, ORG_HOME)
 }
 
-func Profile(ctx *middleware.Context) {
-	ctx.Data["Title"] = "Profile"
-	ctx.Data["PageIsUserProfile"] = true
-
-	uname := ctx.Params(":username")
-	// Special handle for FireFox requests favicon.ico.
-	if uname == "favicon.ico" {
-		ctx.Redirect(setting.AppSubUrl + "/img/favicon.png")
-		return
-	} else if strings.HasSuffix(uname, ".png") {
-		ctx.Error(404)
-		return
-	}
-
-	isShowKeys := false
-	if strings.HasSuffix(uname, ".keys") {
-		isShowKeys = true
-		uname = strings.TrimSuffix(uname, ".keys")
-	}
-
-	u, err := models.GetUserByName(uname)
-	if err != nil {
-		if models.IsErrUserNotExist(err) {
-			ctx.Handle(404, "GetUserByName", err)
-		} else {
-			ctx.Handle(500, "GetUserByName", err)
-		}
-		return
-	}
-
-	// Show SSH keys.
-	if isShowKeys {
-		ShowSSHKeys(ctx, u.Id)
-		return
-	}
-
-	if u.IsOrganization() {
-		showOrgProfile(ctx)
-		return
-	}
-	ctx.Data["Owner"] = u
-
-	tab := ctx.Query("tab")
-	ctx.Data["TabName"] = tab
-	switch tab {
-	case "activity":
-		retrieveFeeds(ctx, u.Id, 0, true)
-		if ctx.Written() {
-			return
-		}
-	default:
-		ctx.Data["Repos"], err = models.GetRepositories(u.Id, ctx.IsSigned && ctx.User.Id == u.Id)
-		if err != nil {
-			ctx.Handle(500, "GetRepositories", err)
-			return
-		}
-	}
-
-	ctx.HTML(200, PROFILE)
-}
-
 func Email2User(ctx *middleware.Context) {
 	u, err := models.GetUserByEmail(ctx.Query("email"))
 	if err != nil {
diff --git a/routers/user/profile.go b/routers/user/profile.go
new file mode 100644
index 0000000000..b531523e65
--- /dev/null
+++ b/routers/user/profile.go
@@ -0,0 +1,145 @@
+// Copyright 2015 The Gogs Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package user
+
+import (
+	"fmt"
+	"strings"
+
+	"github.com/gogits/gogs/models"
+	"github.com/gogits/gogs/modules/base"
+	"github.com/gogits/gogs/modules/middleware"
+	"github.com/gogits/gogs/modules/setting"
+	"github.com/gogits/gogs/routers/repo"
+)
+
+const (
+	FOLLOWERS base.TplName = "user/meta/followers"
+	STARS     base.TplName = "user/meta/stars"
+)
+
+// GetUserByParams returns user whose name is presented in URL paramenter.
+func GetUserByParams(ctx *middleware.Context) *models.User {
+	user, err := models.GetUserByName(ctx.Params(":username"))
+	if err != nil {
+		if models.IsErrUserNotExist(err) {
+			ctx.Error(404)
+		} else {
+			ctx.Handle(500, "GetUserByName", err)
+		}
+		return nil
+	}
+	return user
+}
+
+func Profile(ctx *middleware.Context) {
+	uname := ctx.Params(":username")
+	// Special handle for FireFox requests favicon.ico.
+	if uname == "favicon.ico" {
+		ctx.Redirect(setting.AppSubUrl + "/img/favicon.png")
+		return
+	} else if strings.HasSuffix(uname, ".png") {
+		ctx.Error(404)
+		return
+	}
+
+	isShowKeys := false
+	if strings.HasSuffix(uname, ".keys") {
+		isShowKeys = true
+	}
+
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+
+	// Show SSH keys.
+	if isShowKeys {
+		ShowSSHKeys(ctx, u.Id)
+		return
+	}
+
+	if u.IsOrganization() {
+		showOrgProfile(ctx)
+		return
+	}
+
+	ctx.Data["Title"] = u.DisplayName()
+	ctx.Data["PageIsUserProfile"] = true
+	ctx.Data["Owner"] = u
+
+	tab := ctx.Query("tab")
+	ctx.Data["TabName"] = tab
+	switch tab {
+	case "activity":
+		retrieveFeeds(ctx, u.Id, 0, true)
+		if ctx.Written() {
+			return
+		}
+	default:
+		var err error
+		ctx.Data["Repos"], err = models.GetRepositories(u.Id, ctx.IsSigned && ctx.User.Id == u.Id)
+		if err != nil {
+			ctx.Handle(500, "GetRepositories", err)
+			return
+		}
+	}
+
+	ctx.HTML(200, PROFILE)
+}
+
+func Followers(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	ctx.Data["Title"] = u.DisplayName()
+	ctx.Data["CardsTitle"] = ctx.Tr("user.followers")
+	ctx.Data["PageIsFollowers"] = true
+	ctx.Data["Owner"] = u
+	repo.RenderUserCards(ctx, u.NumFollowers, u.GetFollowers, FOLLOWERS)
+}
+
+func Following(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+	ctx.Data["Title"] = u.DisplayName()
+	ctx.Data["CardsTitle"] = ctx.Tr("user.following")
+	ctx.Data["PageIsFollowing"] = true
+	ctx.Data["Owner"] = u
+	repo.RenderUserCards(ctx, u.NumFollowing, u.GetFollowing, FOLLOWERS)
+}
+
+func Stars(ctx *middleware.Context) {
+
+}
+
+func Action(ctx *middleware.Context) {
+	u := GetUserByParams(ctx)
+	if ctx.Written() {
+		return
+	}
+
+	var err error
+	switch ctx.Params(":action") {
+	case "follow":
+		err = models.FollowUser(ctx.User.Id, u.Id)
+	case "unfollow":
+		err = models.UnfollowUser(ctx.User.Id, u.Id)
+	}
+
+	if err != nil {
+		ctx.Handle(500, fmt.Sprintf("Action (%s)", ctx.Params(":action")), err)
+		return
+	}
+
+	redirectTo := ctx.Query("redirect_to")
+	if len(redirectTo) == 0 {
+		redirectTo = u.HomeLink()
+	}
+	ctx.Redirect(redirectTo)
+}