Problem/Motivation

The SAML authentication settings on the user profile form are added in simplesamlphp_auth_form_user_form_alter(). This field cannot be moved in the interface by a site builder.

Proposed resolution

Add an hook_entity_extra_field_info() implementation, so the SAML authentication settings can be moved on the user profile form.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

The SAML authentication settings can be moved on the user profile form.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 simplesamlphp_auth-user_form_extra_field-2950849-2.patch1.05 KBidebr

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr commented
Status: Active » Needs review
StatusFileSize
new simplesamlphp_auth-user_form_extra_field-2950849-2.patch1.05 KB

Attached patch implements a hook_entity_extra_field_info() implementation, so the SAML authentication settings can be moved on the user profile form.

dakku’s picture

dakku commented

HI @idebr
maybe we should make an admin setting like:
"Allow users to enable/disable saml in profile"

and then implement this functionality in the profile.

ralphvdhoudt’s picture

ralphvdhoudt commented
Status: Needs review » Reviewed & tested by the community

@dakku Currently this isn't an admin setting and teh patch just enabled the relocation of the field

For an admin setting you'd better open a new issue in my opinion

This fix seems RTBC

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Makes sense to me, thanks.

  • Berdir committed ab51ce3 on 8.x-3.x authored by idebr 
    Issue #2950849 by idebr: Implement extra field for SAML authentication...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.