Merge pull request #14874 from eileenmcnaughton/export_1

[REf] [Export] Remove deprecated componentPaymentFields function

Merge pull request #14875 from eileenmcnaughton/export_2

[Ref] [Export] Remove exportComponent function - it's not adding much here

Merge pull request #14854 from eileenmcnaughton/validate

Do not launch raw js alert  jqueryValidation fails

Merge pull request #14895 from mattwire/optiongroup_checkcrash

Don't let optiongroup check crash

Don't let optiongroup check crash

Merge pull request #14879 from eileenmcnaughton/import_gender_test

[REF] [Test] Add test to cover handling of 'gender_id' on import, remove unused code.

Merge pull request #14883 from totten/master-daotest

CRM-20455 - Allow CRM_Core_DAOTest to pass w/packages#259

Merge pull request #14884 from mlutfy/event6

dev/event#6 Follow the 'same email' participants config setting for backend participants

dev/event#6 Follow the 'same email' participants config setting for backend participants

DAOTest - Use docblock for type

This patch is going in as part of some cleanup/refactoring in a mid-cycle
change. Using the docblock allows the type to be communicated without
breaking on mid-cycle contract change.

CRM-20455 - Allow CRM_Core_DAOTest w/packages#259

Merge pull request #14672 from seamuslee001/load_smart_cache_after_deleting_group_contact_records

Invalidate smart group cache for group following deletion of group_c…

Merge pull request #14882 from totten/master-query-event

CRM-20455 - Add specialized event for queries

Remove the js alert message when jqueryValidation fails on frontend pages

It's really really ugly & as discussed without it
a) extensions can still opt to display the errors themselves and
b) the inline errors still appear.

Note this will have no real affect on existing code - the ugliness of this has been
a blocker to getting into front end validation & at this stage only the
omnipay extension is trying to work in this space.
https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/pull/128

I think ideally with Omnipay it DOES makes sense to display messages
near the checkout button - which might be a future step

CRM-20455 - Add specialized event for queries

Overview
--------

The recently merged PR for CRM-20455, https://github.com/civicrm/civicrm-packages/pull/180, uses the
GenericHookEvent for modifying all SQL queries.  GHE is flexible, but it
comes with a slight performance penalty (juggling a few internal arrays).
The penalty is usually too small to care if you're firing one hook. However, with CRM-20455,
we can foresee the event firing hundreds or even thousands of times within a given page-view.

This PR adds a smaller/simpler class for use with `modifyQuery`. There will be a different PR to use it.

Before
------

* Don't have `QueryEvent`

After
-----

* Do have `QueryEvent`

Add test to cover handling of 'gender_id' on import, remove unused code.

I'm just venturing into import to fix a bug & wrote this test to step through the code. Committing as it's passing.

In the process I found that it's impossible to set 'gender' as an import field & only 'gender_id' is
available as a field so removing 'gender'

Merge pull request #14861 from eileenmcnaughton/anet

[test] Catch A.net exception & ignore

Merge pull request #14833 from seamuslee001/ids_ip_logging_improvements

Ensure that if present the HTTP_X_FORWARDED_FOR IP address is used in…

Merge pull request #14878 from civicrm/5.16

5.16 to master

Merge pull request #14876 from eileenmcnaughton/is_front

dev/core#378 Determine front end pages in drupal

Merge pull request #14718 from totten/herb-private-hier

Move log and compilation dirs from "Runtime" to "Paths"

Invalidate smart group cache for group following deletion of group_contact row

Add in function to invalidate smart group cache rather than building it up again when contact removal is deleted

Remove if check

Merge pull request #14873 from eileenmcnaughton/extract

[REF] extract loadSavedMapping

Determine front end pages in drupal

Alternative to https://github.com/civicrm/civicrm-drupal/pull/546/files that I think narrows the scope
since it means we are not changing an existing property

Merge pull request #14862 from mlutfy/event9

dev/event#9 Event Templates: do not set the Start/End dates

[REF] extract loadSavedMapping

