Skip to content
Snippets Groups Projects
Commit 0f850f52 authored by meskio's avatar meskio :tent:
Browse files

Be able to update the role but not the balance when we update member

parent 593b0eb7
Branches
Tags
No related merge requests found
...@@ -179,7 +179,7 @@ func (a *api) UpdateMemberMe(num int, w http.ResponseWriter, req *http.Request) ...@@ -179,7 +179,7 @@ func (a *api) UpdateMemberMe(num int, w http.ResponseWriter, req *http.Request)
} }
member.Num = 0 member.Num = 0
member.Balance = -1 member.Role = ""
m, err := a.updateMember(num, member, true) m, err := a.updateMember(num, member, true)
if err != nil { if err != nil {
if errors.Is(err, gorm.ErrRecordNotFound) { if errors.Is(err, gorm.ErrRecordNotFound) {
...@@ -225,8 +225,8 @@ func (a *api) updateMember(num int, member MemberReq, checkPass bool) (Member, e ...@@ -225,8 +225,8 @@ func (a *api) updateMember(num int, member MemberReq, checkPass bool) (Member, e
if member.Email != "" { if member.Email != "" {
dbMember.Email = member.Email dbMember.Email = member.Email
} }
if member.Balance >= 0 { if member.Role != "" {
dbMember.Balance = member.Balance dbMember.Role = member.Role
} }
if member.Password != "" { if member.Password != "" {
dbMember.PassHash, dbMember.Salt, err = newHashPass(member.Password) dbMember.PassHash, dbMember.Salt, err = newHashPass(member.Password)
......
...@@ -86,7 +86,7 @@ func TestMemberUpdate(t *testing.T) { ...@@ -86,7 +86,7 @@ func TestMemberUpdate(t *testing.T) {
tapi.addTestMember() tapi.addTestMember()
member := testMember member := testMember
member.Balance = 1000 member.Role = "admin"
resp := tapi.doAdmin("PUT", "/member/10", member, nil) resp := tapi.doAdmin("PUT", "/member/10", member, nil)
if resp.StatusCode != http.StatusAccepted { if resp.StatusCode != http.StatusAccepted {
t.Fatal("Can't update member:", resp.Status) t.Fatal("Can't update member:", resp.Status)
...@@ -97,8 +97,8 @@ func TestMemberUpdate(t *testing.T) { ...@@ -97,8 +97,8 @@ func TestMemberUpdate(t *testing.T) {
if resp.StatusCode != http.StatusOK { if resp.StatusCode != http.StatusOK {
t.Error("Can't find the member:", resp.Status) t.Error("Can't find the member:", resp.Status)
} }
if gotMember.Balance != 1000 { if gotMember.Role != "admin" {
t.Error("Wrong balance:", gotMember) t.Error("Wrong role:", gotMember)
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment