عذراً، لكنني لا أوافق. لا أرى كيف يمكن اعتبار هذا طلب ميزة بدلاً من كونه عيباً.
أفهم وجود نقاط نهاية API أخرى يمكن استخدامها. لكن الهدف الرئيسي من /admin/users/sync_sso هو بالضبط الحفاظ على هذا النوع من البيانات متزامناً. إنه يسمح بالفعل بتعيين حقل اسم الحساب — وهذا يعمل. لكنه يفترض أن الاسم مطلوب دائماً ولا يسمح بتعيينه فارغاً. وبالتالي لا يمكن استخدامه للحفاظ على تزامن البيانات.
يبدو أن الكود أدناه يجعله يعمل كما هو متوقع في بيئة الاختبار الخاصة بي، لكن مرة أخرى، لا أشعر بالثقة الكافية لتقديم طلب سحب (PR) في هذه المرحلة. لا تتردد في استخدامه/تعديله، إلخ.
- 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