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

Find Activities Search, the default search option of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be correct. #21595

Merged
merged 1 commit into from
Sep 29, 2021

Conversation

agileware-justin
Copy link
Contributor

Overview

Find Activities Search, the default search option of "With" criteria excludes Activities by default and confuses end users when 0 results are returned.

End users feel less confident using the system as a result and are annoyed by having a default option which is wrong.

Screenshot_20200917_133714

Before

Using the default search option "With" criteria, no Activities are returned in the search results.

After

When "With" criteria is not set, Activities are returned in the search results.

Technical Details

Comments

Agileware Ref: CIVICRM-1563

…th" criteria excludes Activities by default and confuses end users when 0 results are returned
@civibot civibot bot added the master label Sep 24, 2021
@civibot
Copy link

civibot bot commented Sep 24, 2021

(Standard links)

@agileware-justin agileware-justin changed the title CiviCRM, Find Activities, default search options includes "With" criteria which will exclude Activities by default Find Activities Search, the **default search option** of "With" criteria excludes Activities by default and confuses end users when 0 results are returned. Sep 24, 2021
@agileware-justin agileware-justin changed the title Find Activities Search, the **default search option** of "With" criteria excludes Activities by default and confuses end users when 0 results are returned. Find Activities Search, the **default search option** of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be OK. Sep 24, 2021
@agileware-justin agileware-justin changed the title Find Activities Search, the **default search option** of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be OK. Find Activities Search, the **default search option** of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be correct. Sep 24, 2021
@agileware-justin agileware-justin changed the title Find Activities Search, the **default search option** of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be correct. Find Activities Search, the default search option of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be correct. Sep 24, 2021
@demeritcowboy
Copy link
Contributor

It would be awesome if "this one simple trick" fixes it. I remember there being a bug - https://issues.civicrm.org/jira/browse/CRM-7812 and https://issues.civicrm.org/jira/browse/CRM-9065 (the conversation on the second one is mildly amusing in the context of searchkit). At the time it seemed like the issue was buried deep in the query somewhere. If it was just a need to click the "X" I can't believe I overlooked that.

@demeritcowboy demeritcowboy added the merge ready PR will be merged after a few days if there are no objections label Sep 24, 2021
@demeritcowboy demeritcowboy merged commit 7424c78 into civicrm:master Sep 29, 2021
@wmortada
Copy link
Contributor

Wanted to add a note that this caused some unintended consequences if you export data from the results of an activity search. See: https://civicrm.stackexchange.com/questions/41274/missing-data-when-exporting-activities-from-search-results/41275

@demeritcowboy
Copy link
Contributor

I tried an export and I'm not seeing it so just to clarify:

  • Which fields?
  • And is it those fields for all records, or just the records that are now being included that weren't before (the ones without With Contacts)?

@wmortada
Copy link
Contributor

wmortada commented Feb 14, 2022

@demeritcowboy in that particular case it was fields from the related contact - e.g. Organisation Name. The activity fields were exported but we appeared to be missing the related contact data. My theory is that behind the scenes the search does a join on the contact table, but if you don't select 'with' that doesn't happen so the related contact data is missing.

Yes, those fields for all records - the entire column of the spreadsheet was blank.

I'm not sure if there were any records that weren't included previously. That wasn't an issue for this site.

We've seen this on two separate sites so I don't think it is a quirk of data from one organisation.

@davejenx
Copy link
Contributor

Seeing the same issue, also affecting Organisation Name.

@demeritcowboy
Copy link
Contributor

Yes it seems a side-effect of this PR is that anything contact-related no longer defaults to the "With contact", so for exports if you need contact fields to be from the With Contact then you now have to explicitly select "With" on the search page (with the understanding that it will then exclude any activities that do not have a With Contact). Perhaps we should open a lab ticket for discussion?

@davejenx
Copy link
Contributor

Related but slightly different lab ticket: Exporting Activity Contacts Field Mapping page display just shows target.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants