MiczFlor / RPi-Jukebox-RFID / 8668006920

38%

develop: 79%

Pull #2258

github

Fix PlayerMPD.prev/next() when stopped (#2326)

* utils: Add get_config_action

This abstracts away the functionality to resolve a given config option
to an action in a pre-defined dict.

Co-authored-by: Christian Hoffmann <christian@hoffie.info>

* Fix PlayerMPD.prev/next() when stopped

* Avoid MPD-related crashes during all prev/next() calls.
* Explicitly handle prev() in stopped state, configurable via
  `playermpd.stopped_prev_action`.
* Explicitly handle next() in stopped state, configurable via
  `playermpd.stopped_next_action`.
* Explicitly handle next() when reaching the end of the playlist:
  jukebox-daemon will now ignore the action by default (similar to v2).
  It can also be configured to rewind the playlist instead by setting
  the new config option `playermpd.end_of_playlist_next_action: rewind`
  or to stop playing.

Fixes #2294
Fixes #2327

Co-authored-by: pabera <1260686+pabera@users.noreply.github.com>

---------

Co-authored-by: pabera <1260686+pabera@users.noreply.github.com>

56 of 105 new or added lines in 4 files covered. (53.33%)

494 of 1292 relevant lines covered (38.24%)

1.53 hits per line