This is a very 'untouched' extraction for simple replication. Ideally more follow up will be in subsequent PRs (now or down the track

Merge pull request #14871 from eileenmcnaughton/import_post

[IMPORT] [code quality] [REF] Improve readability of variable assignment

Merge pull request #14869 from eileenmcnaughton/import

[IMPORT] [code-quality] Remove usage of nullArray

Merge pull request #14872 from eileenmcnaughton/import_mapper

[IMPORT] reduce php4 support, don't pass by ref when not required

Remove exportComponent function - it's not adding much here

Remove deprecated componentPaymentFields function

Merge pull request #14846 from eileenmcnaughton/ex_pos

[EXPORT] Fix unreleased regression where postal addresses are not suppressed when empty

[IMPORT] reduce php4 support, don't pass by ref when not required

[REF] Improve readability of variable assignment

Merge pull request #14867 from eileenmcnaughton/mapping

[METADATA] Add titles to Mapping xml & DAOs

Merge pull request #14868 from civicrm/5.16

5.16 to master

Remove usage of nullArray

Merge pull request #14860 from eileenmcnaughton/export_1

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

Merge pull request #14866 from alifrumin/5.16.0

Annotating and Sorting 5.16.0 release notes

Add titles to Mapping xml & DAOs

Annotating and Sorting 5.16.0 release notes

dev/event#9 Event Templates: do not set the Start/End dates

Catch A.net exception & ignore

Hopefully we can avoid false negatives like
https://test.civicrm.org/job/CiviCRM-Core-PR/27923/testReport/junit/(root)/CRM_Core_Payment_AuthorizeNetIPNTest/testIPNPaymentRecurNoReceipt/
without losing too much

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

Ensure that if present the HTTP_X_FORWARDED_FOR IP address is used instead of the SERVER_ADDR when logging items from the IDS

Update Core IP address to match IDS Pattern

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

Fix unreleased regression where postal addresses are not suppressed when empty

I had a suspicion about this as the variable showed as unused in my IDE so I upgraded the test that was failing to notice
and it started failing on this.

Bug is that the lines were not being added to the query to suppress empty addresses.

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

(dev/cloud-native#3) Container, Extension Classloader - Change CIVICRM_TEMPLATE_COMPILEDIR to 'civicrm.compile'

The broader PR seeks to make path computation more intuitive, which requires computing the
path to `templates_c` using a function. This PR replaces the reference to `CIVICRM_TEMPLATE_COMPILEDIR`
with a function-call to `Civi::paths()->getPath()`.

Why change these two files in the same commit? Because they're basically doing the same
thing (writing an executable PHP file to the template cache), and the demonstration
of their safety is basically the same.

Is it safe to change this reference to `CIVICRM_TEMPLATE_COMPILEDIR` in
`Civi\Core\Container::loadContainer()` and `CRM_Extension_ClassLoader::register()/::getCacheFile()`?  Yes, I believe so:

* Look at `Civi\Core\Container::boot()`.
* Observe that it initializes services in two general stages:
    * First, the `$bootServices` (`runtime`, `paths`, `userSystem`, etc)
    * Second, the DB/extension-dependent services (`CRM_Utils_Hook`, `CRM_Extension_System`, `loadContainer()`).
* The first stage services (e.g. `paths`) provide enough information
  to process `Civi::paths()->getPath('[civicrm.compile]/foo')`.
* All the modified lines run as part of the *second* stage (i.e. after
  the `$bootServices` have been initialized).

(dev/cloud-native#3) CRM_Core_IDS - Change CIVICRM_TEMPLATE_COMPILEDIR to 'civicrm.compile'

The broader PR seeks to make path computation more intuitive, which requires computing the
path to `templates_c` using a function. This PR replaces the reference to `CIVICRM_TEMPLATE_COMPILEDIR`
with a function-call to `Civi::paths()->getVariable()`.

Is it safe change to make this change in `CRM_Core_IDS`?  Will the
`Civi::paths()` be callable?  Yes, here's why I believe so:

* (Big picture) Civi completes the bootstrap process
  (`CRM_Core_Config::singleton()`) and brings critical services online
  *before* the invoker does any routing or page-processing.  The invoker
  (`CRM_Core_Invoke::runItem()`) is the only thing which uses
  `CRM_Core_IDS`.

* (Little picture) In the changed function
  `CRM_Core_IDS::createBaseConfig()`, observe that it calls
  `CRM_Core_Config::singleton()` before running this modified code.  That's
  a tell-tale sign of code that already relies on having a booted system.

(dev/cloud-native#3) Runtime.php - Remove redundant check

1. Previous commits generally removed any responsibility for
`CIVICRM_TEMPLATE_COMPILEDIR` from `Runtime.php`, so the validation
is longer local/relevant.

2. The `CIVICRM_TEMPLATE_COMPILEDIR` is still used in
`CRM_Utils_File::baseFilePath()` which has a similar validation step.

(dev/cloud-native#3) configAndLogDir,templateCompileDir - Move from "Runtime" to "Paths". Use hierarchy.

This slightly changes the computation of the private log and compilation
directories so that:

* The definition appears more declarative.
* The default is computed on-demand (i.e. later in the process).
* The values can be overriden more intuitively.

Before
------

The properties `$config->configAndLogDir` and `$config->templateCompileDir`
are computed as part of a procedural initialization step in
`CRM_Core_Config_Runtime`.

After
-----

The properties `$config->configAndLogDir` and `$config->templateCompileDir`
are computed on-demand by `Civi\Core\Paths`.

The values of these properties may:

* Be computed automatically in a way that matches the previous output
  (i.e.  `configAndLogDir`=~`CIVICRM_TEMPLATE_COMPILEDIR/../configAndLog`;
  `templateCompileDir`=~`CIVICRM_TEMPLATE_COMPILEDIR`).
* Be customized as a pair (by setting `civicrm.private`)
* Be customized individually (by setting `civicrm.logs` or `civicrm.compile`).