Fix handling of to addresses on graduate school email notifications. …

…The to field can be a comma separated list, be sure we break it apart accordingly.
AppStateESS · Apr 3, 2017 · dfe91d9 · dfe91d9
1 parent 3138e76
commit dfe91d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/class/Email/CancelInternshipNotice.php b/class/Email/CancelInternshipNotice.php
@@ -52,7 +52,7 @@ protected function buildMessage()
 
         // CC the graduate school (for grad level) or the registrar's office (for undergrad level)
         if($this->internship->isGraduate()){
-            $this->cc[] = $this->emailSettings->getGraduateRegEmail();
+            $this->cc[] = explode(',', $this->emailSettings->getGraduateRegEmail()); // NB: Setting is a comma separated array
         } else {
             $this->cc[] = $this->emailSettings->getRegistrarEmail();
         }

diff --git a/class/Email/GradSchoolNotificationEmail.php b/class/Email/GradSchoolNotificationEmail.php
@@ -117,8 +117,7 @@ protected function buildMessage()
             $intlSubject = '';
         }
 
-        $emails = $this->emailSettings->getGradSchoolEmail();
-        $this->to = explode(',', $emails);
+        $this->to = explode(',', $this->emailSettings->getGradSchoolEmail()); // NB: Setting is a comma separated array
 
         $this->subject = 'Internship Approval Needed: ' . $intlSubject . '[' . $this->internship->getBannerId() . '] ' . $this->internship->getFullName();
     }