14.0-b4: Can’t disable shuffle on playlists

[gonzabrusco]

What happened?

It seems that in 14.0-b4 the shuffle option in Playlists is always activated. I wanted to make an specific sequence of effects but the playlists keeps being shuffled. I can’t uncheck "Shuffle" option. When you uncheck it and press Save, if you reload the Playlist the Shuffle option will be still checked.

I'm new to WLED but this seems like a bug. Can anybody confirm?

Thanks

To Reproduce Bug

Just create a Playlist not shuffled. The result will be a shuffled playlist.

Expected Behavior

Respect the playlist order.

Install Method

Binary from WLED.me

What version of WLED?

14.0-b4

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[dosipod]

We have reported the same few days ago but seem the issue is fixed if you use the nightly builds from here https://wled-install.github.io/

[blazoncek]

I am able to save unrandomised playlist.

[softhack007]

I was able to reproduce the behaviour with "main" from today.

It seems to be related to "repeat indefinitely"; if shuffle is unchecked but repeat ind. checked, then the next time you open the presets in UI, it shows "shuffle" checked.

the relevant part of presets.json:

"9":{"playlist":{"ps":[4,6,1],"dur":[200,300,100],"transition":[9,9,7],"repeat":-1,"end":0,"r":0},"on":true,"n":"dada dudu"}

@blazoncek Might have something to do with below part of the code?

https://github.com/Aircoookie/WLED/blob/a2bda5a4dbdc0ada93b66281c179c37b9a6393dc/wled00/playlist.cpp#L102-L106

before "save"

after re-opening in UI

[softhack007]

A workaround is to unckeck "repeat indefinitely", then unckeck "shuffle", and save.

[blazoncek]

It seems to be related to "repeat indefinitely"; if shuffle is unchecked but repeat ind. checked, then the next time you open the presets in UI, it shows "shuffle" checked.

That is a lot more information. Thanks.

[gonzabrusco]

A workaround is to unckeck "repeat indefinitely", then unckeck "shuffle", and save.

But if I need to "repeat indefinitely", how do I make it not shuffled?

[gonzabrusco]

as @softhack007 said, I think it's related to

https://github.com/Aircoookie/WLED/blob/a2bda5a4dbdc0ada93b66281c179c37b9a6393dc/wled00/playlist.cpp#L159

and

https://github.com/Aircoookie/WLED/blob/a2bda5a4dbdc0ada93b66281c179c37b9a6393dc/wled00/playlist.cpp#L102-L106

I have to test it at home. Probably playlist[F("repeat")] is taking a negative value on line 159 and then forcing shuffle on line 105.

[blazoncek]

But if I need to "repeat indefinitely", how do I make it not shuffled?

Edit JSON.