The "Accessible clients" and "Associate a client to this user" sections of the user editing form are missing. This is because the the D7-deprecated hook_user_form() hasn't been replaced with a call to hook_form_FORM_ID_alter(). This also required other code adjustments to work properly. Patching file "hosting_client.access.inc" fixes this. See attached screenshots of the page before and after patching.

CommentFileSizeAuthor
#7 accessible_clients_not_shown-issue_#2857721-comment_#7.patch5.32 KBkfolsom
#3 accessible_clients_patched.jpg36.34 KBkfolsom
#3 accessible_clients_unpatched.jpg19.86 KBkfolsom
#3 accessible_clients_not_shown-issue_#2857721-comment_#3.patch5.32 KBkfolsom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kfolsom created an issue. See original summary.

kfolsom’s picture

kfolsom CreditAttribution: kfolsom commented
Issue summary: View changes
kfolsom’s picture

kfolsom CreditAttribution: kfolsom commented
FileSize
accessible_clients_not_shown-issue_#2857721-comment_#3.patch5.32 KB
accessible_clients_unpatched.jpg19.86 KB
accessible_clients_patched.jpg36.34 KB
kfolsom’s picture

kfolsom CreditAttribution: kfolsom commented
Issue summary: View changes
kienan’s picture

kienan CreditAttribution: kienan commented
Status: Active » Needs review

The patch adds a couple of whitespace errors, but applies and works.

kfolsom’s picture

kfolsom CreditAttribution: kfolsom commented

I'll fix those errors and re-submit the patch. Thanks for reporting it.

Keith

kfolsom’s picture

kfolsom CreditAttribution: kfolsom commented
FileSize
accessible_clients_not_shown-issue_#2857721-comment_#7.patch5.32 KB

Updated patch is attached that fixes the whitespace error.

kienan’s picture

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

  • helmo committed ada8eaa on 7.x-3.x authored by kfolsom 
    Issue #2857721 by kfolsom: "Accessible clients" and "Associate a client...
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed with only a minor code style change.

Status: Fixed » Closed (fixed)

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