Skip to content

Commit db810b2

Browse files
fix bug
1 parent 4a1c3f8 commit db810b2

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

services/user_services.go

+8-5
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,10 @@ func (s *UserService) LoginByUsername(req *dto.LoginByUsernameRequest) (*dto.Tok
5454

5555
}
5656

57-
// Register
5857

58+
// Register
5959
func (s *UserService) RegisterByUsername(req *dto.RegisterUserByUsernameRequest) error {
60-
u := models.User{Username: req.Username,Email: req.Email}
60+
u := models.User{Username: req.Username, Email: req.Email}
6161

6262
exists, err := s.existsByEmail(req.Email)
6363
if err != nil {
@@ -81,20 +81,23 @@ func (s *UserService) RegisterByUsername(req *dto.RegisterUserByUsernameRequest)
8181
return err
8282
}
8383
u.Password = string(hp)
84-
84+
8585
tx := s.database.Begin()
8686
err = tx.Create(&u).Error
8787
if err != nil {
8888
tx.Rollback()
8989
s.logger.Error(logging.Postgres, logging.Rollback, err.Error(), nil)
9090
return err
9191
}
92+
if err != nil {
93+
tx.Rollback()
94+
s.logger.Error(logging.Postgres, logging.Rollback, err.Error(), nil)
95+
return err
96+
}
9297
tx.Commit()
9398
return nil
94-
9599
}
96100

97-
98101
func (s *UserService) existsByEmail(email string) (bool, error) {
99102
var exists bool
100103
if err := s.database.Model(&models.User{}).

0 commit comments

Comments
 (0)