Merge pull request #14860 from eileenmcnaughton/export_1

[REF] [EXPORT] Use columns from processor instead of passing them

Use columns from processor instead of passing them

Merge pull request #14859 from seamuslee001/nfc_audit_update

NFC Update node module versions based on npm audit fix

Merge pull request #14856 from seamuslee001/atomfeeds_cache_convert

Add in Atomfeeds deprecation now that the extension has a new release…

NFC Update node module versions based on npm audit fix

Add in Atomfeeds deprecation now that the extension has a new release with converted cache mechanism

Fix version to match info.xml of new version

Merge pull request #14857 from colemanw/testFormat

Autoformat /tests directory with php short array syntax

Autoformat /tests directory with php short array syntax

Merge pull request #14855 from MikeyMJCO/patch-2

(NFC) Fixing documentation links in readme.

Fixing documentation links in readme.

Merge pull request #14639 from colemanw/i18nCleanup

Multilingual test fix and cleanup

Merge pull request #14716 from eileenmcnaughton/db

Add unit test demonstrating attaching a listener to queries

Merge pull request #14851 from eileenmcnaughton/tt

[REF] [Export] Move temp table creation function to the processor

Merge pull request #14853 from eileenmcnaughton/format

[NFC] code reformatting

Merge pull request #14703 from eileenmcnaughton/act_search_wild

dev/core#538 fix advanced search on activity subject, detail to use wildcards like activity search does

Merge pull request #14658 from MegaphoneJon/mail-46

mail#46 - show label, not value, on contribution custom field tokens

dev/core#538 fix advanced search on activity subject, detail to use wildcards like activity search does

Merge pull request #14735 from mattwire/deprecated_getallpermissions

Remove use of deprecated path in function `CRM_Event_BAO_Event::checkPermission()`

Merge pull request #14843 from mattwire/priceset_to_metadatafield

Switch priceset selector to addField method

Merge pull request #14852 from pradpnayak/SttingsTitle

Changed the title and description of profile_add_to_group_double_optin

Merge pull request #14827 from deepak-srivastava/case-summary-report-case-type-filter

dev/core#1118 correct filteration by case type, re-use parent where()

[NFC] code reformatting

Merge pull request #14845 from colemanw/Set

Set profile greeting fields based on actual contact type

Move temp table creation to the processor

CHanged the title and description of profile_add_to_group_double_optin

dev/core#1118 define filter type for case_type_id

Merge pull request #14848 from eileenmcnaughton/test_fix

[REF][TEST][EXPORT] minor test cleanup & minor cleanup of code it tests

Merge pull request #14849 from eileenmcnaughton/test_up

[NFC] [TEST] code formatting in test class

Merge pull request #14850 from eileenmcnaughton/test_act

[REF] [TEST] cleanup on export activity test

Merge pull request #14838 from eileenmcnaughton/export

[REF] [Export] Stop passing exportParams & sqlColumns around

Merge pull request #14847 from eileenmcnaughton/silly_refs

[REF] Down with php4 compatibility

[REF] [TEST] cleanup on export activity test

[NFC] [TEST] code formatting in test class

[REF][TEST][EXPORT] minor test cleanup & minor cleanup of code it tests

Down with php4 compatibility

Removes a few more cases where parameters are needlessly passed as references

Set profile greeting fields based on actual contact type

Switch priceset selector to addField method

Merge pull request #14841 from chamilwijesooriya/issue-1133

dev/core#1133 Payment method name is displayed instead of label in payment block for Manual payment

dev/core#1133 Payment method name is displayed instead of label in payment block for Manual payment

Merge pull request #14839 from colemanw/Menubar

Menubar - Improve flexibility & remove hardcoded values

Menubar - get rid of hardcoded pixel values in crm.menubar.js

Menubar - Use class instead of id for flexibility

Stop passing exportParams & sqlColumns around

Merge pull request #14835 from eileenmcnaughton/export

