Skip to content

Comments

Wire the Console EntityValueResolver as a service#2193

Open
chalasr wants to merge 1 commit intodoctrine:2.18.xfrom
chalasr:console-entity-resolver
Open

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

Conversation

@chalasr
Copy link
Contributor

@chalasr chalasr commented Jan 3, 2026

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
Copy link
Contributor Author

chalasr commented Feb 5, 2026

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

$this->assertEquals(new MapEntity(null, null, null, [], null, null, null, true, true), $container->get('controller_resolver_defaults'));
}

#[IgnoreDeprecations]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why ignoring deprecations ? This test is meant to cover a new feature, not a deprecated feature. It has no reason to ignore deprecations.

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.

2 participants