Review Request 109470: A patch which allows us to distinguish between cached lyrics and lyrics newly downloaded from the internet.

Matěj Laitl matej at laitl.cz
Wed Apr 17 17:20:40 UTC 2013


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

(Updated April 17, 2013, 5:20 p.m.)


Status
------

This change has been discarded.


Review request for Amarok.


Description
-------

It required modifications, when there is no change in the lyrics downloaded and lyrics retrieved from cache the title display of the lyrics browser changes to "Cached Lyrics" from "Lyrics" so we can tell the difference between old and new. 


Diffs
-----

  CMakeLists.txt d167577 
  src/context/applets/lyrics/LyricsApplet.cpp 2394964 
  src/context/engines/lyrics/LyricsEngine.h b187b73 
  src/context/engines/lyrics/LyricsEngine.cpp 2befa91 
  src/core-impl/collections/nepomukcollection/CMakeLists.txt 642919b 

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


Testing
-------

Its working fine!


Thanks,

mayank jha

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130417/40981030/attachment.html>


More information about the Amarok-devel mailing list