0003307: check.php always checks mail() rather than the config - MantisBT

ID	Project	Category	View Status	Date Submitted	Last Update

0003307	mantisbt	installation	public	2003-08-23 18:18	2004-08-29 02:02

Reporter	vboctor	Assigned To	thraxisp
Priority	normal	Severity	feature	Reproducibility	always
Status	closed	Resolution	fixed
Fixed in Version	0.19.0rc1

Summary	0003307: check.php always checks mail() rather than the config
Description	The admin/check.php checks that the mail() function is working even though the setting in config_inc.php can be to use PHPMailer. The check should be modified to check the configured mode.
Tags	No tags attached.
Attached Files	check_mail.diff (841 bytes) Index: admin/check.php =================================================================== RCS file: /cvsroot/mantisbt/mantisbt/admin/check.php,v retrieving revision 1.8 diff -r1.8 check.php 16c16,17 < @require_once( '../core.php' ); --- > require_once ( dirname( dirname( __FILE__ ) ) . DIRECTORY_SEPARATOR . 'core.php' ); > require_once ( dirname( dirname( __FILE__ ) ) . DIRECTORY_SEPARATOR . 'core/email_api.php' ); 347c348 < $result = mail( config_get( 'administrator_email' ), 'Testing PHP mail() function', 'Your PHP mail settings appear to be correctly set.', 'From: ' . config_get( 'administrator_email' ) . "\n" ); --- > $result = email_send( config_get( 'administrator_email' ), 'Testing PHP mail() function', 'Your PHP mail settings appear to be correctly set.', 'From: ' . config_get( 'administrator_email' ) . "\n" ); check_mail.diff (841 bytes) admin_check.php.gz (3,889 bytes)

grangeway 2004-07-24 08:55 reporter ~0006316	This is probably more important now if we are only going to use phpmailer :)

thraxisp 2004-07-24 20:20 reporter ~0006344 Last edited: 2004-07-25 17:27	Patch and admin/check.php attached to use core email routines to test the mail send. I will apply it if you agree. edited on: 07-25-04 17:27

jlatour 2004-08-06 11:37 reporter ~0006718	Assuming you've tested it, go ahead and apply it. In the future, please send in unified diffs (diff -u) - they're easier to read.

thraxisp 2004-08-06 12:17 reporter ~0006728	fixed in CVS