The main idea is to provide two hook wich permit others modules to alter registration settings when an entity host is created or updated. It's an easy method wich permit to handle several use case as this issue Relative Date for Reminder Messages.

Attached a patch wich provides this 2 hooks (with drupal_alter) on registration_entity_insert() and registration_entity_update() implementations. These hook provide in the $context variable the entity host object.

These hooks are documented in registration.api.php. I also updated this doc with the hook available but not present in this file (hook_registration_status_alter).

CommentFileSizeAuthor
#8 Screen Shot 2018-12-21 at 8.39.00 PM.png53.32 KBChris Matthews
#4 registration-provide-hook-for-altering-registration-settings-2330605-4.patch5.71 KBflocondetoile
#1 registration-provide-hook-for-altering-registration-settings-2330605-2.patch5.6 KBflocondetoile
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
CreditAttribution: flocondetoile commented
Status: Active » Needs review
FileSize
registration-provide-hook-for-altering-registration-settings-2330605-2.patch5.6 KB

The patch attached.
For review.

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
CreditAttribution: flocondetoile commented
Status: Needs review » Active
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
CreditAttribution: flocondetoile commented

The variable $registration_type is not consistent belong the user's role. With user-1 it's a string and with other roles it's an array...This patch handles this issue. The lines changed are :

'registration_type' => is_array($registration_type) ? reset($registration_type) : $registration_type,
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
CreditAttribution: flocondetoile commented
FileSize
registration-provide-hook-for-altering-registration-settings-2330605-4.patch5.71 KB

the patch

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
CreditAttribution: flocondetoile commented
Status: Active » Needs review

Let the bot test it.

semei’s picture

semei CreditAttribution: semei commented

I also think that these hooks are very important to have.

xywei’s picture

xywei CreditAttribution: xywei commented

It helps me a lot,thank you!

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews commented
Version: 7.x-1.3 » 7.x-1.x-dev
Status: Needs review » Needs work
Issue tags: +Needs reroll
FileSize
Screen Shot 2018-12-21 at 8.39.00 PM.png53.32 KB

The patch in #4 does not apply on the latest 7.x-1.x-dev - added needs reroll issue tag.

does not apply

john.oltman’s picture

john.oltman CreditAttribution: john.oltman at SiteBasin, Inc. commented
Status: Needs work » Closed (outdated)