refactor repo.go

diff --git a/routers/repo/branch.go b/routers/repo/branch.go
new file mode 100644 (file)
index 0000000..8c953f2
--- /dev/null
+++ b/routers/repo/branch.go
@@ -0,0 +1,38 @@
+// Copyright 2014 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 repo
+
+import (
+       "github.com/codegangsta/martini"
+       "github.com/gogits/gogs/models"
+       "github.com/gogits/gogs/modules/middleware"
+)
+
+func Branches(ctx *middleware.Context, params martini.Params) {
+       if !ctx.Repo.IsValid {
+               return
+       }
+
+       brs, err := models.GetBranches(params["username"], params["reponame"])
+       if err != nil {
+               ctx.Handle(200, "repo.Branches", err)
+               return
+       } else if len(brs) == 0 {
+               ctx.Handle(404, "repo.Branches", nil)
+               return
+       }
+
+       ctx.Data["Username"] = params["username"]
+       ctx.Data["Reponame"] = params["reponame"]
+
+       if len(params["branchname"]) == 0 {
+               params["branchname"] = "master"
+       }
+       ctx.Data["Branchname"] = params["branchname"]
+       ctx.Data["Branches"] = brs
+       ctx.Data["IsRepoToolbarBranches"] = true
+
+       ctx.HTML(200, "repo/branches")
+}

diff --git a/routers/repo/commit.go b/routers/repo/commit.go
new file mode 100644 (file)
index 0000000..60ee217
--- /dev/null
+++ b/routers/repo/commit.go
@@ -0,0 +1,35 @@
+// Copyright 2014 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 repo
+
+import (
+       "github.com/codegangsta/martini"
+       "github.com/gogits/gogs/models"
+       "github.com/gogits/gogs/modules/middleware"
+)
+
+func Commits(ctx *middleware.Context, params martini.Params) {
+       brs, err := models.GetBranches(params["username"], params["reponame"])
+       if err != nil {
+               ctx.Handle(200, "repo.Commits", err)
+               return
+       } else if len(brs) == 0 {
+               ctx.Handle(404, "repo.Commits", nil)
+               return
+       }
+
+       ctx.Data["IsRepoToolbarCommits"] = true
+       commits, err := models.GetCommits(params["username"],
+               params["reponame"], params["branchname"])
+       if err != nil {
+               ctx.Handle(404, "repo.Commits", nil)
+               return
+       }
+       ctx.Data["Username"] = params["username"]
+       ctx.Data["Reponame"] = params["reponame"]
+       ctx.Data["CommitCount"] = commits.Len()
+       ctx.Data["Commits"] = commits
+       ctx.HTML(200, "repo/commits")
+}

diff --git a/routers/repo/pull.go b/routers/repo/pull.go
new file mode 100644 (file)
index 0000000..d4a12a4
--- /dev/null
+++ b/routers/repo/pull.go
@@ -0,0 +1,20 @@
+// Copyright 2014 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 repo
+
+import (
+       "github.com/codegangsta/martini"
+       "github.com/gogits/gogs/modules/middleware"
+)
+
+func Pulls(ctx *middleware.Context, params martini.Params) {
+       ctx.Data["IsRepoToolbarPulls"] = true
+       if len(params["branchname"]) == 0 {
+               params["branchname"] = "master"
+       }
+
+       ctx.Data["Branchname"] = params["branchname"]
+       ctx.HTML(200, "repo/pulls")
+}

diff --git a/routers/repo/repo.go b/routers/repo/repo.go
index b05ce4a717626d624658b4e0c5f2c7e7e9f4dccf..a055b416c2635ee1778602d6cf23d3ed2f7a7b00 100644 (file)
--- a/routers/repo/repo.go
+++ b/routers/repo/repo.go
@@ -52,33 +52,6 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) {
        ctx.Handle(200, "repo.Create", err)
 }
 
-func Branches(ctx *middleware.Context, params martini.Params) {
-       if !ctx.Repo.IsValid {
-               return
-       }
-
-       brs, err := models.GetBranches(params["username"], params["reponame"])
-       if err != nil {
-               ctx.Handle(200, "repo.Branches", err)
-               return
-       } else if len(brs) == 0 {
-               ctx.Handle(404, "repo.Branches", nil)
-               return
-       }
-
-       ctx.Data["Username"] = params["username"]
-       ctx.Data["Reponame"] = params["reponame"]
-
-       if len(params["branchname"]) == 0 {
-               params["branchname"] = "master"
-       }
-       ctx.Data["Branchname"] = params["branchname"]
-       ctx.Data["Branches"] = brs
-       ctx.Data["IsRepoToolbarBranches"] = true
-
-       ctx.HTML(200, "repo/branches")
-}
-
 func Single(ctx *middleware.Context, params martini.Params) {
        if !ctx.Repo.IsValid {
                return
@@ -279,7 +252,7 @@ func Setting(ctx *middleware.Context, params martini.Params) {
        ctx.HTML(200, "repo/setting")
 }
 
-func SettingPost(ctx *middleware.Context, params martini.Params) {
+func SettingPost(ctx *middleware.Context) {
        if !ctx.Repo.IsOwner {
                ctx.Error(404)
                return
@@ -313,40 +286,6 @@ func SettingPost(ctx *middleware.Context, params martini.Params) {
        }
 }
 
-func Commits(ctx *middleware.Context, params martini.Params) {
-       brs, err := models.GetBranches(params["username"], params["reponame"])
-       if err != nil {
-               ctx.Handle(200, "repo.Commits", err)
-               return
-       } else if len(brs) == 0 {
-               ctx.Handle(404, "repo.Commits", nil)
-               return
-       }
-
-       ctx.Data["IsRepoToolbarCommits"] = true
-       commits, err := models.GetCommits(params["username"],
-               params["reponame"], params["branchname"])
-       if err != nil {
-               ctx.Handle(404, "repo.Commits", nil)
-               return
-       }
-       ctx.Data["Username"] = params["username"]
-       ctx.Data["Reponame"] = params["reponame"]
-       ctx.Data["CommitCount"] = commits.Len()
-       ctx.Data["Commits"] = commits
-       ctx.HTML(200, "repo/commits")
-}
-
-func Pulls(ctx *middleware.Context, params martini.Params) {
-       ctx.Data["IsRepoToolbarPulls"] = true
-       if len(params["branchname"]) == 0 {
-               params["branchname"] = "master"
-       }
-
-       ctx.Data["Branchname"] = params["branchname"]
-       ctx.HTML(200, "repo/pulls")
-}
-
 func Action(ctx *middleware.Context, params martini.Params) {
        var err error
        switch params["action"] {