Merge lp:~mims-michael/nuvola-player/amazon into lp:nuvola-player/2.5.x
Proposed by
Michael Mims
Status: | Superseded |
---|---|
Proposed branch: | lp:~mims-michael/nuvola-player/amazon |
Merge into: | lp:nuvola-player/2.5.x |
Diff against target: |
179 lines (+151/-2) 4 files modified
data/nuvolaplayer/services/amazon/description.html (+9/-0) data/nuvolaplayer/services/amazon/integration.js (+135/-0) data/nuvolaplayer/services/amazon/metadata.conf (+6/-0) src/core/storage.vala (+1/-2) |
To merge this branch: | bzr merge lp:~mims-michael/nuvola-player/amazon |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jiří Janoušek | Needs Fixing | ||
Review via email: mp+103551@code.launchpad.net |
This proposal has been superseded by a proposal from 2012-04-26.
Description of the change
I don't know if there is a need/want for this, but I use Amazon Cloud Player frequently so I've created a service integration module for it. You certainly don't have to use it, but I thought I would share it with you. I've been using it for a little over a day and it seems to be working fine.
To post a comment you must log in.
Thanks for your work, it's certainly valuable contribution to the project. Code looks good (1 minor improvement and 1 cosmetic stuff).
------- -8<---- -------
66 + var state = Nuvola.STATE_NONE;
90 + var state = Nuvola.STATE_NONE;
The second assignment is unnecessary.
------- -8<---- -------
78 + var __meta
91 + var __widgets
121 + var __player
Is there any reason to use "__" prefix? (You can leave it as is if it's your personal preference.)
------- -8<---- -------
175 - var i = uri.last_ index_of_ char('/ '); compute_ for_string( ChecksumType. MD5, uri);
176 - var id = uri.substring(i+1);
177 + var id = Checksum.
I will revert this change, because it will be merged as a separate patch.