Merge pull request #14819 from eileenmcnaughton/export_up

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

Merge pull request #14816 from mlutfy/event8

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

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…

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).

Add getPreview function

Merge pull request #14811 from eileenmcnaughton/ex1

[REF] [Export] clean up incorporation of order by & group by into ExportProcessor

Merge pull request #14812 from eileenmcnaughton/ee

[REF] [Export] More export Structure arrays to processor

Merge pull request #14813 from eileenmcnaughton/update_tests

[REF] [TEST] [Export] Update export tests to reflect new format

Update export tests to reflect new format

Stop passing processor to itself

More export Structure arrays to processor

Move order by to query generator

Move group by to processor

Merge pull request #14810 from eileenmcnaughton/fix_up

Fix upgrade for membership second reminder

Merge pull request #14808 from eileenmcnaughton/export_sql

[REF] [Export] Further  cleanup - construct  sql more concisely

Merge pull request #14809 from eileenmcnaughton/ex_merge

[REF] [Export] move mergeSameAddress to processor class

Make return for query more like sql

Merge pull request #14793 from eileenmcnaughton/update_test

[TEST][EXPORT] Improve unit test on export.

Fix upgrade for membership second reminder

Updates the upgrade from https://github.com/civicrm/civicrm-core/pull/1348 to apply to the merged civi version

[REF] [Export] move mergeSameAddress to processor class

Move more of the query construction into runQuery

Merge pull request #14807 from civicrm/5.16

5.16 to master

Merge pull request #14764 from elisseck/dev/report/15

dev/report/15 Add fix and tests for contact subtype report filter

[Test][REF] Improve membership export test

Merge pull request #14806 from eileenmcnaughton/ex

[REF] [Export] further code cleanup

Merge pull request #14728 from eileenmcnaughton/last_icing

[REF] final cleanup - call bulkCreate from migrate_utils

[TEST][EXPORT] Improve unit test on export (inc add mergeSameAddress test)"

This updates the methodology used to test the membership export. I also made it check a bunch more fields
while I was at it

Don't pass around paymentTableId

Move code wrangling to the start

Merge pull request #14800 from colemanw/mapperKeys

[REF] [EXPORT] Alter CRM_Export_BAO_Export::exportComponents

Merge pull request #13487 from giant-rabbit/17880-after-first-reminder-for-membership-fix

(dev/core#285) Fixed second membership reminder

add tests for contact subtype null and not null report filters

apply reporting#15 fix against master

Merge pull request #14803 from eileenmcnaughton/ex5

[REF] [Export] move build master copy array to ExportProcessor

Alter CRM_Export_BAO_Export::exportComponents to accept standard MappingField format

Merge pull request #14804 from eileenmcnaughton/eee

[REF] [Export] Remove code that seems unused

Merge pull request #14802 from eileenmcnaughton/ex4

[REF] [Export] Move setting of household properties to processor

Merge pull request #14805 from civicrm/5.16

5.16 to master

Merge pull request #14789 from seamuslee001/fast_array_caches

Ensure recently converted groups cache matches previous behabiour my …

[REF] [Export] Remove code that seems unused

I can't find any evidence of this code being re-refenced. We DO reference a similar one for merge_same_address

Merge pull request #14801 from eileenmcnaughton/ee

[REF] [Export] Minor code relocation

[REF] [Export] move build master copy array to ExportProcessor

Merge pull request #14799 from eileenmcnaughton/move_ex

[REF] [Export] Move replace merge tokens to processor class

Move setting of household properties to processor

Minor code relocation

n

Merge pull request #14795 from eileenmcnaughton/ex

[REF] [EXPORT] Stop passing return Properties

Move replace merge tokens to processor class

Stop passing return Properties

Merge pull request #14797 from eileenmcnaughton/ex_address

[REF] [Export] Move function that parses tokens to address processor

[REF] final cleanup - call bulkCreate from migrate_utils

This has a test CRM_Utils_Migrate_ImportExportTest although I had trouble running it without switching to
some hard-coding (at least in isolation). This switches the existing bulk create action over to use bulkCreate

Merge pull request #14796 from eileenmcnaughton/ex2

[REF] [EXPORT] Minor consolidation of weird mergeSameAddreess nightmare code

Merge pull request #14694 from eileenmcnaughton/cust_field_bulk

dev/core#1093 add a bulkCreate action for many customFields in one go

[REF] [Export] Move function that parses tokens to address processor

Minor consolidation of weird mergeSameAddreess nightmare code

Merge pull request #14794 from colemanw/select2map

Use select2 to display field mappings

Merge pull request #14790 from eileenmcnaughton/post

[REF] [EXPORT] cleanup setting of additional postal fields

Merge pull request #14792 from eileenmcnaughton/ex_x

[REF] [EXPORT] [TLA] Update handling of input fields so that the mapping format is accepted.

Use select2 to display field mappings

Update handling of input fields so that the mapping format is accepted.

At this point we are converting to the mapping format AFTEr the function receives the fields but
changing that is the next step

Merge pull request #14787 from eileenmcnaughton/ex1

[REF][Export] Minor cleanup on household merge properties

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

Add in code comment as per Eileen

dev/core#1093 add a bulkSave action for many customFields in one go

Add bulkCreate function for CustomField with a view towards this being a new protocol for how bulk create actions would look from a code POV which we could expose via apiv4

Minor cleanup on household merge properties

[REF] cleanup setting of additional postal fields

Further cleanup

Merge pull request #14780 from eileenmcnaughton/export_test2

[NFC] [REF] [TEST] [EXPORT] Update various export tests to test csv output with new functions

Merge pull request #14788 from eileenmcnaughton/ex2

[REF] [EXPORT] partial cleanup on adding fields to returnProperties based on usage

[REF] partial cleanup on adding fields to returnProperties based on usage

Merge pull request #14785 from colemanw/contactTypes

[EXPORT] Minor fixes to the export form

[NFC][REF][TEST] update im provider export test to use csv output

The move from testing sqlColumns to csv output is deliberate. While refactoring, cleaning up the sqlColumns stuff
having test on it was VERY helpful. But, at the end of the day the csv IS the output and testing that allows
us to improve the mechanism to develop that output

[NFC] [TEST] fix another test for output testing

Improve another test to test csv output

Note we are implictly checking headers so no need to do so separately

Merge pull request #14786 from civicrm/5.16

5.16 to master

Remove irrelevant relationship types from export options

Use function to get contactTypes instead of hardcoded list

Merge pull request #14783 from agh1/5.16.0-releasenotes

5.16.0 release notes first run

Merge pull request #14773 from eileenmcnaughton/export

 [REF] [export] remove chunk of non-functional code

Merge pull request #14774 from eileenmcnaughton/export_again

[REF] [export] Cleaner handling of additional return properties

Merge pull request #14761 from eileenmcnaughton/unsub1

dev/core#1108 Use api call to retrieve mailing_id

5.16.0 release notes: include César Ramos
per
https://github.com/civicrm/civicrm-core/pull/14518#issuecomment-501881835

5.16.0 release notes: add boilerplate

5.16.0 release notes: raw from script

Contributor key: fix invalid YAML

Merge pull request #14767 from eileenmcnaughton/export_2

[REF] initial extraction of loading saved mapping to qf format

Merge pull request #14769 from eileenmcnaughton/ex_test2

[REF] simplify & add tests on getMappingParams

Merge pull request #14781 from civicrm/5.16

5.16 to master

Merge pull request #14775 from eileenmcnaughton/bgm

dev/drupal#75 Drupal8: fix call to languageNegotiationURL() when called from cv