refactor: optimize user status

Author: colin404

internal/nightwatch/watcher/user/fsm.go

@@ -11,12 +11,12 @@ import (
 // The FSM is configured with the following events and callbacks:
 //
 // Events:
-// - UserStatusRegistered -> UserStatusActive
+// - UserStatusRegistered -> UserStatusActived
 // - UserStatusBlacklisted -> UserStatusDisabled
 // - UserStatusDisabled -> UserStatusDeleted
 //
 // Callbacks:
-// - UserStatusActive: Calls the NewActiveUserCallback function to handle the "active user" event.
+// - UserStatusActived: Calls the NewActiveUserCallback function to handle the "active user" event.
 // - UserStatusDisabled: Calls the NewDisableUserCallback function to handle the "disable user" event.
 // - UserStatusDeleted: Calls the NewDeleteUserCallback function to handle the "delete user" event.
 // - UserEventAfterEvent: Calls the NewUserEventAfterEvent function after any user-related event is handled.
@@ -27,17 +27,17 @@ func NewFSM(initial string, w *userWatcher) *fsm.FSM {
 		initial,
 		fsm.Events{
 			// Define status events.
-			{Name: known.UserStatusRegistered, Src: []string{known.UserStatusRegistered}, Dst: known.UserStatusActive},
+			{Name: known.UserStatusRegistered, Src: []string{known.UserStatusRegistered}, Dst: known.UserStatusActived},
 			{Name: known.UserStatusBlacklisted, Src: []string{known.UserStatusBlacklisted}, Dst: known.UserStatusDisabled},
 			// Define need events.
-			{Name: known.UserStatusNeedActive, Src: []string{known.UserStatusNeedActive}, Dst: known.UserStatusActive},
+			{Name: known.UserStatusNeedActive, Src: []string{known.UserStatusNeedActive}, Dst: known.UserStatusActived},
 			{Name: known.UserStatusNeedDisable, Src: []string{known.UserStatusNeedDisable}, Dst: known.UserStatusDisabled},
 			// After disabling the user, they can be deleted, and the FSM will automatically transition to the next deleted state.
 			// I have decided not to delete the user in the code, so the state transition here is commented out.
 			// {Name: known.UserStatusDisabled, Src: []string{known.UserStatusDisabled}, Dst: known.UserStatusDeleted},
 		},
 		fsm.Callbacks{
-			known.UserStatusActive:   NewActiveUserCallback(w.store),
+			known.UserStatusActived:   NewActiveUserCallback(w.store),
 			known.UserStatusDisabled: NewDisableUserCallback(w.store),
 			known.UserStatusDeleted:  NewDeleteUserCallback(w.store),
 			// log, alert, save to stoer, etc for all events.

internal/pkg/known/usercenter/status.go

@@ -8,7 +8,7 @@ const (
 	UserStatusRegistered = "registered"
 	// The user has registered and been verified, and can use the system normally.
 	// Most user operations are performed in this state.
-	UserStatusActive = "active"
+	UserStatusActived = "actived"
 	// The user has entered the incorrect password too many times, and the account has been locked by the system.
 	// The user needs to recover the password or contact the administrator to unlock the account.
 	UserStatusLocked = "locked"
@@ -24,8 +24,8 @@ const (
 )
 
 // Define need status.
-// We can directly update the database to the "Need" state to inform
-// onex-nightwatch of what needs to be done.
+// We can directly update the database to the "Need" state to inform onex-nightwatch of what needs to be done.
+// These statuses are only used for operation and maintenance purposes.
 const (
 	// UserStatusNeedActive informs onex-nightwatch that the user needs to be activated.
 	UserStatusNeedActive = "need_active"

internal/usercenter/model/hook.go

@@ -47,7 +47,7 @@ func (u *UserM) BeforeCreate(tx *gorm.DB) (err error) {
 		return err // Return error if there's a problem with encryption.
 	}
 
-	u.Status = known.UserStatusActive // Set the default status for the user as active.
+	u.Status = known.UserStatusActived // Set the default status for the user as active.
 
 	return nil
 }