From b43cfc5b39c0aeec04b1810f01e57ece96abdd7e Mon Sep 17 00:00:00 2001 From: Unknown Date: Sat, 15 Mar 2014 10:52:14 -0400 Subject: Clean code --- routers/repo/repo.go | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'routers/repo/repo.go') diff --git a/routers/repo/repo.go b/routers/repo/repo.go index 710e5a21..23f8ea10 100644 --- a/routers/repo/repo.go +++ b/routers/repo/repo.go @@ -20,7 +20,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) { return } - if hasErr, ok := ctx.Data["HasError"]; ok && hasErr.(bool) { + if ctx.HasError() { ctx.Render.HTML(200, "repo/create", ctx.Data) return } @@ -30,10 +30,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) { user, err := models.GetUserById(form.UserId) if err != nil { if err.Error() == models.ErrUserNotExist.Error() { - ctx.Data["HasError"] = true - ctx.Data["ErrorMsg"] = "User does not exist" - auth.AssignForm(form, ctx.Data) - ctx.Render.HTML(200, "repo/create", ctx.Data) + ctx.RenderWithErr("User does not exist", "repo/create", &form) return } } @@ -48,10 +45,7 @@ func Create(ctx *middleware.Context, form auth.CreateRepoForm) { } if err.Error() == models.ErrRepoAlreadyExist.Error() { - ctx.Data["HasError"] = true - ctx.Data["ErrorMsg"] = "Repository name has already been used" - auth.AssignForm(form, ctx.Data) - ctx.Render.HTML(200, "repo/create", ctx.Data) + ctx.RenderWithErr("Repository name has already been used", "repo/create", &form) return } -- cgit v1.2.3