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

When a multi-valued ConfigurationSetting has no values set, the default is incorrectly used #936

Open
leonardr opened this issue Apr 23, 2018 · 0 comments

Comments

@leonardr
Copy link
Contributor

Consider a multi-select box such as the one for facets_enabled_available setting.

I had a library in which none of these boxes were checked. This caused problems in other parts of the system, but the value for the setting was correctly stored as "[]". However, in the admin interface, every box was checked, making it difficult to see what the actual problem was.

This is most likely a problem in the circulation manager, rather than the web interface, but it could be in the web interface instead. It's probably due to an if not value instead of if value is not None.

@andrewnshelton andrewnshelton added this to To do in Org_test_CircMgr via automation Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant