Review Request: NepomukRunner: Redirect the 'nepomuk:/res/uuid' urls before opening them

Vishesh Handa me at vhanda.in
Wed Jan 9 18:01:00 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108305/
-----------------------------------------------------------

Review request for Dolphin, Nepomuk and Plasma.


Description
-------

    NepomukRunner: Redirect the 'nepomuk:/res/uuid' urls before opening them
    
    The nepomuksearch kioslave has a special syntax which results in the the
    searched term being displayed in "human terms" without the need for
    stating the url. The nepomuk kioslave does not have this feature. It,
    however, uses the nepomuksearch kioslave internally.
    
    Applications like Dolphin often do not stat a url before displaying it
    in the title bar. This results in users seing an ugly "nepomuk - res -
    uuid" in the title bar instead of the proper display name as set in the
    UDSEntry.
    
    In order to avoid this ugly title bar, we manually redirect all nepomuk
    urls to nepomuksearch before they are opened by the runner.
    
    It's a hack, but it works!
    
    BUG: 287259

@Dolphin people: I just thought you might want to know that this is something that will have to be fixed.


This addresses bug 287259.
    http://bugs.kde.org/show_bug.cgi?id=287259


Diffs
-----

  plasma/generic/runners/nepomuksearch/nepomuksearchrunner.cpp 8075a6b 

Diff: http://git.reviewboard.kde.org/r/108305/diff/


Testing
-------

Yup


Thanks,

Vishesh Handa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130109/ec7e15d6/attachment.html>


More information about the Plasma-devel mailing list