Browse Source

Fix #3315: Release dont use tag creation date (#3374)

* Fix #3315: Release dont use tag creation date

* Simplify code and apply gofmt

* remove useless block (ctx.Repo.GitRepo.GetTag) on EditReleasePost

* apply gofmt on modified files
pull/3369/merge
Thibault Meyer 9 years ago committed by 无闻
parent
commit
30fda0f1ae
  1. 4
      models/release.go
  2. 10
      routers/repo/release.go

4
models/release.go

@ -39,7 +39,9 @@ type Release struct {
} }
func (r *Release) BeforeInsert() { func (r *Release) BeforeInsert() {
r.CreatedUnix = time.Now().Unix() if r.CreatedUnix == 0 {
r.CreatedUnix = time.Now().Unix()
}
} }
func (r *Release) AfterSet(colName string, _ xorm.Cell) { func (r *Release) AfterSet(colName string, _ xorm.Cell) {

10
routers/repo/release.go

@ -167,6 +167,15 @@ func NewReleasePost(ctx *context.Context, form auth.NewReleaseForm) {
return return
} }
var tagCreatedUnix int64
tag, err := ctx.Repo.GitRepo.GetTag(form.TagName)
if err == nil {
commit, err := tag.Commit()
if err == nil {
tagCreatedUnix = commit.Author.When.Unix()
}
}
commit, err := ctx.Repo.GitRepo.GetBranchCommit(form.Target) commit, err := ctx.Repo.GitRepo.GetBranchCommit(form.Target)
if err != nil { if err != nil {
ctx.Handle(500, "GetBranchCommit", err) ctx.Handle(500, "GetBranchCommit", err)
@ -190,6 +199,7 @@ func NewReleasePost(ctx *context.Context, form auth.NewReleaseForm) {
Note: form.Content, Note: form.Content,
IsDraft: len(form.Draft) > 0, IsDraft: len(form.Draft) > 0,
IsPrerelease: form.Prerelease, IsPrerelease: form.Prerelease,
CreatedUnix: tagCreatedUnix,
} }
if err = models.CreateRelease(ctx.Repo.GitRepo, rel); err != nil { if err = models.CreateRelease(ctx.Repo.GitRepo, rel); err != nil {

Loading…
Cancel
Save