<html>

 <body>

  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">

   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">

    <tr>

     <td>

      This is an automatically generated e-mail. To reply, visit:

      <a href="http://git.reviewboard.kde.org/r/102230/">http://git.reviewboard.kde.org/r/102230/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">1. Let the indexer add the nie:InformationElement type

I'm not too sure.

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)

No. Every file is not supposed to be a nie:InformationElement. AFAIK that was added a hack for 4.7 so that we don't break too many things.

3. Use nie:hasPart in the analyser.

I'm not too sure how to go about this. It seems as though nie:hasLogicalPart should accept nfo:DataObjects as well, cause the m3u file is just a text file that contains paths to other files. So, it "logically" contains them, not physically ( as in the case of Messages and attachments )</pre>

 <br />

<p>- Vishesh</p>

<br />

<p>On August 5th, 2011, 8:53 p.m., Sebastian Trueg wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for Nepomuk and Strigi.</div>

<div>By Sebastian Trueg.</div>

<p style="color: grey;"><i>Updated Aug. 5, 2011, 8:53 p.m.</i></p>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>

<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">

 <tr>

  <td>

   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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?</pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>

<ul style="margin-left: 3em; padding-left: 0;">

 <li>lib/lineanalyzers/m3ustreamanalyzer.cpp <span style="color: grey">(dacc15a)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/102230/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>