Thread: Mini Mods - Email reminder for inactive users
View Single Post
  #600  
Old 05-08-2007, 04:53 PM
aam aam is offline
 
Join Date: Apr 2007
Posts: 6
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by kitsch View Post
I've sent out a huge batch of e-mails and to my surprise the From: field of the e-mail has been set incorrectly. It should have been:

admin@gsm-forum.eu

and instead it got set to:

"GSM Forum Europe" <GSM Forum Europe@p2pgsm.com>

(p2pgsm.com being the name of the server the forum runs on). So the users received the mail from europe@p2pgsm.com... Which isn't really cool.
Quote:
Originally Posted by C Braithwaite View Post
Not being funny or anything, but try setting your email address to the correct address in your ACP?
The reason that it is putting that email address is because that is what YOU set it to. any emails sent out from vbulletin will use that email address as the return address/contact address.
I've been playing with your mod (v1.11) and it is very useful, but it needs a bit of tweaking. There is a real problem where the from field is not being generated correctly. I initially thought it must be something to do with the email address in vboptions, but it isn't.

I've noticed that the resultant email "from field" can end up as various things, but generally either "bbtiltle <bbtitle>" or bbtitle<servername>. This can lead to the email being rejected by some mail servers (as the sent email does not have a correct email from header, or a domain indicated and no @).

The problem is due to the way vbmail has been used in remindermail.php. The final email from field is not generated by: (line50)
Code:
$uheaders .= "From: " . $vbulletin->options['bbtitle'] . " Reminder Service <" . $vbulletin->options['webmasteremail'] . ">" . "\r\n";
Instead it is from the $from: (line 69)
Code:
vbmail($toemail, $subject, $message, $notsubscription = false, $from = $vbulletin->options['bbtitle'], $uheaders = '', $username = '');
So all it was generating was a from based on bbtitle. One way to correctly generate a from field containing the board's email is to change $from to use 'webmasteremail': (line 69)

Code:
vbmail($toemail, $subject, $message, $notsubscription = false, $from = $vbulletin->options['webmasteremail'], $uheaders = '', $username = '');
A similar problem can be seen in the reminder report produced (line 79), but this time multiple from email headers are created on the report - one from $from and another from $uheaders.
Reply With Quote
 
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01708 seconds
  • Memory Usage 1,776KB
  • Queries Executed 11 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD_SHOWPOST
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (3)bbcode_code
  • (2)bbcode_quote
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)post_thanks_box
  • (1)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (1)post_thanks_postbit_info
  • (1)postbit
  • (1)postbit_onlinestatus
  • (1)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • reputationlevel
  • showthread
Included Files:
  • ./showpost.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showpost_start
  • bbcode_fetch_tags
  • bbcode_create
  • postbit_factory
  • showpost_post
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • showpost_complete