[mms] Add configuration to allow for SMTP authentication if the curre…

…nt Horde access does not have an authenticated user.
horde · Jul 8, 2014 · 48b100c · 48b100c
1 parent 29a5fa6
commit 48b100c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/framework/Core/lib/Horde/Core/Factory/Mail.php b/framework/Core/lib/Horde/Core/Factory/Mail.php
@@ -97,12 +97,16 @@ public function getConfig()
          * 'auth'. Will remove in create() if final config doesn't require
          * authentication. */
         if (strcasecmp($transport, 'smtp') === 0) {
-            if (!isset($params['username'])) {
-                $params['username'] = $registry->getAuth();
-            }
-            if (!isset($params['password'])) {
-                $params['password'] = $registry->getAuthCredential('password');
+            if ($auth = $registry->getAuth()) {
+                if (!empty($params['username_auth'])) {
+                    $params['username'] = $auth;
+                }
+                if (!empty($params['password_auth'])) {
+                    $params['password'] = $registry->getAuthCredential('password');
+                }
             }
+
+            unset($params['password_auth'], $params['username_auth']);
         }
 
         return array($transport, $params);

diff --git a/framework/Core/package.xml b/framework/Core/package.xml
@@ -39,6 +39,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
+* [mms] Add configuration to allow for SMTP authentication if the current Horde access does not have an authenticated user.
 * [mjr] Fix closing smartmobile dialog pages (Bug #13316).
 * [mjr] Fix issue that could cause multiple tabs to be shown as active when tab_name could be a mix of integers or strings.
  </notes>
@@ -3480,6 +3481,7 @@
    <date>2014-07-01</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
+* [mms] Add configuration to allow for SMTP authentication if the current Horde access does not have an authenticated user.
 * [mjr] Fix closing smartmobile dialog pages (Bug #13316).
 * [mjr] Fix issue that could cause multiple tabs to be shown as active when tab_name could be a mix of integers or strings.
    </notes>