<table><tr><td style="">yurchor added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D17953">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17953#386638" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D17953#386638</a>, <a href="https://phabricator.kde.org/p/ltoscano/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ltoscano</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>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.</p></div>
</blockquote>
<p>Yes, it works (or does not work to the same extent as earlier) for versions KF < 5.52 (tested).</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>(also, could you please point to the issue in 5.52 that was fixed later?)</p></blockquote>
<p>There is no bug in 5.52 (please read the bug report if you have some time, thanks).</p>
<p><a href="https://bugs.kde.org/show_bug.cgi?id=402837" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=402837</a></p>
<p>Long story short:</p>
<p>Before this patch:</p>
<ol class="remarkup-list">
<li class="remarkup-list-item">KF < 5.52, KPat <= 18.12 (tested on Mageia 6, Applications 17.12, KF 5.42):</li>
</ol>
<p>"Help -> Help with <name of the game>" => Main page of the KPat help in KHelpCenter</p>
<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">KF > 5.52, KPat <= 18.12 (tested on Mageia 7, Applications 18.12, KF 5.53):</li>
</ol>
<p>"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")</p>
<p>After this patch:</p>
<ol class="remarkup-list">
<li class="remarkup-list-item">KF < 5.52, KPat <= 18.12 (tested on Mageia 6, Applications 17.12, KF 5.42):</li>
</ol>
<p>"Help -> Help with <name of the game>" => Main page of the KPat help in KHelpCenter</p>
<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">KF > 5.52, KPat <= 18.12 (tested on Mageia 7, Applications 18.12, KF 5.53):</li>
</ol>
<p>"Help -> Help with <name of the game>" => KHelpCenter opens the help section for the <name of the game>.</p>
<p>Thanks again for reading this.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R410 KPatience</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17953">https://phabricator.kde.org/D17953</a></div></div><br /><div><strong>To: </strong>yurchor, KDE Games, aacid<br /><strong>Cc: </strong>ltoscano, kde-games-devel<br /></div>