D16579: Remove support for non-standard APE tag field names
    Alexander Stippich 
    noreply at phabricator.kde.org
       
    Sat Feb 23 15:26:26 GMT 2019
    
    
  
astippich added a comment.
  In D16579#416446 <https://phabricator.kde.org/D16579#416446>, @smithjd wrote:
  
  > 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.
  >
  > 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: https://taglib.org/api/classTagLib_1_1APE_1_1Tag.html#af77a10659fbb0018228420ad6de501e1. 'Disc' is also converted from APE, but you have to dig into the code to find this: https://github.com/taglib/taglib/blob/master/taglib/ape/apetag.cpp#L211
  
  
  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.
  Anyway, most of this special handling of mimetypes goes away with D18826 <https://phabricator.kde.org/D18826>, where TagLib will do the conversion automatically.
REPOSITORY
  R286 KFileMetaData
REVISION DETAIL
  https://phabricator.kde.org/D16579
To: smithjd, astippich, bruns, mgallien
Cc: bruns, astippich, kde-frameworks-devel, #baloo, domson, ashaposhnikov, michaelh, spoorun, ngraham, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190223/dfffedf9/attachment-0001.html>
    
    
More information about the Kde-frameworks-devel
mailing list