Simplify nil check based on code review feedback

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Fix groups not being assigned when registering through OAuth providers
2026-02-17 12:59:57 +00:00 · 2026-02-17 12:58:41 +00:00 · 2026-02-17 12:51:07 +00:00
1 changed files with 7 additions and 0 deletions
--- a/object/user.go
+++ b/object/user.go
@@ -954,6 +954,13 @@ func UpdateUserForAllFields(id string, user *User) (bool, error) {

 	user.UpdatedTime = util.GetCurrentTime()

+	if len(user.Groups) > 0 {
+		_, err = userEnforcer.UpdateGroupsForUser(user.GetId(), user.Groups)
+		if err != nil {
+			return false, err
+		}
+	}
+
 	affected, err := ormer.Engine.ID(core.PK{owner, name}).AllCols().Update(user)
 	if err != nil {
 		return false, err