Correct help link for the new versions of KF5 (>52)
ClosedPublic

Authored by yurchor on Jan 3 2019, 10:22 PM.

Details

Summary

Due to me "broken" KHelpClient in KF 5.52, links in the help menu show error messages now.

BUG: 402837

Test Plan
  1. Start KPat.
  2. Select Freecell.
  3. On the menubar, select "Help -> Help with Freecell".
  4. Help for Freecell should be shown.

Diff Detail

Repository
R410 KPatience
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
yurchor created this revision.Jan 3 2019, 10:22 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald TranscriptJan 3 2019, 10:22 PM
yurchor requested review of this revision.Jan 3 2019, 10:22 PM

Uh, does the issue happen only with KF 5.52?

Uh, does the issue happen only with KF 5.52?

Should be so (it is not tested). Sorry. When I submitted that patch I had tested it for KFind, Kalzium, and KmPlot only. :(

Ok, so does the patched code work also with KF != 5.52? If only 5.52 is bugged and the code is going to work properly with all the other past and future versions, maybe we can just say that 5.52 is broken and be done with that.
(also, could you please point to the issue in 5.52 that was fixed later?)

yurchor added a comment.EditedJan 5 2019, 7:48 AM

Ok, so does the patched code work also with KF != 5.52? If only 5.52 is bugged and the code is going to work properly with all the other past and future versions, maybe we can just say that 5.52 is broken and be done with that.

Yes, it works (or does not work to the same extent as earlier) for versions KF < 5.52 (tested).

(also, could you please point to the issue in 5.52 that was fixed later?)

There is no bug in 5.52 (please read the bug report if you have some time, thanks).

https://bugs.kde.org/show_bug.cgi?id=402837

Long story short:

Before this patch:

  1. KF < 5.52, KPat <= 18.12 (tested on Mageia 6, Applications 17.12, KF 5.42):

"Help -> Help with <name of the game>" => Main page of the KPat help in KHelpCenter

  1. KF > 5.51, KPat <= 18.12 (tested on Mageia 7, Applications 18.12, KF 5.53):

"Help -> Help with <name of the game>" => Error message ("Could not find filename <name of the game>.html in /usr/share/doc/HTML/en//kpat/<name of the game>.html")

After this patch:

  1. KF < 5.52, KPat <= 18.12 (tested on Mageia 6, Applications 17.12, KF 5.42):

"Help -> Help with <name of the game>" => Main page of the KPat help in KHelpCenter

  1. KF > 5.51, KPat <= 18.12 (tested on Mageia 7, Applications 18.12, KF 5.53):

"Help -> Help with <name of the game>" => KHelpCenter opens the help section for the <name of the game>.

Thanks again for reading this.

ltoscano accepted this revision.Jan 5 2019, 11:18 PM

I confirm the behavior before and after the patch, for both KF < 5.52 and current KF (5.54).

Please commit the patch to the Applications/18.12 branch.

This revision is now accepted and ready to land.Jan 5 2019, 11:18 PM
This revision was automatically updated to reflect the committed changes.