Skip to content

Wire the Console EntityValueResolver as a service#2193

Open
chalasr wants to merge 1 commit into
doctrine:3.3.xfrom
chalasr:console-entity-resolver
Open

Wire the Console EntityValueResolver as a service#2193
chalasr wants to merge 1 commit into
doctrine:3.3.xfrom
chalasr:console-entity-resolver

Conversation

@chalasr

@chalasr chalasr commented Jan 3, 2026

Copy link
Copy Markdown
Contributor

This PR enables the EntityValueResolver for Console commands that has been introduced in symfony/symfony#62917.

Needs symfony/symfony#62917 to be merged first

Please let me know if I missed anything.

@chalasr chalasr changed the base branch from 3.2.x to 2.18.x January 3, 2026 19:40
@chalasr chalasr force-pushed the console-entity-resolver branch 7 times, most recently from 0c55ab2 to 13ab18d Compare January 3, 2026 20:06
@chalasr chalasr force-pushed the console-entity-resolver branch from 13ab18d to bc88352 Compare January 25, 2026 15:39
@chalasr

chalasr commented Feb 5, 2026

Copy link
Copy Markdown
Contributor Author

The Symfony PR has been merged so this PR is ready.

Comment thread tests/DependencyInjection/DoctrineExtensionTest.php Outdated
@W0rma

W0rma commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@chalasr Should this PR target 3.2.x instead of 2.18.x since the composer.json of 2.18.x does not allow installing symfony/doctrine-bridge:8.1?

@chalasr chalasr changed the base branch from 2.18.x to 3.3.x June 30, 2026 10:27
@chalasr chalasr force-pushed the console-entity-resolver branch from f6f5465 to a634897 Compare June 30, 2026 10:28
@chalasr

chalasr commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

Retargeted to 3.3.x and rebased. Reviews welcome 🙏

@W0rma

W0rma commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

I tried the changes in this PR in one of my project and it worked like a charm 👍

Comment thread src/DependencyInjection/DoctrineExtension.php Outdated
@chalasr chalasr force-pushed the console-entity-resolver branch from a634897 to 63ac737 Compare July 5, 2026 11:41
@greg0ire greg0ire requested review from derrabus and xabbuh July 5, 2026 16:13
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.

6 participants