<table><tr><td style="">astippich planned changes to this revision.<br />astippich added a comment.
</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/D11365">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11365#236274" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11365#236274</a>, <a href="https://phabricator.kde.org/p/michaelh/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@michaelh</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>This patch should be split.</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Test more properties</li>
<li class="remarkup-list-item">Change return types of ...
<br /><br />
Also 'fix errors' in the title is misleading because currently kfilemetadata works well.</li>
</ol></div>
</blockquote>

<p>Sure, I can do that if it is not a problem that the new tests do not pass (temporarily). It is probably the best idea to create thorough tests for the way we'd like KFileMetaData/taglib to work, as they are lacking in several ways (and hence created the confusion I have had) . Then we can start fixing the errors.<br />
Can we reach a consent how it should behave in the end? e.g. should the result match the valueType in propertyinfo, should there be multiple properties with single strings or stringlists for multiple values? Any other concerns?<br />
<a href="https://phabricator.kde.org/p/mgallien/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mgallien</a>, what's your opinion here?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11365">https://phabricator.kde.org/D11365</a></div></div><br /><div><strong>To: </strong>astippich, Frameworks, Baloo, mgallien, michaelh<br /><strong>Cc: </strong>michaelh, Frameworks, ashaposhnikov, astippich, spoorun, nicolasfella, ngraham, alexeymin<br /></div>