Fix intermittent test fail on NULL array getting contaminated & causing later tests to pass the wrong thing

Merge pull request #14540 from eileenmcnaughton/anettest

Fix A.net to resolve time when using default

Merge pull request #14533 from eileenmcnaughton/deprecate

Deprecate contribution_date as a parameter

Merge pull request #14537 from colemanw/jsDecode

Add CRM_Utils_JS::decode function for decoding js objects

Merge pull request #14524 from eileenmcnaughton/csv

Add csv reader package

Merge pull request #14538 from civicrm/eileenmcnaughton-patch-2

[test] truncate pledge block when cleaning up financial entities

Merge pull request #14535 from colemanw/addressCustom

Address BAO - Handle standard 'custom' param as well as individual fields

Merge pull request #14541 from civicrm/5.15

5.15 to master

Merge pull request #14539 from totten/5.15-tap-yaml

Civi/Test/TAP - Replace Symfony YAML dependency

Civi/Test/TAP - Replace Symfony YAML dependency

This allows two paths:

1. If you have the YAML PECL extension, then that will be used.
   To get the best output, you can install it.
2. If you don't, then it'll output JSON. JSON is considered valid YAML.
   It's not clear that everyone reading the document will handle
   the full range YAML (incl JSON), but... we don't actually have any
   listeners that care to parse the TAP output...

tools/scripts/phpunit - Flip back to preferring 'php' for subprocess

Rationale:

* On the test servers, `php` is wrapper script.
* Outside the test servers, you should be able to use `phpunit` directly
  (rather than calling `tools/scripts/phpunit`), and then it's
  not our problem.

Fix A.net to resolve time when using default

This is presumably passed through to the contribution api & hence it is not so grumpy but when
passed to the activity create fn it is failing when the test is run in isolation (& maybe other scenarios?)
due to 'now' not being parsed

[test] truncate pledge block when cleaning up financial entities

Keep hunting down the test cleanup issue

Add CRM_Utils_JS::decode function for decoding js objects

Merge pull request #14536 from civicrm/5.15

5.15 to master

Merge pull request #14511 from eileenmcnaughton/jungle

Add Unit test for testing smart group flow

Merge pull request #14522 from colemanw/ActivityForm

ActivityForm - Redirect to contact page or activity view in standalone mode

Add Unit test for testing smart group flow

Deprecate contribution_date as a parameter

Merge pull request #14512 from agh1/refactor-before-14349

[REF] CRM_Case_BAO_Case::addcaseActivityLinks to CRM_Case_Selector_Search

Merge pull request #14486 from seamuslee001/contribution_date_search_561

[REF] dev/core#561 Convert Contribution Date field to use date picker…

Merge pull request #14531 from eileenmcnaughton/72

Php 7.2 notices fix on import

Merge pull request #14530 from yashodha/mem_reports

Expose Primary member only/Non primary member only filter in membersh…

Merge pull request #14532 from brianPHM/Updating_contributor_yml

Adding myself to the contributors file

Adding myself to the contributors file

Php 7.2 fix on import

Fixes a known issue where php 7.2 is 'noisy' if you try to count NULL - solution is to use an empty array

Address BAO - Handle standard 'custom' param as well as individual fields

The Address BAO was an oddball in that it took raw unformatted custom field input and processed it.
Other BAOs expect the custom field input to alreay be processed.
This allows it to handle either format.

Expose Primary member only/Non primary member only filter in membership reports

Merge pull request #14523 from kcristiano/cividist

CiviDist fails on BSD flavor of 'cp' with '-r -p' switch to '-R -p'

Add csv reader package

Adds https://csv.thephpleague.com/9.0/ because we can now (since we are php 7.x+ now).

Apart from being a pretty good package it will help with writing tests for csv output

It will also help us cleanup some of our csv code down the way

Merge pull request #14528 from civicrm/5.15

5.15

Merge pull request #14526 from seamuslee001/nfc_fix_template_indent_misc

[NFC] Fix indenting in Misc Setting Template

Merge pull request #14527 from totten/5.15-yaml-dg

(REF) Convert YAML fixtures to JSON. Reset symfony/yaml dependency.

Merge pull request #14505 from seamuslee001/legacy_psr16_multisite

Update PSR16 handling for multisite extension legacy caching group

(REF) Convert YAML fixtures to JSON fixtures. Reset symfony/yaml dependency.

Overview
--------

The test suite includes some tests that read fixture data files (i.e. YAML files). This patch
provides a safer way to address a dependency-management issue with parsing YAML files.

