Review Request 123779: Include the mimetype itself when querying for KFileItemActions::associatedApplications

Martin Klapetek martin.klapetek at gmail.com
Wed May 13 17:05:18 UTC 2015


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

(Updated May 13, 2015, 5:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 37590e2d2bd8965dea13876912afebc2fcf9f565 by Martin Klapetek to branch master.


Bugs: 340326
    http://bugs.kde.org/show_bug.cgi?id=340326


Repository: plasma-framework


Description
-------

This gets the current mimetype _and_ a list of its aliases, because aliases()
do not return the current mimetype in it, so it needs to be prepended,
especially for the cases where there are no aliases to be returned,
this would then just return an empty list, returning the generic action even
when there is a valid mimetype and an associated app.

This fixes bug 340326.


Diffs
-----

  src/plasma/private/associatedapplicationmanager.cpp 01dd319 

Diff: https://git.reviewboard.kde.org/r/123779/diff/


Testing
-------

Folderview now has proper context menu action.


Thanks,

Martin Klapetek

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


More information about the Plasma-devel mailing list