Skip to content

fix(symfony): avoid deprecated translation command probe - #8627

Open
tacman wants to merge 1 commit into
api-platform:mainfrom
tacman:fix/symfony-82-translation-probe
Open

tacman wants to merge 1 commit into
api-platform:mainfrom
tacman:fix/symfony-82-translation-probe

Conversation

@tacman

@tacman tacman commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Fixes #8626.

On Symfony 8.2, API Platform's object-mapper feature probe autoloads the deprecated FrameworkBundle TranslationExtractCommand shim. Probe the command's new Translation component namespace first, retaining the old FrameworkBundle class as a short-circuit fallback for earlier Symfony releases. Keep the existing object-mapper availability checks.

The regression test runs in a separate process, loads the real extension, verifies object-mapper metadata registration, and asserts the deprecated class was not loaded. It fails before the fix and passes afterward.

Validation using Symfony 8.2-dev dependencies:

  • The new regression and common-configuration tests pass: 2 tests, 68 assertions.
  • Running the complete extension test class gives 19 passing tests and one existing failure in testSwaggerUiDisabledConfiguration (Swagger UI processor remains registered). That same failure reproduces with the unchanged upstream extension; it is not suppressed or changed here.
  • Both changed PHP files pass syntax checks; git diff --check passes.

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.

Symfony 8.2: object mapper feature probe loads deprecated translation command

1 participant