Changeset 2129

Timestamp:

Oct 9, 2007, 11:35:31 PM (17 years ago)

Author:

rub

Message:

Fix some little bugs on mail functions

Location:

trunk

Files:

• . (modified) (1 prop)
• include/functions_mail.inc.php (modified) (6 diffs)

trunk

@@ -5 +5 @@
 phpmv2
 map.php
+upgrade65.log

@@ -5 +5 @@
 phpmv2
 map.php
+upgrade65.log

trunk/include/functions_mail.inc.php

@@ -411 +411 @@
       $list[] = $row;
     }
-  }
-
-  foreach ($list as $elem)
-  {
-    $query = '
+
+    foreach ($list as $elem)
+    {
+      $query = '
 SELECT
   u.'.$conf['user_fields']['username'].' as username,
@@ -430 +429 @@
 ;';
 
-    $result = pwg_query($query);
-
-    if (mysql_num_rows($result) > 0)
-    {
-      $Bcc = array();
-      while ($row = mysql_fetch_array($result))
+      $result = pwg_query($query);
+
+      if (mysql_num_rows($result) > 0)
       {
-        if (!empty($row['mail_address']))
+        $Bcc = array();
+        while ($row = mysql_fetch_array($result))
         {
-          array_push($Bcc, format_email($row['username'], $row['mail_address']));
+          if (!empty($row['mail_address']))
+          {
+            array_push($Bcc, format_email($row['username'], $row['mail_address']));
+          }
         }
-      }
-
-      if (count($Bcc) > 0)
-      {
-        switch_lang_to($elem['language']);
-
-        $mail_template = get_mail_template($email_format, $elem);
-        $mail_template->set_filename($tpl_shortname,
-          (empty($dirname) ? '' : $dirname.'/').$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,
+            (empty($dirname) ? '' : $dirname.'/').$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();
+        }
       }
     }
@@ -589 +589 @@
   $content = '';
 
-  if (!isset($conf_mail[$args['email_format']][get_pwg_charset()][$args['template']][$args['theme']]))
+  // key compose of indexes witch allow ti cache mail data
+  $cache_key = $args['email_format'].'-'.$lang_info['code'].'-'.$args['template'].'-'.$args['theme'];
+
+  if (!isset($conf_mail[$cache_key]))
   {
     if (!isset($mail_template))
@@ -650 +653 @@
 
     // what are displayed on the header of each mail ?
-    $conf_mail[$args['email_format']]
-      [get_pwg_charset()]
-      [$args['template']][$args['theme']]['header'] =
-        $mail_template->parse('mail_header', true);
+    $conf_mail[$cache_key]['header'] =
+      $mail_template->parse('mail_header', true);
 
     // what are displayed on the footer of each mail ?
-    $conf_mail[$args['email_format']]
-      [get_pwg_charset()]
-      [$args['template']][$args['theme']]['footer'] =
-        $mail_template->parse('mail_footer', true);
+    $conf_mail[$cache_key]['footer'] =
+      $mail_template->parse('mail_footer', true);
   }
 
   // Header
-  $content.= $conf_mail[$args['email_format']]
-              [get_pwg_charset()]
-              [$args['template']][$args['theme']]['header'];
+  $content.= $conf_mail[$cache_key]['header'];
 
   // Content
@@ -683 +680 @@
 
   // Footer
-  $content.= $conf_mail[$args['email_format']]
-              [get_pwg_charset()]
-              [$args['template']][$args['theme']]['footer'];
+  $content.= $conf_mail[$cache_key]['footer'];
 
   // Close boundary
@@ -700 +695 @@
     global $user;
     @mkdir(PHPWG_ROOT_PATH.'testmail');
-    $filename = PHPWG_ROOT_PATH.'testmail/mail.'.$user['username'];
+    $filename = PHPWG_ROOT_PATH.'testmail/mail.'.$user['username'].'.'.$lang_info['code'].'.'.$args['template'].'.'.$args['theme'];
     if ($args['content_format'] == 'text/plain')
     {

@@ -5 +5 @@
 phpmv2
 map.php
+upgrade65.log