-----------------------------------------------------------
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ý

_______________________________________________
Kde-frameworks-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to