<table><tr><td style="">aleksejshilin retitled this revision from "Prepare for KIO KRecentDocument fix (D10863)" to "Don't skip certain recent documents in kicker and taskmanager".<br />aleksejshilin edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5vaquyewr556w3s/" rel="noreferrer">(Show Details)</a><br />aleksejshilin edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-hzlur6mf5n7pxvs/" rel="noreferrer">(Show Details)</a>
</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/D10890" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Revision D10863 introduced a fix which makes the application stored</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Due to KRecentDocument::add() bug, application name '<app>' could</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">in</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">be stored for</span> recent documents <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">data match the application'</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">instead of it</span>s desktop <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">file. This</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">entry name</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">made all workarounds with cutting </span>'org.kde.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">' prefix out of storage</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><app>'. This issue mostly affects items opened from apps</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">IDs irrelevant.<br />
<br />
However</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">themselves, while items opened from e.g. Dolphin are not affected.<br />
(The bug is fixed by D10863</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">given that release schedules of Plasma Desktop and Frameworks</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but we depend on Frameworks 5.42 which</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">are independent, we can't be sure that the Frameworks version we are<br />
running on does contain the mentioned fix</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">doesn't include it.)<br />
<br />
Kicker and taskmanager used to query for '<app>' agent only</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">so we need to support both</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">thus</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">cases until we depend on the Frameworks version which does</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">excluding items with correct 'org.kde.<app>' agent</span>.<br />
<br />
This commit makes kicker and taskmanager query for both <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><app> (pre-<br />
D10863) and org.kde.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'</span><app><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> (post-D10863) agents. As a bonus, this also</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">' and</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">fixes a bug when items opened from e.g. Dolphin don't show up in recent</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'org.kde.<app>' agents in order to get all recent documents for the</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">documents in kicker's and taskmanager's </span>applicati<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">on context menus</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">on</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">  # Create 'test1' and 'test2' text files.<br />
  # Open 'test1' in Kate from Dolphin.<br />
  # Open 'test2' from Kate itself.<br />
  # Right-click Kate in Kicker/Kickoff and Task Manager and check that both files are present there.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10890" rel="noreferrer">https://phabricator.kde.org/D10890</a></div></div><br /><div><strong>To: </strong>aleksejshilin, Plasma, broulik, hein<br /><strong>Cc: </strong>anthonyfieroni, ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>