- Issue created by @duaelfr
- Merge request !11793Issue #3518513: BulkForm fails to redirect to the expected page if destination... → (Open) created by duaelfr
- 🇫🇷France duaelfr Montpellier, France
Fix and tests pushed in a MR.
Patch attached for composer. Applies on both 11.x and 10.4.x. - 🇮🇳India shivam_tiwari
Need to work here as pipeline is failing related to test coverage.
- 🇳🇱Netherlands Lendude Amsterdam
Since something somewhere would have added that destination to the URL, should we expect this to be passed on the the redirect? Might be out of scope a bit, since ignoring it is certainly better than what happens now.
- 🇫🇷France duaelfr Montpellier, France
@lendude destination is kept after the redirect. I believe it's been made that way in the original issue.
- 🇳🇱Netherlands Lendude Amsterdam
Ok that sounds good then :)
The fails in Drupal\Tests\workspaces\Functional\WorkspaceViewsBulkForm feel like related failures, so moving back to needs work to check those
- 🇫🇷France duaelfr Montpellier, France
I think I found the issue with my test coverage! Crossing fingers ;)
- 🇺🇸United States smustgrave
Seems to be impacting workspaces, seems relevant to the change I believe
- 🇫🇷France duaelfr Montpellier, France
@lendude @smustgrave I'm out of ideas to fix the test. I cannot reproduce this failure locally. Do you have some clues for me?
$ phpunit -c phpunit.xml core/modules/views/tests/src/Functional/BulkFormTest.php PHPUnit 10.5.45 by Sebastian Bergmann and contributors. Runtime: PHP 8.3.15 Configuration: /app/phpunit.xml .. 2 / 2 (100%) HTML output was generated. [...] Time: 00:18.124, Memory: 10.00 MB OK (2 tests, 79 assertions)$ phpunit -c phpunit.xml core/modules/workspaces/tests/src/Functional/WorkspaceViewsBulkFormTest.php PHPUnit 10.5.45 by Sebastian Bergmann and contributors. Runtime: PHP 8.3.15 Configuration: /app/phpunit.xml .. 2 / 2 (100%) HTML output was generated. [...] Time: 00:56.580, Memory: 10.00 MB OK (2 tests, 97 assertions)