rkflx added a comment.

  This fix looks pretty much just like a patch I have lying around somewhere. 
Still I wonder whether we should monkey-patch all present and future users of 
`KFilePlacesModel` instead of fixing a single location? Last time I looked at 
it this was Baloo (e.g. `TimelineProtocol::listDir`, `parseTimelineUrl` etc., 
but I might be wrong on the connection to the `search:/` case).
  In D10245#199667 <https://phabricator.kde.org/D10245#199667>, @rkflx wrote:
  > When fixing the fallout from D8332 <https://phabricator.kde.org/D8332> I 
discovered what you are changing here will propagate to every app using 
`KUrlNavigator`, which is quite common. (Meanwhile I gave up on adding fixes 
like 50e6fa3ffc49 
everywhere, because I realized this has to be solved at Baloo level.)

  R241 KIO


To: broulik, #frameworks, elvisangelaccio
Cc: rkflx, michaelh

Reply via email to