Closed Bug 1346622 Opened 7 years ago Closed 4 years ago

welcome_help.xhtml: wrong hyperlink to SM 2.1 Release Notes

Categories

(SeaMonkey :: Help Documentation, defect)

SeaMonkey 2.46 Branch
Unspecified
All
defect
Not set
normal

Tracking

(seamonkey2.53+ fixed, seamonkey2.57esr? affected)

RESOLVED FIXED
seamonkey 2.74
Tracking Status
seamonkey2.53 + fixed
seamonkey2.57esr ? affected

People

(Reporter: RainerBielefeldNG, Assigned: njsg)

References

Details

(Whiteboard: SM2.53.3)

Attachments

(3 files, 2 obsolete files)

Steps how to reproduce with official  SeaMonkey 2.46 FINAL  (Windows NT 6.1; WOW64; rv:49.0)  Gecko/20100101 Firefox/49.0 Build 20161213183751  (Default Classic Theme)  on German WIN7 64bit:

1. Launch Browser
2. <f1> for Help
   » WELCOME Help page appears
3. Click "Troubleshooting" heading with link
   Bug: opens <http://www.seamonkey-project.org/releases/seamonkey2.1/#troubleshooting> although <http://www.seamonkey-project.org/releases/seamonkey2.46/#troubleshooting> does exist

a) Probably bug is much older than 2.46
b) I do not know how to find source at <https://dxr.mozilla.org/seamonkey/source/>. It's not <https://dxr.mozilla.org/seamonkey/source/suite/locales/en-US/chrome/common/help/welcome_help.xhtml>, there we have a link to 
   <mozilla.org/start/1.5/troubleshooting/> 
   Can someone leave a general hint in the wiki?
c) Source can be found at <https://hg.mozilla.org/>
e) Still REPRODUCIBLE with unzipped installer of  official  en-US SeaMonkey 2.52a1
   (NT 6.1; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0 Build 20170311002651  
   (Default Classic Theme) on German WIN7 64bit
The "Troubleshooting" heading with link actually opens <http://www.seamonkey-project.org/doc/troubleshooting>, which then immediately redirects to <http://www.seamonkey-project.org/releases/seamonkey2.1/#troubleshooting>. The same behavior occurs in any browser when you visit that link.
There isn't a /doc/troubleshooting/ file in <https://hg.mozilla.org/SeaMonkey/seamonkey-project-org/>, and there isn't a link for the page in the sitemap. 

<http://www.seamonkey-project.org/doc/troubleshooting> could be changed to redirect to the most recent version's troubleshooting page, or the "Troubleshooting" heading link could be changed to that of the individual release, but at this point I'm not sure how to implement either of those potential solutions.
Still REPRODUCIBLE with unzipped installer of  UNofficial (by FRG) en-US SeaMonkey 2.52a1 (NT 6.1; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0 Build 20170529113726  (Default Classic Theme) on German WIN7 64bit

NEW due to <https://unofficialseamonkeynews.wordpress.com/2016/08/04/my-unconfirmed-bug-reports/>
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: good-first-bug
Whiteboard: [easyconfirm][good first bug] → [easyconfirm]

The /doc/troubleshooting redirect seems to be defined in src/.htaccess,
https://hg.mozilla.org/SeaMonkey/seamonkey-project-org/file/tip/src/.htaccess#l25

While the redirect itself can be updated, this patch tries to add a separate variable so that the troubleshooting link can work like the release notes one.

And a patch to update the server-side redirect to point to 2.53.1.

Attachment #9145245 - Flags: feedback?
Attachment #9145245 - Flags: feedback? → feedback?(frgrahl)
Attachment #9145241 - Flags: feedback?(iann_bugzilla)
Attachment #9145245 - Flags: feedback?(iann_bugzilla)
Blocks: 1634924
See Also: → 1634924

Reassign due to facts

Assignee: nobody → nunojsg
Status: NEW → ASSIGNED

"The same" as attachment 9145241 [details] [diff] [review] but for l10n packages.

If nothing breaks, this would also solve the problem with outdated "troubleshooting" links that are present in some packages.

Comment on attachment 9145245 [details] [diff] [review]
website src/.htaccess patch to update the redirect [Checked in: Comment 12]

This corrects this issue but I think we need more work on .htaccess as we need to redirect to correct protocol, so if it is http go to http and https to https. Most redirects seem to put everything to http, but this work will need to be in a new bug rather than polluting this one.
Attachment #9145245 - Flags: feedback?(iann_bugzilla) → feedback+
Comment on attachment 9145241 [details] [diff] [review]
Adds a variable for the troubleshooting URL

One more change is needed, if you look at what is in place for app.releaseNotesURL in suite-prefs.js and do the same for app.troubleshootingURL
https://dxr.mozilla.org/comm-esr60/source/suite/app/profile/suite-prefs.js#489

f=me
Attachment #9145241 - Flags: feedback?(iann_bugzilla) → feedback+
Comment on attachment 9145245 [details] [diff] [review]
website src/.htaccess patch to update the redirect [Checked in: Comment 12]

># HG changeset patch
># User Nuno Silva (njsg)
You will need to update your username for the website repo.
Comment on attachment 9145245 [details] [diff] [review]
website src/.htaccess patch to update the redirect [Checked in: Comment 12]

https://hg.mozilla.org/SeaMonkey/seamonkey-project-org/rev/b36c9390cbd915d17294a8968b92ee7f2e7a9144
https://gitlab.com/seamonkey-project/website/-/commit/e045196c8e74c6cb6ad8a908946c52bb2335bf8c
Attachment #9145245 - Attachment description: website src/.htaccess patch to update the redirect → website src/.htaccess patch to update the redirect [Checked in: Comment 12]
Attachment #9145241 - Attachment is obsolete: true

(Adds app.troubleshootingURL to suite-prefs.js)

Attachment #9145298 - Attachment is obsolete: true
Attachment #9145299 - Flags: feedback?(iann_bugzilla)
Comment on attachment 9145299 [details] [diff] [review]
Adds a variable for the troubleshooting URL, v1.1

[Triage Comment]
LGTM r/a=me

You do need to increase the diff context to 8 for patches, either in the default (.hgrc) or on each repo's .hg/hgrc
Attachment #9145299 - Flags: review+
Attachment #9145299 - Flags: feedback?(iann_bugzilla)
Attachment #9145299 - Flags: approval-comm-release+
Attachment #9145299 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/43f9f426e32a
Add a preference for the "troubleshooting" URL and use it in the help. r=IanN

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Comment on attachment 9145245 [details] [diff] [review]
website src/.htaccess patch to update the redirect [Checked in: Comment 12]

Should be 2.53.2 now :)
Attachment #9145245 - Flags: feedback?(frgrahl) → feedback+
Comment on attachment 9145271 [details] [diff] [review]
Troubleshooting URL patch for l10n

[Triage Comment]
r/a=me
Attachment #9145271 - Flags: review+
Attachment #9145271 - Flags: approval-comm-release+
Attachment #9145271 - Flags: approval-comm-esr60+
Blocks: 1641354
Keywords: good-first-bug
Whiteboard: [easyconfirm] → SM2.53.3
You need to log in before you can comment on or make changes to this bug.