Remove use of LOWER from city & street searches and fix international strings test.
authoreileen <emcnaughton@wikimedia.org>
Tue, 23 Oct 2018 19:54:54 +0000 (08:54 +1300)
committereileen <emcnaughton@wikimedia.org>
Fri, 2 Nov 2018 07:46:00 +0000 (20:46 +1300)
commit2fc640828ea218db079f46d6c16b07373c40f751
treee4671e86d93c61fa4a9103b651637db974061833
parent1d9a471dfd15fcbae1efa35c044c84abbd12f623
Remove use of LOWER from city & street searches and fix international strings test.

Adding LOWER to mysql queries makes them slower. lowercasing php strings
breaks under some character sets with some server configs.

In trying to review PR 12364 I dug into the strotolower behaviour. I found that
in the path altered in this PR we are setting the value to lower case but not the
mysql clause. I ran through ContactTest without these lines & found it
only hit this strtolower in one test - testInternationalStrings which searches
by organization name. I found the this test actually failed for me
when I ran it locally - but it passed when I removed the strotlower lines.

Wonder in jenkins finds the same
CRM/Contact/BAO/Query.php
tests/phpunit/CRM/Contact/BAO/QueryTest.php
tests/phpunit/api/v3/ContactTest.php