map instead forEach

vatesfr · May 15, 2023 · 1bb90a4 · 1bb90a4
1 parent 438d102
commit 1bb90a4
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/packages/xo-web/src/common/sorted-table/index.js b/packages/xo-web/src/common/sorted-table/index.js
@@ -193,12 +193,11 @@ const CollapsedActions = decorate([
       },
     },
     computed: {
-      wrappedActions: ({ runningActions }, { actions }) => {
-        actions.forEach((action, index) => {
-          actions[index].isRunning = runningActions.includes(action.label)
-        })
-        return [...actions]
-      },
+      wrappedActions: ({ runningActions }, { actions }) =>
+        actions.map(action => {
+          action.isRunning = runningActions.includes(action.label)
+          return action
+        }),
       dropdownId: generateId,
       actions: ({ wrappedActions: actions }, { items, userData }) =>
         actions.map(({ disabled, grouped, handler, icon, isRunning, label, level, redirectOnSuccess }) => {