Closed
Bug 1497077
Opened 6 years ago
Closed 6 years ago
Convert links, image/iframe sources, form actions to absolute path
Categories
(bugzilla.mozilla.org :: General, task)
Tracking
()
RESOLVED
FIXED
People
(Reporter: kohei, Assigned: kohei)
References
(Blocks 2 open bugs)
Details
Attachments
(3 files)
Before adopting pretty URLs like /bug/ID, /user/ID or whatever, the current links with a relative path like `query.cgi` have to be converted to an absolute path like `/query.cgi`.
Assignee | ||
Comment 1•6 years ago
|
||
But some installations are not on the root, e.g. https://www.w3.org/Bugs/Public/ so it has to be `[% urlbase FILTER html %]query.cgi` ideally excluding the origin part.
Assignee | ||
Comment 2•6 years ago
|
||
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Merged to master.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
Follow-up PRs also merged to master. Let’s move on to Bug 1468848.
Assignee | ||
Comment 8•6 years ago
|
||
I have found that some redirects are relative URLs, not prefixed with `urlbase` (now those could be mostly replaced with `basepath`). It’s difficult to find all the relative stuff in the complex codebase, but I’ll try to scan further.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Updated•6 years ago
|
Status: REOPENED → ASSIGNED
Summary: Convert links to absolute path → Convert links, redirects, image/iframe sources, form actions to absolute path
Assignee | ||
Comment 9•6 years ago
|
||
I’ll fix redirects in a separate bug.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Summary: Convert links, redirects, image/iframe sources, form actions to absolute path → Convert links, image/iframe sources, form actions to absolute path
Assignee | ||
Updated•6 years ago
|
Type: enhancement → task
Assignee | ||
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•