issue 1022. fix missing _route on live components batch requests

[arrabiata-asanz]

Q	A
Bug fix?	yes
New feature?	no
Docs?	no
Issues	Fix #1022
License	MIT

The _route param get overwritten for Live Component Batch Requests, persist this param to the sub requests

Symfony\UX\LiveComponent\Controller\BatchActionController

$request->duplicate(attributes: [
                '_controller' => [$serviceId, $name],
                '_component_action_args' => $action['args'] ?? [],
                '_mounted_component' => $_mounted_component,
                '_live_component' => $serviceId,
            ]

$response = $this->kernel->handle($subRequest, HttpKernelInterface::SUB_REQUEST, false);

Duplicate overwrites all attributes in the sub request, missing one important attribute _route, add it to the array in order to know where the request comes from.

[arrabiata-asanz]

i think the failed tests don't apply to the changes in this PR. (Deprecation warning and Exception Messages Formatting)

[Kocal]

Thanks for the PR! I'm totally not familiar with LiveComponent, but:

1. can we add a test preventing any regression?
2. I think we should also pass _route_params, no?

[Kocal]

i think the failed tests don't apply to the changes in this PR. (Deprecation warning and Exception Messages Formatting)

Yes you can ignore the 8.4 dev job, but for fabbot, it would be nice to fix it (even if not related :) )

[smnandre]

Hmm.. could we also add some tests for this?

[Copilot]

Pull Request Overview

This PR fixes the issue where the _route attribute was omitted in live component batch requests, ensuring that it is preserved during sub-request duplication.

• Added the missing '_route' attribute in the request duplication for batch processing.

Comments suppressed due to low confidence (1)

src/LiveComponent/src/Controller/BatchActionController.php:41

• Consider verifying that '_route' is not null or documenting the behavior when it is absent to avoid potential issues in downstream sub-request handling.

'_route' => $request->attributes->get('_route'),