Cannot add songs with " ' " in their filenames about mympd HOT 12 CLOSED

Does it work, if you click on the song title?

from mympd.

If i click on the song title it will be added successfully.

from mympd.

Thanks for your confirmation. I think I found the bug

from mympd.

👍

from mympd.

Should now be fixed in devel. Can you test it?

from mympd.

SyntaxError: JSON.parse: unterminated string at line 1 column 133 of the JSON data[Learn More] mympd.min.js:121:1
showMenu/</<
http://192.168.0.206/js/mympd.min.js:121:1

Same error jc...

from mympd.

var cmd = JSON.parse(dh.replace(/'/g, '"'));

If i replace minified with normal js the Web Console indicate this line when i hit "Append to queue".

I do not know if it helps somewhere.

from mympd.

Oh you have another one here:

el.addEventListener('shown.bs.popover', function(event) {
    document.getElementsByClassName('popover-content')[0].addEventListener('click', function(event) {
        event.preventDefault();
        event.stopPropagation();
        if (event.target.nodeName == 'A') {
            var dh = event.target.getAttribute('data-href');
            if (dh) {
                **var cmd = JSON.parse(dh.replace(/\'/g, '"'));**

from mympd.

var cmd = JSON.parse(dh);
is here right. But this doesn't fix the bug.
It seems that saving the ouptut of JSON.stringify can't be saved in a data-* attribute in the dom, if it contains ' or " characters in values.

Encoding the output in base64 works. I commited the fix in devel.

from mympd.

Perfect jc

Case closed!

Thank you.

from mympd.

Hi JC,

Is this the same issue?

I tried to add an album with " ' " in title... Sgt. Pepper's - Beatles

from mympd.

Is an another issue, please open a new issue and attach debug logs of myMPD and verbose logs of mpd. If JavaScript errors occurs these also.

from mympd.