When using the broadcast message form, I believe it would be very helpful to offer the following options for the recipients:

All signups
Users that have attended
Users that have not attended
Users without a recorded attendance

Attached is a patch that does so.

-------

On a side note, I believe the use of constants should be introduced for attendance status, rather than 1, 0, and NULL.

I had to do:

if (!($signup->attended === '1')) {

Where I should be doing:

if ($signup->attended == SIGNUP_ATTENDANCE_YES) {

CommentFileSizeAuthor
signup_broadcast-attendance-options.patch2.73 KBmstef

Comments

avpaderno’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
Issue summary: View changes
Status: Needs review » Closed (outdated)

I am closing this issue, since it's for a Drupal version no longer supported.