From: Jérôme Benoit <jerome.benoit@sap.com>
Date: Fri, 17 Apr 2026 00:39:28 +0000 (+0200)
Subject: fix(ui-web): remove orphaned CSS classes and guard onFinally in useExecuteAction
X-Git-Tag: cli@v4.5.0~49
X-Git-Url: https://git.piment-noir.org/?a=commitdiff_plain;h=b01978f7e4eeb6c11136109a8b98871c72f4ce17;p=e-mobility-charging-stations-simulator.git

fix(ui-web): remove orphaned CSS classes and guard onFinally in useExecuteAction

- Remove dead connectors-table__row/column classes from CSConnector.vue
- Wrap onFinally callback in try/catch inside .finally() to prevent
  swallowing the original rejection error
---

diff --git a/ui/web/src/components/charging-stations/CSConnector.vue b/ui/web/src/components/charging-stations/CSConnector.vue
index 531c8078..e0beffdf 100644
--- a/ui/web/src/components/charging-stations/CSConnector.vue
+++ b/ui/web/src/components/charging-stations/CSConnector.vue
@@ -1,21 +1,21 @@
 <template>
-  <tr class="connectors-table__row">
-    <td class="connectors-table__column">
+  <tr>
+    <td>
       {{ evseId != null ? `${evseId}/${connectorId}` : connectorId }}
     </td>
-    <td class="connectors-table__column">
+    <td>
       {{ connector.status ?? EMPTY_VALUE_PLACEHOLDER }}
     </td>
-    <td class="connectors-table__column">
+    <td>
       {{ connector.locked === true ? 'Yes' : 'No' }}
     </td>
-    <td class="connectors-table__column">
+    <td>
       {{ connector.transactionStarted === true ? `Yes (${connector.transactionId})` : 'No' }}
     </td>
-    <td class="connectors-table__column">
+    <td>
       {{ atgStatus?.start === true ? 'Yes' : 'No' }}
     </td>
-    <td class="connectors-table__column">
+    <td>
       <StateButton
         :active="connector.locked === true"
         :off="() => unlockConnector()"
diff --git a/ui/web/src/composables/Utils.ts b/ui/web/src/composables/Utils.ts
index 301b89ec..7afeee00 100644
--- a/ui/web/src/composables/Utils.ts
+++ b/ui/web/src/composables/Utils.ts
@@ -160,7 +160,13 @@ export const useExecuteAction = (emit?: (event: 'need-refresh') => void) => {
         emit?.('need-refresh')
         return $toast.success(successMsg)
       })
-      .finally(onFinally)
+      .finally(() => {
+        try {
+          onFinally?.()
+        } catch (error: unknown) {
+          console.error('Error in onFinally callback:', error)
+        }
+      })
       .catch((error: unknown) => {
         $toast.error(errorMsg)
         console.error(`${errorMsg}:`, error)