Corrections following recommendations

This commit is contained in:
Antoine GIRARD 2016-01-28 20:49:05 +01:00
parent 81e5722bcc
commit b7b30cd85e
5 changed files with 39 additions and 37 deletions

View file

@ -288,9 +288,9 @@ func (repo *Repository) GetMirror() (err error) {
return err return err
} }
func (repo *Repository) GetBranch(br string) (_ *Branch, err error) { func (repo *Repository) GetBranch(br string) (*Branch, error) {
if(!git.IsBranchExist(repo.RepoPath(), br)){ if(!git.IsBranchExist(repo.RepoPath(), br)){
return nil, errors.New("Branch do not exist"); return nil, fmt.Errorf("Branch does not exist: %s", br);
} }
return &Branch{ return &Branch{
Path: repo.RepoPath(), Path: repo.RepoPath(),
@ -298,7 +298,7 @@ func (repo *Repository) GetBranch(br string) (_ *Branch, err error) {
},nil },nil
} }
func (repo *Repository) GetBranches() (_ []*Branch, err error) { func (repo *Repository) GetBranches() ([]*Branch, error) {
return GetBranchesByPath(repo.RepoPath()) return GetBranchesByPath(repo.RepoPath())
} }

View file

@ -1,3 +1,7 @@
// Copyright 2016 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 models package models
import ( import (
@ -30,10 +34,6 @@ func GetBranchesByPath(path string) ([]*Branch, error) {
return Branches, nil return Branches, nil
} }
func GetBranchesByRepo(user,repo string) ([]*Branch, error) {
return GetBranchesByPath(RepoPath(user, repo))
}
func (br *Branch) GetCommit() (*git.Commit, error) { func (br *Branch) GetCommit() (*git.Commit, error) {
gitRepo, err := git.OpenRepository(br.Path) gitRepo, err := git.OpenRepository(br.Path)
if err != nil { if err != nil {

View file

@ -185,7 +185,7 @@ func RegisterRoutes(m *macaron.Macaron) {
m.Get("/archive/*", repo.GetArchive) m.Get("/archive/*", repo.GetArchive)
m.Group("/branches", func() { m.Group("/branches", func() {
m.Get("",repo.ListBranches) m.Get("",repo.ListBranches)
m.Get("/:id",repo.GetBranch) m.Get("/:branchname",repo.GetBranch)
}) })
m.Group("/keys", func() { m.Group("/keys", func() {
m.Combo("").Get(repo.ListDeployKeys). m.Combo("").Get(repo.ListDeployKeys).

View file

@ -16,26 +16,6 @@ import (
"github.com/gogits/gogs/modules/setting" "github.com/gogits/gogs/modules/setting"
) )
// ToApiBranch converts user to its API format.
func ToApiBranch(b *models.Branch,c *git.Commit) *api.Branch {
return &api.Branch{
Name: b.Name,
Commit: ToApiCommit(c),
}
}
// ToApiCommit converts user to its API format.
func ToApiCommit(c *git.Commit) *api.PayloadCommit {
return &api.PayloadCommit{
ID: c.ID.String(),
Message: c.Message(),
URL: "Not implemented",
Author: &api.PayloadAuthor{
Name: c.Committer.Name,
Email: c.Committer.Email,
/* UserName: c.Committer.UserName, */
},
}
}
// ToApiUser converts user to its API format. // ToApiUser converts user to its API format.
func ToApiUser(u *models.User) *api.User { func ToApiUser(u *models.User) *api.User {
return &api.User{ return &api.User{
@ -71,6 +51,28 @@ func ToApiRepository(owner *models.User, repo *models.Repository, permission api
} }
} }
// ToApiBranch converts user to its API format.
func ToApiBranch(b *models.Branch,c *git.Commit) *api.Branch {
return &api.Branch{
Name: b.Name,
Commit: ToApiCommit(c),
}
}
// ToApiCommit converts user to its API format.
func ToApiCommit(c *git.Commit) *api.PayloadCommit {
return &api.PayloadCommit{
ID: c.ID.String(),
Message: c.Message(),
URL: "Not implemented",
Author: &api.PayloadAuthor{
Name: c.Committer.Name,
Email: c.Committer.Email,
/* UserName: c.Committer.UserName, */
},
}
}
// ToApiPublicKey converts public key to its API format. // ToApiPublicKey converts public key to its API format.
func ToApiPublicKey(apiLink string, key *models.PublicKey) *api.PublicKey { func ToApiPublicKey(apiLink string, key *models.PublicKey) *api.PublicKey {
return &api.PublicKey{ return &api.PublicKey{

View file

@ -1,4 +1,4 @@
// Copyright 2014 The Gogs Authors. All rights reserved. // Copyright 2016 The Gogs Authors. All rights reserved.
// Use of this source code is governed by a MIT-style // Use of this source code is governed by a MIT-style
// license that can be found in the LICENSE file. // license that can be found in the LICENSE file.
@ -14,32 +14,32 @@ import (
// Temporary: https://gist.github.com/sapk/df64347ff218baf4a277#get-a-branch // Temporary: https://gist.github.com/sapk/df64347ff218baf4a277#get-a-branch
// https://github.com/gogits/go-gogs-client/wiki/Repositories-Branches#get-a-branch // https://github.com/gogits/go-gogs-client/wiki/Repositories-Branches#get-a-branch
func GetBranch(ctx *middleware.Context) { func GetBranch(ctx *middleware.Context) {
//Getting the branch requested // Getting the branch requested
branch, err := ctx.Repo.Repository.GetBranch(ctx.Params(":id")) branch, err := ctx.Repo.Repository.GetBranch(ctx.Params(":branchname"))
if err != nil { if err != nil {
ctx.APIError(500, "Repository.GetBranch", err) ctx.APIError(500, "Repository.GetBranch", err)
return return
} }
//Getting the last commit of the branch // Getting the last commit of the branch
c, err := branch.GetCommit() c, err := branch.GetCommit()
if err != nil { if err != nil {
ctx.APIError(500, "Branch.GetCommit", err) ctx.APIError(500, "Branch.GetCommit", err)
return return
} }
//Converting to API format and send payload // Converting to API format and send payload
ctx.JSON(200, convert.ToApiBranch(branch,c)) ctx.JSON(200, convert.ToApiBranch(branch,c))
} }
// Temporary: https://gist.github.com/sapk/df64347ff218baf4a277#list-branches // Temporary: https://gist.github.com/sapk/df64347ff218baf4a277#list-branches
// https://github.com/gogits/go-gogs-client/wiki/Repositories-Branches#list-branches // https://github.com/gogits/go-gogs-client/wiki/Repositories-Branches#list-branches
func ListBranches(ctx *middleware.Context) { func ListBranches(ctx *middleware.Context) {
//Listing of branches // Listing of branches
Branches, err := ctx.Repo.Repository.GetBranches() Branches, err := ctx.Repo.Repository.GetBranches()
if err != nil { if err != nil {
ctx.APIError(500, "Repository.GetBranches", err) ctx.APIError(500, "Repository.GetBranches", err)
return return
} }
//Getting the last commit of each branch // Getting the last commit of each branch
apiBranches := make([]*api.Branch, len(Branches)) apiBranches := make([]*api.Branch, len(Branches))
for i := range Branches { for i := range Branches {
c, err := Branches[i].GetCommit() c, err := Branches[i].GetCommit()
@ -47,9 +47,9 @@ func ListBranches(ctx *middleware.Context) {
ctx.APIError(500, "Branch.GetCommit", err) ctx.APIError(500, "Branch.GetCommit", err)
return return
} }
//Converting to API format // Converting to API format
apiBranches[i] = convert.ToApiBranch(Branches[i],c) apiBranches[i] = convert.ToApiBranch(Branches[i],c)
} }
//Sending the payload // Sending the payload
ctx.JSON(200, &apiBranches) ctx.JSON(200, &apiBranches)
} }