We can now store additional field types, e.g. string_long. It would be nice to alter the data prior to save.

For example, I get an array of json objects that I'd like to keep up-to-date in a user profile field. Currently, it will only save the first item in the array, with an alter hook I can prepare the data to be saved as a string.

CommentFileSizeAuthor
#4 allow_other_modules_to-2912214-4.patch2.62 KBsanduhrs
openid_connect--allow-alter-of-claim.patch542 byteschrisrockwell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chrisrockwell created an issue. See original summary.

chrisrockwell’s picture

chrisrockwell CreditAttribution: chrisrockwell commented
Status: Active » Needs review
interX’s picture

interX CreditAttribution: interX as a volunteer commented

Any reason why images can not be altered before saving? I can imagine a use case for that too.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
FileSize
allow_other_modules_to-2912214-4.patch2.62 KB

Changed the signature a bit and added documentation.
Please have a look if this is still working for you.

Mario Steinitz’s picture

Mario Steinitz
Location Hong Kong Central
CreditAttribution: Mario Steinitz at SHORELESS Limited commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Closed (fixed) » Needs work

We should not alter public APIs in the beta phase.. The older hook_openid_connect_save_userinfo() must be re-instated, and marked deprecated.

  • jcnventura authored 08b87c5 on 8.x-1.x 
    Issue #2912214 by jcnventura: Restore deprecated...
jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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