Skip to content

[26.0] Fix nested DatasetCollectionElement in dynamic options#22287

Merged
jmchilton merged 2 commits intogalaxyproject:release_26.0from
mvdbeek:fix-nested-dce-dynamic-option-filter
Mar 29, 2026
Merged

[26.0] Fix nested DatasetCollectionElement in dynamic options#22287
jmchilton merged 2 commits intogalaxyproject:release_26.0from
mvdbeek:fix-nested-dce-dynamic-option-filter

Conversation

@mvdbeek
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek commented Mar 28, 2026

When a DatasetCollectionElement containing an LDDA or nested child collection (rather than an HDA) was used as a reference for dynamic option filtering, _get_ref_data() failed to normalize it into a list, causing "TypeError: 'DatasetCollectionElement' object is not iterable".

Use DatasetCollectionElement.dataset_instances which correctly returns a list of dataset instances for all element types (HDA, LDDA, or nested collections).

Fixes #22099

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

mvdbeek added 2 commits March 28, 2026 10:26
When a DatasetCollectionElement containing an LDDA or nested child
collection (rather than an HDA) was used as a reference for dynamic
option filtering, _get_ref_data() failed to normalize it into a list,
causing "TypeError: 'DatasetCollectionElement' object is not iterable".

Use DatasetCollectionElement.dataset_instances which correctly returns
a list of dataset instances for all element types (HDA, LDDA, or
nested collections).

Fixes galaxyproject#22099
@github-actions github-actions Bot added this to the 26.1 milestone Mar 28, 2026
@jmchilton jmchilton merged commit 7cbe36c into galaxyproject:release_26.0 Mar 29, 2026
55 of 58 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies Mar 29, 2026
@github-actions
Copy link
Copy Markdown

This PR was merged without a "kind/" label, please correct.

@nsoranzo nsoranzo deleted the fix-nested-dce-dynamic-option-filter branch March 30, 2026 00:13
@ahmedhamidawan ahmedhamidawan modified the milestones: 26.1, 26.0 Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants