search

kimlai / melikeyradio

Simple webradio/musicblog engine.
1 stars 1 forks source link

Get a default playlist when none has been defined #23

Closed nclavaud closed 5 years ago

nclavaud commented 10 years ago

If playlist_id is not set in the app parameters, the main page will fail with error code 500. This fix will retrieve the playlist with the lowest id instead when no playlist_id has been set.

Fixes issue #21

kimlai commented 10 years ago

This seems like a lot of code for a missing parameter. Maybe it'd be better to make the melikeyradio:currentPlaylist:update a bit more robust, and call it when loading the fixtures.

Plus, this won't handle the case where there simply is no playlist.

kimlai commented 10 years ago

Or maybe using a parameter for this is simply not a good approach. I dunno.

nclavaud commented 10 years ago

Sure it's a hack. Best option IMO would be to be able to star / flag the main playlist.

nclavaud commented 5 years ago

So old :wave: