From ae6cfd01ad37f32bc234659226ad358c97cfde87 Mon Sep 17 00:00:00 2001
From: demeritcowboy <demeritcowboy@hotmail.com>
Date: Fri, 1 Jul 2022 20:50:38 -0400
Subject: [PATCH] add comment

---
 Civi/Core/SqlTriggers.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Civi/Core/SqlTriggers.php b/Civi/Core/SqlTriggers.php
index a6af10858f..91b5cb9438 100644
--- a/Civi/Core/SqlTriggers.php
+++ b/Civi/Core/SqlTriggers.php
@@ -118,6 +118,7 @@ class SqlTriggers {
             $template_values,
             $value['sql']
           );
+          // @todo See https://github.com/civicrm/civicrm-core/pull/23926#discussion_r912298758. It's not clear 'variables' is used anywhere, and isn't documented or unit tested.
           $variables = str_replace($template_params,
             $template_values,
             $value['variables'] ?? ''
-- 
2.25.1