From 112c4a6def00af1c78406e6c8333cb670ccac7c8 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Fri, 5 Jan 2024 16:56:08 -0400
Subject: [PATCH 01/15] feature/FOUR-12702

---
 .../components/ProcessInfo.vue                |  2 +-
 .../components/ProcessOptions.vue             |  4 +-
 .../components/optionsMenu/ButtonsStart.vue   | 47 ++++++++++++++++++-
 3 files changed, 48 insertions(+), 5 deletions(-)

diff --git a/resources/js/processes-catalogue/components/ProcessInfo.vue b/resources/js/processes-catalogue/components/ProcessInfo.vue
index 4c638e15c0..e1dce5220f 100644
--- a/resources/js/processes-catalogue/components/ProcessInfo.vue
+++ b/resources/js/processes-catalogue/components/ProcessInfo.vue
@@ -14,7 +14,7 @@
         />
       </b-col>
       <b-col cols="3">
-        <process-options :process="process" />
+        <process-options :process="process" :user="currentUserId"/>
       </b-col>
     </div>
     <b-col cols="12">
diff --git a/resources/js/processes-catalogue/components/ProcessOptions.vue b/resources/js/processes-catalogue/components/ProcessOptions.vue
index 3153e864f8..53a0a86eaf 100644
--- a/resources/js/processes-catalogue/components/ProcessOptions.vue
+++ b/resources/js/processes-catalogue/components/ProcessOptions.vue
@@ -1,6 +1,6 @@
 <template>
   <div>
-    <buttons-start :process="process" />
+    <buttons-start :process="process" :currentUser="user"/>
     <process-counter :process="process" />
     <b-button v-if="createdFromWizardTemplate" class="mt-2 wizard-link" variant="link" @click="getHelperProcess">
       <img src="../../../img/wizard-icon.svg" :alt="$t('Guided Template Icon')" />
