ynerant/synchronize-dolibarr-to-ldap
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't add group entries if we aren't in a group

Browse Source
This commit is contained in:
Emmy D'Anello 2025-02-18 15:08:23 +01:00
parent 94dd261d15
commit 27f3145857
Signed by: ynerant
GPG Key ID: 3A75C55819C8CF85
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
main.py
View File

@ -104,6 +104,8 @@ def append_extra_group_fields_to_ldap_user(ldap_user: WritableEntry, dolibarr_us
user_id = dolibarr_user['id']
dolibarr_groups: list[dict] = dolibarr_client.get_user_groups_uid(user_id)
if not isinstance(dolibarr_groups, list):
dolibarr_groups = []
if oldgroupid:
dolibarr_groups = [group for group in dolibarr_groups if group['id'] != oldgroupid]
if newgroupid:
@ -111,6 +113,9 @@ def append_extra_group_fields_to_ldap_user(ldap_user: WritableEntry, dolibarr_us
if new_group:
dolibarr_groups = [group for group in dolibarr_groups if group['id'] != new_group['id']] + [new_group]
if not dolibarr_groups:
return
for extra_field in config.LDAP_USERS_EXTRA_FIELDS:
dolibarr_attrs, ldap_attr = extra_field.split(':')
dolibarr_attrs = dolibarr_attrs.split('|')