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

dev/core#1888 and dev/core#1885 - Fatal error on advanced search and warnings and missing group display on contact form #17888

Merged
merged 1 commit into from
Jul 20, 2020

Conversation

demeritcowboy
Copy link
Contributor

@demeritcowboy demeritcowboy commented Jul 18, 2020

Overview

This reverts #13958 since it is causing some problems:

  1. On advanced search choose a group => Fatal error.
  2. When editing a contact that does have a group set, the group does not display and E_WARNING.
  3. When saving a contact without a group set, E_WARNING.

If somebody wants to work on those then I'm happy to close this PR but I'm not really willing to spend more time on it. I started here if you want to pick it up: https://github.com/civicrm/civicrm-core/pull/17877/files

https://lab.civicrm.org/dev/core/-/issues/1888

https://lab.civicrm.org/dev/core/-/issues/1885

@civibot
Copy link

civibot bot commented Jul 18, 2020

(Standard links)

@civibot civibot bot added the master label Jul 18, 2020
@eileenmcnaughton
Copy link
Contributor

I'm Ok with a straight revert for now

@demeritcowboy
Copy link
Contributor Author

Just noticing I messed up one line somehow. PR coming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants