remove unnecessary abstraction

This commit is contained in:
dhax 2017-10-12 16:51:46 +02:00
parent eb928912af
commit 81aa0b651e
2 changed files with 4 additions and 5 deletions

View file

@ -116,8 +116,7 @@ func (rs *AccountResource) create(w http.ResponseWriter, r *http.Request) {
return return
} }
acc := data.Account if err := rs.Store.Create(data.Account); err != nil {
if err := rs.Store.Create(acc); err != nil {
switch err.(type) { switch err.(type) {
case validation.Errors: case validation.Errors:
render.Render(w, r, ErrValidation(ErrAccountValidation, err)) render.Render(w, r, ErrValidation(ErrAccountValidation, err))
@ -126,7 +125,7 @@ func (rs *AccountResource) create(w http.ResponseWriter, r *http.Request) {
render.Render(w, r, ErrInvalidRequest(err)) render.Render(w, r, ErrInvalidRequest(err))
return return
} }
render.Respond(w, r, newAccountResponse(acc)) render.Respond(w, r, newAccountResponse(data.Account))
} }
func (rs *AccountResource) get(w http.ResponseWriter, r *http.Request) { func (rs *AccountResource) get(w http.ResponseWriter, r *http.Request) {
@ -142,7 +141,7 @@ func (rs *AccountResource) update(w http.ResponseWriter, r *http.Request) {
return return
} }
if err := rs.Store.Update(data.Account); err != nil { if err := rs.Store.Update(acc); err != nil {
switch err.(type) { switch err.(type) {
case validation.Errors: case validation.Errors:
render.Render(w, r, ErrValidation(ErrAccountValidation, err)) render.Render(w, r, ErrValidation(ErrAccountValidation, err))

View file

@ -103,7 +103,7 @@ func (rs *AccountResource) update(w http.ResponseWriter, r *http.Request) {
return return
} }
if err := rs.Store.Update(data.Account); err != nil { if err := rs.Store.Update(acc); err != nil {
switch err.(type) { switch err.(type) {
case validation.Errors: case validation.Errors:
render.Render(w, r, ErrValidation(err)) render.Render(w, r, ErrValidation(err))