Fix CurrentPathStack::getPath() documentation that says it has no leading / (and make it easier to actually get that?)

Created on 21 February 2015, over 9 years ago
Updated 31 January 2023, over 1 year ago

Problem/Motivation

I'm a bit confused by #2372507-74: Remove _system_path from $request->attributes β†’ .4 (leading /) and the changes based on that. At least, we need to fix the documentation in CurrentPathStack::getPath() because that still says it returns it without a leading /, and given that it is 50/50 when we need a leading / it, maybe there should be an easier way to get the path without leading /?

IMHO, leading / was always a good way to differentiate between a url string (which is yet another thing, as commented above, it also includes the subfolder you're in) and the path. We need to better explain who should be using it and when, the current explanation there reads like an excuse for it's existence :)

Proposed resolution

Remaining tasks

User interface changes

API changes

πŸ› Bug report
Status

Needs work

Version

9.5

Component
Request processingΒ  β†’

Last updated about 10 hours ago

No maintainer
Created by

πŸ‡¨πŸ‡­Switzerland Berdir Switzerland

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024