bugfix/FOUR-26343: Completed tasks can be modified in the task preview pane.

resources/js/tasks/components/TaskListRowButtons.vue

@@ -5,7 +5,7 @@
         <slot name="body">
           <span v-for="(button, index) in buttons" :key="index">
             <b-button :id="button.id + rowIndex"
-                      v-if="button.show"
+                      v-if="shouldShow(button)"
                       aria-label="button.title"
                       @click="onClick(button)"
                       variant="light"
@@ -19,6 +19,7 @@
                    :alt="button.title"/>
             </b-button>
             <b-tooltip
+              v-if="shouldShow(button)"
               :target="button.id + rowIndex"
               :title="button.title"
               custom-class="task-hover-tooltip"
@@ -27,7 +28,7 @@
               boundary="viewport"
               :no-fade="true"
               />
-            <div v-if="index < buttons.length - 1 && button.show"
+            <div v-if="index < buttons.length - 1 && shouldShow(button)"
                  class="task-vertical-separator">
             </div>
           </span>
@@ -51,6 +52,12 @@
       showButtons: null
     },
     methods: {
+      shouldShow(button) {
+        if (typeof button.show === "function") {
+          return button.show(this.row);
+        }
+        return button.show;
+      },
       show() {
         this.triggerFloatingButtons(() => {
           if (!this.showButtons) {

resources/js/tasks/components/TasksList.vue

@@ -348,7 +348,7 @@ export default {
           click: this.previewTasks,
           icon: "fas fa-eye",
           title: this.$t("Preview"),
-          show: !this.verifyURL("saved-searches") || false,
+          show: (row) => this.shouldShowPreviewButton(row),
         },
         {
           id: "openCaseButton",
@@ -444,6 +444,7 @@ export default {
             record.process_request,
             record,
           );
+          record.task_status = record.status;
           record.status = this.formatStatus(record);
           record.assignee = this.formatAvatar(record.user);
           record.request = this.formatRequest(record);
@@ -759,6 +760,16 @@ export default {
       }
       return isInUrl;
     },
+    /**
+     * Hide Preview for completed tasks (API status CLOSED) and on normal saved searches.
+     * Use task_status because status is replaced with badge HTML in the data watcher.
+     */
+    shouldShowPreviewButton(row) {
+      if (this.verifyURL("saved-searches") && !this.processesIManage) {
+        return false;
+      }
+      return row?.task_status !== "CLOSED";
+    },
     /**
      * This method is used in PMColumnFilterPopoverCommonMixin.js
      */