The function fieldactions_send_email_to_email_action creates a dblog entry of 'Sent email to %node_owner_mail', when it should be the email address entered into an email widget field.

So in line 523,

      if (drupal_mail('fieldactions_general', 'fieldactions_mail_to_email', $field_value['email'], $language, $params, $from, TRUE)) {
        watchdog('fieldactions', 'Sent email to %recipient', array('%recipient' => $vars['%node_owner_mail']));
      }
      else {
        watchdog('error', 'Unable to send email to %recipient', array('%recipient' => $vars['%node_owner_mail']));
      }

should be changed to

      if (drupal_mail('fieldactions_general', 'fieldactions_mail_to_email', $field_value['email'], $language, $params, $from, TRUE)) {
        watchdog('fieldactions', 'Sent email to %recipient', array('%recipient' => $field_value['email']));
      }
      else {
        watchdog('error', 'Unable to send email to %recipient', array('%recipient' => $field_value['email']));
      }
CommentFileSizeAuthor
#1 fieldactions-547062-1.patch828 bytesjajathejazzcat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jajathejazzcat’s picture

FileSize
828 bytes

Here is a patch.

deekayen’s picture

Status: Active » Fixed

committed to HEAD and DRUPAL-6--1

Status: Fixed » Closed (fixed)

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