- Status changed to Needs review
about 1 year ago 2:34pm 28 September 2023 - last update
about 1 year ago 89 pass, 12 fail The last submitted patch, 3: 3319509-3-fix_redirect.patch, failed testing. View results →
- last update
8 months ago 118 pass - 🇧🇾Belarus dewalt
With a patch in #3 there is an error if no destination query param is provided on Translations page
InvalidArgumentException: The internal path component 'node/179158/translations' is invalid. Its path component must have a leading slash, e.g. internal:/foo. in Drupal\Core\Url::fromInternalUri() (line 416 of core/lib/Drupal/Core/Url.php).
Patch below process path after UrlHelper::parse(), which supports both URIs with and without leading slash.
- last update
8 months ago 118 pass - 🇧🇾Belarus dewalt
One more case:
Having Drupal installed in subfolder, going from content overview page to translation page (URL looks like http://example.com/web/node/2/translations?destination=/web/admin/content), after submitting translation I go to 404 page (http://example.com/web/web/admin/content).
The next patch from the child solves this issue too.
- Assigned to Pavlos.Dontas
- 🇬🇷Greece Pavlos.Dontas
I'll be reviewing the patch from #8 🐛 Redirect to page 404 after requesting a translation Needs review as part of the GreeceSpringSprint2024 →
- Issue was unassigned.
- 🇬🇷Greece Pavlos.Dontas
After installing tmgmt on a fresh D10 installation and following the provided steps, I couldn't reproduce the problem. Step 2 is redundant since its already included in step 1.