Should UrlHelper::parse() unify the leading slash for local URLs?

Created on 21 May 2024, 7 months ago
Updated 17 July 2024, 5 months ago

Problem/Motivation

I just ran into a case in a redirect contrib module: πŸ› "Delete redirects defined in the spreadsheet" option without functionality? Active where UrlHelper::parse() is used.

Drupal 8+ expects leading slashes for internal paths, but currently that's not represented or unified in that helper method.

So currently the output for
/node/123
is
/node/123

and for
node/123
is
node/123

and I think it makes sense to discuss if this internal Drupal helper method shouldn't better returned the unified Drupal path /node/123 always with the leading slash, despite the input!

In this example it leads to two different check sums for the same path in Drupal.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ› Bug report
Status

Active

Version

11.0 πŸ”₯

Component
BaseΒ  β†’

Last updated about 12 hours ago

Created by

πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024