Skip to content

Commit

Permalink
Small bug fix -- wasn't sending back the addeStreamItem
Browse files Browse the repository at this point in the history
  • Loading branch information
MeoMix committed Jun 18, 2014
1 parent df8f404 commit e7d4196
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion src/js/background/collection/streamItems.js
Original file line number Diff line number Diff line change
Expand Up @@ -240,6 +240,8 @@
if (playOnAdd || options.markFirstActive) {
createdStreamItems[0].save({ active: true });
}

return createdStreamItems;
},

deactivateAllExcept: function(changedStreamItem) {
Expand Down Expand Up @@ -356,10 +358,12 @@
else if (radioEnabled) {
var randomRelatedSong = this.getRandomRelatedSong();

this.addSongs(randomRelatedSong, {
var addedSongs = this.addSongs(randomRelatedSong, {
// Mark as active after adding it to deselect other active items and ensure it is visible visually.
markFirstActive: true
});

nextItem = addedSongs[0];
}
// If the active item was deleted and there's nothing to advance forward to -- activate the previous item and pause.
// This should go AFTER radioEnabled check because it feels good to skip to the next one when deleting last with radio turned on.
Expand Down
2 changes: 1 addition & 1 deletion src/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"manifest_version": 2,
"name": "__MSG_extensionName__",
"short_name": "Streamus",
"version": "0.142",
"version": "0.143",

"default_locale": "en",
"description": "__MSG_extensionDescription__",
Expand Down

0 comments on commit e7d4196

Please sign in to comment.