326617 - Hidden "site-tools" footer should link to https://doctor.mozilla.com instead of doctor-test

Status: RESOLVED FIXED

(www.mozilla.org :: General, defect)

Description

[Reed Loden [:reed]]

Currently, all pages on www.mozilla.com have a link to http://doctor-test.mozilla.org hidden in the footer. The real doctor for the www.mozilla.com pages is https://doctor.mozilla.com, not doctor-test.

For reference sake, this footer is contained in mozilla-com/lib/wrapper.tmpl.

Comment 1

[Reed Loden [:reed]]

Basically, the link should be:
<li><a href="https://doctor.mozilla.com/doctor.cgi?action=edit&amp;file=[% sourcefile %]">[% strings.Edit %]</a></li>

Notice that this new version of doctor does not need the "mozilla-com/html/" at the front of the filename. If it is added (as it is now), it will not work. As an example, https://doctor.mozilla.com/doctor.cgi?action=edit&file=firefox/releases/1.5.0.1.html edits http://www.mozilla.com/firefox/releases/1.5.0.1.html.

Comment 2

[Reed Loden [:reed]]

(In reply to comment #1)
er, I meant "mozilla-com/src/" instead of "mozilla-com/html/".

Comment 3

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

(In reply to comment #1)
> Notice that this new version of doctor does not need the "mozilla-com/html/" at
> the front of the filename. If it is added (as it is now), it will not work. As

Looks OK to me with it.

Comment 4

[victory <[e-mail]>]

Attachment: fix what reed mentions

Comment 5

[Reed Loden [:reed]]

Comment on attachment 211348 [details] [diff] [review]
fix what reed mentions

As per bug 326341, https://doctor.mozilla.com should be used (https instead of http).

Comment 6

[Steven Garrity [:sgarrity]]

I've checked this in (with the https link). It might have to wait on a full site rebuild before you start to see it on the live site.

Comment 7

[Reed Loden [:reed]]

Comment on attachment 211348 [details] [diff] [review]
fix what reed mentions

As per steven's check-in of the patch (with http->https) [revision 1.19 of mozilla-com/lib/wrapper.tmpl].