Random subfolder play #1175
Open
Random subfolder play #1175
+618
−34
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…1182 and combined it with random subfolder play (MiczFlor#1175). Also added the possibility to play all or just a single subfolder.
|
Hi all, just added skip subfolder functionality as requested in #1182 Unfortunately, it required some more mangling with rfid_trigger_play.sh, but it should not be noticed by surroundings. |
Karli2727
suggested changes
Aug 10, 2022
Thanks for your comment. |
s-martin
reviewed
Apr 8, 2024
| # replace subfolder slashes with " % " | ||
| PLAYLISTPATH="${PLAYLISTSFOLDERPATH}/${FOLDER//\//\ %\ }.m3u" | ||
| PLAYLISTNAME="${FOLDER//\//\ %\ }" | ||
| echo "" > "${PLAYLISTPATH}" |
There was a problem hiding this comment.
Suggested change
| echo "" > "${PLAYLISTPATH}" | |
| echo "" >> "${PLAYLISTPATH}" |
Added https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/1175/files#r1556369182 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reason for improvement:
See #1170
Missing functionality to play a random subfolder when a card is assigned to the top folder and the conf is set accordingly.
Proposal:
Minor changes and additions necessary.
Possible breaking change: changed logic to check for second swipe: Instead to check for LASTPLAYLIST now LASTID is used. LASTID is basically the folder from CARDID, separating the really played folder from the one assigned to the card.
Beside this, the LASTPLAYLIST is still there and being set. I think this will not break anything and still enable a thourough secondswipe check.
Backend
New parameter introduced in "folder.conf", FOLDERSHUFFLE to save setting for folder. Also used in WebUI.
The button is shown if there is at least one subfolder. With this button, also a new api file is introduced.