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

Jump to bottom

Drop solr_field local in favor of field_name #1621

Merged
merged 1 commit into from Jan 23, 2017
Merged

Drop solr_field local in favor of field_name #1621

merged 1 commit into from Jan 23, 2017

Conversation

jcoyne
Copy link
Member

@jcoyne jcoyne commented Jan 23, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.583% when pulling 25e5efc on drop_solr_field into 8fb25e4 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.583% when pulling 25e5efc on drop_solr_field into 8fb25e4 on master.

@cbeer cbeer merged commit cc8baab into master Jan 23, 2017
@cbeer cbeer deleted the drop_solr_field branch January 23, 2017 22:08
@cbeer cbeer added this to Done in Blacklight 7.0 Release Feb 1, 2017
dkinzer added a commit to dkinzer/blacklight_range_limit that referenced this pull request Apr 29, 2018
@dkinzer
Replace solr_field with field_name.
1e27456 
When upgrading a blacklight site from BL-6.14.1 to BL-7 I got the
following error from this gem:

```
undefined local variable or method solr_field
``

This is because solr_field has been removed from blacklight:
projectblacklight/blacklight#1621

This commit replaces the calls to solr_field that caused the error
I was seeing with field_name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbeer cbeer cbeer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Blacklight 7.0 Release
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jcoyne @coveralls @cbeer