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
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
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 CreditAttribution: eMPee584 commented

yeah guess so..

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid 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
CreditAttribution: greggles commented

It also needs a check_plain on it.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Assigned: Unassigned » Dave Reid
Status: Needs work » Needs review
FileSize
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
CreditAttribution: Dave Reid 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.