From 7eaa080b6fa8f6e1ae17a406449b9e46db56b1ac Mon Sep 17 00:00:00 2001
From: scottrigby <scott@basekamp.com>
Date: Sun, 29 May 2011 12:23:10 -0400
Subject: [PATCH] Issue #615748 by frjo: update Signup Status Mailer to send to anonymous user

---
 .../signup_status_mailer.module                    |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/modules/signup_status_mailer/signup_status_mailer.module b/modules/signup_status_mailer/signup_status_mailer.module
index 20e3f66..160eb30 100644
--- a/modules/signup_status_mailer/signup_status_mailer.module
+++ b/modules/signup_status_mailer/signup_status_mailer.module
@@ -117,12 +117,15 @@ function signup_status_mailer_nodeapi(&$node, $op, $arg) {
 function signup_status_mailer_signup_status($action, $signup) {
   $params = array();
   $node = node_load($signup->nid);
+  $user_mail = _signup_get_email($signup);
+  $user_name = _signup_get_username($signup);
+  $account = user_load($signup->uid);
 
   // See if there are any admin notifications configured for this site.
   $notify_emails = variable_get('signup_status_mailer_admin_notify_emails', array());
   if (!empty($notify_emails)) {
     $params['node'] = $node;
-    $params['account'] = $account = user_load($signup->uid);
+    $params['account'] = $account;
     $params['signup'] = $signup;
     $params['subject'] = variable_get('signup_status_mailer_admin_notify_subject', '');
     $params['body'] = variable_get('signup_status_mailer_admin_notify_body', '');
@@ -155,9 +158,6 @@ function signup_status_mailer_signup_status($action, $signup) {
     $mail_key .= '_default';
   }
 
-  if (empty($account)) {
-    $account = user_load($signup->uid);
-  }
   $language = user_preferred_language($account);
   if (empty($params)) {
     $params['node'] = $node;
@@ -168,11 +168,11 @@ function signup_status_mailer_signup_status($action, $signup) {
   $params['body'] = $settings['body'];
 
   // Actually try to send it and log what happened.
-  if (drupal_mail('signup_status_mailer', $mail_key, $account->mail, $language, $params)) {
-    watchdog('signup_status_mail', 'Signup status message sent to %name at %mail for %action %status', array('%name' => $account->name, '%mail' => $account->mail, '%action' => $action, '%status' => $status_name), WATCHDOG_NOTICE, l(t('view'), 'node/'. $node->nid .'/signups'));
+  if (drupal_mail('signup_status_mailer', $mail_key, $user_mail, $language, $params)) {
+    watchdog('signup_status_mail', 'Signup status message sent to %name at %mail for %action %status', array('%name' => $user_name, '%mail' => $user_mail, '%action' => $action, '%status' => $status_name), WATCHDOG_NOTICE, l(t('view'), 'node/'. $node->nid .'/signups'));
   }
   else {
-    watchdog('signup_status_mail', 'Error sending signup status message to %name at %mailfor %action %status', array('%name' => $account->name, '%mail' => $mail, '%action' => $action, '%status' => $status_name), WATCHDOG_ERROR, l(t('view'), 'node/'. $node->nid .'/signups'));
+    watchdog('signup_status_mail', 'Error sending signup status message to %name at %mail for %action %status', array('%name' => $user_name, '%mail' => $user_mail, '%action' => $action, '%status' => $status_name), WATCHDOG_ERROR, l(t('view'), 'node/'. $node->nid .'/signups'));
   }
 }
 
-- 
1.7.2.3