From e892c376ce54b89ffc2436cd3da3d65cf5429761 Mon Sep 17 00:00:00 2001
From: Arlen Johnson <arlen@sphericalcowgroup.com>
Date: Thu, 3 Aug 2023 16:56:49 -0400
Subject: [PATCH] Improve row actions for Provisioning Status page (CFM-280)
 (#114)

---
 app/resources/locales/en_US/operation.po     |  3 +
 app/templates/ProvisioningTargets/status.php | 75 +++++++++++---------
 app/webroot/css/co-base.css                  |  3 +
 3 files changed, 46 insertions(+), 35 deletions(-)

diff --git a/app/resources/locales/en_US/operation.po b/app/resources/locales/en_US/operation.po
index 2268a90d6..52a618c5d 100644
--- a/app/resources/locales/en_US/operation.po
+++ b/app/resources/locales/en_US/operation.po
@@ -126,6 +126,9 @@ msgstr "Make Primary"
 msgid "provision"
 msgstr "Provision Now"
 
+msgid "provision.confirm"
+msgstr "Are you sure you want to run provisioning?"
+
 msgid "provisioning.status"
 msgstr "Provisioning Status"
 
diff --git a/app/templates/ProvisioningTargets/status.php b/app/templates/ProvisioningTargets/status.php
index 3781cbbf5..406ad3a14 100644
--- a/app/templates/ProvisioningTargets/status.php
+++ b/app/templates/ProvisioningTargets/status.php
@@ -53,16 +53,10 @@
 
 <!-- Our view is similar to index.php -->
 <div class="table-container">
-  <?php
-    $indexTableClasses = 'index-table list-mode';
-    if (!empty($rowActions)) {
-      $indexTableClasses .= ' with-actions';
-    }
-  ?>
-  <table id="<?= 'provisioningstatus-table'; ?>" class="<?= $indexTableClasses; ?>">
+  <table id="provisioningstatus-table" class="index-table list-mode with-actions">
     <thead>
       <tr>
-        <th class="actions"></th>
+        <td class="actions"></td>
         <th><?= __d('controller', 'ProvisioningTargets', [1]); ?></th>
         <th><?= __d('field', 'status'); ?></th>
         <th><?= __d('field', 'comment'); ?></th>
@@ -74,34 +68,45 @@
     </tbody>
       <?php foreach($vv_provisioning_statuses as $p): ?>
       <tr>
-        <td>
-<!-- JIRA to merge this with menuActions, also to throw up "Are you sure?" -->
-          <a class="field-actions-menu" href="<?= 
-            $this->Url->build([
-              'controller' => $vv_primary_link_model,
-              'action' => 'provision',
-              $vv_primary_link_obj->id,
-              '?' => [
-                'provisioning_target_id' => $p['target']->id
+        <td class="actions">
+          <div class="field-actions">
+<!-- CFM-305 -- JIRA to throw up "Are you sure?" - also to simplify all the below -->
+          <?php
+            // Build the row actions
+            $action_args = array();
+            $action_args['vv_attr_id'] =  $p['target']->id;
+            $action_args['vv_actions'] = [
+              [
+                'order' => $this->Menu->getMenuOrder('Default'),
+                'icon' => 'start',
+                'url' => $this->Url->build([
+                  'controller' => $vv_primary_link_model,
+                  'action' => 'provision',
+                  $vv_primary_link_obj->id,
+                  '?' => [
+                    'provisioning_target_id' => $p['target']->id
+                  ]
+                ]),
+                'label' => __d('operation', 'provision')
+              ],
+              [
+                'order' => $this->Menu->getMenuOrder('Default'),
+                'icon' => 'history',
+                'url' => $this->Url->build([
+                  'controller' => 'provisioning_history_records',
+                  'action' => 'index',
+                  '?' => [
+                    'provisioning_target_id' => $p['target']->id,
+                    StringUtilities::entityToForeignKey($vv_primary_link_obj) => $vv_primary_link_obj->id
+                  ]
+                ]),
+                'label' => __d('controller', 'ProvisioningHistoryRecords', [99])
               ]
-            ]);
-          ?>">
-            <em class="material-icons">start</em>
-          <?= __d('operation', 'provision'); ?>
-          </a>
-          <a class="field-actions-menu" href="<?= 
-            $this->Url->build([
-              'controller' => 'provisioning_history_records',
-              'action' => 'index',
-              '?' => [
-                'provisioning_target_id' => $p['target']->id,
-                StringUtilities::entityToForeignKey($vv_primary_link_obj) => $vv_primary_link_obj->id
-              ]
-            ]);
-          ?>">
-            <em class="material-icons">history</em>
-          <?= __d('controller', 'ProvisioningHistoryRecords', [99]); ?>
-          </a>
+            ];
+  
+            print $this->element('menuAction', $action_args);
+          ?>
+          </div>
         </td>
         <td><?= $p['target']->description; ?></td>
         <td><?= __d('enumeration', 'ProvisioningStatusEnum.'.$p['status']); ?></td>
diff --git a/app/webroot/css/co-base.css b/app/webroot/css/co-base.css
index f0447caa9..a2ff7122d 100644
--- a/app/webroot/css/co-base.css
+++ b/app/webroot/css/co-base.css
@@ -1550,6 +1550,9 @@ th, td {
 th {
   background-color: var(--cmg-color-bg-003);
 }
+thead td.actions {
+  background-color: var(--cmg-color-bg-003);
+}
 td {
   border-bottom: 1px solid var(--cmg-color-bg-005);
 }