Changeset 1914 for trunk/include/functions_mail.inc.php

Timestamp:

03/16/07 13:08:25 (6 years ago)

Author:

rub

Message:

Fix bug: Don't invoke mail function when there aren't recipient.

Files:

• trunk/include/functions_mail.inc.php (modified) (4 diffs)

trunk/include/functions_mail.inc.php

@@ -6 +6 @@
 // | Copyright (C) 2006-2007 Ruben ARNAUD - team@phpwebgallery.net         |
 // +-----------------------------------------------------------------------+
-// | branch        : BSF (Best So Far)
-// | file          : $RCSfile$
+// | file          : $Id$
 // | last update   : $Date$
 // | last modifier : $Author$
@@ -291 +290 @@
   }
 
-  $keyargs_content_admin_info = array
-  (
-    get_l10n_args('Connected user: %s', $user['username']),
-    get_l10n_args('IP: %s', $_SERVER['REMOTE_ADDR']),
-    get_l10n_args('Browser: %s', $_SERVER['HTTP_USER_AGENT'])
-  );
-
-  switch_lang_to($conf['default_language']);
-
-  $return = pwg_mail
-  (
-    '',
-    array
+  if (count($admins) > 0)
+  {
+    $keyargs_content_admin_info = array
     (
-      'Bcc' => $admins,
-      'subject' => '['.$conf['gallery_title'].'] '.l10n_args($keyargs_subject),
-      'content' => 
-         l10n_args($keyargs_content)."\n\n"
-        .l10n_args($keyargs_content_admin_info)."\n",
-      'content_format' => 'text/plain'
-    )
-  ) and $return;
-
-  switch_lang_back();
+      get_l10n_args('Connected user: %s', $user['username']),
+      get_l10n_args('IP: %s', $_SERVER['REMOTE_ADDR']),
+      get_l10n_args('Browser: %s', $_SERVER['HTTP_USER_AGENT'])
+    );
+
+    switch_lang_to($conf['default_language']);
+
+    $return = pwg_mail
+    (
+      '',
+      array
+      (
+        'Bcc' => $admins,
+        'subject' => '['.$conf['gallery_title'].'] '.l10n_args($keyargs_subject),
+        'content' => 
+           l10n_args($keyargs_content)."\n\n"
+          .l10n_args($keyargs_content_admin_info)."\n",
+        'content_format' => 'text/plain'
+      )
+    ) and $return;
+
+    switch_lang_back();
+  }
 
   return $return;
@@ -400 +402 @@
       }
 
-      switch_lang_to($elem['language']);
-
-      $mail_template = get_mail_template($email_format, $elem);
-      $mail_template->set_filename($tpl_shortname, 
-        (IN_ADMIN ? 'admin/' : '').$tpl_shortname.'.tpl');
-      $mail_template->assign_vars($assign_vars);
-
-      $return = pwg_mail
-      (
-        '',
-        array
+      if (count($Bcc) > 0)
+      {
+        switch_lang_to($elem['language']);
+
+        $mail_template = get_mail_template($email_format, $elem);
+        $mail_template->set_filename($tpl_shortname, 
+          (IN_ADMIN ? 'admin/' : '').$tpl_shortname.'.tpl');
+        $mail_template->assign_vars($assign_vars);
+
+        $return = pwg_mail
         (
-          'Bcc' => $Bcc,
-          'subject' => l10n_args($keyargs_subject),
-          'email_format' => $email_format,
-          'content' => $mail_template->parse($tpl_shortname, true),
-          'content_format' => $email_format,
-          'template' => $elem['template'],
-          'theme' => $elem['theme']
-        )
-      ) and $return;
-
-      switch_lang_back();
+          '',
+          array
+          (
+            'Bcc' => $Bcc,
+            'subject' => l10n_args($keyargs_subject),
+            'email_format' => $email_format,
+            'content' => $mail_template->parse($tpl_shortname, true),
+            'content_format' => $email_format,
+            'template' => $elem['template'],
+            'theme' => $elem['theme']
+          )
+        ) and $return;
+
+        switch_lang_back();
+      }
     }
   }
@@ -452 +457 @@
   global $conf, $conf_mail, $lang_info, $page;
 
+  if (empty($to) and empty($args['Cc']) and empty($args['Bcc']))
+  {
+    return true;
+  }
+  
   if (!isset($conf_mail))
   {