i had a customized version of token running till now, this was the only change patch-worthy..
easy to implement but don't know if node_load gives too much of a performance penalty?

CommentFileSizeAuthor
#5 321487-token-comment-node-title.patch1.83 KBdave reid
token-add-comment-node-title.patch1.17 KBeMPee584

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Needs work

Yes, I think this would be better as a 

$title = db_result(db_query("SELECT title FROM {node} WHERE nid = %d", $comment->nid));
eMPee584’s picture

eMPee584 commented

yeah guess so..

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Category: bug » feature

This isn't a bug, it's a feature request. :)

greggles’s picture

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

It also needs a check_plain on it.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: Unassigned » dave reid
Status: Needs work » Needs review
StatusFileSize
new 321487-token-comment-node-title.patch1.83 KB

Revised patch with comment-node-title-raw and comment-node-title.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

This passed locally for me, so committed to CVS, and committing to the token upgrade path in D7.
http://drupal.org/cvs?commit=388308
http://drupal.org/cvs?commit=388310

Status: Fixed » Closed (fixed)

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