Review Request: [GSoC] Amarok integration with gpodder.net

[Lucas Gomes]

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

(Updated Sept. 24, 2011, 2:25 p.m.)


Review request for Amarok, Stefan Derkits and Bart Cerneels.


Changes
-------

Removed some debug output from PlaylistBrowserModel. 
Added my name to the copyright header of every file I did a substantial change. 
Removed SyncRelationStorage::slaves(), since it's not needed anymore.


Summary
-------

The attached diff is the work done during this summer to integrate gpodder.net web service with Amarok.

Added features:

Podcast subscription synchronisation.
Podcast status synchronisation.
Browse through podcasts suggested by gpodder.net.
Browse through gpodder.net top recommended podcasts.


Diffs (updated)
-----

  src/CMakeLists.txt 962070d 
  src/browsers/playlistbrowser/PlaylistBrowserModel.h 76ccb10 
  src/browsers/playlistbrowser/PlaylistBrowserModel.cpp ff18d83 
  src/browsers/playlistbrowser/PodcastModel.h ce1a86f 
  src/browsers/playlistbrowser/PodcastModel.cpp 4aad518 
  src/browsers/playlistbrowser/QtGroupingProxy.cpp e0b4e7a 
  src/core-impl/playlists/types/file/PlaylistFileSupport.cpp 2dcc0cd 
  src/core-impl/playlists/types/file/m3u/M3UPlaylist.cpp c64cb97 
  src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp 0a3c99a 
  src/core-impl/podcasts/gpodder/GpodderPodcastMeta.h PRE-CREATION 
  src/core-impl/podcasts/gpodder/GpodderPodcastMeta.cpp PRE-CREATION 
  src/core-impl/podcasts/gpodder/GpodderProvider.h PRE-CREATION 
  src/core-impl/podcasts/gpodder/GpodderProvider.cpp PRE-CREATION 
  src/core-impl/podcasts/sql/SqlPodcastMeta.h 0110911 
  src/core-impl/podcasts/sql/SqlPodcastMeta.cpp e88bc9e 
  src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 83e9b99 
  src/core/playlists/Playlist.h f88b070 
  src/core/podcasts/PodcastMeta.h 21364b4 
  src/core/podcasts/PodcastMeta.cpp fb82fa8 
  src/main.cpp 7306e84 
  src/playlistmanager/PlaylistManager.h 54d63b1 
  src/playlistmanager/PlaylistManager.cpp 333ff17 
  src/playlistmanager/SyncRelationStorage.h 2a2681d 
  src/playlistmanager/SyncRelationStorage.cpp b151243 
  src/playlistmanager/SyncedPlaylist.h 54914c7 
  src/playlistmanager/SyncedPlaylist.cpp c8383b5 
  src/playlistmanager/SyncedPodcast.h PRE-CREATION 
  src/playlistmanager/SyncedPodcast.cpp PRE-CREATION 
  src/playlistmanager/file/KConfigSyncRelStore.h 4674f66 
  src/playlistmanager/file/KConfigSyncRelStore.cpp a857593 
  src/playlistmanager/file/PlaylistFileProvider.cpp 728030d 
  src/playlistmanager/sql/SqlPlaylist.cpp b2a9ca5 
  src/services/gpodder/CMakeLists.txt 97f14b3 
  src/services/gpodder/GpodderConfigWidget.ui PRE-CREATION 
  src/services/gpodder/GpodderService.h fac508a 
  src/services/gpodder/GpodderService.cpp e0cb02f 
  src/services/gpodder/GpodderServiceConfig.h PRE-CREATION 
  src/services/gpodder/GpodderServiceConfig.cpp PRE-CREATION 
  src/services/gpodder/GpodderServiceModel.h 1fbc90d 
  src/services/gpodder/GpodderServiceModel.cpp d899456 
  src/services/gpodder/GpodderServiceSettings.h PRE-CREATION 
  src/services/gpodder/GpodderServiceSettings.cpp PRE-CREATION 
  src/services/gpodder/GpodderTreeItem.h 5698d5b 
  src/services/gpodder/GpodderTreeItem.cpp bfa216e 
  src/services/gpodder/amarok_service_gpodder_config.desktop PRE-CREATION 

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


Testing
-------

This patch is the result of git diff for the range of several commits prior to HEAD, rebased on current master, and should build. Everything is working as expected and there aren't any known issues.


Thanks,

Lucas

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