Review Request 125993: Implemented lastfmimagefetcher as a Plugin
Aditya Dev Sharma
aditya.sharma15696 at gmail.com
Wed Dec 23 09:20:08 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125993/
-----------------------------------------------------------
(Updated Dec. 23, 2015, 9:20 a.m.)
Review request for Plasma, Ashish Bansal and Bhushan Shah.
Changes
-------
Minor changes in renaming stuff
Repository: plasma-mediacenter
Description (updated)
-------
Implemented it like the other datasources.
Signal newMedia from MediaLibrary is absorbed by handleLastFmNewMedia . handleLastFmNewMedia then calls fetchImage for each newMedia in the newMediaList.
After the image is fetched, gotImage emits the signal imageFetched (from mediaLibrary). This signal is absorbed by signalUpdate in PmcMetaDataModel.
Diffs (updated)
-----
libs/mediacenter/pmcmetadatamodel.cpp 2fe6efe
libs/mediacenter/medialibrary.h 45744c1
libs/mediacenter/lastfmimagefetcher.cpp e005077
libs/mediacenter/lastfmimagefetcher.h ecff37e
libs/mediacenter/CMakeLists.txt 5a13449
datasources/lastfm/lastfmimagefetcher.desktop PRE-CREATION
datasources/lastfm/CMakeLists.txt PRE-CREATION
datasources/CMakeLists.txt a7171b7
autotests/lastfmimagefetchertest.cpp abb3fd6
autotests/CMakeLists.txt 04ba1c3
autotests/lastfmimagefetchertest.h eb7cb29
Diff: https://git.reviewboard.kde.org/r/125993/diff/
Testing
-------
Builds 100%.
Works as expected.
Thanks,
Aditya Dev Sharma
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151223/8ef191c7/attachment.html>
More information about the Plasma-devel
mailing list