removeUserTracks can now pass IDs and Spotify URIs

eddiemoore · Sep 17, 2014 · 6a5ce09 · 6a5ce09
1 parent 58c27dd
commit 6a5ce09
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 2 deletions.
diff --git a/dist/angular-spotify.min.js b/dist/angular-spotify.min.js
diff --git a/src/angular-spotify.js b/src/angular-spotify.js
@@ -344,6 +344,10 @@
         };
 
         NgSpotify.prototype.removeUserTracks = function (tracks) {
+          tracks = angular.isString(tracks) ? tracks.split(',') : tracks;
+          angular.forEach(tracks, function (value, index) {
+            tracks[index] = value.indexOf('spotify:') > -1 ? value.split(':')[2] : value;
+          });
           return this.api('/me/tracks', 'DELETE', {
             ids: tracks.toString()
           }, null, {

diff --git a/test/spec/angular-spotify.spec.js b/test/spec/angular-spotify.spec.js
@@ -1349,6 +1349,22 @@ describe('angular-spotify', function () {
           });
         });
 
+        it('should be able to pass Spotify URIs', function () {
+          spyOn(Spotify, 'api');
+
+          Spotify.setAuthToken('TESTING');
+
+          Spotify.removeUserTracks(['spotify:track:0udZHhCi7p1YzMlvI4fXoK','spotify:track:3SF5puV5eb6bgRSxBeMOk9']);
+
+          expect(Spotify.api).toHaveBeenCalled();
+          expect(Spotify.api).toHaveBeenCalledWith('/me/tracks', 'DELETE', {
+            ids: '0udZHhCi7p1YzMlvI4fXoK,3SF5puV5eb6bgRSxBeMOk9'
+          }, null, {
+            'Authorization': 'Bearer TESTING',
+            'Content-Type': 'application/json'
+          });
+        });
+
       });
 
     });