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