Merge pull request #6435 from owncloud/fix-disable-users

fix disable users by group
owncloud · Jun 5, 2023 · 3385b07 · 3385b07
2 parents 80b4e85 + 58d8c1a
commit 3385b07
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/changelog/unreleased/fix-disable-user.md b/changelog/unreleased/fix-disable-user.md
@@ -0,0 +1,5 @@
+Bugfix: Fix error message when disabling users
+
+When we disable users by adding them to a group we do not need to update the user entry.
+
+https://github.com/owncloud/ocis/pull/6435
diff --git a/services/graph/pkg/identity/ldap.go b/services/graph/pkg/identity/ldap.go
@@ -1178,7 +1178,7 @@ func (i *LDAP) updateAccountEnabledState(logger log.Logger, accountEnabled bool,
 		} else {
 			err = i.disableUser(logger, e.DN)
 		}
-		updateNeeded = true
+		updateNeeded = false
 	}
 
 	return updateNeeded, err