[REF] [Export] Simplify setting of address strings

Simplify setting of address strings

Merge pull request #14834 from seamuslee001/dev_core_1056

dev/core#1056 Remove unneded schema file

Merge pull request #14830 from eileenmcnaughton/export

dev/core#1120 remove multiple export handling

dev/core#1056 Remove unneded schema file

Merge pull request #14831 from eileenmcnaughton/format

[NFC] [TEST] Reformat arrays in test classes

Merge pull request #14832 from mattwire/profileformsession

Use singleton to get session instead of relying on ->_session being set elsewhere

Use singleton to get session instead of relying on ->_session being set elsewhere

Reformat arrays in tests

Merge pull request #14828 from seamuslee001/deprecation_notices

Add in Deprecation warnings on Cache functons

dev/core#1120 remove multiple export handling

Update Unit tests to use Civi facade

Merge pull request #14829 from colemanw/1093

dev/core#1093 Make bulkSave defaults optional and ensure correct post hook is called

Merge pull request #14737 from seamuslee001/convert_contribution_recur_date_fields

[REF] Convert contribution_recur dates to datepicker from jcalendar

Merge pull request #14824 from demeritcowboy/case-singleton-url

dev/core#1115 - fix invalid url in singleton/max_instances warning in civicase

Merge pull request #14822 from eileenmcnaughton/export

[REF] [Export] remove another confusing parameter

dev/core#1093 Make bulkSave defaults optional and ensure correct post hook is called

fix singleton warning url

Add in Deprecation warnings on Cache functons

Convert contribution_recur dates to datepicker from jcalendar

Handle unique title in whereclauseSingle as well

Merge pull request #14825 from eileenmcnaughton/recur

[REF] Cleanup up handling of dates for Recurring & Contribution date fields in query class

dev/core#1118 correct filteration by case type, re-use parent where()

Cleanup contribution & recurring contribution date field handling in query object

This gets us to the point where all date fields are handled the same way, which is now tested. Now that recurring
contribution fields have unique keys we can handle them as metadata

Add a swag of test cover on contribution & contribution recur date fields

Merge pull request #14823 from seamuslee001/master

Remove additional custom fields deletegroup functions

Remove additional custom fields deletegroup functions

Add comment about state of GenerateReportData file

Merge pull request #14583 from seamuslee001/convert_custom_fields_cache_group

Convert the contact fields cache group to standard cache backend

Merge pull request #14820 from eileenmcnaughton/recur

Add unique names and unique title for recurrings.

Ensure recently converted groups cache matches previous behabiour my setting withArray as fast for it

Also ensure contact fields uses fastArrayCache

Replicate current practice of resetting of ACL and System caches sensibly

Ensure ACL and other caches are reset as well when flushing ACL Cache

Convert the contact fields cache group to standard cache backend

Merge pull request #14582 from seamuslee001/convert_custom_data_cache_group

Convert Custom Data cache group to be using standard cache backend

Merge pull request #14821 from civicrm/5.16

5.16 to master

[REF] [Export] remove another confusing parameter

Merge pull request #14819 from eileenmcnaughton/export_up

[REF] [Export] Stop passing export params to the merge function

Merge pull request #14722 from JKingsnorth/fix-public-url-link-in-browse

Fix public URL link with hashed URLs in mailing browse screen

Merge pull request #14816 from mlutfy/event8

dev/event#8 Event Cart: save Participant custom field data

Add unique names and unique title for recurrings.

Per
discussion https://github.com/civicrm/civicrm-core/pull/14737/files#r300889057
we hace a need sometimes to display context dependent titles & context independent titles.

This adds unique_title for the purpose

Merge pull request #14815 from vingle/patch-7

Remove the only two defined fonts from selectors

Merge pull request #14675 from seamuslee001/cleanup_prev_next_set_item

[REF] Cleanup usage of CRM_Core_BAO_PrevNextCache::setItem and deprec…

