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

core#1411: Advanced Search crashes when some form values start with 1 #15918

Merged
merged 1 commit into from
Nov 21, 2019

Conversation

MegaphoneJon
Copy link
Contributor

RC version of #15914.

@civibot
Copy link

civibot bot commented Nov 21, 2019

(Standard links)

@civibot civibot bot added the 5.20 label Nov 21, 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 i couldn't replicate a hard fail but i was able to confirm the query wasn't working right, After applying the patch the search returned expected results.
  • Developer standards

@MegaphoneJon
Copy link
Contributor Author

@seamuslee001 You're right, sorry, it doesn't crash, it just doesn't return search results. That was sloppy of me, and the Gitlab issue has a more descriptive title.

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

Successfully merging this pull request may close these issues.

2 participants