Skip to content

fix(symfony): drop translation command probe for object mapper - #8726

Merged
soyuka merged 2 commits into
api-platform:4.4from
soyuka:fix/translation-extract-gate-8725
Oct 19, 2026
Merged

soyuka merged 2 commits into
api-platform:4.4from
soyuka:fix/translation-extract-gate-8725

Conversation

@soyuka

@soyuka soyuka commented Oct 19, 2026

Copy link
Copy Markdown
Member

Summary

Follows up #8627. FrameworkBundle 8.2's TranslationExtractCommand extends a class of the optional symfony/translation package, so class_exists() on it is a fatal error when that package is missing. The probe only stood for "framework-bundle >= 7.3"; a Composer conflict on <7.4 now guarantees it (matches the ^7.4 floor of the other Symfony dependencies) and the object mapper gate keeps only willBeAvailable.

The new conflict (symfony/framework-bundle: <7.4, replacing the root 6.4.6 || 7.0.6 one and added to src/Symfony/composer.json) is the only constraint change.

Test plan

  • testObjectMapperGateDoesNotAutoloadTranslationCommands fails on the previous ApiPlatformExtension, passes now.
  • Replaces testObjectMapperDoesNotLoadDeprecatedTranslationCommand, which asserted a subset and was skipped without the 8.2 command.

Fixes #8725

FrameworkBundle 8.2's TranslationExtractCommand extends a class of the optional symfony/translation
package, so probing it with class_exists() is a fatal error when that package is missing. The probe
only stood for "framework-bundle >= 7.3", which a composer conflict on <7.4 now guarantees, matching
the ^7.4 floor of every other Symfony dependency. Follows up api-platform#8627.

Fixes api-platform#8725
@soyuka
soyuka merged commit 1ea9db8 into api-platform:4.4 Oct 19, 2026
117 checks passed
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