Review Request 127912: Add 2 methods to gain more info about a mimetype or protocol

Martin Kostolný clearmartin at zoho.com
Mon May 23 04:49:03 UTC 2016


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

(Updated May 23, 2016, 4:49 a.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks, Krusader, David Edmundson, David Faure, and Marco Martin.


Repository: kio


Description
-------

KProtocolManager has a method called protocolForArchiveMimetype. It gives one protocol for given mimetype even if there exist more then one protocol capable of handling this mimetype.

I suggest adding a method called protocolsForArchiveMimetype (note the "s") which would give us all protocols that can handle given mimetype. I also suggest to add a method for giving all archive mimetypes for given protocol.

This request is based on a potential need of some apps to prioritize one protocol over others (specifically krarc:/ from Krusader over other archive-handling protocols).


Diffs
-----

  src/core/kprotocolmanager.h 13b8c07 
  src/core/kprotocolmanager.cpp 9a0a96f 

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


Testing
-------

I've tested both methods through Krusader code. Partially covered in this phabricator task: https://phabricator.kde.org/T2401


Thanks,

Martin Kostolný

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160523/afd4d369/attachment.html>


More information about the Kde-frameworks-devel mailing list