[v10.4.x] Chore: Remove extra sql select from the Insert function of userimpl.store (#87164)

Chore: Remove extra sql select from the Insert function of userimpl.store (#87060)

Remove getAnyUserType

(cherry picked from commit 1cb3f332a1)
This commit is contained in:
Misi
2024-05-01 10:12:22 +02:00
committed by GitHub
parent edde0fa815
commit 5dfc92cdec
-21
View File
@@ -83,11 +83,6 @@ func (ss *sqlStore) Insert(ctx context.Context, cmd *user.User) (int64, error) {
return 0, err
}
// verify that user was created and cmd.ID was updated with the actual new userID
_, err = ss.getAnyUserType(ctx, cmd.ID)
if err != nil {
return 0, err
}
return cmd.ID, nil
}
@@ -735,19 +730,3 @@ func (ss *sqlStore) Search(ctx context.Context, query *user.SearchUsersQuery) (*
})
return &result, err
}
// getAnyUserType searches for a user record by ID. The user account may be a service account.
func (ss *sqlStore) getAnyUserType(ctx context.Context, userID int64) (*user.User, error) {
usr := user.User{ID: userID}
err := ss.db.WithDbSession(ctx, func(sess *db.Session) error {
has, err := sess.Get(&usr)
if err != nil {
return err
}
if !has {
return user.ErrUserNotFound
}
return nil
})
return &usr, err
}