11
Mithrandir
Re: Add to Notification Option

You have both {X_PNAME} in your mail_template AND a $tags['X_PNAME'] = something; line in your php file?

12
phatty
Re: Add to Notification Option
  • 2004/4/20 19:20

  • phatty

  • Just popping in

  • Posts: 70

  • Since: 2004/3/9 2


Just figured that out.... sorry for the bump everyone...

Working great now...

Thanks to all

Jeff

13
phatty
Re: Add to Notification Option
  • 2004/4/20 19:29

  • phatty

  • Just popping in

  • Posts: 70

  • Since: 2004/3/9 2


Just figured that out.... sorry for the bump everyone...

Working great now...

Thanks to all

Jeff

14
greggo
Re: Add to Notification Option
  • 2005/8/19 20:43

  • greggo

  • Just popping in

  • Posts: 7

  • Since: 2003/3/20


Quote:

Mithrandir wrote:
You have both {X_PNAME} in your mail_template AND a $tags['X_PNAME'] = something; line in your php file?
I've tried $tags['X_PNAME'] = $myts->stripSlashesGPC($forumpost->uname()); and it doesn't work. In fact, it messes up my post.php page...which doesn't redirect after a post is made, just a blank screen. I had made an additional change so that the subject and content could be in the notification email, which was to simply move
$tags['POST_CONTENT'] = $myts->stripSlashesGPC($_POST['message']);
        
$tags['POST_NAME'] = $myts->stripSlashesGPC($_POST['subject']);
up a few lines so they'd appear in the email as well. Any ideas?

15
xgarb
Add thread subject to Notification Option
  • 2005/12/2 17:08

  • xgarb

  • Not too shy to talk

  • Posts: 154

  • Since: 2003/3/30


you need to move three lines....


$myts =& MyTextSanitizer::getInstance();
$tags['POST_CONTENT'] = $myts->stripSlashesGPC($HTTP_POST_VARS['message']);
$tags['POST_NAME'] = $myts->stripSlashesGPC($HTTP_POST_VARS['subject']);


and then in notification.php

add
$subject $subject." : ".$tags['POST_NAME'];
above
$notification->notifyUser($template_dir$template$subject$tags);

(about line 673)

to get the thread subject in the email.

I've done mimimal testing and only just know what I'm doing so test it!

Thanks,

xgarb