@@ -25,7 +25,7 @@ import WizardHelperProcessModal from '../../components/templates/WizardHelperPro
 
 export default {
   components: { ButtonsStart, ProcessCounter, ChartSaveSearch, WizardHelperProcessModal },
-  props: ["process"],
+  props: ["process", "user"],
   computed: {
     createdFromWizardTemplate() {
       return this.process?.properties?.wizardTemplateUuid ? true : false;
diff --git a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
index 62344550f6..8a7286c587 100644
--- a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
+++ b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
@@ -62,9 +62,11 @@
 </template>
 
 <script>
+import { integer } from '@processmaker/screen-builder';
+
 
 export default {
-  props: ["process"],
+  props: ["process", "currentUser"],
   data() {
     return {
       processEvents: [],
@@ -87,7 +89,21 @@ export default {
             const webEntry = JSON.parse(event.config).web_entry;
             event.webEntry = webEntry;
           }
-          this.processEvents.push(event);
+          switch (event.assignment) {
+            case "user":
+              if (this.currentUser === Number(event.assignedUsers)){
+                this.processEvents.push(event);
+              }
+              break;
+            case "group":
+              this.checkUsersGroup(event, event.assignedGroups);
+              break;
+            case "process_manager":
+              if (this.currentUser === this.process.manager_id){
+                this.processEvents.push(event);
+              }
+              break;
+          }
         }
       });
       if (this.processEvents.length <= 1) {
@@ -98,6 +114,33 @@ export default {
         }
       }
     },
+    /**
+     * check if currentUser is member of a group
+     */
+    checkUsersGroup(event, groupId) {
+      ProcessMaker.apiClient
+        .get(`groups/${groupId}/users`)
+        .then((response) => {
+          const users = response.data.data;
+          users.forEach((user) => {
+            if(user.id === this.currentUser) {
+              this.processEvents.push(event);
+              if (this.processEvents.length > 1) {
+                  this.havelessOneStartEvent = false;
+                  this.startEvent = 0;
+              }
+            }
+          })
+        });
+      ProcessMaker.apiClient
+        .get(`groups/${groupId}/groups`)
+        .then((response) => {
+          const groups = response.data.data;
+          groups.forEach((group) => {
+            this.checkUsersGroup(group.id);
+          })
+        });
+    },
     /**
      * Start new request
      */

From c0a2ce378701ed76e230c008b7a61b5ddc694f2b Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Mon, 8 Jan 2024 09:17:50 -0400
Subject: [PATCH 02/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     | 116 ++++++++++++++++++
 .../components/optionsMenu/ButtonsStart.vue   |  63 ++--------
 routes/api.php                                |   1 +
 3 files changed, 126 insertions(+), 54 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 95c613af0d..214b8c2797 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -15,6 +15,7 @@
 use ProcessMaker\Events\TemplateUpdated;
 use ProcessMaker\Exception\TaskDoesNotHaveUsersException;
 use ProcessMaker\Facades\WorkflowManager;
+use ProcessMaker\Http\Controllers\Api\GroupController;
 use ProcessMaker\Http\Controllers\Api\TemplateController;
 use ProcessMaker\Http\Controllers\Controller;
 use ProcessMaker\Http\Resources\ApiCollection;
@@ -25,6 +26,7 @@
 use ProcessMaker\Jobs\ExportProcess;
 use ProcessMaker\Jobs\ImportProcess;
 use ProcessMaker\Models\Bookmark;
+use ProcessMaker\Models\Group;
 use ProcessMaker\Models\Process;
 use ProcessMaker\Models\ProcessPermission;
 use ProcessMaker\Models\Screen;
@@ -214,6 +216,79 @@ public function show(Request $request, Process $process)
         return new Resource($process);
     }
 
+    /**
+     * Display the specified resource.
+     *
+     * @param $process
+     *
+     * @return Response
+     *
+     * @OA\Get(
+     *     path="/processes/{processId}/start_events",
+     *     summary="Get start events of a process by Id",
+     *     operationId="getStartEventsProcessById",
+     *     tags={"Processes"},
+     *     @OA\Parameter(
+     *         description="ID of process to return",
+     *         in="path",
+     *         name="processId",
+     *         required=true,
+     *         @OA\Schema(
+     *           type="integer",
+     *         )
+     *     ),
+     *     @OA\Parameter(ref="#/components/parameters/include"),
+     *     @OA\Response(
+     *         response=200,
+     *         description="Successfully found the start events process",
+     *         @OA\JsonContent(
+     *             type="object",
+     *             @OA\Property(
+     *                 property="data",
+     *                 type="array",
+     *                 @OA\Items(ref="#/components/schemas/ProcessStartEvents"),
+     *             ),
+     *             @OA\Property(
+     *                 property="meta",
+     *                 type="object",
+     *                 ref="#/components/schemas/metadata",
+     *             ),
+     *         ),)
+     *     ),
+     * )
+     */
+    public function startEvents(Request $request, Process $process)
+    {
+        $startEvents = [];
+        $currentUser = Auth::user()->id;
+        foreach ($process->start_events as $event) {
+            if (count($event->eventDefinitions) === 0) {
+                if ($event->config) {
+                    $webEntry = json_decode($event->config)->web_entry;
+                    $event->webEntry = $webEntry;
+                }
+                switch ($event->assignment) {
+                    case "user":
+                        if ($currentUser === (int)$event->assignedUsers){
+                            $startEvents[] = $event;
+                        }
+                    break;
+                    case "group":
+                        if ($this->checkUsersGroup($event, $event->assignedGroups, $request)){
+                            $startEvents[] = $event;
+                        }
+                    break;
+                    case "process_manager":
+                        if ($currentUser === $process->manager_id){
+                            $startEvents[] = $event;
+                        }
+                    break;
+                }
+            }
+        }
+        return new Resource($startEvents);
+    }
+
     /**
      * Store a newly created resource in storage.
      *
@@ -1434,6 +1509,47 @@ protected function getRequestFilterBy(Request $request, array $searchableColumns
         return $where;
     }
 
+    /**
+     * check if currentUser is member of a group
+     *
+     * @param object $event
+     * @param Group $groupId
+     * @param Request $request
+     *
+     * @return bool
+     */
+    protected function checkUsersGroup($event, Group $groupId, Request $request)
+    {
+        $currentUser = Auth::user()->id;
+        try {
+            $response = (new GroupController(new Group()))->users($groupId, $request);
+            $users = $response->data->data;
+
+            foreach ($users as $user) {
+                if($user->id === $currentUser) {
+                    return true;
+                }
+            }
+            return false;
+        } catch (\Exception $error) {
+            return ['error' => $error->getMessage()];
+        }
+
+        try {
+            $response = (new GroupController(new Group()))->groups($groupId, $request);
+            $groups = $response->data->data;
+
+            foreach ($groups as $group) {
+                if ($this->checkUsersGroup($event, $group->id, $request)) {
+                    return true;
+                }
+            }
+            return false;
+        } catch (\Exception $error) {
+            return ['error' => $error->getMessage()];
+        }
+    }
+
     /**
      * Get included relationships.
      *
diff --git a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
index 8a7286c587..4322d49797 100644
--- a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
+++ b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
@@ -82,63 +82,18 @@ export default {
      * get start events for dropdown Menu
      */
     getStartEvents() {
-      const startEvents = this.process.start_events;
-      startEvents.forEach((event) => {
-        if (event.eventDefinitions.length === 0) {
-          if (event.config) {
-            const webEntry = JSON.parse(event.config).web_entry;
-            event.webEntry = webEntry;
-          }
-          switch (event.assignment) {
-            case "user":
-              if (this.currentUser === Number(event.assignedUsers)){
-                this.processEvents.push(event);
-              }
-              break;
-            case "group":
-              this.checkUsersGroup(event, event.assignedGroups);
-              break;
-            case "process_manager":
-              if (this.currentUser === this.process.manager_id){
-                this.processEvents.push(event);
-              }
-              break;
-          }
-        }
-      });
-      if (this.processEvents.length <= 1) {
-        const event = this.processEvents[0] ?? {};
-        if (!event.webEntry) {
-          this.havelessOneStartEvent = true;
-          this.startEvent = event.id ?? 0;
-        }
-      }
-    },
-    /**
-     * check if currentUser is member of a group
-     */
-    checkUsersGroup(event, groupId) {
+      this.processEvents = [];
       ProcessMaker.apiClient
-        .get(`groups/${groupId}/users`)
+        .get(`processes/${this.process.id}/start_events`)
         .then((response) => {
-          const users = response.data.data;
-          users.forEach((user) => {
-            if(user.id === this.currentUser) {
-              this.processEvents.push(event);
-              if (this.processEvents.length > 1) {
-                  this.havelessOneStartEvent = false;
-                  this.startEvent = 0;
-              }
+          this.processEvents = response.data.data;
+          if (this.processEvents.length <= 1) {
+            const event = this.processEvents[0] ?? {};
+            if (!event.webEntry) {
+              this.havelOneStartEvent = true;
+              this.startEvent = event.id ?? 0;
             }
-          })
-        });
-      ProcessMaker.apiClient
-        .get(`groups/${groupId}/groups`)
-        .then((response) => {
-          const groups = response.data.data;
-          groups.forEach((group) => {
-            this.checkUsersGroup(group.id);
-          })
+          }
         });
     },
     /**
diff --git a/routes/api.php b/routes/api.php
index 2615761d6d..c38ab60305 100644
--- a/routes/api.php
+++ b/routes/api.php
@@ -122,6 +122,7 @@
     // Processes
     Route::get('processes', [ProcessController::class, 'index'])->name('processes.index')->middleware('can:view-processes');
     Route::get('processes/{process}', [ProcessController::class, 'show'])->name('processes.show')->middleware('can:view-processes,process');
+    Route::get('processes/{process}/start_events', [ProcessController::class, 'startEvents'])->name('processes.start.events')->middleware('can:view-processes,process');
     Route::post('processes/{process}/export', [ProcessController::class, 'export'])->name('processes.export')->middleware('can:export-processes,process');
     Route::get('processes/{process}/bpmn', [ProcessController::class, 'downloadBpmn'])->name('processes.export.bpmn')->middleware('can:view-processes,process');
     Route::post('processes/import', [ProcessController::class, 'import'])->name('processes.import')->middleware('can:import-processes');

From 4b52d3cfbc2c94f9c21dba32ebe3248e091ae7e5 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Mon, 8 Jan 2024 15:38:05 -0400
Subject: [PATCH 03/15] feature/FOUR-12702

---
 .../Http/Controllers/Api/ProcessController.php     | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 214b8c2797..7112209c64 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -262,19 +262,19 @@ public function startEvents(Request $request, Process $process)
         $startEvents = [];
         $currentUser = Auth::user()->id;
         foreach ($process->start_events as $event) {
-            if (count($event->eventDefinitions) === 0) {
-                if ($event->config) {
-                    $webEntry = json_decode($event->config)->web_entry;
-                    $event->webEntry = $webEntry;
+            if (count($event["eventDefinitions"]) === 0) {
+                if (array_key_exists("config", $event)) {
+                    $webEntry = json_decode($event["config"])->web_entry;
+                    $event["webEntry"] = $webEntry;
                 }
-                switch ($event->assignment) {
+                switch ($event["assignment"]) {
                     case "user":
-                        if ($currentUser === (int)$event->assignedUsers){
+                        if ($currentUser === (int)$event["assignedUsers"]){
                             $startEvents[] = $event;
                         }
                     break;
                     case "group":
-                        if ($this->checkUsersGroup($event, $event->assignedGroups, $request)){
+                        if ($this->checkUsersGroup($event, $event["assignedGroups"], $request)){
                             $startEvents[] = $event;
                         }
                     break;

From 57a127a1606087bfbb7f2f6d3da5e5cdccaa8db0 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Mon, 8 Jan 2024 15:43:36 -0400
Subject: [PATCH 04/15] feature/FOUR-12702

---
 ProcessMaker/Http/Controllers/Api/ProcessController.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 7112209c64..8efca5956d 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -274,7 +274,7 @@ public function startEvents(Request $request, Process $process)
                         }
                     break;
                     case "group":
-                        if ($this->checkUsersGroup($event, $event["assignedGroups"], $request)){
+                        if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
                             $startEvents[] = $event;
                         }
                     break;

From 344998cfbb193cdfa0ed51431a20b891d6f99463 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Tue, 9 Jan 2024 17:22:29 -0400
Subject: [PATCH 05/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     | 58 ++++++++++---------
 .../components/optionsMenu/ButtonsStart.vue   |  5 +-
 2 files changed, 35 insertions(+), 28 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 8efca5956d..ab2a68b6df 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -267,26 +267,32 @@ public function startEvents(Request $request, Process $process)
                     $webEntry = json_decode($event["config"])->web_entry;
                     $event["webEntry"] = $webEntry;
                 }
-                switch ($event["assignment"]) {
-                    case "user":
-                        if ($currentUser === (int)$event["assignedUsers"]){
-                            $startEvents[] = $event;
+                if (!Auth::user()->is_administrator) {
+                    if (array_key_exists("assignment", $event)){
+                        switch ($event["assignment"]) {
+                            case "user":
+                                if ($currentUser === (int)$event["assignedUsers"]){
+                                    $startEvents[] = $event;
+                                }
+                                break;
+                            case "group":
+                                if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
+                                    $startEvents[] = $event;
+                                }
+                            break;
+                            case "process_manager":
+                                if ($currentUser === $process->manager_id){
+                                    $startEvents[] = $event;
+                                }
+                            break;
                         }
-                    break;
-                    case "group":
-                        if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
-                            $startEvents[] = $event;
-                        }
-                    break;
-                    case "process_manager":
-                        if ($currentUser === $process->manager_id){
-                            $startEvents[] = $event;
-                        }
-                    break;
+                    } 
+                } else {
+                    $startEvents[] = $event;
                 }
             }
         }
-        return new Resource($startEvents);
+        return new ApiCollection($startEvents);
     }
 
     /**
@@ -1513,41 +1519,41 @@ protected function getRequestFilterBy(Request $request, array $searchableColumns
      * check if currentUser is member of a group
      *
      * @param object $event
-     * @param Group $groupId
+     * @param int $groupId
      * @param Request $request
      *
      * @return bool
      */
-    protected function checkUsersGroup($event, Group $groupId, Request $request)
+    protected function checkUsersGroup($event, int $groupId, Request $request)
     {
         $currentUser = Auth::user()->id;
+        $group = Group::find($groupId);
         try {
-            $response = (new GroupController(new Group()))->users($groupId, $request);
-            $users = $response->data->data;
+            $response = (new GroupController(new Group()))->users($group, $request);
+            $users = $response->all();
 
             foreach ($users as $user) {
-                if($user->id === $currentUser) {
+                if($user->resource->id === $currentUser) {
                     return true;
                 }
             }
-            return false;
         } catch (\Exception $error) {
             return ['error' => $error->getMessage()];
         }
 
         try {
-            $response = (new GroupController(new Group()))->groups($groupId, $request);
-            $groups = $response->data->data;
+            $response = (new GroupController(new Group()))->groups($group, $request);
+            $groups = $response->all();
 
             foreach ($groups as $group) {
-                if ($this->checkUsersGroup($event, $group->id, $request)) {
+                if ($this->checkUsersGroup($event, $group->resource->id, $request)) {
                     return true;
                 }
             }
-            return false;
         } catch (\Exception $error) {
             return ['error' => $error->getMessage()];
         }
+        return false;
     }
 
     /**
diff --git a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
index 4322d49797..c076a3b2d0 100644
--- a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
+++ b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
@@ -16,6 +16,7 @@
       data-toggle="dropdown"
       aria-haspopup="true"
       aria-expanded="false"
+      @click="getStartEvents()"
     >
       <span class="pl-3 pr-4"> {{ $t('Start this process') }} </span>
     </button>
@@ -89,8 +90,8 @@ export default {
           this.processEvents = response.data.data;
           if (this.processEvents.length <= 1) {
             const event = this.processEvents[0] ?? {};
-            if (!event.webEntry) {
-              this.havelOneStartEvent = true;
+            if (!("webEntry" in event)) {
+              this.havelessOneStartEvent = true;
               this.startEvent = event.id ?? 0;
             }
           }

From f6ed407df6ea2a7fc572663fbd5c12598f44c5a2 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Tue, 9 Jan 2024 17:27:59 -0400
Subject: [PATCH 06/15] feature/FOUR-12702

---
 resources/js/processes-catalogue/components/ProcessInfo.vue   | 2 +-
 .../js/processes-catalogue/components/ProcessOptions.vue      | 4 ++--
 .../components/optionsMenu/ButtonsStart.vue                   | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/resources/js/processes-catalogue/components/ProcessInfo.vue b/resources/js/processes-catalogue/components/ProcessInfo.vue
index 1717dba31d..8daff32e9c 100644
--- a/resources/js/processes-catalogue/components/ProcessInfo.vue
+++ b/resources/js/processes-catalogue/components/ProcessInfo.vue
@@ -14,7 +14,7 @@
         />
       </b-col>
       <b-col cols="3">
-        <process-options :process="process" :user="currentUserId"/>
+        <process-options :process="process" />
       </b-col>
     </div>
     <b-col cols="12">
diff --git a/resources/js/processes-catalogue/components/ProcessOptions.vue b/resources/js/processes-catalogue/components/ProcessOptions.vue
index 8fa363e004..707aa8e939 100644
--- a/resources/js/processes-catalogue/components/ProcessOptions.vue
+++ b/resources/js/processes-catalogue/components/ProcessOptions.vue
@@ -1,6 +1,6 @@
 <template>
   <div>
-    <buttons-start :process="process" :currentUser="user"/>
+    <buttons-start :process="process"/>
     <process-counter :process="process" />
     <b-button v-if="createdFromWizardTemplate" class="mt-2 wizard-link" variant="link" @click="getHelperProcess">
       <img src="../../../img/wizard-icon.svg" :alt="$t('Guided Template Icon')" />
@@ -25,7 +25,7 @@ import WizardHelperProcessModal from '../../components/templates/WizardHelperPro
 
 export default {
   components: { ButtonsStart, ProcessCounter, ChartSaveSearch, WizardHelperProcessModal },
-  props: ["process", "user"],
+  props: ["process"],
   computed: {
     createdFromWizardTemplate() {
       return this.process?.properties?.wizardTemplateUuid ? true : false;
diff --git a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
index c076a3b2d0..dc0744c5ca 100644
--- a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
+++ b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
@@ -67,7 +67,7 @@ import { integer } from '@processmaker/screen-builder';
 
 
 export default {
-  props: ["process", "currentUser"],
+  props: ["process"],
   data() {
     return {
       processEvents: [],

From 0949bca8429225138d26a55465dccda339a54ad8 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Tue, 9 Jan 2024 17:30:00 -0400
Subject: [PATCH 07/15] feature/FOUR-12702

---
 resources/js/processes-catalogue/components/ProcessOptions.vue | 2 +-
 .../components/optionsMenu/ButtonsStart.vue                    | 3 ---
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/resources/js/processes-catalogue/components/ProcessOptions.vue b/resources/js/processes-catalogue/components/ProcessOptions.vue
index 707aa8e939..114bb49954 100644
--- a/resources/js/processes-catalogue/components/ProcessOptions.vue
+++ b/resources/js/processes-catalogue/components/ProcessOptions.vue
@@ -6,7 +6,7 @@
       <img src="../../../img/wizard-icon.svg" :alt="$t('Guided Template Icon')" />
       {{ $t('Re-run Wizard') }}
     </b-button>
-    <chart-save-search :process="process"/>
+    <chart-save-search :process="process" />
 
     <wizard-helper-process-modal
       v-if="createdFromWizardTemplate"
diff --git a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
index dc0744c5ca..008b4e0383 100644
--- a/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
+++ b/resources/js/processes-catalogue/components/optionsMenu/ButtonsStart.vue
@@ -63,9 +63,6 @@
 </template>
 
 <script>
-import { integer } from '@processmaker/screen-builder';
-
-
 export default {
   props: ["process"],
   data() {

From 929e0d7275618b2a23ff8619303bbe56aedaaea4 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Wed, 10 Jan 2024 08:52:11 -0400
Subject: [PATCH 08/15] feature/FOUR-12702

---
 resources/js/processes-catalogue/components/ProcessOptions.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resources/js/processes-catalogue/components/ProcessOptions.vue b/resources/js/processes-catalogue/components/ProcessOptions.vue
index 114bb49954..b61f629876 100644
--- a/resources/js/processes-catalogue/components/ProcessOptions.vue
+++ b/resources/js/processes-catalogue/components/ProcessOptions.vue
@@ -1,6 +1,6 @@
 <template>
   <div>
-    <buttons-start :process="process"/>
+    <buttons-start :process="process" />
     <process-counter :process="process" />
     <b-button v-if="createdFromWizardTemplate" class="mt-2 wizard-link" variant="link" @click="getHelperProcess">
       <img src="../../../img/wizard-icon.svg" :alt="$t('Guided Template Icon')" />

From 925ebf38ae9228caff5714ce931ce6f5b02dc40e Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Wed, 10 Jan 2024 09:12:54 -0400
Subject: [PATCH 09/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     | 43 +++++++++----------
 routes/api.php                                |  3 +-
 2 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index ab2a68b6df..9afac574ef 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -267,26 +267,24 @@ public function startEvents(Request $request, Process $process)
                     $webEntry = json_decode($event["config"])->web_entry;
                     $event["webEntry"] = $webEntry;
                 }
-                if (!Auth::user()->is_administrator) {
-                    if (array_key_exists("assignment", $event)){
-                        switch ($event["assignment"]) {
-                            case "user":
-                                if ($currentUser === (int)$event["assignedUsers"]){
-                                    $startEvents[] = $event;
-                                }
-                                break;
-                            case "group":
-                                if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
-                                    $startEvents[] = $event;
-                                }
-                            break;
-                            case "process_manager":
-                                if ($currentUser === $process->manager_id){
-                                    $startEvents[] = $event;
-                                }
+                if (!Auth::user()->is_administrator && array_key_exists("assignment", $event)) {
+                    switch ($event["assignment"]) {
+                        case "user":
+                            if ($currentUser === (int)$event["assignedUsers"]){
+                                $startEvents[] = $event;
+                            }
                             break;
-                        }
-                    } 
+                        case "group":
+                            if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
+                                $startEvents[] = $event;
+                            }
+                        break;
+                        case "process_manager":
+                            if ($currentUser === $process->manager_id){
+                                $startEvents[] = $event;
+                            }
+                        break;
+                    }
                 } else {
                     $startEvents[] = $event;
                 }
@@ -1528,13 +1526,14 @@ protected function checkUsersGroup($event, int $groupId, Request $request)
     {
         $currentUser = Auth::user()->id;
         $group = Group::find($groupId);
+        $response = false; 
         try {
             $response = (new GroupController(new Group()))->users($group, $request);
             $users = $response->all();
 
             foreach ($users as $user) {
                 if($user->resource->id === $currentUser) {
-                    return true;
+                    $response = true;
                 }
             }
         } catch (\Exception $error) {
@@ -1547,13 +1546,13 @@ protected function checkUsersGroup($event, int $groupId, Request $request)
 
             foreach ($groups as $group) {
                 if ($this->checkUsersGroup($event, $group->resource->id, $request)) {
-                    return true;
+                    $response = true;
                 }
             }
         } catch (\Exception $error) {
             return ['error' => $error->getMessage()];
         }
-        return false;
+        return $response;
     }
 
     /**
diff --git a/routes/api.php b/routes/api.php
index 0aa0eba192..8d39ffc46e 100644
--- a/routes/api.php
+++ b/routes/api.php
@@ -125,7 +125,8 @@
     // Processes
     Route::get('processes', [ProcessController::class, 'index'])->name('processes.index')->middleware('can:view-processes');
     Route::get('processes/{process}', [ProcessController::class, 'show'])->name('processes.show')->middleware('can:view-processes,process');
-    Route::get('processes/{process}/start_events', [ProcessController::class, 'startEvents'])->name('processes.start.events')->middleware('can:view-processes,process');
+    Route::get('processes/{process}/start_events', 
+        [ProcessController::class, 'startEvents'])->name('processes.start.events')->middleware('can:view-processes,process');
     Route::post('processes/{process}/export', [ProcessController::class, 'export'])->name('processes.export')->middleware('can:export-processes,process');
     Route::get('processes/{process}/bpmn', [ProcessController::class, 'downloadBpmn'])->name('processes.export.bpmn')->middleware('can:view-processes,process');
     Route::post('processes/import', [ProcessController::class, 'import'])->name('processes.import')->middleware('can:import-processes');

From 570f85ef6a42bfe9d18acbfb18f668ff74a1d1c6 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Wed, 10 Jan 2024 09:41:11 -0400
Subject: [PATCH 10/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     | 60 +++++++++++--------
 routes/api.php                                |  6 +-
 2 files changed, 40 insertions(+), 26 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 9afac574ef..3091f4a30d 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -260,32 +260,17 @@ public function show(Request $request, Process $process)
     public function startEvents(Request $request, Process $process)
     {
         $startEvents = [];
-        $currentUser = Auth::user()->id;
+        $currentUser = Auth::user();
         foreach ($process->start_events as $event) {
             if (count($event["eventDefinitions"]) === 0) {
                 if (array_key_exists("config", $event)) {
                     $webEntry = json_decode($event["config"])->web_entry;
                     $event["webEntry"] = $webEntry;
                 }
-                if (!Auth::user()->is_administrator && array_key_exists("assignment", $event)) {
-                    switch ($event["assignment"]) {
-                        case "user":
-                            if ($currentUser === (int)$event["assignedUsers"]){
-                                $startEvents[] = $event;
-                            }
-                            break;
-                        case "group":
-                            if ($this->checkUsersGroup($event, (int)$event["assignedGroups"], $request)){
-                                $startEvents[] = $event;
-                            }
-                        break;
-                        case "process_manager":
-                            if ($currentUser === $process->manager_id){
-                                $startEvents[] = $event;
-                            }
-                        break;
-                    }
-                } else {
+                if (
+                    $this->checkUserCanStartProcess($event, $currentUser->id, $process, $request) ||
+                    Auth::user()->is_administrator
+                ) {
                     $startEvents[] = $event;
                 }
             }
@@ -1512,21 +1497,48 @@ protected function getRequestFilterBy(Request $request, array $searchableColumns
 
         return $where;
     }
+    /**
+     * check if currentUser can start the request
+     *
+     * @param array $event
+     * @param int $currentUser
+     * @param Process $process
+     * @param Request $request
+     *
+     * @return bool
+     */
+    protected function checkUserCanStartProcess($event, $currentUser, $process, $request)
+    {
+        $response = false;
+        if (array_key_exists("assignment", $event)) {
+            switch ($event["assignment"]) {
+                case "user":
+                    $response = $currentUser === (int)$event["assignedUsers"];
+                    break;
+                case "group":
+                    $response = $this->checkUsersGroup((int)$event["assignedGroups"], $request);
+                break;
+                case "process_manager":
+                    $response = $currentUser === $process->manager_id;
+                break;
+            }
+        }
+        return $response;
+    }
 
     /**
      * check if currentUser is member of a group
      *
-     * @param object $event
      * @param int $groupId
      * @param Request $request
      *
      * @return bool
      */
-    protected function checkUsersGroup($event, int $groupId, Request $request)
+    protected function checkUsersGroup(int $groupId, Request $request)
     {
         $currentUser = Auth::user()->id;
         $group = Group::find($groupId);
-        $response = false; 
+        $response = false;
         try {
             $response = (new GroupController(new Group()))->users($group, $request);
             $users = $response->all();
@@ -1545,7 +1557,7 @@ protected function checkUsersGroup($event, int $groupId, Request $request)
             $groups = $response->all();
 
             foreach ($groups as $group) {
-                if ($this->checkUsersGroup($event, $group->resource->id, $request)) {
+                if ($this->checkUsersGroup($group->resource->id, $request)) {
                     $response = true;
                 }
             }
diff --git a/routes/api.php b/routes/api.php
index 8d39ffc46e..ac84ea49c6 100644
--- a/routes/api.php
+++ b/routes/api.php
@@ -125,8 +125,10 @@
     // Processes
     Route::get('processes', [ProcessController::class, 'index'])->name('processes.index')->middleware('can:view-processes');
     Route::get('processes/{process}', [ProcessController::class, 'show'])->name('processes.show')->middleware('can:view-processes,process');
-    Route::get('processes/{process}/start_events', 
-        [ProcessController::class, 'startEvents'])->name('processes.start.events')->middleware('can:view-processes,process');
+    Route::get('processes/{process}/start_events',
+        [ProcessController::class, 'startEvents'])
+        ->name('processes.start.events')
+        ->middleware('can:view-processes,process');
     Route::post('processes/{process}/export', [ProcessController::class, 'export'])->name('processes.export')->middleware('can:export-processes,process');
     Route::get('processes/{process}/bpmn', [ProcessController::class, 'downloadBpmn'])->name('processes.export.bpmn')->middleware('can:view-processes,process');
     Route::post('processes/import', [ProcessController::class, 'import'])->name('processes.import')->middleware('can:import-processes');

From 905d5cef6ef285811a8d1be6a3c73dab599aadc9 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Thu, 11 Jan 2024 10:43:26 -0400
Subject: [PATCH 11/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     |  1 +
 tests/Feature/Api/ProcessTest.php             | 40 +++++++++++++++++++
 2 files changed, 41 insertions(+)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index 3091f4a30d..da904cad84 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -261,6 +261,7 @@ public function startEvents(Request $request, Process $process)
     {
         $startEvents = [];
         $currentUser = Auth::user();
+        dd($process, 'porsiiaaaa');
         foreach ($process->start_events as $event) {
             if (count($event["eventDefinitions"]) === 0) {
                 if (array_key_exists("config", $event)) {
diff --git a/tests/Feature/Api/ProcessTest.php b/tests/Feature/Api/ProcessTest.php
index 4f0bd1824c..cb2f4231f7 100644
--- a/tests/Feature/Api/ProcessTest.php
+++ b/tests/Feature/Api/ProcessTest.php
@@ -1294,4 +1294,44 @@ public function testDeleteMediaImages()
 
         $this->assertDatabaseMissing('media', ['id' => $mediaImagen->id]);
     }
+
+    // Test Get start events 
+    public function testGetStartEvents()
+    {
+        $this->user = User::factory()->create([
+            'is_administrator' => false,
+        ]);
+        $this->user->permissions()->attach(Permission::byName('view-processes'));
+
+        $process = Process::factory()->create(
+            [
+                'name' => 'Test SartEvents',
+                'start_events' => [
+                    [
+                    'assignment' => 'user',
+                    'assignedUsers' => $this->user->id
+                    ],
+                    [
+                    'assignment' => 'user',
+                    'assignedUsers' => '9'
+                    ]
+                ]
+            ]
+        );
+        $process->start_events = [
+            [
+                'assignment' => 'user',
+                'assignedUsers' => $this->user->id
+            ],
+            [
+            'assignment' => 'user',
+            'assignedUsers' => '9'
+            ]
+        ];
+        $route = route('api.processes.start.events', $process);
+
+        $response = $this->apiCall('GET', $route);
+        $response->assertStatus(200);
+        dd($response->getData()->data);
+    }
 }

From f7cbb85c4707a9dd55565ac506a7b0a0a0f221e6 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Thu, 11 Jan 2024 17:43:04 -0400
Subject: [PATCH 12/15] feature/FOUR-12702

---
 ProcessMaker/Http/Controllers/Api/ProcessController.php | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index da904cad84..d14e4cbc0a 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -261,7 +261,6 @@ public function startEvents(Request $request, Process $process)
     {
         $startEvents = [];
         $currentUser = Auth::user();
-        dd($process, 'porsiiaaaa');
         foreach ($process->start_events as $event) {
             if (count($event["eventDefinitions"]) === 0) {
                 if (array_key_exists("config", $event)) {
@@ -1514,10 +1513,14 @@ protected function checkUserCanStartProcess($event, $currentUser, $process, $req
         if (array_key_exists("assignment", $event)) {
             switch ($event["assignment"]) {
                 case "user":
-                    $response = $currentUser === (int)$event["assignedUsers"];
+                    if (array_key_exists("assignedUsers", $event)) {
+                        $response = $currentUser === (int)$event["assignedUsers"];
+                    }
                     break;
                 case "group":
-                    $response = $this->checkUsersGroup((int)$event["assignedGroups"], $request);
+                    if (array_key_exists("assignedGroups", $event)) {
+                        $response = $this->checkUsersGroup((int)$event["assignedGroups"], $request);
+                    }
                 break;
                 case "process_manager":
                     $response = $currentUser === $process->manager_id;

From e8582d10f7ae0ff9136884a79f4c19d29de6dc3d Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Thu, 11 Jan 2024 17:50:03 -0400
Subject: [PATCH 13/15] feature/FOUR-12702

---
 tests/Feature/Api/ProcessTest.php | 40 -------------------------------
 1 file changed, 40 deletions(-)

diff --git a/tests/Feature/Api/ProcessTest.php b/tests/Feature/Api/ProcessTest.php
index cb2f4231f7..4f0bd1824c 100644
--- a/tests/Feature/Api/ProcessTest.php
+++ b/tests/Feature/Api/ProcessTest.php
@@ -1294,44 +1294,4 @@ public function testDeleteMediaImages()
 
         $this->assertDatabaseMissing('media', ['id' => $mediaImagen->id]);
     }
-
-    // Test Get start events 
-    public function testGetStartEvents()
-    {
-        $this->user = User::factory()->create([
-            'is_administrator' => false,
-        ]);
-        $this->user->permissions()->attach(Permission::byName('view-processes'));
-
-        $process = Process::factory()->create(
-            [
-                'name' => 'Test SartEvents',
-                'start_events' => [
-                    [
-                    'assignment' => 'user',
-                    'assignedUsers' => $this->user->id
-                    ],
-                    [
-                    'assignment' => 'user',
-                    'assignedUsers' => '9'
-                    ]
-                ]
-            ]
-        );
-        $process->start_events = [
-            [
-                'assignment' => 'user',
-                'assignedUsers' => $this->user->id
-            ],
-            [
-            'assignment' => 'user',
-            'assignedUsers' => '9'
-            ]
-        ];
-        $route = route('api.processes.start.events', $process);
-
-        $response = $this->apiCall('GET', $route);
-        $response->assertStatus(200);
-        dd($response->getData()->data);
-    }
 }

From ed72dc7c9933e8b222a84eb5445bbc0a36d41c98 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Thu, 11 Jan 2024 17:50:17 -0400
Subject: [PATCH 14/15] feature/FOUR-12702

---
 tests/Feature/Api/ProcessTest.php | 40 +++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/tests/Feature/Api/ProcessTest.php b/tests/Feature/Api/ProcessTest.php
index 4f0bd1824c..cb2f4231f7 100644
--- a/tests/Feature/Api/ProcessTest.php
+++ b/tests/Feature/Api/ProcessTest.php
@@ -1294,4 +1294,44 @@ public function testDeleteMediaImages()
 
         $this->assertDatabaseMissing('media', ['id' => $mediaImagen->id]);
     }
+
+    // Test Get start events 
+    public function testGetStartEvents()
+    {
+        $this->user = User::factory()->create([
+            'is_administrator' => false,
+        ]);
+        $this->user->permissions()->attach(Permission::byName('view-processes'));
+
+        $process = Process::factory()->create(
+            [
+                'name' => 'Test SartEvents',
+                'start_events' => [
+                    [
+                    'assignment' => 'user',
+                    'assignedUsers' => $this->user->id
+                    ],
+                    [
+                    'assignment' => 'user',
+                    'assignedUsers' => '9'
+                    ]
+                ]
+            ]
+        );
+        $process->start_events = [
+            [
+                'assignment' => 'user',
+                'assignedUsers' => $this->user->id
+            ],
+            [
+            'assignment' => 'user',
+            'assignedUsers' => '9'
+            ]
+        ];
+        $route = route('api.processes.start.events', $process);
+
+        $response = $this->apiCall('GET', $route);
+        $response->assertStatus(200);
+        dd($response->getData()->data);
+    }
 }

From 05b419976c5453eee043a153b37f98889529d945 Mon Sep 17 00:00:00 2001
From: Luciana Nunez <luciana.nunez@processmaker.com>
Date: Thu, 11 Jan 2024 22:19:44 -0400
Subject: [PATCH 15/15] feature/FOUR-12702

---
 .../Controllers/Api/ProcessController.php     | 42 ++++++++++---------
 tests/Feature/Api/ProcessTest.php             | 40 ------------------
 2 files changed, 22 insertions(+), 60 deletions(-)

diff --git a/ProcessMaker/Http/Controllers/Api/ProcessController.php b/ProcessMaker/Http/Controllers/Api/ProcessController.php
index d14e4cbc0a..e813317ccb 100644
--- a/ProcessMaker/Http/Controllers/Api/ProcessController.php
+++ b/ProcessMaker/Http/Controllers/Api/ProcessController.php
@@ -1543,30 +1543,32 @@ protected function checkUsersGroup(int $groupId, Request $request)
         $currentUser = Auth::user()->id;
         $group = Group::find($groupId);
         $response = false;
-        try {
-            $response = (new GroupController(new Group()))->users($group, $request);
-            $users = $response->all();
-
-            foreach ($users as $user) {
-                if($user->resource->id === $currentUser) {
-                    $response = true;
+        if (isset($group)){
+            try {
+                $response = (new GroupController(new Group()))->users($group, $request);
+                $users = $response->all();
+    
+                foreach ($users as $user) {
+                    if($user->resource->id === $currentUser) {
+                        $response = true;
+                    }
                 }
+            } catch (\Exception $error) {
+                return ['error' => $error->getMessage()];
             }
-        } catch (\Exception $error) {
-            return ['error' => $error->getMessage()];
-        }
-
-        try {
-            $response = (new GroupController(new Group()))->groups($group, $request);
-            $groups = $response->all();
-
-            foreach ($groups as $group) {
-                if ($this->checkUsersGroup($group->resource->id, $request)) {
-                    $response = true;
+    
+            try {
+                $response = (new GroupController(new Group()))->groups($group, $request);
+                $groups = $response->all();
+    
+                foreach ($groups as $group) {
+                    if ($this->checkUsersGroup($group->resource->id, $request)) {
+                        $response = true;
+                    }
                 }
+            } catch (\Exception $error) {
+                return ['error' => $error->getMessage()];
             }
-        } catch (\Exception $error) {
-            return ['error' => $error->getMessage()];
         }
         return $response;
     }
diff --git a/tests/Feature/Api/ProcessTest.php b/tests/Feature/Api/ProcessTest.php
index cb2f4231f7..4f0bd1824c 100644
--- a/tests/Feature/Api/ProcessTest.php
+++ b/tests/Feature/Api/ProcessTest.php
@@ -1294,44 +1294,4 @@ public function testDeleteMediaImages()
 
         $this->assertDatabaseMissing('media', ['id' => $mediaImagen->id]);
     }
-
-    // Test Get start events 
-    public function testGetStartEvents()
-    {
-        $this->user = User::factory()->create([
-            'is_administrator' => false,
-        ]);
-        $this->user->permissions()->attach(Permission::byName('view-processes'));
-
-        $process = Process::factory()->create(
-            [
-                'name' => 'Test SartEvents',
-                'start_events' => [
-                    [
-                    'assignment' => 'user',
-                    'assignedUsers' => $this->user->id
-                    ],
-                    [
-                    'assignment' => 'user',
-                    'assignedUsers' => '9'
-                    ]
-                ]
-            ]
-        );
-        $process->start_events = [
-            [
-                'assignment' => 'user',
-                'assignedUsers' => $this->user->id
-            ],
-            [
-            'assignment' => 'user',
-            'assignedUsers' => '9'
-            ]
-        ];
-        $route = route('api.processes.start.events', $process);
-
-        $response = $this->apiCall('GET', $route);
-        $response->assertStatus(200);
-        dd($response->getData()->data);
-    }
 }