CRM-19385 revert changes to order by id on prevnext_cache.
authoreileen <emcnaughton@wikimedia.org>
Wed, 29 Mar 2017 05:51:24 +0000 (18:51 +1300)
committereileen <emcnaughton@wikimedia.org>
Wed, 29 Mar 2017 05:51:24 +0000 (18:51 +1300)
commitd481902f6bf8110aef323847c6de70cc28cf2944
tree4b87fc683824208b5e090189d96ecee37bb34a49
parentd46d3eaa0bd4c21d4a4347ee5b1147159885eaea
CRM-19385 revert changes to order by id on prevnext_cache.

After PR #10019 was merged @bhahumanists reported on #9801 that smart group ordering was broken. The
report is lacking detail and I could not replicate it, but am reverting the search changes for safety.

After this revert sites can still experiment with dropping/altering the primary key on
-civicrm_cache
-civicrm_group_contact_cache
-civicrm_acl_cache

But not civicrm_prevnext_cache.

Although I could not replicate the search issue it seems to me that our Ubuntu test suite
is failing where the other one is not on search-related items, suggesting that
for some reason Ubuntu does not default to the natural search order, due to
some config or other difference
CRM/Contact/BAO/Query.php
CRM/Core/BAO/PrevNextCache.php