Skip to content

fix(symfony): separate route priority - #8529

Merged
soyuka merged 1 commit into
api-platform:mainfrom
soyuka:fix/route-priority
Sep 12, 2026
Merged

fix(symfony): separate route priority#8529
soyuka merged 1 commit into
api-platform:mainfrom
soyuka:fix/route-priority

Conversation

@soyuka

@soyuka soyuka commented Sep 12, 2026

Copy link
Copy Markdown
Member

Summary

PR #8309 passed the existing operation priority to Symfony's route collection. That field controls operation ordering within a resource, so the change altered route matching for resources that rely on operation priorities.

This adds the distinct nullable routePriority metadata field, wires it through HTTP operations and YAML/XML metadata, and uses it when registering Symfony routes. Existing priority behavior is preserved.

CI analysis

The merged PR branch introduced route-ordering failures in the PostgreSQL, MySQL, MongoDB, Symfony 8.1, Symfony dev, PHP 8.2, event-listener, no-deprecations, and component matrix jobs. The failures included the prioritized-operation regression and route matches returning the wrong resource representation.

The remaining failed checks on the merged branch were pre-existing: api-platform/test dependency resolution in the minimal Symfony jobs, Unknown named parameter $detail in state/serializer jobs, and the dependent Codecov project check.

Test plan

  • Focused ApiLoaderTest passes.
  • PHPStan passes for changed source files.
  • PHP CS Fixer passes.
  • The broader functional matrix remains covered by CI.

@soyuka
soyuka force-pushed the fix/route-priority branch 2 times, most recently from d9c2dfd to 606edbe Compare September 12, 2026 06:23
Include the new nullable routePriority field in the XML and YAML

extractor test expectations.
@soyuka
soyuka merged commit 5ad54c9 into api-platform:main Sep 12, 2026
111 of 117 checks passed
@soyuka
soyuka deleted the fix/route-priority branch September 12, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant