Navigation Menu

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#1293 Still output csv file on export, even if no rows are in it #15440

Merged
merged 1 commit into from Oct 8, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Oct 8, 2019

Overview

Fixes a bug where output is confusing when there are no rows

Before

  1. Do a contact search.
  2. Select contacts so that none of them have a street address, the Do Not Mail choice, or deceased. E.g. pick a couple contacts where all they have is name and/or email.
  3. Choose export from the dropdown.
  4. Choose selected fields.
  5. Check the box that says Exclude contacts with "do not mail" privacy, no street address, or who are deceased.
  6. On the next page choose a field like Do Not Mail.
  7. When you export you just get a blank screen and there's no errors anywhere. It's because there is no actual error, it's just that there are now zero contacts to export, but it also doesn't give you a zero-length file to download.

After

Step 7 works.

@civibot
Copy link

civibot bot commented Oct 8, 2019

(Standard links)

@civibot civibot bot added the master label Oct 8, 2019
@eileenmcnaughton eileenmcnaughton changed the title Still output csv file, even if no rows are in it dev/core#1293 Still output csv file on export, even if no rows are in it Oct 8, 2019
@eileenmcnaughton eileenmcnaughton changed the base branch from master to 5.19 October 8, 2019 07:35
@civibot civibot bot added 5.19 and removed master labels Oct 8, 2019
@seamuslee001
Copy link
Contributor

(CiviCRM Review Template WORD-1.2)

  • General standards
    • (r-explain) Pass
    • (r-user) Pass
    • (r-doc) Pass
    • (r-run) Pass replicated the issue and confirms that this fixes it and outputs an empty CSV which seems appropriate in this case
  • Developer standards

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