D16197: provide a list of supported mimetypes for embeddedimagedata

[Stefan Brüns]

bruns requested changes to this revision.
bruns added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> embeddedimagedatatest.cpp:48
> +    testAudioFile = testFilePath("test.opus");
> +    QCOMPARE(imageData.readMimetypes().contains(mimeDb.mimeTypeForFile(testAudioFile).name()),true);
> +    images = imageData.imageData(testAudioFile);

QVERIFY(condition)

> embeddedimagedata.cpp:73
>  
> +QStringList EmbeddedImageData::readMimetypes() const
> +{

`supportedMimeTypes()` ?
capital `T`, see `ImageType` and `QMimeType`

REPOSITORY
  R286 KFileMetaData

REVISION DETAIL
  https://phabricator.kde.org/D16197

To: astippich, bruns, mgallien
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181016/d2150480/attachment-0001.html>