Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spotify cache and normalization customization variables don't work if their value is not defined #626

Closed
laytonhayes opened this issue Jan 25, 2023 · 3 comments · Fixed by #628
Labels
bug Something isn't working

Comments

@laytonhayes
Copy link
Contributor

laytonhayes commented Jan 25, 2023

The configuration variables SOUND_SPOTIFY_DISABLE_NORMALISATION and SOUND_SPOTIFY_ENABLE_CACHE only work if their value is defined. It is not clear in the support docs that any value has to be declared.

Additional context
In /plugins/spotify/start.sh these variable values are checked for null or not null using -z. If the variable is set but no value declared it functions as if there is no variable at all.

if [[ -z "$SOUND_SPOTIFY_DISABLE_NORMALISATION" ]]; then
  set -- "$@" \
    --enable-volume-normalisation
fi 

Suggestions
Update the support docs to reflect that a value has to be set to something OR check to see if that variable exist, even if it's set to null

@laytonhayes laytonhayes added the bug Something isn't working label Jan 25, 2023
@maggie44
Copy link
Contributor

Great catch. Do you think that is what could be happening here: #620?

Would you be able to submit a PR?

@laytonhayes
Copy link
Contributor Author

@Maggie0002, I'll test and see if that's what's causing the issue with #620. I should be able to submit a PR in the next few days.

@laytonhayes
Copy link
Contributor Author

@Maggie0002 - I've created PR #628 for the Spotify variable fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants