From cb8fb2dfc208edd42a725ca6737fffd3aa5beabf Mon Sep 17 00:00:00 2001 From: Jannis Mattheis Date: Wed, 28 Feb 2018 19:46:53 +0100 Subject: [PATCH] Remove unnecessary code --- api/user.go | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/api/user.go b/api/user.go index 3f1ea3b..a92a0b9 100644 --- a/api/user.go +++ b/api/user.go @@ -47,16 +47,12 @@ func (a *UserAPI) GetCurrentUser(ctx *gin.Context) { func (a *UserAPI) CreateUser(ctx *gin.Context) { user := model.UserExternalWithPass{} if err := ctx.Bind(&user); err == nil { - if len(user.Pass) == 0 { - ctx.AbortWithError(400, errors.New("password may not be empty")) + internal := a.toInternal(&user, []byte{}) + if a.DB.GetUserByName(internal.Name) == nil { + a.DB.CreateUser(internal) + ctx.JSON(200, toExternal(internal)) } else { - internal := a.toInternal(&user, []byte{}) - if a.DB.GetUserByName(internal.Name) == nil { - a.DB.CreateUser(internal) - ctx.JSON(200, toExternal(internal)) - } else { - ctx.AbortWithError(400, errors.New("username already exists")) - } + ctx.AbortWithError(400, errors.New("username already exists")) } } }