Before
------

* In the last public release (`5.14`), the `symfony/yaml` package was not included with `civicrm-core`.
  However, it was loaded during testing by virtue of running `phpunit6`.
* In the `5.15` RC code, `symfony/yaml` was added -- so that the test suite worked in `phpunit6`.
  However, this also added `symfony/yaml` to the general runtime distribution - which is liable to create conflicts
  with Civi extensions and CMS modules that also bundle `symfony/yaml`. (It's a popular package.)

After
-----

* The test suite no longer uses YAML, and the build no longer requires `symfony/yaml`. Instead, it uses JSON.

[NFC] Fix indenting in Misc Setting Template

CiviDist fails on BSD flavor of 'cp' with '-r -p' switch to '-R -p'

Add in deprecated notice about code path being removed at end of 2019

ActivityForm - Redirect to contact page or activity view in standalone mode

Merge pull request #14501 from mattwire/attachment_description

Display description next to 'paperclip' file icon - usually the filename

Merge pull request #14513 from eileenmcnaughton/export_sc_just_change

dev/core#1015 fix regression on exporting soft credits - more robust …

Merge pull request #14518 from colemanw/paymentEditFix

dev/core#965 Fix destination in payment edit

Merge pull request #14517 from eileenmcnaughton/test_fix

Fix test which fails when run in isolation.

Merge pull request #14471 from eileenmcnaughton/tests

[NFC] test cleanup. Uses CRM_Core_Exceptions, properly outputs unfiltered results

Merge pull request #14520 from civicrm/5.15

5.15

Merge pull request #14484 from seamuslee001/l18n_improved_schema

Improve I18n schema by including comments and default and NOT NULL or…

Merge pull request #14154 from greenpeace-cee/fix-quicksearch-font

Fix placeholder font in Quicksearch

Merge pull request #14516 from eileenmcnaughton/except

[test] Add PrematureExitException to support unit testing where the code would otherwise exit

Merge pull request #13966 from agileware/CIVICRM-1168

Fixed visibility logic on Price field options.

dev/core#965 Fix destination in payment edit

Fix test which fails when run in isolation.

There is a mystery contamination in our test suite somewhere where this test passes in a
suite & not by itself - this fixes although the fact this passes when not alone is
probably behind a few hard to find issues

Merge pull request #14250 from totten/master-select-null

CRM_Utils_SQL_* - Properly interpolate NULL values

Add PrematureExitException to support unit testing where the code would otherwise exit

Bugfix - getting id from hash (#14508)

* Bugfix - getting id from hash

* Styling changes, using single quotes as preferred.

* Correcting public link when hashed url is set

* Bugfix : correcting if statement. Public link setting the hash instead of id when 'hashed' setting is on.

* Coding standards changes. Spacing after close parenthesis and space after keyword

Merge pull request #14506 from seamuslee001/lab_core_1039

dev/core#1039 Make the contact details in the contact summary screen …

Merge pull request #14510 from totten/master-relax-get-to-it

Civi\Angular\ChangeSet - Relax debug-mode consistency check

dev/core#1015 fix regression on exporting soft credits - more robust version

This applies a full fix to a regression we fixed with a less robust fix due to test issues
https://github.com/civicrm/civicrm-core/pull/13748/commits/7f59431188688d4697bff55738fe17996abe442f#diff-e54381bfdf51e31cab376c71ca0d66ffR2716
removed a space that was required when exporting soft credits. The fix makes the addition of a space in
a more sensible way (doesn't rely on it being done correctly in dozens of places

[NFC] test cleanup. Uses CRM_Core_Exceptions, properly outputs unfiltered results

The comment blocks should show all exceptions thrown by a function so trying to stdise a little.

Also fixed failed output of unfiltered results when retrieval fails

[REF] CRM_Case_BAO_Case::addcaseActivityLinks to CRM_Case_Selector_Search

Civi\Angular\ChangeSet - Relax debug-mode consistency check

Overview
--------

The Angular subsystem *sometimes* includes a consistency-check. The consistency check ensures that an HTML document is
read/written in consistency format (i.e. `encode(decode($html)) === $html`). However, the check is not often run, and
it sometimes produces errors on consistency issues which we don't care about.

This approach here is to relaxes the checks in `civicrm-core` and reproduce them in separate developer-oriented tooling.

Before
------

* If an Angular HTML partial is not altered (`hook_civicrm_alterAngular`), then the partial is *not* checked for consistency.
* If a site is in production/non-debug mode, then the partial is *not* checked for consistency.
* If a site is in debug mode, then the partial *is* checked for consistency.

After
-----

* The partial is not checked for consistency.
* A task is recorded on the [Afform roadmap](https://github.com/totten/afform/blob/master/docs/roadmap.md) to include
  better consistency checks in the `afform_auditor` extension.

Merge pull request #13959 from mlutfy/setMessageError

dev/drupal#63 Drupal8: override setMessage(), because drupal_set_message is deprecated

Merge pull request #14507 from yashodha/mem_reports

Expose Primary member only/Non primary member only filter in membersh…

Expose Primary member only/Non primary member only filter in membership reports

dev/core#1039 Make the contact details in the contact summary screen popup on merge screens non bold a-la contact summary screen

Update to Monish's suggestion

Update PSR16 handling for multisite extension legacy caching group

Fix version to compare against

Merge pull request #14500 from mattwire/extract_recaptcha

REF Deduplicate recaptcha handling code

Merge pull request #14504 from seamuslee001/ref_remove_unneded_search_code_contirbution_recur

Remove duplicated code in contribution recur search build function

Merge pull request #14492 from colemanw/sub_type

Fix deletion of contact sub_type in api4

Update contact_sub_type to ignore null values for consistency

For better or worse, every other field gets ignored by bao::add functions when passed as null,
so this test had an incorrect assumption about how it was supposed to work.

Remove duplicated code in contribution recur search build function

Merge pull request #14496 from mattwire/extract_overridedefaultcurrency

REF Extract override default currency function

Merge pull request #14444 from eileenmcnaughton/export_up

[REF] Minor code cleanup on string concatenation

Merge pull request #14436 from eileenmcnaughton/pay

Fix proportional test to test Payment.create & for the test to make more sense

Deduplicate recaptcha handling

Extract override default currency function

Merge pull request #14498 from mattwire/extract_paypalexpress_preprocess

REF: Extract preProcess paypalexpress

Merge pull request #14497 from mattwire/commentsandformat

NFC Comments and formatting only

Merge pull request #14474 from colemanw/placeholderUrl

dev/drupal#21 Remove asterisks from placeholder url

Display description next to 'paperclip' file icon - usually the filename

Extract preProcess paypalexpress

Comments and formatting only

Merge pull request #14494 from colemanw/customPc

Add a couple customField pseudoconstants

Remove asterisks from placeholder url

These symbols get escaped by some CMSs which messes up CRM.url

Merge pull request #13916 from alifrumin/caserelstocore

dev/core#530 Make a_b relationships available as case roles

Add a couple customField pseudoconstants

Merge pull request #14452 from eileenmcnaughton/payment_activity

Create payment activity when creating a payment via the api, test

Merge pull request #14478 from eileenmcnaughton/contact_search

[REF] Move sort_name definition to searchFieldMetadata

Merge pull request #14493 from seamuslee001/more_free

Remove more free calls

[REF] dev/core#561 Convert Contribution Date field and contribution cancel date field to use date picker on search form

Update ContributionAggregate search as well as it uses the field name contribution_date as well as core search

Add in Upgrade Step to handle change in export and profiles and update default return properties and tests and api for the new unique field names

Convert Cancel Date field as well

Inlcude schema changes

Fix test

Fix some more reports

Update generated data

Revert namespace change to contribution

Remove more free calls

Merge pull request #14485 from seamuslee001/domain_uniqueness_cache_key

Add in uniqueness to cache keys to mitigate clashes on multisite inst…

Merge pull request #14431 from eileenmcnaughton/pledge_search

dev/core#561 Convert pledge search form to use metadata functions

Merge pull request #14290 from MegaphoneJon/json-pp-logging

financial#55: support IPN logging on processors that send JSON data as POST

Merge pull request #14489 from aydun/core_1026

Fix core#1026 where an email address synced from a CMS contact is created with no location field.

Merge pull request #14491 from pradpnayak/ActivityPre

Object id is always NULL in ore hook in update activity mode

Extend location_type_id test to api4

Fix deletion of contact sub_type in api4

Fixed Checkstyle Warnings

Merge pull request #14488 from seamuslee001/test_dev_core_889

dev/core#889 Add test for changing fee selection to 0 and then record…

When syncing to UF record, use API instead of DAO to create email
 - with the addition of a default location in this change, this fixes core#1026

Bonus code tidy: if UF email matches the contact primary email, avoid unnecessary update.
If update is needed, use API not direct SQL.