<table><tr><td style="">bruns added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10694">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10694#inline-62627">View Inline</a><span style="color: #4b4d51; font-weight: bold;">michaelh</span> wrote in <span style="color: #4b4d51; font-weight: bold;">epubextractor.cpp:85</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I think we should port away from libepub. Multiple titles result in one ';'-joined string.<br />
Also it seems to be unmaintained.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The joined titles is the fault of this epubextractor AFAICS - see fetchMetadataString</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10694#inline-62628">View Inline</a><span style="color: #4b4d51; font-weight: bold;">michaelh</span> wrote in <span style="color: #4b4d51; font-weight: bold;">epubextractor.cpp:97</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Right, this inconsistency is intentional, and it needs discussion. That's why I added a comment in <a href="https://phabricator.kde.org/D12197" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12197</a> which was probably overlooked.<br />
DC and IDPF aren't very clear on how to use <tt style="background: #ebebeb; font-size: 13px;">dc:subject</tt>. Calibre interprets it as tags, My impression is, that most provider also do. Hence I prefer to use <tt style="background: #ebebeb; font-size: 13px;">Property::Keywords</tt> only because it comes closest imo. That change would not really be breaking as currently <tt style="background: #ebebeb; font-size: 13px;">Property::Subject</tt> is one large string joined with ';'.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Distinction between Subject and Keywords typically is keywords are just a bunch of words without further specification, while subject, as specified by DC, and as used by e.g. libraries, are taken from filed specific catalogs.</p>

<p style="padding: 0; margin: 8px;">Baloos properties documentation specifically mentions dc:subject for Properties:Subject.</p>

<p style="padding: 0; margin: 8px;">One of the file formats which has both, keywords and subject, is ODF, which uses dc:subject and meta:keywords.</p>

<p style="padding: 0; margin: 8px;">DC specifies for <em>any</em> property:</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">Recommendation 5. Multiple property values should be encoded by repeating the XML element for that property.</p></blockquote>

<p style="padding: 0; margin: 8px;">My opinion is to <strong>always</strong> use Properties::Subject for dc:subject (as documented for baloo), and add each property instance individually. If properties are already messed up in the originating document, there is nothing we can do, but we should not make things worse.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>BRANCH</strong><div><div>multi-subject</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10694">https://phabricator.kde.org/D10694</a></div></div><br /><div><strong>To: </strong>michaelh, mgallien, dfaure<br /><strong>Cc: </strong>bruns, astippich, Frameworks, ashaposhnikov, michaelh, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp<br /></div>