Add comment variables to send email action

Excellent - works for me.

Thank you

Feature request go to HEAD and then ported.

It is a bug report, not a feature request.

First of all: The system_mail function includes some taxonomy variables but the action settings form's description does not present these. (%term_name, %term_description, %term_id, %vocabulary_name, %vocabulary_description, %vocabulary_id) These are the hidden variables.

Next: The system_send_email_action function gives the comment's data(in the context) but doesn't use it(just the cid, to load the node). It's a bug. When I make a trigger for new comment, I can't make informative email which contains the comment data. I just make a simple message which contains node data. When anybody comments my blog I read 'You got a new comment'. This is unusable! I want to make a more informative mail that contains the comment body itself. Drupal 6 has this feature but I can't use it because the system module has this bug.

System message is the same.

I do it.

http://drupal.org/node/323325

pp

Well, to me it looks like pp's comment includes #2 and expands on that with related bug fixes. Also, #2 includes testing code (error_log()), so that is not up for consideration as-is.

Just ran across this issue while searching. Bumping cause it's old.

It seems there is not anymore interest in this feature, or it has already covered by another report.

I still have interest in it! - I have to re-patch my Drupal installs every time a new version comes out.

I would love this, or a feature like this, to be included in Drupal as I'd like to be able to read the comment text without having to login to my site to see that the comment was spam or not for approval.

Please review this patch! I am afraid it's not a perfect patch, but it works.

1, Let's talk about which variables are necessary and which aren't.
2, If you see my little module that resolves this problem ( http://drupal.org/node/323325#comment-1417786 ) you see the following three lines are unnecessary because the $params['context'] included the comment data.

  // Include comment info if this is a comment
  if (isset($comment)) {
    $params['comment'] = $comment;
  }

But I think the code is nicer with it. You can use $params['comment'] instead of $params['context']['comment'].

pp

It work's fine, but I saw 5 "Hunk message"s.

I rerolled the patch.

Please read #12

pp

Should not it better to wait until token.module is ported into Drupal, before to proceed with this one? if Token get ported in Drupal, as it seems it is happening, this patch should be updated, and applied again.

OK. It is postponed until #113614: Add centralized token/placeholder substitution to core is in the core.

the item this was postponed on is complete.

I wonder how it can be ported back now that the idea is to make it dependent on 7.x feature.

Is there no way we can get this functionality back-ported to 6.x w/o doing it ourselves every time we want it? I guess we could dupe the system module email action and create one that offers the comment variables but that seems sorta' silly.

Use BlogAid (http://ftp.drupal.org/files/projects/blogaid-6.x-2.x-dev.tar.gz) or mini module http://drupal.org/node/323325#comment-1417786

pp

#14: 287315-2.patch queued for re-testing.

subscribe