From 3f9312f21c4b6272a58bc2d359915d7a969e2f16 Mon Sep 17 00:00:00 2001
From: demeritcowboy <demeritcowboy@hotmail.com>
Date: Wed, 14 Jul 2021 11:06:05 -0400
Subject: [PATCH] missing type

---
 CRM/Logging/ReportDetail.php           | 6 +++---
 templates/CRM/Logging/ReportDetail.tpl | 1 +
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/CRM/Logging/ReportDetail.php b/CRM/Logging/ReportDetail.php
index ad16340ec6..99ed827d14 100644
--- a/CRM/Logging/ReportDetail.php
+++ b/CRM/Logging/ReportDetail.php
@@ -103,9 +103,9 @@ class CRM_Logging_ReportDetail extends CRM_Report_Form {
     }
 
     $this->_columnHeaders = [
-      'field' => ['title' => ts('Field')],
-      'from' => ['title' => ts('Changed From')],
-      'to' => ['title' => ts('Changed To')],
+      'field' => ['title' => ts('Field'), 'type' => CRM_Utils_Type::T_STRING],
+      'from' => ['title' => ts('Changed From'), 'type' => CRM_Utils_Type::T_STRING],
+      'to' => ['title' => ts('Changed To'), 'type' => CRM_Utils_Type::T_STRING],
     ];
   }
 
diff --git a/templates/CRM/Logging/ReportDetail.tpl b/templates/CRM/Logging/ReportDetail.tpl
index bd95dd3a73..3b562c64f0 100644
--- a/templates/CRM/Logging/ReportDetail.tpl
+++ b/templates/CRM/Logging/ReportDetail.tpl
@@ -9,6 +9,7 @@
 *}
 <div class="crm-block crm-content-block crm-report-form-block">
   {if $rows}
+    {* todo: Is `raw` ever assigned to the template and why would it make a difference as to whether this div is present? The revert confirmation is handled (awkwardly) lower down by javascript $revertConfirm *}
     {if $raw}
       <div class="status">
         <dl>
-- 
2.25.1