Change default for CIVICRM_CONTAINER_CACHE to simplify admin/developer experience

In discussion of cache-related PRs for Civi 5.3, there were a few
reports/issues from developers observing `ServiceNotFoundException`.  This
is because there's not much awareness about how service-definitions are
cached.  It shouldn't be a significant issue for production systems running
vanilla code, but for admins and developers (who juggle patches/branches),
it can cause confusion/support-issues/false-failures.  This PR aims to
reduce those.

(This is a follow-up/substitute for #12401.)

Before
------
* The default value of `CIVICRM_CONTAINER_CACHE` is `always`.

After
-----
* The default value of `CIVICRM_CONTAINER_CACHE` is `auto`.

Technical Details
-----------------
The Symfony container essentially stores a list of "services".  In some
Symfony-based apps, building the list of services can be time consuming, so
it's common to cache this.

In Civi, this cache physically appears as
`files/civicrm/templates_c/CachedCiviContainer.*.php`.  The constant
`CIVICRM_CONTAINER_CACHE` determines how Civi manages the cache.  There are
three available policies:

* `never`: This means we never use the cache.  You never have to worry about
  staleness.  But it means we have to fire `hook_civicrm_container` on every
  page-request, and we go through any container-compilation tasks.
  This would have the fewest support-issues for devs and advanced admins.

* `always`: This means we always use whatever is in the cache.  It never
  (automatically) rebuilds the cache or checks freshness...  if you make
  change, then you must flush the cache explicitly.  This should be the
  fastest in production.

* `auto`: This means we typically use the cache (avoiding
  hooks/recompilation), but it has to `stat()` a half-dozen files to check
  the modification time.  (To wit: if the timestamp on
  `Civi/Core/Container.php` changes, then we discard the cache.)

Since performance is a consideration, I did some light benchmarking on my
laptop (fetching a basic public Civi page 100 times across 3 threads).

https://gist.github.com/totten/ec4bffd723afb7967aec56a3040b9ca3

In these results, the `never` policy appears to be ~15-20ms slower than
`auto` or `always`. `auto` is only ~2ms slower than `always`.

The other consideration is accuracy -- `auto` will usually re-compile if you
make a change, but there are some edge-cases where you must still flush
manually.  (In particular -- when you first implement
`hook_civicrm_container` in a new extension, it might not be aware of the
new extension.  And extensions need to call `$container->addResource()`.)

However, overall, `auto` is a pretty good compromise that's almost as fast
you can get and works out-of-the-box for many dev/admin scenarios.

Merge pull request #12409 from eileenmcnaughton/paydata

dev/core/issues/211 Fix mis-allocation of financial transactions when editing payment method on a completed payment

Add late changes

Fix mis-allocation of financial transactions

Per

https://lab.civicrm.org/dev/core/issues/211 there seems to be an issue with assigning entity_financial_trxn
entries the wrong financial_trxn_id - I cannot find any recent change that would have caused this
(unless it's simply the greater use of the payment edit block) - it has been reported as working in
5.0.2.

Merge pull request #12399 from eileenmcnaughton/jamie

dev/core/#/229 Fix fatal error on send test mail

Merge pull request #12398 from agh1/5.3.0-releasenotes

5.3.0 release notes

Fix fatal error on send test mail

Merge pull request #12400 from seamuslee001/dev_core_227

dev/core#227 Fix issue where on some extened groups the multiple reco…

Add release notes for 5.2.2

Add release notes for 5.2.1

5.3.0 release notes: combined, organized, added contributors, boilerplate

Also removed redundant issues from 5.2.1 and 5.2.2

5.3.0 release notes: my half of the issues

5.3.0 release notes: Alice part of release notes

dev/core#227 Fix issue where on some extened groups the multiple records checkbox doesn't show this causes empty string to be passed in which was causing problems when working out if we needed to make changes to the custom value table

5.3.0 release notes: raw from script

Merge pull request #12305 from eileenmcnaughton/dedupe_limit

Re-instate Dedupe limit functionality & fix select toggle functionality

Merge pull request #12382 from eileenmcnaughton/jaap

Fix mis-reporting of false booleans in reports

Fix mis-reporting of false booleans in reports

Merge pull request #12333 from pradpnayak/185

dev/core/issues/189, fixed smarty error

fixed smarty error

Merge pull request #12326 from pradpnayak/185

dev/core/issues/185 “Print selected rows” option missing in Event Participant listing dropdown actions

Wrong key assigned to task action

Re-instate batch limit

Pass cachekey around more & criteria less

It turned out the toggleDuplicates was not working when criteria was set as the validation rule didn't work.

Passing around cacheKey
is easier to validate and we know the cache willbe created at the point of toggle. Use cacheKey instead
in url

Merge pull request #12277 from seamuslee001/dev_core_163

dev/core#163 Improve inclusion of disabled groups when getting all ma…

dev/core#163 Improve way of getting all groups for use in getting all mailing accessable by a user by removing is_active filter alltogether

Merge pull request #12293 from colemanw/arrayUtil

Improve CRM_Utils_Array::recursiveBuild to work with existing arrays.

Improve CRM_Utils_Array::recursiveBuild to work with existing arrays.

Merge pull request #12287 from eileenmcnaughton/5.3

core/#170 minimal fix for fatal on soft_credit field

core/#170 minimal fix for fatal on soft_credit field

Merge pull request #12278 from eileenmcnaughton/5.3

Fix incorrect operator on previous Export fix

Fix incorrect operator on previous Export fix

Set version to 5.3.beta1

Merge pull request #12275 from JMAConsulting/dev_mail_11-1

dev/mail#11 add CRM_Mailing_BAO_MailingJob::del()

Merge pull request #12193 from eileenmcnaughton/dedupe

Fix non-display of conflicts after batch dedupe from dedupe screen

dev/mail#11 add CRM_Mailing_BAO_MailingJob::del()

Merge pull request #12274 from civicrm/5.2

5.2 to master

Merge pull request #12273 from eileenmcnaughton/notices

dev/core#158 group contacts list and export limits results to 500 records

Fix e-notices on export coming from https://github.com/civicrm/civicrm-core/pull/12110/files

Merge pull request #12266 from jitendrapurohit/core-154

dev/core#154 - Can't edit related records when current employer has a…

Merge pull request #10630 from tschuettler/CRM-CRM-20841

CRM-20841 - Dedupe - Show on_hold, is_bulkmail or signature merge…

Merge pull request #12269 from civicrm/5.2

5.2.0 merge to master

5.2.0 release notes (#12100)

* 5.2.0 release notes: raw from script

* 5.2.0 release notes: added boilerplate

* 5.2.0 release notes: first pass of release notes

* 5.2.0 release notes: updated contributors

* 5.2.0 release notes: final edits

* 5.2.0 release notes: update with late RC changes

* Add release notes for 5.1.2

* release-notes - Update 5.1.2

* Fix typo in release-notes for 5.1.2

dev/core#154 - Can't edit related records when current employer has a pending membership

check for pending status instead of pay later

Merge pull request #12259 from seamuslee001/dev_core_163

dev/core#163 Fix issue where disabling a group would block access to …

Merge pull request #12267 from colemanw/dev/core/#152

dev/core/#152 - AdvMulti-Select cleanup

Merge pull request #12265 from agileware/Accessibility-Issue-9

Accessibility #9: Regression - js error due to alertDismissal setting fixed.

dev/core/#152 - AdvMulti-Select cleanup

Accessibility #9: Regression - js error due to alertDismissal setting fixed.

Merge pull request #12264 from civicrm/5.2

5.2 merge to maste

Merge pull request #12262 from JMAConsulting/dev_mail_13-rc

dev/mail#13 - All members should not be excluded from Smart unsubscribe group

Merge pull request #12255 from hosseinamin/master

fix related to issue dev/core#127, in depth overlap of filter related contacts

Merge pull request #10341 from colemanw/CRM-20565

CRM-20565 - Improve ajax dedupe lookups on contact add form

Merge pull request #12257 from eileenmcnaughton/relationship_type

Fix unreleased regression - fatal when editing relationship type Employer

Fix code style - use elseif

Fix for dev/mail#13 - All members should not be excluded from Smart unsubscribe group.

dev/mail#13 : Added unit test

Tidy up comments

Make getWhereString function sane by not appending a limit clause

Fix display of conflicts on duplicate screen.

There are 2 bulk dedupe scenarios
1) dedupe selected
2) dedupe all

In the case of the former, but not the latter, the display to which contacts
are redirected should be filted by 'selected'

Currently the selected filter is being applied whenever conflicts is true, resulting in an
empty form in the latter case.

This fix disambiguates the 2 concepts & attempts to rationalise & clarify related parameters in the flow.

Merge pull request #12258 from eileenmcnaughton/alpha

Add 'Alphanumeric' rule type

Merge pull request #12238 from colemanw/dev/core/#152

dev/core/#152 - Remove AdvMulti-Select custom field type

Merge pull request #12243 from eileenmcnaughton/custom_group

Fix enotice when updating a custom group with is_multiple = 1

dev/core#163 Fix issue where disabling a group would block access to any mailing reports that group was used for

Add 'Alphanumeric' rule type

This type is now available when reading GET parameters with
`CRM_Utils_Request::retrieve()` and it offers improved security over the
widely used 'String' type by being strict enough to reject just about
any conceivable attack payload, while still accepting relatively simple
strings.

Merge pull request #12252 from seamuslee001/dev_core_160

dev/core#160 Set the import progress widget to poll the server every …

Fix unrelease regression - fatal when editing relationship type Employer

Merge pull request #12256 from colemanw/CRM-20621

CRM-20621 - Initial cleanup toward performance improvements

Merge pull request #12254 from jitendrapurohit/core-162

dev/core#162 - Use checksum to access user dashboard

CRM-20621 - Initial cleanup toward performance improvements

Merge pull request #12241 from michaelmcandrew/dev/core#157

ensure that the indexed column is not an FK before deleting the index

fix related to issue dev/core#127, in depth overlap of filter related contacts

ensure that the indexed column is not an FK before deleting the index

Merge pull request #12248 from colemanw/tagsets

Auto-open tagest fields

Merge pull request #12249 from hosseinamin/master

fix of issue dev/core#127 (at gitlab), incorrect cache records for smart groups

Merge pull request #12114 from jitendrapurohit/membership-2

dev/membership#2 - Add membership start date and registration start/end date to schedule reminder

dev/core#162 - Use checksum to access user dashboard

fix for issue dev/core#127, incorrect cache records for smart groups, with testunit

Additional unit test

dev/core#160 Set the import progress widget to poll the server every 5s rather than every 5ms

Merge pull request #12226 from JMAConsulting/dev_core_147

dev/core#147 Use location type machine name as table alias instead of label

Merge pull request #12245 from sritechie26/master

getCaseRoles not working when supplied relationship id.  civicrm_relationship table name changed to alias name rel because its changed in the actual query

Merge pull request #12247 from JMAConsulting/dev_mail_12

dev/mail#12 Incorrect Total Count on mail summary report

Auto-open tagest fields

Merge pull request #11407 from cividesk/CRM-21553

CRM-21553 : profile duplicate field validation for field with primary location type

dev/mail#12 Incorrect Total Count on mail summary report

dev/core#147 Use location type m/c name as table alias instead of label

civicrm_relationship table name changed to alias name rel, because its changed in the actual query but not in this extra Where condition

Merge pull request #12225 from jitendrapurohit/core-149

dev/core#149 - Fatal Error on customvalue get api

additional unit test fixes

Fix enotice when updating a custom group with is_multiple = 1

When updating a group with is_multiple = 1 there should be no change unless ['is_multiple']
is set but existing code is pretty confusing.

Merge pull request #12240 from michaelmcandrew/dev/core#156

Only set defaults when creating a custom field (not when editing one)

Merge pull request #12242 from eileenmcnaughton/up

Move upgrade message to right file

Only set defaults when creating the custom field

Move upgrade message to right file

Merge pull request #12211 from eileenmcnaughton/price

Preliminary tidy up on PriceSet form.

Merge pull request #12158 from agileware/Accessibility-Issue-4

Accessibility #4: Make alerts accessible, Added new setting under misc.

Merge pull request #11672 from omarabuhussein/CRM-21769-improve-unsupported-locale-warning

CRM-21769: Show 'unsupported locale for parsing' warning only when enabling address parsing

Accessibility #4: Make alerts accessible, Added new setting under misc.

dev/core#61 Split Edit Message Templates Permission (#11974)

* dev/core#61 split edit message templates permission