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

[#4554] ClausePresenter: don't attempt to call #display_label on a non-existant field configuration #4559

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

sandbergja
Copy link
Member

This fixes an error that appeared when a user specified a search field that does not exist. It is essentially the same as upstream fix projectblacklight/blacklight#3442, so if that is accepted and we start using a Blacklight release that includes it, we can remove our customized version.

Closes #4554

…n-existant field configuration

This fixes an error that appeared when a user specified a search field that
does not exist.  It is essentially the same as upstream fix projectblacklight/blacklight#3442,
so if that is accepted and we start using a Blacklight release that includes
it, we can remove our customized version.
@coveralls
Copy link

Coverage Status

coverage: 96.691% (+0.001%) from 96.69%
when pulling 567bf37 on i4554
into c3d3d84 on main.

@kevinreiss kevinreiss merged commit 4e84f4f into main Nov 11, 2024
14 checks passed
@kevinreiss kevinreiss deleted the i4554 branch November 11, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Advanced Search Form Search Raises NoMethodError: private method `display_label' called for nil:NilClass
3 participants