I wanted to be able to add a link to the outgoing email allowing people to reply to a comment with a single click, so i added a variable !comment_reply_url to the array of available variables. Pretty simple patch and thought you might find it useful

CommentFileSizeAuthor
#8 314630_cid_nid_tokens.patch3.48 KBgreggles
reply_link.patch433 bytesgdd
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented

Thanks for the patch. I think this would be better implemented by supporting the token module #273112: Provide realname support in the existing token system

Then people could just use something like:

[site-url]/comment/reply/[nid]/[cid]

To get the same thing. Right?

gdd’s picture

gdd
he/him
Primary language English
Location Portland, OR
CreditAttribution: gdd commented

If token support was implemented, then yes that would absolutely be the way to go about it.

zilla’s picture

zilla CreditAttribution: zilla commented

very nice. is this part of the current dev?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented

@zilla - when I commit things I update them to fixed. If it were part of dev you could see it in http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/comment_not...

So, no, it's not.

zilla’s picture

zilla CreditAttribution: zilla commented

thanks, gotta start tracking the dev in cvs because there are too many threads to follow and tracking the details on the project page makes no sense...

aclight’s picture

aclight CreditAttribution: aclight commented
Version: 6.x-1.0 » 7.x-1.x-dev

Patch in #0 applies successfully to 5.x-2.x-dev and 6.x-1.x dev. I think this is a good feature request. I guess it needs to be decided first whether or not comment_notify will require token module. I don't know what effect Dries's comment #273112-4: Provide realname support in the existing token system has on this decision.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented

He seems to be ok with token in general these days: http://drupal.org/node/113614#comment-1185431 (note also that paul.lovvik, works for acquia, is the main contributor to that thread these days).

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Title: Add 'reply' link as an option for email » Add nid and cid replacements to allow things like a reply link
FileSize
314630_cid_nid_tokens.patch3.48 KB

How about this more flexible alternative that gets the same goal, but in a more flexible way?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Needs review » Patch (to be ported)

Now committed - http://drupal.org/cvs?commit=290170

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Version: 7.x-1.x-dev » 5.x-2.x-dev
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Patch (to be ported) » Closed (fixed)

This module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.