Prevent duplication of actions within appendAction Stimulus method

TrestleAdmin · Dec 4, 2024 · 98db106 · 98db106
1 parent f324bb5
commit 98db106
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/app/assets/bundle/trestle/admin.js b/app/assets/bundle/trestle/admin.js
diff --git a/frontend/js/controllers/application_controller.js b/frontend/js/controllers/application_controller.js
@@ -5,7 +5,11 @@ import { fetchWithErrorHandling, fetchTurboStream } from '../core/fetch'
 export default class extends Controller {
   appendAction (event, method, element = this.element) {
     const actions = this.actionsList
-    actions.push(`${event}->${this.identifier}#${method}`)
+    const newAction = `${event}->${this.identifier}#${method}`
+
+    if (!actions.includes(newAction)) {
+      actions.push(newAction)
+    }
 
     element.dataset.action = actions.join(' ')
   }