Problem/Motivation

The event registration reminders go to all registered people at the moment, regardless of whether they are waitlisted or not

Proposed resolution

In file recurring_events/modules/recurring_events_registration/modules/recurring_events_reminders/recurring_events_reminders.module line: 156.

Change: $registrants = $registration_creation_service->retrieveRegisteredParties();

To: $registrants = $registration_creation_service->retrieveRegisteredParties(TRUE, FALSE);

Background info:

From recurring_events/modules/recurring_events_registration/src/RegistrationCreationService.php line 170 onwards

/**
* Retrieve all registered parties.
*
* @param bool $include_nonwaitlisted
* Whether or not to include non-waitlisted registrants.
* @param bool $include_waitlisted
* Whether or not to include waitlisted registrants.
* @param int $uid
* The user ID for whom to retrieve registrants.
*
* @return array
* An array of registrants.
*/
public function retrieveRegisteredParties($include_nonwaitlisted = TRUE, $include_waitlisted = TRUE, $uid = FALSE) { ....

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

simonknight created an issue. See original summary.

plopesc made their first commit to this issue’s fork.

plopesc’s picture

Status: Active » Needs review

Created MR with suggested changes. Could you please confirm that works as expected?

pfrenssen made their first commit to this issue’s fork.

  • pfrenssen committed a4966c08 on 2.0.x authored by plopesc
    Issue #3535470 by plopesc, pfrenssen, simonknight: Registration...
pfrenssen’s picture

Status: Needs review » Fixed

Thanks for the fix!

  • pfrenssen committed a4966c08 on 3.0.x authored by plopesc
    Issue #3535470 by plopesc, pfrenssen, simonknight: Registration...

Status: Fixed » Closed (fixed)

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