Browse Source

Merge pull request #52 from IamTaoChen/fix/bug

fix: cannot delete user
1 year ago
parent
commit
0d6db0d2a1
1 changed files with 3 additions and 4 deletions
  1. 3 4
      service/user.go

+ 3 - 4
service/user.go

@@ -215,12 +215,11 @@ func (us *UserService) Delete(u *model.User) error {
215 215
 		tx.Rollback()
216 216
 		return err
217 217
 	}
218
+	tx.Commit()
218 219
 	// 删除关联的peer
219
-	if err := AllService.PeerService.EraseUserId(u.Id); err != nil {
220
-		tx.Rollback()
221
-		return err
220
+	return AllService.PeerService.EraseUserId(u.Id); err != nil {
221
+		return errors.New("User deleted successfully, but failed to unlink peer.")
222 222
 	}
223
-	tx.Commit()
224 223
 	return nil
225 224
 }
226 225