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

Timestamp:

Mar 18, 2008, 11:02:33 PM (16 years ago)

Author:

rub

Message:

Resolved issue 0000815: Email format standardization

Merge branch-1_7 2282:2283 into BSF

File:

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

trunk/include/functions_mail.inc.php

@@ -53 +53 @@
 
 /*
+ * Returns the name of the mail sender :
+ *
+ * @return string
+ */
+function get_mail_sender_name()
+{
+  global $conf;
+
+  return (empty($conf['mail_sender_name']) ? $conf['gallery_title'] : $conf['mail_sender_name']);
+}
+
+/*
  * Returns an array of mail configuration parameters :
  *
@@ -82 +94 @@
   // name of the webmaster is the title of the gallery
   $conf_mail['formated_email_webmaster'] =
-    format_email($conf['gallery_title'], $conf_mail['email_webmaster']);
+    format_email(get_mail_sender_name(), $conf_mail['email_webmaster']);
 
   $conf_mail['boundary_key'] = generate_key(32);
@@ -97 +109 @@
 function format_email($name, $email)
 {
-  global $conf;
-
   // Spring cleaning
   $cvt_email = trim(preg_replace('#[\n\r]+#s', '', $email));
-
-  if ($conf['enabled_format_email'])
-  {
-    // Spring cleaning
-    $cvt_name = trim(preg_replace('#[\n\r]+#s', '', $name));
-
-    if ($cvt_name!="")
-    {
-      $cvt_name = encode_mime_header(
-                '"'
-                .addcslashes($cvt_name,'"')
-                .'"');
-      $cvt_name .= ' ';
-    }
-
-    if (strpos($cvt_email, '<') === false)
-    {
-      return $cvt_name.'<'.$cvt_email.'>';
-    }
-    else
-    {
-      return $cvt_name.$cvt_email;
-    }
+  $cvt_name = trim(preg_replace('#[\n\r]+#s', '', $name));
+
+  if ($cvt_name!="")
+  {
+    $cvt_name = encode_mime_header(
+              '"'
+              .addcslashes($cvt_name,'"')
+              .'"');
+    $cvt_name .= ' ';
+  }
+
+  if (strpos($cvt_email, '<') === false)
+  {
+    return $cvt_name.'<'.$cvt_email.'>';
   }
   else
   {
-    return $cvt_email;
-  }
+    return $cvt_name.$cvt_email;
+  }
+}
+
+/**
+ * Returns an email address list with minimal email string
+ *
+ * @param string with email list (email separated by comma)
+ */
+function get_strict_email_list($email_list)
+{
+  $result = array();
+  $list = explode(',', $email_list);
+  foreach ($list as $email)
+  {
+    if (strpos($email, '<') !== false)
+    {
+       $email = preg_replace('/.*<(.*)>.*/i', '$1', $email);
+    }
+    $result[] = trim($email);
+  }
+
+  return implode(',', $result);
 }
 
@@ -507 +529 @@
  *
  * @param:
- *   - to: receiver(s) of the mail.
+ *   - to: receiver(s) of the mail (list separated by comma).
  *   - args: function params of mail function:
  *       o from: sender [default value webmaster email]
@@ -546 +568 @@
   }
 
-  if (!empty($to))
-  {
-    $to = format_email('', $to);
-  }
-
   if (empty($args['from']))
   {
@@ -597 +614 @@
   {
     $headers.= 'To: undisclosed-recipients: ;'."\n";
+  }
+  else
+  {
+    $headers.= 'To: '.$to."\n";
   }
 
@@ -720 +741 @@
     trigger_event('send_mail',
       false, /* Result */
-      trigger_event('send_mail_to', $to),
+      trigger_event('send_mail_to', get_strict_email_list($to)),
       trigger_event('send_mail_subject', $cvt_subject),
       trigger_event('send_mail_content', $content),