From 05365583a455d8c4753cd06075e23ce6fdec8afa Mon Sep 17 00:00:00 2001
From: Yaroslav <91559310+yaroslavafenkin@users.noreply.github.com>
Date: Fri, 25 Oct 2024 01:18:32 +0300
Subject: [PATCH] [JENKINS-73956] Extract inline event handlers from
 `LegacyJobConfigMigrationMonitor/manage.jelly` (#241)

---
 .../manage.jelly                              | 24 +++++------
 .../resources.js                              | 40 +++++++++----------
 2 files changed, 32 insertions(+), 32 deletions(-)

diff --git a/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/manage.jelly b/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/manage.jelly
index 1a1b0b61..f000c79a 100644
--- a/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/manage.jelly
+++ b/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/manage.jelly
@@ -39,23 +39,23 @@ THE SOFTWARE.
                     <a href="#"
                        class="action"
                        title="${%SelectAll.title}"
-                       onclick="selectAll(this);return false;">${%SelectAll}</a>
+                       data-action="select-all">${%SelectAll}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllMigratable.title}"
-                       onclick="selectAllMigratable(this);return false;">${%SelectAllMigratable}</a>
+                       data-action="select-migratable">${%SelectAllMigratable}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllValid.title}"
-                       onclick="selectAllValid(this);return false;">${%SelectAllValid}</a>
+                       data-action="select-all-valid">${%SelectAllValid}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllInvalid.title}"
-                       onclick="selectAllInvalid(this);return false;">${%SelectAllInvalid}</a>
+                       data-action="select-all-invalid">${%SelectAllInvalid}</a>
                 </div>
                 <div class="action-panel">
                     <span class="yui-button">
-                        <button class="action-migrate-selected" onclick="confirmAndMigrateAllSelected(this);"
+                        <button class="action action-migrate-selected"
                                 data-url="${rootURL}/${it.url}/migrateAllSelected"
                                 data-confirm-template="${%MigrateAllSelected_confirm}"
                                 data-nothing-selected="${%MigrateAllSelected_nothing}">
@@ -63,7 +63,7 @@ THE SOFTWARE.
                         </button>
                     </span>
                     <span class="yui-button">
-                        <button class="action-ignore-selected" onclick="confirmAndIgnoreAllSelected(this);"
+                        <button class="action action-ignore-selected"
                                 data-url="${rootURL}/${it.url}/ignoreAllSelected"
                                 data-confirm-template="${%IgnoreAllSelected_confirm}"
                                 data-nothing-selected="${%IgnoreMigrateAllSelected_nothing}">
@@ -187,24 +187,24 @@ THE SOFTWARE.
                     <a href="#"
                        class="action"
                        title="${%SelectAll.title}"
-                       onclick="selectAll(this);return false;">${%SelectAll}</a>
+                       data-action="select-all">${%SelectAll}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllMigratable.title}"
-                       onclick="selectAllMigratable(this);return false;">${%SelectAllMigratable}</a>
+                       data-action="select-migratable">${%SelectAllMigratable}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllValid.title}"
-                       onclick="selectAllValid(this);return false;">${%SelectAllValid}</a>
+                       data-action="select-all-valid">${%SelectAllValid}</a>
                     <a href="#"
                        class="action"
                        title="${%SelectAllInvalid.title}"
-                       onclick="selectAllInvalid(this);return false;">${%SelectAllInvalid}</a>
+                       data-action="select-all-invalid">${%SelectAllInvalid}</a>
                 </div>
                 
                 <div class="action-panel">
                     <span class="yui-button">
-                        <button class="action-migrate-selected" onclick="confirmAndMigrateAllSelected(this);"
+                        <button class="action action-migrate-selected"
                                 data-url="${rootURL}/${it.url}/migrateAllSelected"
                                 data-confirm-template="${%MigrateAllSelected_confirm}"
                                 data-nothing-selected="${%MigrateAllSelected_nothing}">
@@ -212,7 +212,7 @@ THE SOFTWARE.
                         </button>
                     </span>
                     <span class="yui-button">
-                        <button class="action-ignore-selected" onclick="confirmAndIgnoreAllSelected(this);"
+                        <button class="action action-ignore-selected"
                                 data-url="${rootURL}/${it.url}/ignoreAllSelected"
                                 data-confirm-template="${%IgnoreAllSelected_confirm}"
                                 data-nothing-selected="${%IgnoreMigrateAllSelected_nothing}">
diff --git a/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/resources.js b/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/resources.js
index 9ac00f44..8bbcecf7 100644
--- a/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/resources.js
+++ b/src/main/resources/hudson/plugins/copyartifact/monitor/LegacyJobConfigMigrationMonitor/resources.js
@@ -21,18 +21,6 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
-function selectAll(anchor){
-    _selectAllThat(anchor, '.checkbox-line');
-}
-function selectAllMigratable(anchor){
-    _selectAllThat(anchor, '.checkbox-line.migratable-line.valid-line');
-}
-function selectAllValid(anchor){
-    _selectAllThat(anchor, '.checkbox-line.valid-line');
-}
-function selectAllInvalid(anchor){
-    _selectAllThat(anchor, '.checkbox-line.invalid-line');
-}
 
 function _selectAllThat(anchor, selector){
     var parent = anchor.closest('.legacy-copy-artifact');
@@ -67,14 +55,6 @@ function checkTheDesiredOne(allCheckBoxes, concernedCheckBoxes){
     }
 }
 
-function confirmAndIgnoreAllSelected(button){
-    confirmAndSendRequest(button);
-}
-
-function confirmAndMigrateAllSelected(button){
-    confirmAndSendRequest(button);
-}
-
 function confirmAndSendRequest(button){
     var parent = button.closest('.legacy-copy-artifact');
     var allCheckBoxes = parent.querySelectorAll('.checkbox-line');
@@ -181,5 +161,25 @@ function onCheckChanged(checkBox){
                 return false;
             }
         );
+
+        const actionSelectors = {
+            "select-all": '.checkbox-line',
+            "select-migratable": '.checkbox-line.migratable-line.valid-line',
+            "select-all-valid": '.checkbox-line.valid-line',
+            "select-all-invalid": '.checkbox-line.invalid-line'
+        }
+
+        document.querySelectorAll("a.action").forEach(anchor => {
+            anchor.addEventListener("click", (event) => {
+                event.preventDefault();
+                _selectAllThat(event.target, actionSelectors[event.target.dataset.action]);
+            });
+        });
+
+        document.querySelectorAll(".action-panel button.action").forEach((button) => {
+            button.addEventListener("click", (event) => {
+                confirmAndSendRequest(event.target);
+            });
+        });
     });
 })();