[Nepomuk] Review Request: Use nie:hasPart instead of nie:hasLogicalPart in m3u analyser

[Sebastian Trueg]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102230/
-----------------------------------------------------------

Review request for Nepomuk and Strigi.


Summary
-------

The range of nie:hasLogicalPart is nie:InformationElement. As far as Nepomuk is concerned any local file is only a nfo:FileDataObject by itself. Thus, indexing m3u files fails with an invalid range error.
Now there are basically three ways to solve this:
1. Let the indexer add the nie:InformationElement type
2. Let Nepomuk assume every local file is a nie:InformationElement anyway (actually this is what the file indexer already does so we might think about doing it in the Data Management Service as well)
3. Use nie:hasPart in the analyser.
The latter is what this patch does. But actually I would rather like to think this through before pushing this. I cannot wrap my head around it: does the m3u file refer to the files on disk or the information elements inside the files?


Diffs
-----

  lib/lineanalyzers/m3ustreamanalyzer.cpp dacc15a 

Diff: http://git.reviewboard.kde.org/r/102230/diff


Testing
-------


Thanks,

Sebastian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20110805/8f2e808c/attachment.html>