Changeset 10 for Common/Mail.php

Timestamp:

Apr 7, 2020, 8:52:35 PM (5 years ago)

Author:

chronos

Message:

• Modified: Improved code formatting.

File:

• Common/Mail.php (modified) (9 diffs)

Common/Mail.php

@@ -69 +69 @@
   function Organization($org)
   {
-    if(trim($org != '')) $this->xheaders['Organization'] = $org;
+    if (trim($org != '')) $this->xheaders['Organization'] = $org;
   }
 
   function Priority($Priority)
   {
-    if(!intval($Priority)) return(false);
-
-    if(!isset($this->priorities[$Priority - 1])) return(false);
+    if (!intval($Priority)) return false;
+
+    if (!isset($this->priorities[$Priority - 1])) return false;
 
     $this->xheaders['X-Priority'] = $this->priorities[$Priority - 1];
-    return(true);
+    return true;
   }
 
@@ -96 +96 @@
   function Send()
   {
-    if(count($this->Bodies) == 0) throw new Exception(T('Mail message need at least one text body'));
+    if (count($this->Bodies) == 0) throw new Exception(T('Mail message need at least one text body'));
 
     $Body = $this->BuildAttachment($this->BuildBody());
@@ -103 +103 @@
     $this->Headers['CC'] = '';
     $this->Headers['BCC'] = '';
-    foreach($this->Recipients as $Index => $Recipient)
-    {
-      if($Recipient['Type'] == 'SendCombined')
-      {
-        if($Index > 0) $To .= ', ';
+    foreach ($this->Recipients as $Index => $Recipient)
+    {
+      if ($Recipient['Type'] == 'SendCombined')
+      {
+        if ($Index > 0) $To .= ', ';
         $To .= $Recipient['Address'];
       } else
-      if($Recipient['Type'] == 'Send')
-      {
-        if($Index > 0) $To .= ', ';
+      if ($Recipient['Type'] == 'Send')
+      {
+        if ($Index > 0) $To .= ', ';
         $To .= $Recipient['Name'].' <'.$Recipient['Address'].'>';
       } else
-      if($Recipient['Type'] == 'Copy')
-      {
-        if($Index > 0) $this->Headers['CC'] .= ', ';
+      if ($Recipient['Type'] == 'Copy')
+      {
+        if ($Index > 0) $this->Headers['CC'] .= ', ';
         $this->Headers['CC'] .= $Recipient['Name'].' <'.$To['Address'].'>';
       } else
-      if($Recipient['Type'] == 'HiddenCopy')
-      {
-        if($Index > 0) $this->Headers['BCC'] .= ', ';
+      if ($Recipient['Type'] == 'HiddenCopy')
+      {
+        if ($Index > 0) $this->Headers['BCC'] .= ', ';
         $this->Headers['BCC'] .= $Recipient['Name'].' <'.$To['Address'].'>';
       }
     }
-    if($To == '') throw new Exception(T('Mail message need at least one recipient address'));
+    if ($To == '') throw new Exception(T('Mail message need at least one recipient address'));
 
     $this->Headers['Mime-Version'] = '1.0';
 
-    if($this->AgentIdent != '') $this->Headers['X-Mailer'] = $this->AgentIdent;
-    if($this->ReplyTo != '') $this->Headers['Reply-To'] = $this->ReplyTo;
-    if($this->From != '') $this->Headers['From'] = $this->From;
+    if ($this->AgentIdent != '') $this->Headers['X-Mailer'] = $this->AgentIdent;
+    if ($this->ReplyTo != '') $this->Headers['Reply-To'] = $this->ReplyTo;
+    if ($this->From != '') $this->Headers['From'] = $this->From;
 
     $Headers = '';
-    foreach($this->Headers as $Header => $Value)
-    {
-      if(($Header != 'Subject') and ($Value != '')) $Headers .= $Header.': '.$Value."\n";
+    foreach ($this->Headers as $Header => $Value)
+    {
+      if (($Header != 'Subject') and ($Value != '')) $Headers .= $Header.': '.$Value."\n";
     }
 
     $this->Subject = strtr($this->Subject, "\r\n", '  ');
 
-    if($this->Subject == '') throw new Exception(T('Mail message missing Subject'));
+    if ($this->Subject == '') throw new Exception(T('Mail message missing Subject'));
 
 
     $res = mail($To, $this->Subject, $Body, $Headers);
-    return($res);
+    return $res;
   }
 
   function ValidEmail($Address)
   {
-    if(ereg(".*<(.+)>", $Address, $regs)) $Address = $regs[1];
-    if(ereg("^[^@  ]+@([a-zA-Z0-9\-]+\.)+([a-zA-Z0-9\-]{2}|net|com|gov|mil|org|edu|int)\$", $Address))
-      return(true);
-      else return(false);
+    if (ereg(".*<(.+)>", $Address, $regs)) $Address = $regs[1];
+    if (ereg("^[^@  ]+@([a-zA-Z0-9\-]+\.)+([a-zA-Z0-9\-]{2}|net|com|gov|mil|org|edu|int)\$", $Address))
+      return true;
+      else return false;
   }
 
   function CheckAdresses($Addresses)
   {
-    foreach($Addresses as $Address)
-    {
-      if(!$this->ValidEmail($Address))
+    foreach ($Addresses as $Address)
+    {
+      if (!$this->ValidEmail($Address))
   throw new Exception(sprintf(T('Mail message invalid address %s'), $Address));
     }
@@ -168 +168 @@
   private function ContentEncoding($Charset)
   {
-    if($Charset != CHARSET_ASCII) return('8bit');
-      else return('7bit');
+    if ($Charset != CHARSET_ASCII) return '8bit';
+      else return '7bit';
   }
 
   private function BuildAttachment($Body)
   {
-    if(count($this->Attachments) > 0)
+    if (count($this->Attachments) > 0)
     {
       $this->Headers['Content-Type'] = "multipart/mixed;\n boundary=\"PHP-mixed-".$this->Boundary."\"";
@@ -181 +181 @@
       $Result .= $Body;
 
-      foreach($this->Attachments as $Attachment)
+      foreach ($this->Attachments as $Attachment)
       {
         $FileName = $Attachment['FileName'];
@@ -187 +187 @@
         $ContentType = $Attachment['FileType'];
         $Disposition = $Attachment['Disposition'];
-        if($Attachment['Type'] == 'File')
+        if ($Attachment['Type'] == 'File')
         {
-          if(!file_exists($FileName))
+          if (!file_exists($FileName))
             throw new Exception(sprintf(T('Mail message attached file %s can\'t be found'), $FileName));
           $Data = file_get_contents($FileName);
         } else
-        if($Attachment['Type'] == 'Data') $Data = $Attachment['Data'];
+        if ($Attachment['Type'] == 'Data') $Data = $Attachment['Data'];
           else $Data = '';
 
@@ -203 +203 @@
       }
     } else $Result = $Body;
-    return($Result);
+    return $Result;
   }
 
@@ -209 +209 @@
   {
     $Result = '';
-    if(count($this->Bodies) > 1)
+    if (count($this->Bodies) > 1)
     {
       $this->Headers['Content-Type'] = 'multipart/alternative; boundary="PHP-alt-'.$this->Boundary.'"';
@@ -221 +221 @@
 
 
-    foreach($this->Bodies as $Body)
-    {
-      if(count($this->Bodies) > 1) $Result .= "\n\n".'--PHP-alt-'.$this->Boundary."\n";
+    foreach ($this->Bodies as $Body)
+    {
+      if (count($this->Bodies) > 1) $Result .= "\n\n".'--PHP-alt-'.$this->Boundary."\n";
       $Result .= 'Content-Type: '.$Body['Type'].'; charset='.$Body['Charset'].
         "\nContent-Transfer-Encoding: ".$this->ContentEncoding($Body['Charset'])."\n\n".$Body['Content']."\n";
     }
-    return($Result);
+    return $Result;
   }
 }