[REF] Move code closer to where it is used
authoreileen <emcnaughton@wikimedia.org>
Mon, 18 Jan 2021 01:57:12 +0000 (14:57 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 18 Jan 2021 01:57:12 +0000 (14:57 +1300)
Simply moves a few lines of code so the actual usage is apparent

CRM/Core/BAO/MessageTemplate.php

index e32ab9d4cabd876f870a81d951be385c6e7304fb..922fbbd26a13f62e4d38944e9f5f39bb850d94c6 100644 (file)
@@ -471,14 +471,12 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
 
     // replace tokens in the three elements (in subject as if it was the text body)
     $domain = CRM_Core_BAO_Domain::getDomain();
-    $hookTokens = [];
+
     $mailing = new CRM_Mailing_BAO_Mailing();
     $mailing->subject = $mailContent['subject'];
     $mailing->body_text = $mailContent['text'];
     $mailing->body_html = $mailContent['html'];
     $tokens = $mailing->getTokens();
-    CRM_Utils_Hook::tokens($hookTokens);
-    $categories = array_keys($hookTokens);
 
     $contactID = $params['contactId'] ?? NULL;
 
@@ -540,6 +538,9 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
       );
       $contact = $contactArray[$contactID];
 
+      $hookTokens = [];
+      CRM_Utils_Hook::tokens($hookTokens);
+      $categories = array_keys($hookTokens);
       $mailContent['subject'] = CRM_Utils_Token::replaceHookTokens($mailContent['subject'], $contact, $categories, TRUE);
       $mailContent['text'] = CRM_Utils_Token::replaceHookTokens($mailContent['text'], $contact, $categories, TRUE);
       $mailContent['html'] = CRM_Utils_Token::replaceHookTokens($mailContent['html'], $contact, $categories, TRUE);