Review Request 124746: [KDesktopFile] add readMimeType

[David Faure]

> On Aug. 18, 2015, 9:47 p.m., David Faure wrote:
> > I'm still very surprised by the use case. How are you getting the full path to an application .desktop file without using the KService framework? It sounds like parsing all application desktop files by hand, which sounds horrible ;-)   ... 
> > 
> > 
> > (and yes readMimeTypes() would be a better name if we end up deciding in favour of this; the desktop field is an historic misnaming, no point in propagating that to C++ developers)
> 
> Kai Uwe Broulik wrote:
>     It's for the icon widget (Review 124472) so I can check whether the app actually accepts the mime type I'm going to drop onto it. I already have the KDesktopFile anyway since its absolute path is stored in the applet's config fwict.

Ah OK. Possibly a special case indeed, but yeah I guess I'm thinking too much filedialog/filemanager while in your case, you're actually starting from app desktop file indeed.


- David


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


On Aug. 15, 2015, 5:38 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124746/
> -----------------------------------------------------------
> 
> (Updated Aug. 15, 2015, 5:38 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kconfig
> 
> 
> Description
> -------
> 
> This allows to get the MimeType field of a desktop file to determin whether eg. the given application would accept a given mime type.
> 
> I'm not too happy about the name because "readMimeType" suggests it's a single item but then it's consisteht with the key name.
> 
> 
> Diffs
> -----
> 
>   src/core/kdesktopfile.cpp 254b59a 
>   autotests/kdesktopfiletest.cpp 1a9c1c3 
>   src/core/kdesktopfile.h 2190051 
> 
> Diff: https://git.reviewboard.kde.org/r/124746/diff/
> 
> 
> Testing
> -------
> 
> Test passes
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150822/064ff78a/attachment-0001.html>