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

fix(donor): allow multiple search queries #2807

Closed
samsmith89 opened this issue Feb 13, 2018 · 2 comments
Closed

fix(donor): allow multiple search queries #2807

samsmith89 opened this issue Feb 13, 2018 · 2 comments

Comments

@samsmith89
Copy link

Issue Overview

When a search is conducted for a donor in the Donors admin section it returns accurate results. When you hit the clear "x" at the end of the field, or if you delete the entry and search again. It returns with "no items found."

Expected Behavior

The user should have the ability to make a search query, and then be able to make another query without having to reload the page.

Current Behavior

When a query is executed the results return fine. If you make another query it returns with "no items found." This is also the case if I try to search within the results that have already been returned.

gif-test

Steps to Reproduce (for bugs)

  1. Install Give
  2. Go to the admin panel and select donations.
  3. Search for a donor within the system.
  4. Once the results return, delete the entry, or hit the clear and attempt to make another search.
  5. Confirm that "no items found" displays.

WordPress Environment

WordPress Environment

Home URL: |   | http://pumpscenario.local
Site URL: |   | http://pumpscenario.local
WP Version: |   | 4.9.4
WP Multisite: |   | –
WP Memory Limit: |   | 256 MB
WP Debug Mode: |   | –
WP Cron: |   |  
Language: |   | en_US
Permalink Structure: |   | /%year%/%monthnum%/%day%/%postname%/
Show on Front: |   | posts
Table Prefix: |   | wpv0_
Table Prefix Length: |   | 5
Table Prefix Status: |   | Acceptable
Admin AJAX: |   | Accessible
Registered Post Statuses: |   | publish, future, draft, pending, private, trash, auto-draft, inherit, refunded, failed, revoked, cancelled, abandoned, processing, preapproval, give_subscription

Server Environment
Hosting Provider: DBH: localhost, SRV: pumpscenario.local
TLS Connection: Connection uses TLS 1.2
TLS Rating: Probably Okay
Server Info: Apache/2.4.10 (Debian)
PHP Version: 7.0.3
PHP Post Max Size: 1,000 MB
PHP Time Limit: 900
PHP Max Input Vars: 3000
PHP Max Upload Size: 1,000 MB
cURL Version: 7.38.0, OpenSSL/1.0.1t - We recommend a minimum cURL version of 7.40.
SUHOSIN Installed: –
MySQL Version: 5.6.34
Default Timezone is UTC:
fsockopen/cURL:
SoapClient:
DOMDocument:
gzip:
GD Graphics Library:
Multibyte String:
Remote Post:
Remote Get:

WordPress Environment

Home URL: |   | http://pumpscenario.local
Site URL: |   | http://pumpscenario.local
WP Version: |   | 4.9.4
WP Multisite: |   | –
WP Memory Limit: |   | 256 MB
WP Debug Mode: |   | –
WP Cron: |   |  
Language: |   | en_US
Permalink Structure: |   | /%year%/%monthnum%/%day%/%postname%/
Show on Front: |   | posts
Table Prefix: |   | wpv0_
Table Prefix Length: |   | 5
Table Prefix Status: |   | Acceptable
Admin AJAX: |   | Accessible
Registered Post Statuses: |   | publish, future, draft, pending, private, trash, auto-draft, inherit, refunded, failed, revoked, cancelled, abandoned, processing, preapproval, give_subscription

@kevinwhoffman kevinwhoffman changed the title fix: donor search won't allow additional searches fix(donor): allow multiple search queries Feb 13, 2018
@ravinderk
Copy link
Collaborator

@samsmith89 This has been fixed. Please review it in release/2.0.3 branch.

dne0szfzmv

@samsmith89
Copy link
Author

@ravinderk I was able to test it and everything is working great. Thank you.

DevinWalker added a commit that referenced this issue Feb 15, 2018
* master: (185 commits)
  doc(release): ran pre-release gulp
  doc(release-changelog): remove line break and dupe entries
  doc(release): added change log, ran gulp, new POT file for release 2.0.3
  fix(chosen-style): remove bubble button look from chosen dropdowns in 2.0.3
  fix(float-labels-style): improve theme compatiblity by only targetting fl-label inputs #2340
  fix(i18n-string): use date_i18n() rather than date() for localization
  doc(release): prepare changelog and bump version for 2.0.3 release
  fix(doc-link): open doc link in new window
  fix(setting): use div to wrap setting field instead p
  fix(emails): update email message setting field desc
  fix(form): add data attribute to manage donate button label
  fix(email): replace only first occurrence of meta type
  fix(email): remove error_log
  style(email): update function desc
  feat(email): add email tag meta_*
  fix(shortcodes): display forms with publish status only
  style: improve code formatting
  fix(donation-form): improve if condition
  Fix #2807
  fix(assets/js): Update amount to precision
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants