[SOLVED] moOde 9.0.5 and 9.0.7: Adding radio station to playlist does not work

[kurt1970]

No, before pressung the "Add" button, there are no error messages.

[kurt1970]

Hi @Tim Curtis, were you able to make progress on this?

[Tim Curtis]

Indeed, yes.

I've identified the bug causing the variable to be overwritten. It will occur any time the Queue is updated for example play/pause, next item, radio station transitions to next track, etc.

I'll fix it for upcoming 9.0.7 and I'll prolly post a patch that you can install when I get it tested on my end in the next couple of days.

[kurt1970]

Which is brilliant news! Tx a million! Let me know, and I'm happy to test.
Btw, in what time zone are you?

[Tim Curtis]

USA, Michigan (The Great Lakes State!)
Eastern Time Zone. It's around 2:30 PM as I post this.

[Tim Curtis]

For testing:

Code:

sudo wget -q https://moodeaudio.org/test/lib.min.js -O /var/www/js/lib.min.js
sudo wget -q https://moodeaudio.org/test/main.min.js -O /var/www/js/main.min.js

The easiest way to test the fix:

1. Do an Add to playlist for a station or track but don't click the Add button
2. MPC PLAY followed by MPC NEXT which will force the Queue to update
3. Click Add and confirm that both notifications are displayed
4. Navigate to the playlist and view or edit it to confirm that the item was added

[kurt1970]

Get worked.

Stupid Q: what does "MPC PLAY" and "MPC NEXT" mean?

[Tim Curtis]

Enter the commands via SSH so you can control MPD while leaving the WebUI sitting at that add to playlist screen.

[kurt1970]

ok. Did that. It tried 20x and got half of them failed. I mean, half of them didn't show the 2nd message, and were not saved in the PL

[Tim Curtis]

Sounds like a stale Browser cache that still has the old JS.

Tomorrow I'll enable the console.log debug statements (to make it easier to see whats in the variables) and then post a new set of min files.