When creating multiple CAS users via /admin/people/create/cas-bulk, the users are created, and associated with the CAS user and such, but the email address defaults to:
[cas_username]@
Which is a malformed email address, and therefore causes other issues in Drupal.

There is a setting in the CAS settings form for "Email hostname" (The email domain name used to combine with the username to form the user's email address.) but it is only available when "Automatically register users" is checked. When it's checked, the domain name is added appropriately.

I propose pulling "Email hostname" out of the visibility rules of "Automatically register users", so that it's always available.

CommentFileSizeAuthor
#5 3106985-5.patch5.28 KBbkosborne
#4 3106985-4.patch6.11 KBbkosborne
#3 bulk-add-email-3106985-3.patch900 bytessegovia94
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Shawn DeArmond created an issue. See original summary.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented

Shawn, what about if on the bulk add form, we added a field where an admin can input an email address to use for the users? We only need to present it if the module is not already configured to use the hardcoded email for all users.

This keeps the admin interface a bit cleaner I think. It's less confusing to explain.

segovia94’s picture

segovia94 CreditAttribution: segovia94 as a volunteer commented
FileSize
bulk-add-email-3106985-3.patch900 bytes

Here's a patch to at least fall back to the email hostname if and account email hostname is not set.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
Status: Active » Needs review
FileSize
3106985-4.patch6.11 KB

Here's a solution based on my proposal.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
FileSize
3106985-5.patch5.28 KB

Re-rolled

  • bkosborne committed cc6e84a on 8.x-1.x 
    Issue #3106985 by bkosborne, segovia94: When bulk-adding CAS users,...
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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