Merge pull request #17977 from davelopez/24.0_fix_output_filtering_ma…

…rkdown_directives

[24.0] Fix filtering workflow outputs by type in markdown directives

client/src/components/Markdown/MarkdownToolBox.vue

@@ -259,18 +259,25 @@ export default {
                 });
             return steps;
         },
-        getOutputs() {
+        getOutputs(filterByType = undefined) {
             const outputLabels = [];
             this.steps &&
                 Object.values(this.steps).forEach((step) => {
                     step.workflow_outputs.forEach((workflowOutput) => {
                         if (workflowOutput.label) {
-                            outputLabels.push(workflowOutput.label);
+                            if (!filterByType || this.stepOutputMatchesType(step, workflowOutput, filterByType)) {
+                                outputLabels.push(workflowOutput.label);
+                            }
                         }
                     });
                 });
             return outputLabels;
         },
+        stepOutputMatchesType(step, workflowOutput, type) {
+            return Boolean(
+                step.outputs.find((output) => output.name === workflowOutput.output_name && output.type === type)
+            );
+        },
         getArgumentTitle(argumentName) {
             return (
                 argumentName[0].toUpperCase() +
@@ -331,13 +338,13 @@ export default {
         onHistoryDatasetId(argumentName) {
             this.selectedArgumentName = argumentName;
             this.selectedType = "history_dataset_id";
-            this.selectedLabels = this.getOutputs();
+            this.selectedLabels = this.getOutputs("data");
             this.selectedShow = true;
         },
         onHistoryCollectionId(argumentName) {
             this.selectedArgumentName = argumentName;
             this.selectedType = "history_dataset_collection_id";
-            this.selectedLabels = this.getOutputs();
+            this.selectedLabels = this.getOutputs("collection");
             this.selectedShow = true;
         },
         onWorkflowId(argumentName) {