I got it to work too
I got it to work too
No idea what are you talking about, as the email subject contains thanked post subject as well as email body does, the latter also contains the link to the thanked post.
rxu wrote: ↑Wed Jul 19, 2017 4:19 pmNo idea what are you talking about, as the email subject contains thanked post subject
['quote]
Not in my installation. I mentioned this before here
I am out of town at the moment, but I can send you a screen cap if you want.
Yes, this works as advertised.as well as email body does, the latter also contains the link to the thanked post.
Subject: {THANKS_SUBG} — "{POST_SUBJECT}"
line, so "No email subject specified" case shouldn't be possible.I can confirm that even with the correct Email template, "No Email subject specified" DOES happen.rxu wrote: ↑Wed Jul 19, 2017 6:26 pm Original email template is this https://github.com/rxu/thanks_for_posts ... thanks.txt so what is yours?
It hasSubject: {THANKS_SUBG} — "{POST_SUBJECT}"
line, so "No email subject specified" case shouldn't be possible.
Hi rxu, it happens all the time every time.
Code: Select all
public function get_email_template_variables()
{
$username = $this->user_loader->get_username($this->get_data('poster_id'), 'username');
return array(
'THANKS_SUBG' => htmlspecialchars_decode($this->language->lang('THANKS_PM_SUBJECT_'. $this->get_data('lang_act'))),
'USERNAME' => htmlspecialchars_decode($this->user->data['username']),
'POST_SUBJECT' => htmlspecialchars_decode(censor_text($this->get_data('post_subject'))),
'POST_THANKS' => htmlspecialchars_decode($this->language->lang('THANKS_PM_MES_'. $this->get_data('lang_act'))),
'POSTER_NAME' => htmlspecialchars_decode($username),
'U_POST_THANKS' => generate_board_url() . '/viewtopic.' . $this->php_ext . "?p={$this->item_id}#p{$this->item_id}",
);
}
I've only been using the extension for a couple of weeks or so and mostly it's just me testing it...giving likes, not getting many...but it has happened every time I've received a notification.
As I have already stated, that part that rxu mentioned is exactly as it should be and is correct and can therefore be eliminated from the problem. That is why I was wondering if the code I've highlighted - which looks as though it should insert the sent Email notification subject line - isn't correct.