123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982 |
- From 56af7e99a819758c6bedc6d745a9f810f3fc050d Mon Sep 17 00:00:00 2001
- From: Gogs <gogitservice@gmail.com>
- Date: Wed, 19 Mar 2014 21:57:55 +0800
- Subject: [PATCH] fork render
- ---
- .gopmfile | 1 -
- modules/base/template.go | 5 +
- modules/middleware/auth.go | 4 +-
- modules/middleware/context.go | 28 ++---
- modules/middleware/render.go | 286 ++++++++++++++++++++++++++++++++++++++++++
- modules/middleware/repo.go | 6 +-
- routers/dashboard.go | 4 +-
- routers/repo/repo.go | 10 +-
- routers/repo/single.go | 32 ++---
- routers/user/setting.go | 24 ++--
- routers/user/user.go | 43 +++----
- templates/base/footer.tmpl | 4 +-
- web.go | 3 +-
- 13 files changed, 367 insertions(+), 83 deletions(-)
- create mode 100644 modules/middleware/render.go
- diff --git a/.gopmfile b/.gopmfile
- index 4c05ae4..9e2440f 100644
- --- a/.gopmfile
- +++ b/.gopmfile
- @@ -4,7 +4,6 @@ path=github.com/gogits/gogs
- [deps]
- github.com/codegangsta/cli=
- github.com/codegangsta/martini=
- -github.com/martini-contrib/render=
- github.com/martini-contrib/sessions=
- github.com/Unknwon/com=
- github.com/Unknwon/cae=
- diff --git a/modules/base/template.go b/modules/base/template.go
- index db79340..23d3d27 100644
- --- a/modules/base/template.go
- +++ b/modules/base/template.go
- @@ -6,7 +6,9 @@ package base
-
- import (
- "container/list"
- + "fmt"
- "html/template"
- + "time"
- )
-
- func Str2html(raw string) template.HTML {
- @@ -40,6 +42,9 @@ var TemplateFuncs template.FuncMap = map[string]interface{}{
- "AppDomain": func() string {
- return Domain
- },
- + "LoadTimes": func(startTime time.Time) string {
- + return fmt.Sprint(time.Since(startTime).Nanoseconds()/1e6) + "ms"
- + },
- "AvatarLink": AvatarLink,
- "str2html": Str2html,
- "TimeSince": TimeSince,
- diff --git a/modules/middleware/auth.go b/modules/middleware/auth.go
- index 84bbc7f..ed847dd 100644
- --- a/modules/middleware/auth.go
- +++ b/modules/middleware/auth.go
- @@ -13,7 +13,7 @@ func SignInRequire(redirect bool) martini.Handler {
- return func(ctx *Context) {
- if !ctx.IsSigned {
- if redirect {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- }
- return
- } else if !ctx.User.IsActive {
- @@ -28,7 +28,7 @@ func SignInRequire(redirect bool) martini.Handler {
- func SignOutRequire() martini.Handler {
- return func(ctx *Context) {
- if ctx.IsSigned {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- }
- }
- }
- diff --git a/modules/middleware/context.go b/modules/middleware/context.go
- index 7eaf665..c97c329 100644
- --- a/modules/middleware/context.go
- +++ b/modules/middleware/context.go
- @@ -7,26 +7,24 @@ package middleware
- import (
- "fmt"
- "net/http"
- + "time"
-
- "github.com/codegangsta/martini"
- - "github.com/martini-contrib/render"
- "github.com/martini-contrib/sessions"
-
- "github.com/gogits/gogs/models"
- "github.com/gogits/gogs/modules/auth"
- - "github.com/gogits/gogs/modules/base"
- "github.com/gogits/gogs/modules/log"
- )
-
- // Context represents context of a request.
- type Context struct {
- + *Render
- c martini.Context
- p martini.Params
- Req *http.Request
- Res http.ResponseWriter
- Session sessions.Session
- - Data base.TmplData
- - Render render.Render
- User *models.User
- IsSigned bool
-
- @@ -62,27 +60,25 @@ func (ctx *Context) RenderWithErr(msg, tpl string, form auth.Form) {
- ctx.Data["HasError"] = true
- ctx.Data["ErrorMsg"] = msg
- auth.AssignForm(form, ctx.Data)
- - ctx.Render.HTML(200, tpl, ctx.Data)
- + ctx.HTML(200, tpl, ctx.Data)
- }
-
- // Handle handles and logs error by given status.
- func (ctx *Context) Handle(status int, title string, err error) {
- log.Error("%s: %v", title, err)
- if martini.Dev == martini.Prod {
- - ctx.Render.HTML(500, "status/500", ctx.Data)
- + ctx.HTML(500, "status/500", ctx.Data)
- return
- }
-
- ctx.Data["ErrorMsg"] = err
- - ctx.Render.HTML(status, fmt.Sprintf("status/%d", status), ctx.Data)
- + ctx.HTML(status, fmt.Sprintf("status/%d", status), ctx.Data)
- }
-
- // InitContext initializes a classic context for a request.
- func InitContext() martini.Handler {
- return func(res http.ResponseWriter, r *http.Request, c martini.Context,
- - session sessions.Session, rd render.Render) {
- -
- - data := base.TmplData{}
- + session sessions.Session, rd *Render) {
-
- ctx := &Context{
- c: c,
- @@ -90,7 +86,6 @@ func InitContext() martini.Handler {
- Req: r,
- Res: res,
- Session: session,
- - Data: data,
- Render: rd,
- }
-
- @@ -99,16 +94,17 @@ func InitContext() martini.Handler {
- ctx.User = user
- ctx.IsSigned = user != nil
-
- - data["IsSigned"] = ctx.IsSigned
- + ctx.Data["IsSigned"] = ctx.IsSigned
-
- if user != nil {
- - data["SignedUser"] = user
- - data["SignedUserId"] = user.Id
- - data["SignedUserName"] = user.LowerName
- + ctx.Data["SignedUser"] = user
- + ctx.Data["SignedUserId"] = user.Id
- + ctx.Data["SignedUserName"] = user.LowerName
- }
-
- + ctx.Data["PageStartTime"] = time.Now()
- +
- c.Map(ctx)
- - c.Map(data)
-
- c.Next()
- }
- diff --git a/modules/middleware/render.go b/modules/middleware/render.go
- new file mode 100644
- index 0000000..8a54183
- --- /dev/null
- +++ b/modules/middleware/render.go
- @@ -0,0 +1,286 @@
- +// 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.
- +
- +// foked from https://github.com/martini-contrib/render/blob/master/render.go
- +package middleware
- +
- +import (
- + "bytes"
- + "encoding/json"
- + "fmt"
- + "html/template"
- + "io"
- + "io/ioutil"
- + "net/http"
- + "os"
- + "path/filepath"
- + "time"
- +
- + "github.com/codegangsta/martini"
- +
- + "github.com/gogits/gogs/modules/base"
- +)
- +
- +const (
- + ContentType = "Content-Type"
- + ContentLength = "Content-Length"
- + ContentJSON = "application/json"
- + ContentHTML = "text/html"
- + ContentXHTML = "application/xhtml+xml"
- + defaultCharset = "UTF-8"
- +)
- +
- +var helperFuncs = template.FuncMap{
- + "yield": func() (string, error) {
- + return "", fmt.Errorf("yield called with no layout defined")
- + },
- +}
- +
- +type Delims struct {
- + Left string
- +
- + Right string
- +}
- +
- +type RenderOptions struct {
- + Directory string
- +
- + Layout string
- +
- + Extensions []string
- +
- + Funcs []template.FuncMap
- +
- + Delims Delims
- +
- + Charset string
- +
- + IndentJSON bool
- +
- + HTMLContentType string
- +}
- +
- +type HTMLOptions struct {
- + Layout string
- +}
- +
- +func Renderer(options ...RenderOptions) martini.Handler {
- + opt := prepareOptions(options)
- + cs := prepareCharset(opt.Charset)
- + t := compile(opt)
- + return func(res http.ResponseWriter, req *http.Request, c martini.Context) {
- + var tc *template.Template
- + if martini.Env == martini.Dev {
- +
- + tc = compile(opt)
- + } else {
- +
- + tc, _ = t.Clone()
- + }
- +
- + rd := &Render{res, req, tc, opt, cs, base.TmplData{}, time.Time{}}
- +
- + rd.Data["TmplLoadTimes"] = func() string {
- + if rd.startTime.IsZero() {
- + return ""
- + }
- + return fmt.Sprint(time.Since(rd.startTime).Nanoseconds()/1e6) + "ms"
- + }
- +
- + c.Map(rd.Data)
- + c.Map(rd)
- + }
- +}
- +
- +func prepareCharset(charset string) string {
- + if len(charset) != 0 {
- + return "; charset=" + charset
- + }
- +
- + return "; charset=" + defaultCharset
- +}
- +
- +func prepareOptions(options []RenderOptions) RenderOptions {
- + var opt RenderOptions
- + if len(options) > 0 {
- + opt = options[0]
- + }
- +
- + if len(opt.Directory) == 0 {
- + opt.Directory = "templates"
- + }
- + if len(opt.Extensions) == 0 {
- + opt.Extensions = []string{".tmpl"}
- + }
- + if len(opt.HTMLContentType) == 0 {
- + opt.HTMLContentType = ContentHTML
- + }
- +
- + return opt
- +}
- +
- +func compile(options RenderOptions) *template.Template {
- + dir := options.Directory
- + t := template.New(dir)
- + t.Delims(options.Delims.Left, options.Delims.Right)
- +
- + template.Must(t.Parse("Martini"))
- +
- + filepath.Walk(dir, func(path string, info os.FileInfo, err error) error {
- + r, err := filepath.Rel(dir, path)
- + if err != nil {
- + return err
- + }
- +
- + ext := filepath.Ext(r)
- + for _, extension := range options.Extensions {
- + if ext == extension {
- +
- + buf, err := ioutil.ReadFile(path)
- + if err != nil {
- + panic(err)
- + }
- +
- + name := (r[0 : len(r)-len(ext)])
- + tmpl := t.New(filepath.ToSlash(name))
- +
- + for _, funcs := range options.Funcs {
- + tmpl.Funcs(funcs)
- + }
- +
- + template.Must(tmpl.Funcs(helperFuncs).Parse(string(buf)))
- + break
- + }
- + }
- +
- + return nil
- + })
- +
- + return t
- +}
- +
- +type Render struct {
- + http.ResponseWriter
- + req *http.Request
- + t *template.Template
- + opt RenderOptions
- + compiledCharset string
- +
- + Data base.TmplData
- +
- + startTime time.Time
- +}
- +
- +func (r *Render) JSON(status int, v interface{}) {
- + var result []byte
- + var err error
- + if r.opt.IndentJSON {
- + result, err = json.MarshalIndent(v, "", " ")
- + } else {
- + result, err = json.Marshal(v)
- + }
- + if err != nil {
- + http.Error(r, err.Error(), 500)
- + return
- + }
- +
- + r.Header().Set(ContentType, ContentJSON+r.compiledCharset)
- + r.WriteHeader(status)
- + r.Write(result)
- +}
- +
- +func (r *Render) JSONString(v interface{}) (string, error) {
- + var result []byte
- + var err error
- + if r.opt.IndentJSON {
- + result, err = json.MarshalIndent(v, "", " ")
- + } else {
- + result, err = json.Marshal(v)
- + }
- + if err != nil {
- + return "", err
- + }
- + return string(result), nil
- +}
- +
- +func (r *Render) renderBytes(name string, binding interface{}, htmlOpt ...HTMLOptions) (*bytes.Buffer, error) {
- + opt := r.prepareHTMLOptions(htmlOpt)
- +
- + if len(opt.Layout) > 0 {
- + r.addYield(name, binding)
- + name = opt.Layout
- + }
- +
- + out, err := r.execute(name, binding)
- + if err != nil {
- + return nil, err
- + }
- +
- + return out, nil
- +}
- +
- +func (r *Render) HTML(status int, name string, binding interface{}, htmlOpt ...HTMLOptions) {
- + r.startTime = time.Now()
- +
- + out, err := r.renderBytes(name, binding, htmlOpt...)
- + if err != nil {
- + http.Error(r, err.Error(), http.StatusInternalServerError)
- + return
- + }
- +
- + r.Header().Set(ContentType, r.opt.HTMLContentType+r.compiledCharset)
- + r.WriteHeader(status)
- + io.Copy(r, out)
- +}
- +
- +func (r *Render) HTMLString(name string, binding interface{}, htmlOpt ...HTMLOptions) (string, error) {
- + if out, err := r.renderBytes(name, binding, htmlOpt...); err != nil {
- + return "", err
- + } else {
- + return out.String(), nil
- + }
- +}
- +
- +func (r *Render) Error(status int) {
- + r.WriteHeader(status)
- +}
- +
- +func (r *Render) Redirect(location string, status ...int) {
- + code := http.StatusFound
- + if len(status) == 1 {
- + code = status[0]
- + }
- +
- + http.Redirect(r, r.req, location, code)
- +}
- +
- +func (r *Render) Template() *template.Template {
- + return r.t
- +}
- +
- +func (r *Render) execute(name string, binding interface{}) (*bytes.Buffer, error) {
- + buf := new(bytes.Buffer)
- + return buf, r.t.ExecuteTemplate(buf, name, binding)
- +}
- +
- +func (r *Render) addYield(name string, binding interface{}) {
- + funcs := template.FuncMap{
- + "yield": func() (template.HTML, error) {
- + buf, err := r.execute(name, binding)
- +
- + return template.HTML(buf.String()), err
- + },
- + }
- + r.t.Funcs(funcs)
- +}
- +
- +func (r *Render) prepareHTMLOptions(htmlOpt []HTMLOptions) HTMLOptions {
- + if len(htmlOpt) > 0 {
- + return htmlOpt[0]
- + }
- +
- + return HTMLOptions{
- + Layout: r.opt.Layout,
- + }
- +}
- diff --git a/modules/middleware/repo.go b/modules/middleware/repo.go
- index db29bd1..a12212a 100644
- --- a/modules/middleware/repo.go
- +++ b/modules/middleware/repo.go
- @@ -30,7 +30,7 @@ func RepoAssignment(redirect bool) martini.Handler {
- user, err = models.GetUserByName(params["username"])
- if err != nil {
- if redirect {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- return
- }
- ctx.Handle(200, "RepoAssignment", err)
- @@ -42,7 +42,7 @@ func RepoAssignment(redirect bool) martini.Handler {
-
- if user == nil {
- if redirect {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- return
- }
- ctx.Handle(200, "RepoAssignment", errors.New("invliad user account for single repository"))
- @@ -55,7 +55,7 @@ func RepoAssignment(redirect bool) martini.Handler {
- repo, err := models.GetRepositoryByName(user, params["reponame"])
- if err != nil {
- if redirect {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- return
- }
- ctx.Handle(200, "RepoAssignment", err)
- diff --git a/routers/dashboard.go b/routers/dashboard.go
- index d5358f0..6c194ad 100644
- --- a/routers/dashboard.go
- +++ b/routers/dashboard.go
- @@ -15,10 +15,10 @@ func Home(ctx *middleware.Context) {
- return
- }
- ctx.Data["PageIsHome"] = true
- - ctx.Render.HTML(200, "home", ctx.Data)
- + ctx.HTML(200, "home", ctx.Data)
- }
-
- func Help(ctx *middleware.Context) {
- ctx.Data["PageIsHelp"] = true
- - ctx.Render.HTML(200, "help", ctx.Data)
- + ctx.HTML(200, "help", ctx.Data)
- }
- diff --git a/routers/repo/repo.go b/routers/repo/repo.go
- index 61bf47c..947643a 100644
- --- a/routers/repo/repo.go
- +++ b/routers/repo/repo.go
- @@ -17,7 +17,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) {
- if ctx.Req.Method == "GET" {
- ctx.Data["LanguageIgns"] = models.LanguageIgns
- ctx.Data["Licenses"] = models.Licenses
- - ctx.Render.HTML(200, "repo/create", ctx.Data)
- + ctx.HTML(200, "repo/create", ctx.Data)
- return
- }
-
- @@ -25,7 +25,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) {
- form.Language, form.License, form.Visibility == "private", form.InitReadme == "on")
- if err == nil {
- log.Trace("%s Repository created: %s/%s", ctx.Req.RequestURI, ctx.User.LowerName, form.RepoName)
- - ctx.Render.Redirect("/"+ctx.User.Name+"/"+form.RepoName, 302)
- + ctx.Redirect("/"+ctx.User.Name+"/"+form.RepoName, 302)
- return
- } else if err == models.ErrRepoAlreadyExist {
- ctx.RenderWithErr("Repository name has already been used", "repo/create", &form)
- @@ -36,7 +36,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) {
-
- func SettingPost(ctx *middleware.Context) {
- if !ctx.Repo.IsOwner {
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
-
- @@ -44,7 +44,7 @@ func SettingPost(ctx *middleware.Context) {
- case "delete":
- if len(ctx.Repo.Repository.Name) == 0 || ctx.Repo.Repository.Name != ctx.Query("repository") {
- ctx.Data["ErrorMsg"] = "Please make sure you entered repository name is correct."
- - ctx.Render.HTML(200, "repo/setting", ctx.Data)
- + ctx.HTML(200, "repo/setting", ctx.Data)
- return
- }
-
- @@ -55,5 +55,5 @@ func SettingPost(ctx *middleware.Context) {
- }
-
- log.Trace("%s Repository deleted: %s/%s", ctx.Req.RequestURI, ctx.User.LowerName, ctx.Repo.Repository.LowerName)
- - ctx.Render.Redirect("/", 302)
- + ctx.Redirect("/", 302)
- }
- diff --git a/routers/repo/single.go b/routers/repo/single.go
- index d64248d..171b095 100644
- --- a/routers/repo/single.go
- +++ b/routers/repo/single.go
- @@ -27,7 +27,7 @@ func Branches(ctx *middleware.Context, params martini.Params) {
- ctx.Handle(200, "repo.Branches", err)
- return
- } else if len(brs) == 0 {
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
-
- @@ -38,7 +38,7 @@ func Branches(ctx *middleware.Context, params martini.Params) {
- ctx.Data["Branches"] = brs
- ctx.Data["IsRepoToolbarBranches"] = true
-
- - ctx.Render.HTML(200, "repo/branches", ctx.Data)
- + ctx.HTML(200, "repo/branches", ctx.Data)
- }
-
- func Single(ctx *middleware.Context, params martini.Params) {
- @@ -57,11 +57,11 @@ func Single(ctx *middleware.Context, params martini.Params) {
- brs, err := models.GetBranches(params["username"], params["reponame"])
- if err != nil {
- log.Error("repo.Single(GetBranches): %v", err)
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- } else if len(brs) == 0 {
- ctx.Data["IsBareRepo"] = true
- - ctx.Render.HTML(200, "repo/single", ctx.Data)
- + ctx.HTML(200, "repo/single", ctx.Data)
- return
- }
-
- @@ -72,7 +72,7 @@ func Single(ctx *middleware.Context, params martini.Params) {
- params["branchname"], params["commitid"], treename)
- if err != nil {
- log.Error("repo.Single(GetReposFiles): %v", err)
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
- ctx.Data["Username"] = params["username"]
- @@ -94,7 +94,7 @@ func Single(ctx *middleware.Context, params martini.Params) {
- params["branchname"], params["commitid"])
- if err != nil {
- log.Error("repo.Single(GetCommit): %v", err)
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
- ctx.Data["LastCommit"] = commit
- @@ -130,12 +130,12 @@ func Single(ctx *middleware.Context, params martini.Params) {
- ctx.Data["Treenames"] = treenames
- ctx.Data["IsRepoToolbarSource"] = true
- ctx.Data["Files"] = files
- - ctx.Render.HTML(200, "repo/single", ctx.Data)
- + ctx.HTML(200, "repo/single", ctx.Data)
- }
-
- func Setting(ctx *middleware.Context, params martini.Params) {
- if !ctx.Repo.IsOwner {
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
-
- @@ -143,11 +143,11 @@ func Setting(ctx *middleware.Context, params martini.Params) {
- brs, err := models.GetBranches(params["username"], params["reponame"])
- if err != nil {
- log.Error("repo.Setting(GetBranches): %v", err)
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- } else if len(brs) == 0 {
- ctx.Data["IsBareRepo"] = true
- - ctx.Render.HTML(200, "repo/setting", ctx.Data)
- + ctx.HTML(200, "repo/setting", ctx.Data)
- return
- }
-
- @@ -158,7 +158,7 @@ func Setting(ctx *middleware.Context, params martini.Params) {
-
- ctx.Data["Title"] = title + " - settings"
- ctx.Data["IsRepoToolbarSetting"] = true
- - ctx.Render.HTML(200, "repo/setting", ctx.Data)
- + ctx.HTML(200, "repo/setting", ctx.Data)
- }
-
- func Commits(ctx *middleware.Context, params martini.Params) {
- @@ -167,7 +167,7 @@ func Commits(ctx *middleware.Context, params martini.Params) {
- ctx.Handle(200, "repo.Commits", err)
- return
- } else if len(brs) == 0 {
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
-
- @@ -175,19 +175,19 @@ func Commits(ctx *middleware.Context, params martini.Params) {
- commits, err := models.GetCommits(params["username"],
- params["reponame"], params["branchname"])
- if err != nil {
- - ctx.Render.Error(404)
- + ctx.Error(404)
- return
- }
- ctx.Data["Commits"] = commits
- - ctx.Render.HTML(200, "repo/commits", ctx.Data)
- + ctx.HTML(200, "repo/commits", ctx.Data)
- }
-
- func Issues(ctx *middleware.Context) {
- ctx.Data["IsRepoToolbarIssues"] = true
- - ctx.Render.HTML(200, "repo/issues", ctx.Data)
- + ctx.HTML(200, "repo/issues", ctx.Data)
- }
-
- func Pulls(ctx *middleware.Context) {
- ctx.Data["IsRepoToolbarPulls"] = true
- - ctx.Render.HTML(200, "repo/pulls", ctx.Data)
- + ctx.HTML(200, "repo/pulls", ctx.Data)
- }
- diff --git a/routers/user/setting.go b/routers/user/setting.go
- index 66289b6..719e0c1 100644
- --- a/routers/user/setting.go
- +++ b/routers/user/setting.go
- @@ -24,13 +24,13 @@ func Setting(ctx *middleware.Context, form auth.UpdateProfileForm) {
- ctx.Data["Owner"] = user
-
- if ctx.Req.Method == "GET" {
- - ctx.Render.HTML(200, "user/setting", ctx.Data)
- + ctx.HTML(200, "user/setting", ctx.Data)
- return
- }
-
- // below is for POST requests
- if hasErr, ok := ctx.Data["HasError"]; ok && hasErr.(bool) {
- - ctx.Render.HTML(200, "user/setting", ctx.Data)
- + ctx.HTML(200, "user/setting", ctx.Data)
- return
- }
-
- @@ -45,7 +45,7 @@ func Setting(ctx *middleware.Context, form auth.UpdateProfileForm) {
- }
-
- ctx.Data["IsSuccess"] = true
- - ctx.Render.HTML(200, "user/setting", ctx.Data)
- + ctx.HTML(200, "user/setting", ctx.Data)
- log.Trace("%s User setting updated: %s", ctx.Req.RequestURI, ctx.User.LowerName)
- }
-
- @@ -55,7 +55,7 @@ func SettingPassword(ctx *middleware.Context, form auth.UpdatePasswdForm) {
- ctx.Data["IsUserPageSettingPasswd"] = true
-
- if ctx.Req.Method == "GET" {
- - ctx.Render.HTML(200, "user/password", ctx.Data)
- + ctx.HTML(200, "user/password", ctx.Data)
- return
- }
-
- @@ -82,7 +82,7 @@ func SettingPassword(ctx *middleware.Context, form auth.UpdatePasswdForm) {
- }
-
- ctx.Data["Owner"] = user
- - ctx.Render.HTML(200, "user/password", ctx.Data)
- + ctx.HTML(200, "user/password", ctx.Data)
- log.Trace("%s User password updated: %s", ctx.Req.RequestURI, ctx.User.LowerName)
- }
-
- @@ -95,7 +95,7 @@ func SettingSSHKeys(ctx *middleware.Context, form auth.AddSSHKeyForm) {
- if err != nil {
- ctx.Data["ErrorMsg"] = err
- log.Error("ssh.DelPublicKey: %v", err)
- - ctx.Render.JSON(200, map[string]interface{}{
- + ctx.JSON(200, map[string]interface{}{
- "ok": false,
- "err": err.Error(),
- })
- @@ -109,13 +109,13 @@ func SettingSSHKeys(ctx *middleware.Context, form auth.AddSSHKeyForm) {
- if err = models.DeletePublicKey(k); err != nil {
- ctx.Data["ErrorMsg"] = err
- log.Error("ssh.DelPublicKey: %v", err)
- - ctx.Render.JSON(200, map[string]interface{}{
- + ctx.JSON(200, map[string]interface{}{
- "ok": false,
- "err": err.Error(),
- })
- } else {
- log.Trace("%s User SSH key deleted: %s", ctx.Req.RequestURI, ctx.User.LowerName)
- - ctx.Render.JSON(200, map[string]interface{}{
- + ctx.JSON(200, map[string]interface{}{
- "ok": true,
- })
- }
- @@ -125,7 +125,7 @@ func SettingSSHKeys(ctx *middleware.Context, form auth.AddSSHKeyForm) {
- // Add new SSH key.
- if ctx.Req.Method == "POST" {
- if hasErr, ok := ctx.Data["HasError"]; ok && hasErr.(bool) {
- - ctx.Render.HTML(200, "user/publickey", ctx.Data)
- + ctx.HTML(200, "user/publickey", ctx.Data)
- return
- }
-
- @@ -157,7 +157,7 @@ func SettingSSHKeys(ctx *middleware.Context, form auth.AddSSHKeyForm) {
- ctx.Data["PageIsUserSetting"] = true
- ctx.Data["IsUserPageSettingSSH"] = true
- ctx.Data["Keys"] = keys
- - ctx.Render.HTML(200, "user/publickey", ctx.Data)
- + ctx.HTML(200, "user/publickey", ctx.Data)
- }
-
- func SettingNotification(ctx *middleware.Context) {
- @@ -165,7 +165,7 @@ func SettingNotification(ctx *middleware.Context) {
- ctx.Data["Title"] = "Notification"
- ctx.Data["PageIsUserSetting"] = true
- ctx.Data["IsUserPageSettingNotify"] = true
- - ctx.Render.HTML(200, "user/notification", ctx.Data)
- + ctx.HTML(200, "user/notification", ctx.Data)
- }
-
- func SettingSecurity(ctx *middleware.Context) {
- @@ -173,5 +173,5 @@ func SettingSecurity(ctx *middleware.Context) {
- ctx.Data["Title"] = "Security"
- ctx.Data["PageIsUserSetting"] = true
- ctx.Data["IsUserPageSettingSecurity"] = true
- - ctx.Render.HTML(200, "user/security", ctx.Data)
- + ctx.HTML(200, "user/security", ctx.Data)
- }
- diff --git a/routers/user/user.go b/routers/user/user.go
- index d702f2b..4203007 100644
- --- a/routers/user/user.go
- +++ b/routers/user/user.go
- @@ -9,8 +9,6 @@ import (
- "strings"
-
- "github.com/codegangsta/martini"
- - "github.com/martini-contrib/render"
- - "github.com/martini-contrib/sessions"
-
- "github.com/gogits/gogs/models"
- "github.com/gogits/gogs/modules/auth"
- @@ -35,7 +33,7 @@ func Dashboard(ctx *middleware.Context) {
- return
- }
- ctx.Data["Feeds"] = feeds
- - ctx.Render.HTML(200, "user/dashboard", ctx.Data)
- + ctx.HTML(200, "user/dashboard", ctx.Data)
- }
-
- func Profile(ctx *middleware.Context, params martini.Params) {
- @@ -70,19 +68,19 @@ func Profile(ctx *middleware.Context, params martini.Params) {
- ctx.Data["Repos"] = repos
- }
-
- - ctx.Render.HTML(200, "user/profile", ctx.Data)
- + ctx.HTML(200, "user/profile", ctx.Data)
- }
-
- func SignIn(ctx *middleware.Context, form auth.LogInForm) {
- ctx.Data["Title"] = "Log In"
-
- if ctx.Req.Method == "GET" {
- - ctx.Render.HTML(200, "user/signin", ctx.Data)
- + ctx.HTML(200, "user/signin", ctx.Data)
- return
- }
-
- if hasErr, ok := ctx.Data["HasError"]; ok && hasErr.(bool) {
- - ctx.Render.HTML(200, "user/signin", ctx.Data)
- + ctx.HTML(200, "user/signin", ctx.Data)
- return
- }
-
- @@ -99,14 +97,13 @@ func SignIn(ctx *middleware.Context, form auth.LogInForm) {
-
- ctx.Session.Set("userId", user.Id)
- ctx.Session.Set("userName", user.Name)
- -
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- }
-
- -func SignOut(r render.Render, session sessions.Session) {
- - session.Delete("userId")
- - session.Delete("userName")
- - r.Redirect("/")
- +func SignOut(ctx *middleware.Context) {
- + ctx.Session.Delete("userId")
- + ctx.Session.Delete("userName")
- + ctx.Redirect("/")
- }
-
- func SignUp(ctx *middleware.Context, form auth.RegisterForm) {
- @@ -114,7 +111,7 @@ func SignUp(ctx *middleware.Context, form auth.RegisterForm) {
- ctx.Data["PageIsSignUp"] = true
-
- if ctx.Req.Method == "GET" {
- - ctx.Render.HTML(200, "user/signup", ctx.Data)
- + ctx.HTML(200, "user/signup", ctx.Data)
- return
- }
-
- @@ -127,7 +124,7 @@ func SignUp(ctx *middleware.Context, form auth.RegisterForm) {
- }
-
- if ctx.HasError() {
- - ctx.Render.HTML(200, "user/signup", ctx.Data)
- + ctx.HTML(200, "user/signup", ctx.Data)
- return
- }
-
- @@ -157,7 +154,7 @@ func SignUp(ctx *middleware.Context, form auth.RegisterForm) {
- if base.Service.RegisterEmailConfirm {
- auth.SendRegisterMail(u)
- }
- - ctx.Render.Redirect("/user/login")
- + ctx.Redirect("/user/login")
- }
-
- func Delete(ctx *middleware.Context) {
- @@ -166,7 +163,7 @@ func Delete(ctx *middleware.Context) {
- ctx.Data["IsUserPageSettingDelete"] = true
-
- if ctx.Req.Method == "GET" {
- - ctx.Render.HTML(200, "user/delete", ctx.Data)
- + ctx.HTML(200, "user/delete", ctx.Data)
- return
- }
-
- @@ -186,12 +183,12 @@ func Delete(ctx *middleware.Context) {
- return
- }
- } else {
- - ctx.Render.Redirect("/")
- + ctx.Redirect("/")
- return
- }
- }
-
- - ctx.Render.HTML(200, "user/delete", ctx.Data)
- + ctx.HTML(200, "user/delete", ctx.Data)
- }
-
- const (
- @@ -202,7 +199,7 @@ const (
- func Feeds(ctx *middleware.Context, form auth.FeedsForm) {
- actions, err := models.GetFeeds(form.UserId, form.Page*20, false)
- if err != nil {
- - ctx.Render.JSON(500, err)
- + ctx.JSON(500, err)
- }
-
- feeds := make([]string, len(actions))
- @@ -210,19 +207,19 @@ func Feeds(ctx *middleware.Context, form auth.FeedsForm) {
- feeds[i] = fmt.Sprintf(TPL_FEED, base.ActionIcon(actions[i].OpType),
- base.TimeSince(actions[i].Created), base.ActionDesc(actions[i], ctx.User.AvatarLink()))
- }
- - ctx.Render.JSON(200, &feeds)
- + ctx.JSON(200, &feeds)
- }
-
- func Issues(ctx *middleware.Context) {
- - ctx.Render.HTML(200, "user/issues", ctx.Data)
- + ctx.HTML(200, "user/issues", ctx.Data)
- }
-
- func Pulls(ctx *middleware.Context) {
- - ctx.Render.HTML(200, "user/pulls", ctx.Data)
- + ctx.HTML(200, "user/pulls", ctx.Data)
- }
-
- func Stars(ctx *middleware.Context) {
- - ctx.Render.HTML(200, "user/stars", ctx.Data)
- + ctx.HTML(200, "user/stars", ctx.Data)
- }
-
- func Activate(ctx *middleware.Context) {
- diff --git a/templates/base/footer.tmpl b/templates/base/footer.tmpl
- index 841af68..fe0ea93 100644
- --- a/templates/base/footer.tmpl
- +++ b/templates/base/footer.tmpl
- @@ -3,7 +3,9 @@
- <footer id="footer">
- <div class="container footer-wrap">
- <p>© 2014 Gogs · ver {{AppVer}} ·
- - <i class="fa fa-github"></i><a target="_blank" href="https://github.com/gogits/gogs">GitHub</a>
- + All: {{LoadTimes .PageStartTime}} ·
- + Tmpl: {{call .TmplLoadTimes}} ·
- + <i class="fa fa-github"></i><a target="_blank" href="https://github.com/gogits/gogs">GitHub</a> ·
- </p>
- <p class="desc"></p>
- </div>
- diff --git a/web.go b/web.go
- index e519fe1..fe3596e 100644
- --- a/web.go
- +++ b/web.go
- @@ -12,7 +12,6 @@ import (
-
- "github.com/codegangsta/cli"
- "github.com/codegangsta/martini"
- - "github.com/martini-contrib/render"
- "github.com/martini-contrib/sessions"
-
- "github.com/gogits/binding"
- @@ -64,7 +63,7 @@ func runWeb(*cli.Context) {
- m := newMartini()
-
- // Middlewares.
- - m.Use(render.Renderer(render.Options{Funcs: []template.FuncMap{base.TemplateFuncs}}))
- + m.Use(middleware.Renderer(middleware.RenderOptions{Funcs: []template.FuncMap{base.TemplateFuncs}}))
-
- // TODO: should use other store because cookie store is not secure.
- store := sessions.NewCookieStore([]byte("secret123"))
- --
- 1.8.4.1
|