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#771 - Smart group with uf_group_id does not load contacts wi… #13742

Merged
merged 1 commit into from Mar 18, 2019

Conversation

jitendrapurohit
Copy link
Contributor

…th same search profile

Overview

Smart group with uf_group_id does not load contacts with same search profile

Before

Steps to reproduce -

  • Create a search profile adding some basic fields first name, last name and email.
  • Open Advanced Search form, select this profile and add some filters.
  • create a smart group of the resulted contacts.
  • When you come back from the process after clicking the Done button on the last screen, the profile is not pre-selected by default.
  • uf_group_id is not stored as a formvalue in civicrm_saved_search table.

After

Fixed. uf_group_id is correctly stored and retrieved from civicrm_saved_search table.

Comments

Gitlab - https://lab.civicrm.org/dev/core/issues/771

@civibot
Copy link

civibot bot commented Mar 1, 2019

(Standard links)

@civibot civibot bot added the master label Mar 1, 2019
@jitendrapurohit
Copy link
Contributor Author

test this please

1 similar comment
@jitendrapurohit
Copy link
Contributor Author

test this please

@eileenmcnaughton eileenmcnaughton added this to Main Review to-do-list in review board Mar 18, 2019
@eileenmcnaughton
Copy link
Contributor

I was able to replicate this & verify it. Small sensible change, adds a test - good to merge!

@eileenmcnaughton eileenmcnaughton merged commit 9b9ba5a into civicrm:master Mar 18, 2019
review board automation moved this from Main Review to-do-list to done Mar 18, 2019
@jitendrapurohit jitendrapurohit deleted the core-771 branch March 18, 2019 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants