From e83671332195a7c8c21c21a5a1871d7dca600704 Mon Sep 17 00:00:00 2001
From: Pradeep Nayak <pradpnayak@gmail.com>
Date: Fri, 21 Oct 2022 19:59:44 +0100
Subject: [PATCH] Use API4 to get message templates over DAO function

---
 CRM/Core/BAO/MessageTemplate.php | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/CRM/Core/BAO/MessageTemplate.php b/CRM/Core/BAO/MessageTemplate.php
index 977b91b250..bdb9a9d644 100644
--- a/CRM/Core/BAO/MessageTemplate.php
+++ b/CRM/Core/BAO/MessageTemplate.php
@@ -202,19 +202,18 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate implemen
    * @return array
    */
   public static function getMessageTemplates($all = TRUE, $isSMS = FALSE) {
-    $msgTpls = [];
 
-    $messageTemplates = new CRM_Core_DAO_MessageTemplate();
-    $messageTemplates->is_active = 1;
-    $messageTemplates->is_sms = $isSMS;
+    $messageTemplates = \Civi\Api4\MessageTemplate::get()
+      ->addSelect('id', 'msg_title')
+      ->addWhere('is_active', '=', TRUE)
+      ->addWhere('is_sms', '=', $isSMS);
 
     if (!$all) {
-      $messageTemplates->workflow_id = 'NULL';
-    }
-    $messageTemplates->find();
-    while ($messageTemplates->fetch()) {
-      $msgTpls[$messageTemplates->id] = $messageTemplates->msg_title;
+      $messageTemplates->addWhere('workflow_id', 'IS NULL');
     }
+
+    $msgTpls = array_column((array) $messageTemplates->execute(), 'msg_title', 'id');
+
     asort($msgTpls);
     return $msgTpls;
   }
-- 
2.25.1