Merge lp:~rodrigo-moya/rhythmbox-ubuntuone/activate-on-u1ms-links into lp:rhythmbox-ubuntuone

Proposed by Rodrigo Moya
Status: Merged
Approved by: Alejandro J. Cura
Approved revision: 95
Merged at revision: 95
Proposed branch: lp:~rodrigo-moya/rhythmbox-ubuntuone/activate-on-u1ms-links
Merge into: lp:rhythmbox-ubuntuone
Diff against target: 11 lines (+2/-0)
1 file modified
umusicstore/MusicStoreWidget.py (+2/-0)
To merge this branch: bzr merge lp:~rodrigo-moya/rhythmbox-ubuntuone/activate-on-u1ms-links
Reviewer Review Type Date Requested Status
Alejandro J. Cura (community) Approve
Stuart Langridge (community) Approve
Review via email: mp+37723@code.launchpad.net

Commit message

Activate U1 music store when getting a u1ms:// link

Description of the change

Activate U1 music store when getting a u1ms:// link

To post a comment you must log in.
Revision history for this message
Stuart Langridge (sil) :
review: Approve
Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

To test this, before using this branch, run:

rhythmbox u1ms://stores.7digital.com/corporate_1/artists/govt-mule/the-deepest-end-live-in-concert-1/

and see it not activate the music store

after this:

sudo cp umusicstore/MusicStoreWidget.py /usr/lib/rhythmbox/plugins/umusicstore

and then, again:

rhythmbox u1ms://stores.7digital.com/corporate_1/artists/govt-mule/the-deepest-end-live-in-concert-1/

Revision history for this message
Alejandro J. Cura (alecu) wrote :

Works as advertised.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'umusicstore/MusicStoreWidget.py'
2--- umusicstore/MusicStoreWidget.py 2010-09-24 09:16:29 +0000
3+++ umusicstore/MusicStoreWidget.py 2010-10-06 10:52:42 +0000
4@@ -295,6 +295,8 @@
5 return False
6 uri_to_use = uri.replace("u1ms://", "http://")
7 print "Calling u1musicstore plugin with %s" % uri_to_use
8+ shell = self.get_property("shell")
9+ shell.props.sourcelist.select(self)
10 self.browser.load_store_link(uri_to_use)
11 return True
12

Subscribers

People subscribed via source and target branches