Merge pull request #8310 from ProcessMaker/bugfix/FOUR-23617

FOUR-23617: When reloading the page, the completed status filter show all clear

Author: ryancooley

ProcessMaker/Traits/TaskControllerIndexMethods.php

@@ -27,17 +27,6 @@ private function indexBaseQuery($request)
 
         $query = ProcessRequestToken::exclude(['data']);
 
-        // If all_inbox is true and user has process requests, filter to only show the latest process
-        if ($request->has('all_inbox') && $request->input('all_inbox') === 'false') {
-            $latestProcessRequest = ProcessRequestToken::where('user_id', auth()->id())
-                ->orderBy('created_at', 'desc')
-                ->first();
-
-            if ($latestProcessRequest) {
-                $query->where('process_id', $latestProcessRequest->process_id);
-            }
-        }
-
         $query = $query->with([
             'processRequest' => function ($q) use ($includeData) {
                 if (!$includeData) {

resources/js/tasks/components/ListMixin.js

@@ -93,19 +93,6 @@ const ListMixin = {
         if (this.additionalIncludes) {
           include.push(...this.additionalIncludes);
         }
-        const currentUrl = window.location.href;
-        const path = new URL(currentUrl).pathname.split("/").pop();
-
-        let getAllTasksInbox = "";
-        if (path === "inbox") {
-          if (this.$parent.allInbox) {
-            getAllTasksInbox = "&all_inbox=true";
-          } else {
-            getAllTasksInbox = "&all_inbox=false";
-          }
-        } else {
-          getAllTasksInbox = "&all_inbox=true";
-        }
         // Load from our api client
         ProcessMaker.apiClient
           .get(
@@ -115,7 +102,6 @@ const ListMixin = {
                 this.perPage + this.sumCards
               }${filterParams}${this.getSortParam()}&non_system=true` +
               `&processesIManage=${this.processesIManage ? "true" : "false"}` +
-              getAllTasksInbox +
               advancedFilter +
               this.columnsQuery,
             {

resources/js/tasks/components/ParticipantHomeScreen.vue

@@ -281,7 +281,11 @@ export default {
     },
     getAllTasks() {
       this.selectedProcess = "inbox";
-      this.allInbox = true;
+      if (this.$route.name !== 'inbox') {
+        this.$router.push({
+          name: "inbox",
+        });
+      }
       this.$refs.processesDashboardsMenu?.clearSelection();
       this.callingTaskList();
     },

resources/js/tasks/router.js

@@ -21,6 +21,16 @@ const router = new VueRouter({
         ellipsisPermission: window.ProcessMaker.ellipsisPermission
       })
     },
+    {
+      path: "",
+      name: "inbox",
+      component: Process,
+      props: route => ({
+        processId: null,
+        process: null,
+        ellipsisPermission: window.ProcessMaker.ellipsisPermission
+      })
+    },
     {
       path: "/dashboard/:dashboardId",
       name: "dashboard",