dfaure added a comment.
Yes, but then startsWith() would be even more correct (and slightly faster) than contains(). And application/vnd.oasis.opendocument.text isn't plain text, so contains("text") should also be changed to startsWith("text/"). REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D7948 To: ngraham, #frameworks, nicolasfella, dfaure, kossebau, vhanda Cc: #frameworks