<table><tr><td style="">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/D16579">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/D16579#416446" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16579#416446</a>, <a href="https://phabricator.kde.org/p/smithjd/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@smithjd</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I realise that with all the fields referenced here having at least one standard fieldname parsed now that complaining may seem to be flogging a dead horse, but I'd like to re-iterate that identically-purposed fields should not be encouraged. This is at best a specialized re-purposing that really shouldn't be. Maintaining a branch / fork for this is probably appropriate.</p>

<p>Lastly, I'd also like to point out the TagLib documentation, that specifically mentions a conversion from 'Album Artist' in 'AlbumArtist' for compatibility with other tags: <a href="https://taglib.org/api/classTagLib_1_1APE_1_1Tag.html#af77a10659fbb0018228420ad6de501e1" class="remarkup-link" target="_blank" rel="noreferrer">https://taglib.org/api/classTagLib_1_1APE_1_1Tag.html#af77a10659fbb0018228420ad6de501e1</a>. 'Disc' is also converted from APE, but you have to dig into the code to find this: <a href="https://github.com/taglib/taglib/blob/master/taglib/ape/apetag.cpp#L211" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/taglib/taglib/blob/master/taglib/ape/apetag.cpp#L211</a></p></div>
</blockquote>

<p>I just double-checked, TagLib allows both. It converts the "Disc" field to the "Disc Number" in the property map, but also successfully reads from "Disc Number" field directly.<br />
Anyway, most of this special handling of mimetypes goes away with <a href="https://phabricator.kde.org/D18826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18826</a>, where TagLib will do the conversion automatically.</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/D16579">https://phabricator.kde.org/D16579</a></div></div><br /><div><strong>To: </strong>smithjd, astippich, bruns, mgallien<br /><strong>Cc: </strong>bruns, astippich, kde-frameworks-devel, Baloo, domson, ashaposhnikov, michaelh, spoorun, ngraham, abrahams<br /></div>