[Svn] r3906 - in trunk/roundcubemail: . installer program/include

trac at roundcube.net trac at roundcube.net
Wed Aug 25 21:09:14 CEST 2010


Author: alec
Date: 2010-08-25 14:09:13 -0500 (Wed, 25 Aug 2010)
New Revision: 3906

Modified:
   trunk/roundcubemail/CHANGELOG
   trunk/roundcubemail/installer/test.php
   trunk/roundcubemail/program/include/rcube_smtp.php
Log:
- Fix SMTP test in Installer (#1486952)


Modified: trunk/roundcubemail/CHANGELOG
===================================================================
--- trunk/roundcubemail/CHANGELOG	2010-08-25 06:20:00 UTC (rev 3905)
+++ trunk/roundcubemail/CHANGELOG	2010-08-25 19:09:13 UTC (rev 3906)
@@ -7,6 +7,7 @@
 - Fixes in SQL init script + added update script for MSSQL database
 - Remove redundant date in syslog messages (#1486945)
 - Fix contacts list page controls when a group is selected (#1486946)
+- Fix SMTP test in Installer (#1486952)
 
 RELEASE 0.4
 -----------

Modified: trunk/roundcubemail/installer/test.php
===================================================================
--- trunk/roundcubemail/installer/test.php	2010-08-25 06:20:00 UTC (rev 3905)
+++ trunk/roundcubemail/installer/test.php	2010-08-25 19:09:13 UTC (rev 3906)
@@ -276,7 +276,8 @@
       $send_headers = $mail_object->headers($headers);
 
       $SMTP = new rcube_smtp();
-      $SMTP->connect();
+      $SMTP->connect(rcube_parse_host($RCI->getprop('smtp_server')),
+        $RCI->getprop('smtp_port'), $CONFIG['smtp_user'], $CONFIG['smtp_pass']);
 
       $status = $SMTP->send_mail($headers['From'], $headers['To'],
           ($foo = $mail_object->txtHeaders($send_headers)), $body);

Modified: trunk/roundcubemail/program/include/rcube_smtp.php
===================================================================
--- trunk/roundcubemail/program/include/rcube_smtp.php	2010-08-25 06:20:00 UTC (rev 3905)
+++ trunk/roundcubemail/program/include/rcube_smtp.php	2010-08-25 19:09:13 UTC (rev 3906)
@@ -38,21 +38,16 @@
 
 
   /**
-   * Object constructor
-   *
-   * @param 
-   */
-  function __construct()
-  {
-  }
-
-
-  /**
    * SMTP Connection and authentication
    *
+   * @param string Server host
+   * @param string Server port
+   * @param string User name
+   * @param string Password
+   *
    * @return bool  Returns true on success, or false on error
    */
-  public function connect()
+  public function connect($host=null, $port=null, $user=null, $pass=null)
   {
     $RCMAIL = rcmail::get_instance();
   
@@ -64,10 +59,10 @@
   
     // let plugins alter smtp connection config
     $CONFIG = $RCMAIL->plugins->exec_hook('smtp_connect', array(
-      'smtp_server' => $RCMAIL->config->get('smtp_server'),
-      'smtp_port'   => $RCMAIL->config->get('smtp_port', 25),
-      'smtp_user'   => $RCMAIL->config->get('smtp_user'),
-      'smtp_pass'   => $RCMAIL->config->get('smtp_pass'),
+      'smtp_server' => $host ? $host : $RCMAIL->config->get('smtp_server'),
+      'smtp_port'   => $port ? $port : $RCMAIL->config->get('smtp_port', 25),
+      'smtp_user'   => $user ? $user : $RCMAIL->config->get('smtp_user'),
+      'smtp_pass'   => $pass ? $pass : $RCMAIL->config->get('smtp_pass'),
       'smtp_auth_type' => $RCMAIL->config->get('smtp_auth_type'),
       'smtp_helo_host' => $RCMAIL->config->get('smtp_helo_host'),
       'smtp_timeout'   => $RCMAIL->config->get('smtp_timeout'),

_______________________________________________
http://lists.roundcube.net/mailman/listinfo/svn



More information about the Svn mailing list