D10694: epubextractor: Handle multiple subjects better

[Matthieu Gallien]

mgallien added a comment.


  In D10694#210409 <https://phabricator.kde.org/D10694#210409>, @michaelh wrote:
  
  > @mgallien Then they won't benefit from this until a file is reindexed. 
  >  Users with baloo disabled will benefit, because in that case `baloo_filemetadata_temp_extractor` will extract the data on-the-fly.
  >  Reindexing just the ebooks can be done with:
  >
  >   find ~/EBooks/ -type f -name "*.epub" | { mapfile -t; balooctl clear "${MAPFILE[@]}";  balooctl index "${MAPFILE[@]}"; }
  >  
  >
  
  
  Could you also add a test for the compatibility with current unmodified code ? I would like to be sure that we do not introduce any incompatibility.
  
  I am still not convinced we should change the extractors without an automatic solution for the reindexing case. You will have to ensure everybody can use the property both as a string and a stringlist forever.

REPOSITORY
  R286 KFileMetaData

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

To: michaelh, mgallien, dfaure
Cc: #frameworks, ashaposhnikov, michaelh, spoorun, navarromorales, isidorov, nicolasfella, firef, andrebarros, alexeymin, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180220/c6ad4a1a/attachment-0001.html>