[imp] 'smtphost' in servers.php don't fully work

Michael M Slusarz slusarz at mail.curecanti.org
Wed Feb 2 09:12:55 PST 2005


Quoting Лукьянов Владимир <lichnoye at macbox.ru>:

> If anybody interesting, we solve this problem by changing next lines 
> to ../horde/imp/lib/Sessions.php
>
> was
>
>            if ($conf['mailer']['type'] == 'smtp') {
>                if (!empty($settings['smtphost'])) {
>                    $_SESSION['imp']['smtphost'] = $settings['smtphost'];
>                }
>                if (!empty($settings['smtpport'])) {
>                    $_SESSION['imp']['smtpport'] = $settings['smtpport'];
>                }
>
> became
>
>            if ($conf['mailer']['type'] == 'smtp') {
>                $_SESSION['imp']['smtphost'] = isset($ptr['smtphost']) 
> ? $ptr['smtphost'] : null;
> //                if (!empty($settings['smtphost'])) {
> //                    $_SESSION['imp']['smtphost'] = $settings['smtphost'];
> //                }
>                $_SESSION['imp']['smtpport'] = isset($ptr['smtpport']) 
> ? $ptr['smtpport'] : null;
> //                if (!empty($settings['smtpport'])) {
> //                    $_SESSION['imp']['smtpport'] = $settings['smtpport'];
> //                }

Close, but not quite (you can't use isset()).  I've implemented a 
working solution and committed to both HEAD and 4.0.2.  Thanks for the 
report.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]


More information about the imp mailing list