-
Notifications
You must be signed in to change notification settings - Fork 21
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
Fixes #3946 Whitescreen if az_person not installed. #3947
Conversation
This should not affect 2.11.x |
I think to prevent other similar issues we may want to check if the field exists on the bundle, rather than checking if the module is enabled. |
Co-authored-by: Joe Parsons <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work in cases where az_person
does not exist, and field_az_netid
does not exist. Thanks @trackleft !
Co-authored-by: Joe Parsons <[email protected]>
Description
Checking if az_person is enabled before counting on az_person fields.
Release notes
Related issues
Closes #3946
How to test
Types of changes
Arizona Quickstart (install profile, custom modules, custom theme)
Drupal core
Drupal contrib projects
Checklist