Review Request: "Album Artist" field support.

Sergey Ivanov 123kash at gmail.com
Mon Nov 1 00:27:22 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100084/
-----------------------------------------------------------

(Updated 2010-10-31 23:27:22.039845)


Review request for Amarok.


Changes
-------

Update to make It apply on latest revision.


Summary
-------

Some try to handle "Album Artist" data (ability to store/load It in/from file). Support of updating/setting "Album Artist" for albums in SQLCollection.


Diffs (updated)
-----

  src/core-impl/capabilities/timecode/TimecodeEditCapability.h 286cb3f 
  src/core-impl/capabilities/timecode/TimecodeEditCapability.cpp 9a69079 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.h 7df9225 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.cpp f3142e1 
  src/core-impl/collections/proxycollection/ProxyCollectionMeta.cpp 2008336 
  src/core-impl/collections/sqlcollection/CapabilityDelegateImpl.cpp d984391 
  src/core-impl/collections/sqlcollection/SqlMeta.h a89ad6d 
  src/core-impl/collections/sqlcollection/SqlMeta.cpp 19b4a7e 
  src/core-impl/meta/file/File.h 9ab364b 
  src/core-impl/meta/file/File.cpp 70f2b91 
  src/core-impl/meta/proxy/MetaProxy.h 3bae966 
  src/core-impl/meta/proxy/MetaProxy.cpp 450cd6e 
  src/core/capabilities/EditCapability.h ba91ef9 
  src/core/meta/support/MetaUtility.cpp 534a61a 
  src/dialogs/TagDialog.cpp f0e671a 
  tests/core-impl/collections/proxycollection/TestProxyCollectionMeta.cpp 97e0281 

Diff: http://git.reviewboard.kde.org/r/100084/diff


Testing
-------

Tested as I could. Works fine.
Found out that I have ~5 different "Best Of" albums of different artists, but not the only 1 as I thought before. :)


Thanks,

Sergey

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20101031/6320e6c2/attachment-0001.htm 


More information about the Amarok-devel mailing list