Désolé, mais je ne suis pas d’accord. Je ne vois pas comment cela peut être une demande de fonctionnalité et non un bogue.
Je comprends qu’il existe d’autres points de terminaison d’API utilisables. Mais l’objectif principal de /admin/users/sync_sso est précisément de maintenir ce type de données, eh bien, synchronisées. Il permet déjà de définir le champ nom du compte — cela fonctionne. Cependant, il suppose que le nom est toujours un champ obligatoire et ne permet pas de le laisser vide. Il ne peut donc pas être utilisé pour maintenir les données synchronisées.
Le code ci-dessous semble fonctionner comme prévu dans mon bac à sable, mais encore une fois, je ne me sens pas assez confiant pour soumettre une PR pour le moment. N’hésitez pas à l’utiliser, l’adapter, etc.
- if SiteSetting.sso_overrides_name && user.name != name && name.present?
- user.name = name || User.suggest_name(username.blank? ? email : username)
+ if SiteSetting.sso_overrides_name && user.name != name
+ if SiteSetting.full_name_required && name.present?
+ user.name = name || User.suggest_name(username.blank? ? email : username)
+ else
+ user.name = name
+ end