Review Request 130013: Make PlainTextExtractor match "text/plain" mimetype
Igor Poboiko
igor.poboiko at gmail.com
Fri Mar 17 08:23:53 UTC 2017
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/130013/
-----------------------------------------------------------
(Updated March 17, 2017, 8:23 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Anthony Fieroni.
Changes
-------
Submitted with commit f9c8bb29b2cf5d46ff68c0261393084d1ff16ece by Igor Poboiko to branch master.
Repository: kfilemetadata
Description
-------
After commit 7c7e985a4678fef5f5d0dd8faa9b9cb42e3844b4 (see https://git.reviewboard.kde.org/r/129720/), PlainTextExtractor no longer matches ANY of the text/ mimetypes.
This broke completely Baloo indexing e.g. simple plain text files.
Introduced check however allows to provide "text/plain" as supported mimetype for the extractor and hope that everything containing plain text will be inherited from it.
Diffs
-----
autotests/CMakeLists.txt 5ab742b
autotests/extractorcollectiontest.cpp PRE-CREATION
src/externalextractor.cpp 05f0645
src/extractors/plaintextextractor.cpp 26e1247
Diff: https://git.reviewboard.kde.org/r/130013/diff/
Testing
-------
KFileMetaData compiles.
Baloo indexes plain text files.
Everybody is happy.
Thanks,
Igor Poboiko
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170317/b036f8f4/attachment.html>
More information about the Kde-frameworks-devel
mailing list