From 15ed06b415483bbd9bfe3e561fd0c9324d9952a4 Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <emcnaughton@wikimedia.org>
Date: Thu, 14 Oct 2021 21:21:04 +1300
Subject: [PATCH] Fix hookTokens to be clearable outside the class

Tests can't unset the proerty in the class easily but just using the static makes it easier
---
 CRM/Contact/Tokens.php       | 25 +++++++------------------
 CRM/Utils/Hook/UnitTests.php |  3 +++
 2 files changed, 10 insertions(+), 18 deletions(-)

diff --git a/CRM/Contact/Tokens.php b/CRM/Contact/Tokens.php
index 03c4d37db1..7df4720e93 100644
--- a/CRM/Contact/Tokens.php
+++ b/CRM/Contact/Tokens.php
@@ -32,13 +32,6 @@ class CRM_Contact_Tokens extends CRM_Core_EntityTokens {
     return 'Contact';
   }
 
-  /**
-   * Tokens defined by the legacy hook.
-   *
-   * @var array
-   */
-  protected $hookTokens;
-
   /**
    * @inheritDoc
    */
@@ -249,7 +242,7 @@ class CRM_Contact_Tokens extends CRM_Core_EntityTokens {
    */
   public function evaluateLegacyHookTokens(TokenValueEvent $e): void {
     $messageTokens = $e->getTokenProcessor()->getMessageTokens();
-    if (!array_intersect(array_keys($this->getHookTokens()), array_keys($messageTokens))) {
+    if (empty($messageTokens) || !array_intersect(array_keys($this->getHookTokens()), array_keys($messageTokens))) {
       return;
     }
 
@@ -694,17 +687,13 @@ class CRM_Contact_Tokens extends CRM_Core_EntityTokens {
    * @return array
    */
   protected function getHookTokens(): array {
-    if ($this->hookTokens === NULL) {
-      if (isset(Civi::$statics[__CLASS__]['hook_tokens'])) {
-        $this->hookTokens = Civi::$statics[__CLASS__]['hook_tokens'];
-      }
-      else {
-        $this->hookTokens = [];
-        \CRM_Utils_Hook::tokens($this->hookTokens);
-        Civi::$statics[__CLASS__]['hook_tokens'] = $this->hookTokens;
-      }
+    if (isset(Civi::$statics[__CLASS__]['hook_tokens'])) {
+      return Civi::$statics[__CLASS__]['hook_tokens'];
     }
-    return $this->hookTokens;
+    $tokens = [];
+    \CRM_Utils_Hook::tokens($tokens);
+    Civi::$statics[__CLASS__]['hook_tokens'] = $tokens;
+    return $tokens;
   }
 
 }
diff --git a/CRM/Utils/Hook/UnitTests.php b/CRM/Utils/Hook/UnitTests.php
index 7b07c1b5ce..71babb86f8 100644
--- a/CRM/Utils/Hook/UnitTests.php
+++ b/CRM/Utils/Hook/UnitTests.php
@@ -55,6 +55,9 @@ class CRM_Utils_Hook_UnitTests extends CRM_Utils_Hook {
    */
   public function setHook(string $hook, $callable): void {
     $this->adhocHooks[$hook] = $callable;
+    if (strpos($hook, 'token') !== FALSE) {
+      unset(Civi::$statics['CRM_Contact_Tokens']['hook_tokens']);
+    }
   }
 
   /**
-- 
2.25.1