Changeset 25361

Timestamp:

Nov 6, 2013, 8:12:18 PM (11 years ago)

Author:

mistic100

Message:

feature 2995: New email template
add function get_clean_recipients_list() allowing big flexibility for recipients lists

File:

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

trunk/include/functions_mail.inc.php

@@ -114 +114 @@
   if (is_array($input))
   {
+    if (!isset($input['name']))
+    {
+      $input['name'] = '';
+    }
     return $input;
   }
@@ -132 +136 @@
   }
 }
+
+/**
+ * Return a clean array of hashmaps (email, name) from various inputs
+ * - comma separated list
+ * - array of emails
+ * - single hashmap (email[, name])
+ * @since 2.6
+ *
+ * @param mixed $data
+ * @return string[][]
+ */
+function get_clean_recipients_list($data)
+{
+  if (empty($data))
+  {
+    return array();
+  }
+  else if (is_array($data))
+  {
+    $values = array_values($data);
+    if (!is_array($values[0]))
+    {
+      $keys = array_keys($data);
+      if (is_int($keys[0]))
+      { // simple array of emails
+        foreach ($data as &$item)
+        {
+          $item = array(
+            'email' => $item,
+            'name' => '',
+            );
+        }
+        unset($item);
+      }
+      else
+      { // hashmap of one recipient
+        $data = array(unformat_email($data));
+      }
+    }
+    else
+    { // array of hashmaps
+      $data = array_map('unformat_email', $data);
+    }
+  }
+  else
+  {
+    $data = explode(',', $data);
+    $data = array_map('unformat_email', $data);
+  }
   
+  return $data;
+}
 
 /**
@@ -155 +210 @@
   return implode(',', array_unique($result));
 }
-
 
 /**
@@ -494 +548 @@
   $mail = new PHPMailer;
 
-  $recipients = !is_array($to) ? explode(',', $to) : $to;
-  foreach ($recipients as $recipient)
-  {
-    $recipient = unformat_email($recipient);
+  foreach (get_clean_recipients_list($to) as $recipient)
+  {
     $mail->addAddress($recipient['email'], $recipient['name']);
   }
@@ -532 +584 @@
   if (!empty($args['Cc']))
   {
-    foreach ($args['Cc'] as $cc)
-    {
-      $cc = unformat_email($cc);
-      $mail->addCC($cc['email'], $cc['name']);
+    foreach (get_clean_recipients_list($args['Cc']) as $recipient)
+    {
+      $mail->addCC($recipient['email'], $recipient['name']);
     }
   }
 
   // Bcc
+  $Bcc = get_clean_recipients_list(@$args['Bcc']);
   if ($conf_mail['send_bcc_mail_webmaster'])
   {
-    $args['Bcc'][] = get_webmaster_mail_address();
-  }
-  if (!empty($args['Bcc']))
-  {
-    foreach ($args['Bcc'] as $bcc)
-    {
-      $bcc = unformat_email($bcc);
-      $mail->addBCC($bcc['email'], $bcc['name']);
+    $Bcc[] = get_webmaster_mail_address();
+  }
+  if (!empty($Bcc))
+  {
+    foreach ($Bcc as $recipient)
+    {
+      $mail->addBCC($recipient['email'], $recipient['name']);
     }
   }