All users were logged out of Bugzilla on October 13th, 2018

wiki.subpages() include redirects, it should not

RESOLVED FIXED

Status

P1
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: trevorhemail-mozbug, Unassigned)

Tracking

Details

(Whiteboard: bug)

(Reporter)

Description

6 years ago
On this page https://developer.mozilla.org/en-US/docs/Mozilla , there are two list items for Mozilla Framework Based on Templates (MFBT), one links to https://developer.mozilla.org/en-US/docs/Mozilla/MFBT which is the actual page, the other links to https://developer.mozilla.org/en-US/docs/Mozilla/Mozilla_Framework_Based_on_Templates_(MFBT) which is a redirect.
As much as possible, let's try to fix wiki.tree and page.subpage stuff in the KumaScript templates so that #mdndev isn't blocking them.
(Reporter)

Comment 2

6 years ago
As far as I am aware it is not possible to tell in KumaScript if a page is a redirect.
Right, this should be done serverside

Comment 4

6 years ago
Need to add this fix to a sprint
Priority: -- → P1
Whiteboard: bug
Fixed with today's PR merge
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Comment 6

6 years ago
Unfortunately I am still seeing redirects in the list.
I don't know that this has hit production yet....
I'm still seeing redirects; see https://developer.mozilla.org/en-US/docs/Mozilla for example. Note that both .../MFBT and the longer name show up; one of those is a redirect to the other.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Also, the sitemap is including redirects, and should not be.
This is affecting my work on getting the Apps hierarchy, among others, prepared for MWC. It would be good to get it resolved very soon.
Marked it as ready in Kanbanery.
Yeah, so the reason this is happening is because the migration I created was run as a no-op forward because we were afraid running it could take down the site / bomb out.  We need to create a likewise raw SQL statement to do this conversion.
Fixed once we ran the migration.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.