simplify dependency ID parsing

jenkinsci · Nov 12, 2024 · fd147b5 · fd147b5
1 parent b3e23da
commit fd147b5
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.../hudson/plugin/buildpipeline/extension/BuildCardExtension/build-card-template-onclicks.js b/.../hudson/plugin/buildpipeline/extension/BuildCardExtension/build-card-template-onclicks.js
@@ -4,8 +4,7 @@ function parseDataAttributes(container) {
         buildExtId: container.dataset.rerunBuildExtId,
         dependencyIds: container.dataset.rerunBuildDependencyIds
             .split(",")
-            .filter(Boolean)
-            .map(id => parseInt(id, 10))
+            .map(id => parseInt(id))
     };
 }
 
@@ -17,8 +16,7 @@ function parseTriggerAttributes(container) {
         projectName: container.dataset.projectName,
         dependencyIds: container.dataset.dependencyIds
             .split(",")
-            .filter(Boolean)
-            .map(id => parseInt(id, 10))
+            .map(id => parseInt(id))
     };
 }
 
@@ -30,12 +28,14 @@ function handleDialogClick(dataContainer) {
 
 function handleRerunClick(dataContainer) {
     const { id, buildExtId, dependencyIds } = parseDataAttributes(dataContainer);
+    console.log("dependencyIds", dependencyIds);
     buildPipeline.showSpinner(id);
     buildPipeline.rerunBuild(id, buildExtId, dependencyIds);
 }
 
 function handleTriggerBuild(dataContainer) {
     const { id, upstreamProjectName, upstreamBuildNumber, projectName, dependencyIds } = parseTriggerAttributes(dataContainer);
+    console.log("dependencyIds", dependencyIds);
     buildPipeline.showSpinner(id);
     buildPipeline.triggerBuild(id, upstreamProjectName, upstreamBuildNumber, projectName, dependencyIds);
 }