From b26eba9f4799432765cdd59487579f9df84538b7 Mon Sep 17 00:00:00 2001
From: Arlen Johnson <arlen@sphericalcowgroup.com>
Date: Fri, 10 May 2024 13:43:51 -0400
Subject: [PATCH] UI updates for Message Templates (CFM-390) (#199)

* Add Message Templates to menu panels (CFM-390)

* Show hide message template body fields based on format (CFM-390)
---
 app/src/Controller/DashboardsController.php |  2 +-
 app/templates/MessageTemplates/fields.inc   | 45 +++++++++++++++++++--
 app/templates/element/menuPanel.php         | 26 +++++++++---
 3 files changed, 64 insertions(+), 9 deletions(-)

diff --git a/app/src/Controller/DashboardsController.php b/app/src/Controller/DashboardsController.php
index 4875fc867..c06e4ee40 100644
--- a/app/src/Controller/DashboardsController.php
+++ b/app/src/Controller/DashboardsController.php
@@ -103,7 +103,7 @@ public function configuration() {
         'action'        => 'index'
       ],
       __d('controller', 'MessageTemplates', [99]) => [
-        'icon'          => 'message',
+        'icon'          => 'email',
         'controller'    => 'message_templates',
         'action'        => 'index'
       ],
diff --git a/app/templates/MessageTemplates/fields.inc b/app/templates/MessageTemplates/fields.inc
index a88f5e691..6a7f49e68 100644
--- a/app/templates/MessageTemplates/fields.inc
+++ b/app/templates/MessageTemplates/fields.inc
@@ -25,6 +25,30 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 ?>
+  
+<script>
+  // JS specific to these fields
+  
+  function updateGadgets(isPageLoad) {
+    // Hide or show gadgets according to current state
+    var format = document.getElementById('format').value;
+    
+    if(format == '<?= \App\Lib\Enum\MessageFormatEnum::HTML ?>') {
+      hideFields(['body-text'], isPageLoad);
+      showFields(['body-html'], isPageLoad);
+    } else if(format == '<?= \App\Lib\Enum\MessageFormatEnum::PlainText ?>') {
+      hideFields(['body-html'], isPageLoad);
+      showFields(['body-text'], isPageLoad);
+    } else {
+      showFields(['body-html','body-text'], isPageLoad);
+    }
+  }
+  
+  function jsLocalOnLoad() {
+    updateGadgets(true);
+  }
+</script>
+  
 <?php
 // This view does not support read-only
 if($vv_action == 'add' || $vv_action == 'edit') {
@@ -32,11 +56,26 @@ if($vv_action == 'add' || $vv_action == 'edit') {
              'description',
              'status',
              'context',
-             'format',
+           ] as $field) {
+    print $this->element('form/listItem', [
+      'arguments' => [
+        'fieldName' => $field
+      ]]);
+  }
+
+  print $this->element('form/listItem', [
+    'arguments' => [
+      'fieldName' => 'format',
+      'fieldOptions' => [
+        'onChange' => 'updateGadgets()'
+      ],
+      'fieldType' => 'select'
+    ]
+  ]);
+  
+  foreach ([
              'subject',
-// XXX auto show/hide based on format
              'body_text',
-// XXX auto show/hide based on format
              'body_html',
              'cc',
              'bcc',
diff --git a/app/templates/element/menuPanel.php b/app/templates/element/menuPanel.php
index 8790380c3..5ee0ca11a 100644
--- a/app/templates/element/menuPanel.php
+++ b/app/templates/element/menuPanel.php
@@ -434,16 +434,16 @@
               ?>
               <a href="<?= $menuUrl ?>">
                 <em class="material-icons" aria-hidden="true">widgets</em>
-                Types
+                <span class="menu-panel-link-text"><?= __d('controller', 'Types', [99]) ?></span>
               </a>
             </li>
           </ul>
         </li>
       </ul>
 
-      <?php /* XXX The "Personalization" links are disabled until needed, but we will leave them in the code to 
+      <?php /* XXX Most "Personalization" links are disabled until needed, but we will leave them in the code to 
                provide hints to where they belong. The description ('co.configuration.panel.personalization.desc')
-               reads "Dashboards, custom text, and theming". When such things are ready, place them here.
+               reads "Dashboards, custom text, and theming". When such things are ready, place them here. */ ?>
  
       <ul class="menu-panel-links">
         <li>
@@ -455,8 +455,24 @@
             <li><a href="#"><em class="material-icons" aria-hidden="true">navigation</em> CO Navigation Links</a></li>
             <li><a href="#"><em class="material-icons" aria-hidden="true">dashboard</em> Dashboards</a></li>
             <li><a href="#"><em class="material-icons" aria-hidden="true">book</em> Dictionaries</a></li>
-            <li><a href="#"><em class="material-icons" aria-hidden="true">translate</em> Localizations</a></li>
-            <li><a href="#"><em class="material-icons" aria-hidden="true">email</em> Message Templates</a></li>
+            <li><a href="#"><em class="material-icons" aria-hidden="true">translate</em> Localizations</a></li> */ ?>
+            <li>
+              <?php
+                $menuUrl = $this->Url->build(
+                  ['plugin'       => null,
+                   'controller'   => 'message_templates',
+                   'action'       => 'index',
+                   '?'            => [
+                     'co_id' => $vv_cur_co->id
+                   ]]
+                );
+              ?>
+              <a href="<?= $menuUrl ?>">
+                <em class="material-icons" aria-hidden="true">email</em>
+                <span class="menu-panel-link-text"><?= __d('controller', 'MessageTemplates', [99]) ?></span>
+              </a>
+            </li>
+            <?php /* More placeholders:
             <li><a href="#"><em class="material-icons" aria-hidden="true">room_service</em> Self Service Permissions</a></li>
             <li><a href="#"><em class="material-icons" aria-hidden="true">wallpaper</em> Themes</a></li>
             * / ? >