1
0
mirror of https://github.com/SuperBFG7/ympd synced 2024-11-30 00:19:54 +00:00

Fix broken add track action

Fix the bug I've introduced in my PR #78, breaking the add
track/playlist action. We have to decode URIs before sending them back
to our socket!

fixes #80
This commit is contained in:
Virgil Dupras 2015-07-13 14:31:56 -04:00
parent fe44fca184
commit 698f573a65

View File

@ -243,7 +243,7 @@ function webSocketConnect() {
"<span class=\"glyphicon glyphicon-" + glyphicon + "\"></span></a>") "<span class=\"glyphicon glyphicon-" + glyphicon + "\"></span></a>")
.find('a').click(function(e) { .find('a').click(function(e) {
e.stopPropagation(); e.stopPropagation();
socket.send(onClickAction + "," + $(this).parents("tr").attr("uri")); socket.send(onClickAction + "," + decodeURI($(this).parents("tr").attr("uri")));
$('.top-right').notify({ $('.top-right').notify({
message:{ message:{
text: $('td:nth-child(2)', $(this).parents("tr")).text() + " added" text: $('td:nth-child(2)', $(this).parents("tr")).text() + " added"
@ -264,7 +264,7 @@ function webSocketConnect() {
app.setLocation("#/browse/0/"+$(this).attr("uri")); app.setLocation("#/browse/0/"+$(this).attr("uri"));
break; break;
case 'song': case 'song':
socket.send("MPD_API_ADD_TRACK," + $(this).attr("uri")); socket.send("MPD_API_ADD_TRACK," + decodeURI($(this).attr("uri")));
$('.top-right').notify({ $('.top-right').notify({
message:{ message:{
text: $('td:nth-child(2)', this).text() + " added" text: $('td:nth-child(2)', this).text() + " added"
@ -272,7 +272,7 @@ function webSocketConnect() {
}).show(); }).show();
break; break;
case 'plist': case 'plist':
socket.send("MPD_API_ADD_PLAYLIST," + $(this).attr("uri")); socket.send("MPD_API_ADD_PLAYLIST," + decodeURI($(this).attr("uri")));
$('.top-right').notify({ $('.top-right').notify({
message:{ message:{
text: "Playlist " + $('td:nth-child(2)', this).text() + " added" text: "Playlist " + $('td:nth-child(2)', this).text() + " added"