Bug #590

type mismatch phpmailer->SMTPDebug vs. smtp->do_debug

Added by Anonymous almost 15 years ago. Updated almost 15 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Common
Target version:0.68

Description

using glpi-trunk-2006-06-13.

line 529 of /lib/phpmailer/class.phpmailer.php has:

$this->smtp->do_debug = $this->SMTPDebug;

where SMTPDebug is boolean, default 'false' (from line 199: var $SMTPDebug = false;)

but in /lib/phpmailer/class.smtp.php

do_debug is integer, default '0' (from line 62: $this->do_debug = 0;)
and various lines of class.smtp.php compare do_debug as integer. of course this condition (do_debug >= <some integer>) always evaluates to 'true'!

not sure if it is better to make phpmailer->SMTPDebug to be integer, or to make smtp->do_debug to be boolean. /inc/mailing.class.php uses boolean SMTPDebug as well.

History

#1 Updated by moyo almost 15 years ago

Are you sure that it is not due to the force DEBUG mode activate in config.php line 280 for development purpose ?
$cfg_glpidebug=DEBUG_MODE;
need to be comment to use GLPI in normal mode.

#2 Updated by almost 15 years ago

hm I thought I had commented that and also updated debug field of glpi_config table to '1' just to be sure.

but let me check, thanks!!

#3 Updated by admin almost 15 years ago

apologies, you are quite correct. I set to NORMAL_MODE and class.smtp does not output debug anymore. please close as non-issue. thank you!

#4 Updated by moyo almost 15 years ago

  • Status changed from New to Closed

Also available in: Atom PDF