Ensure recently converted groups cache matches previous behabiour my setting withArray as fast for it

Ensure custom data cache use the fastArray Cache

Convert Custom Data cache group to be using standard cache backend

Merge pull request #14581 from seamuslee001/convert_navigation_cache

Convert Navigation cache group to current cache defition system

Stop passing export params to the merge function

We have a string which does or doesn't have merge tokens in it - we don't need  to determine this

Merge pull request #14817 from eileenmcnaughton/export_up

[TEST] export - add unit test covering merge to same address addressee handling

Merge pull request #14818 from totten/master-magic-cache

MagicMerge - Fix ephemeral overrides for aliased properties

Convert static variable to class property (should prevent test leakage"

CRM_Core_Config_MagicMerge - Fix thread-local updates for aliased properties

Overview
--------

`CRM_Core_Config_MagicMerge` allows properties to be temporarily modified
(for the duration the object's lifespan). However, this behavior does
not work correctly if the property-name uses aliasing.

The PR addresses a test-failure that became visible in #14718, and it adds
test-coverage for some typical and atypical examples.

Before
------

* (Vast majority of properties) If a property has a simple name (e.g.
  `maxFileSize`), then `$cache`-reads and `$cache`-writes are consistent.
* (Handful of properties) If a property has a split name (e.g.
  `maxAttachments`, `max_attachments`), then `$cache`-reads and
  `$cache`-writes are *not* consistent.  Writes basically don't work.

After
-----

* Aliased properties work the same as normal/non-aliased.

Technical Details
-----------------

To understand the change, you should skim `MagicMerge` and observe
a few things:

* In `getPropertyMap()`,  observe that:
    * (a) Most properties have a singular name -- for example, `maxFileSize`
      has one name, which will be the same in the high-level facade (`$config->maxFileSize`)
      and in the underlying data-store (`settings->get('maxFileSize')`).
    * (b) Some properties are aliased. Ex: `$config->maxAttachments`
      corresponds to underlying item `settings->get('max_attachments')`.
* In `__get()` and `__set()`, note that properties are loaded initally from
  the underlying data-store once; thereafter, any reads or writes go to
  `$this->cache`. That's a thread-local place that stores temporary revisions.

To see the cache consistency problem, specifically note that:

* `__get()` consistently references `$this->cache[$k]`
* `__set()` was confused; at the start, it used `$this->cache[$k]`,
  and later it used `$this->cache[$name]`.

This PR adds a unit-test which ensures that the thread-local/cached value
works consistently.

Consolidate handling of greeting label

[TEST] export - add unit test covering merge to same address addressee handling

Ensure recently converted groups cache matches previous behabiour my setting withArray as fast for it

Ensure that navigation cache is using fastArray

Convert Navigation cache group to current cache defition system

Also reset memory and system caches when resetting navigation

Additonal place to cause extra flush

Add in utility function for resetting ACL and System Level Caches

Use cache utility function

[REF] Cleanup usage of CRM_Core_BAO_PrevNextCache::setItem and deprecate passing in array of values

Fix conversion from old style format to new and add in unit test of conversion and confirm it stores data correctly in the table

Reformat Function as per Patrick's comments

Update doc block and add in comment detailing the change in function param

Fix Test

Merge pull request #14782 from eileenmcnaughton/export_wip

[EXPORT] add getPreview function

dev/event#8 Event Cart: save Participant custom field data

Remove 'font-family: Arial/etc' on two selectors

For some reason, thesae two selectors have hardcode Arial/etc as the font for form legends and H3 elements. Therefore while a front-end contribution page or profile will inherit the site fonts, but in these two instances it will use Arial/etc.

This can be overriden by the user, but given that there is no other use of Arial in CiviCRM, doesn't seem to make much sense. When theming front-end CiviCRM forms/pages these two elements have to be reset with "font-family: inherit". My guess is it's legacy from a much earlier version of the system (but in making this proposal perhaps the real reason will emerge).