- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
This needs an issue summary update.
What steps were taken?
What's the proposed solution"Think we should find what caused this issue as this seems to patching the symptom and not the cause.
This will need a test case as a bug.
- 🇩🇪Germany Anybody Porta Westfalica
Confirming the issue with Drupal 9.5.5 + PHP 8.1.17 when viewing a node.
Perhaps in Drupal 10.0.x this is already fixed and can be backported?
- Open on Drupal.org →Environment: PHP 8.1 & MySQL 5.7last update
about 2 years ago Not currently mergeable. - @slydevil opened merge request.
- Open on Drupal.org →Environment: PHP 8.1 & MySQL 5.7last update
about 2 years ago Not currently mergeable. - last update
about 2 years ago 28,504 pass - last update
about 2 years ago Patch Failed to Apply - last update
about 2 years ago 28,504 pass - 🇺🇸United States josephcheek Provo, UT
@slydevel made a patch for D10.1 that applies for me (whereas #15 does not). It took me a while to find it so here's the link, hoping to save others some time fixing this issue on D10.1.
https://git.drupalcode.org/issue/drupal-3308707/-/commit/acdedacad5af5af...
- last update
about 2 years ago 29,453 pass - Status changed to Needs review
about 2 years ago 1:09pm 28 July 2023 - 🇺🇦Ukraine dmytro-aragorn Lutsk
No need to call
rawurlencode()for non-existing or empty $path variable. - 🇬🇧United Kingdom longwave UK
We should write automated tests to cover this case so we don't cause a regression in the future.
Can someone who has experienced this error please write down the steps to reproduce it, preferably starting from "Install Drupal" if possible. We can use those instructions to write a test case.
- Status changed to Postponed: needs info
about 2 years ago 9:33pm 28 July 2023 - 🇧🇷Brazil fadonascimento
Thanks @josephcheek the patch #17 🐛 Deprecated function - Files: UrlGenerator.php and AliasPathProcessor Postponed: needs info works as expected in Druapl 10.1
- 🇧🇪Belgium xaa Brussels
I saw similar issue on a d10.2.0 (upgraded from 9.5.11) after adding URL alias (/admin/config/search/path/add). Patch #20 seems fixed the issue here.
trying to create alias:
- System path: /node
- URL alias: / - 🇬🇧United Kingdom sittard
We were are also getting this on Drupal 10.2.6 for us we had some paths which appeared to be missing the alias value. Not 100% sure why or how these empty alias existed, they were all taxonomy terms.
We were able to delete the paths via the /admin/config/search/path interface and solve the problem.
- 🇺🇸United States smustgrave
Just following up if anyone can provide steps to reproduce this issue? If no steps are provided probably will close out in 3 months.