Skip to content

Respect SystemParam::validate_param for observers and other non-executor system runners #15394

Closed
@MiniaczQ

Description

@MiniaczQ

What problem does this solve or what need does it fill?

Currently only schedule executors respect the SystemParam::validate_param to prevent panics when running systems with unavailable resources.
As a follow up to #15276 this should be implemented for other places that run systems like:

  • observers,
  • manually executed systems through &mut World

What solution would you like?

Check validate_param directly before running the systems and if it fails, don't run them.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-ECSEntities, components, systems, and eventsC-FeatureA new feature, making something new possibleD-ModestA "normal" level of difficulty; suitable for simple features or challenging fixesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!X-UncontroversialThis work is generally agreed upon

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions