Peter M. Groen / knowledgetree

Commit 9aed007847cd8772cc4c0daa69decc8f10880645

Authored by Paul Barrett
1 parent 0d731da2

KTS-4223. Altered email to multiple recipients to prevent sending to a cumulativ… 

…e list rather than individual recipients.

Fixed

"email to external addresses spams recipients"

Committed by: Paul Barrett

Reviewed by: Megan Watson
Inline Side-by-side
Showing 1 changed file with 2 additions and 2 deletions
plugins/ktstandard/KTEmail.php
  View file @9aed007
@@ -138,7 +138,7 @@ function sendManualEmails($aEmailAddresses, &$aUserEmails, &$aEmailErrors) { @@ -138,7 +138,7 @@ function sendManualEmails($aEmailAddresses, &$aUserEmails, &$aEmailErrors) {
138 function sendExternalEmails($aEmailAddresses, $iDocumentID, $sDocumentName, $sComment, &$aEmailErrors){ 138 function sendExternalEmails($aEmailAddresses, $iDocumentID, $sDocumentName, $sComment, &$aEmailErrors){
139 global $default; 139 global $default;
140 $oSendingUser = User::get($_SESSION['userID']); 140 $oSendingUser = User::get($_SESSION['userID']);
141 - 141 +
142 // Create email content 142 // Create email content
143 /* 143 /*
144 $sMessage = '<font face="arial" size="2">'; 144 $sMessage = '<font face="arial" size="2">';
@@ -176,10 +176,10 @@ function sendExternalEmails($aEmailAddresses, $iDocumentID, $sDocumentName, $sCo @@ -176,10 +176,10 @@ function sendExternalEmails($aEmailAddresses, $iDocumentID, $sDocumentName, $sCo
176 $sEmail = $oSendingUser->getEmail(); 176 $sEmail = $oSendingUser->getEmail();
177 $sEmailFrom = $oSendingUser->getName(); 177 $sEmailFrom = $oSendingUser->getName();
178 } 178 }
179 - $oEmail = new Email($sEmail, $sEmailFrom);  
180 179
181 $iCounter = 0; 180 $iCounter = 0;
182 foreach ($aEmailAddresses as $sAddress){ 181 foreach ($aEmailAddresses as $sAddress){
  182 + $oEmail = new Email($sEmail, $sEmailFrom);
183 if(validateEmailAddress($sAddress)){ 183 if(validateEmailAddress($sAddress)){
184 // Add to list of addresses 184 // Add to list of addresses
185 $sDestEmails .= (empty($sDestEmails)) ? $sAddress : ', '.$sAddress; 185 $sDestEmails .= (empty($sDestEmails)) ? $sAddress : ', '.$sAddress;