<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: dfaure, Frameworks.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />ngraham requested review of this revision.
</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/D18778">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>If <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">System Settings</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Applications</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Default Applications</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Web Browser</span></span></span> is using the setting "In an application based on the contents of the URL", then lookup can fail and pass the URL to the wrong app when no default app is set in <tt style="background: #ebebeb; font-size: 13px;">kdeglobals</tt> and <tt style="background: #ebebeb; font-size: 13px;">mimeapps.list</tt> has no <tt style="background: #ebebeb; font-size: 13px;">x-scheme-handler/https</tt> or <tt style="background: #ebebeb; font-size: 13px;">x-scheme-handler/http</tt> entries, but does have an entry for <tt style="background: #ebebeb; font-size: 13px;">text/html</tt>.</p>

<p>This patch makes KRun additionally fall back to reading from <tt style="background: #ebebeb; font-size: 13px;">text/html</tt> in <tt style="background: #ebebeb; font-size: 13px;">mimeapps.list</tt> if it's present.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Can now open URLs when using "In an application based on the contents of the URL when <tt style="background: #ebebeb; font-size: 13px;">mimeapps.list</tt> only has an entry for <tt style="background: #ebebeb; font-size: 13px;">text/html</tt>.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>more-mimeapps-fallbacks (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18778">https://phabricator.kde.org/D18778</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/widgets/krun.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, dfaure, Frameworks<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>