Fix row ID input not being added for delete form

This fixes an issue where the hidden "id" element wasn't added
because the CRM_Core_Action::DELETE handler has an early return.

Fix row ID not being set in contribution/participant form

Fix event participants getting overwritten

dev/financial#50 - Fix contributions getting overwritten

This fixes an issue where contributions may be overwritten when users
open an existing contribution after opening the "Record Contribution"
form (but before actually submitting it).

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

[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

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

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

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

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.

Set a default location in Email.create API call.  Add test for this.

Default location used to create the email is the default LocationType

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

Merge pull request #14103 from jitendrapurohit/core-889

dev/core#889 - Refund throws a fatal error if the main contribution a…

dev/core#530 CiviCase: when creating a new reltype assume client is A

dev/core#530 CiviCase: using label instead of name to be consistent and commenting why

dev/core#530 CiviCase: fixing tests to match new convention

dev/core#530 CiviCase: save new case roles in correct direction for xml and ui

dev/core#530 CiviCase: don't assume that clients or case role contacts are individuals

dev/core#530 CiviCase: translate case role direction coming from xml

dev/core#530 CiviCase: fix Manage Case print report duplicate case roles rows

dev/core#530 CiviCase: fix count on the case dashboard

dev/core#530 CiviCase: making roles available in either direction

dev/core#889 Add test for changing fee selection to 0 and then recording refund

Merge pull request #14425 from eileenmcnaughton/form_clean

[Form cleanup] remove form classes & tpls for Event Component settings & Multisite

Add in uniqueness to cache keys to mitigate clashes on multisite installs

Improve I18n schema by including comments and default and NOT NULL or NULL statements to be more consistant with non lingual schema

Minor improvement

Merge pull request #14483 from civicrm/5.15

5.15

Merge pull request #14468 from seamuslee001/e2e_asset_builder_test_wordpress

Add CRM_Utils_System::sendResponse(). Fix AssetBuilder's status-code on WP.

Incorporate Tim's Cleanup

Merge pull request #14482 from seamuslee001/update_exception_dao

[NFC] Update Exception DAO class

Update Exception DAO class

Slimplify functions down as per Tim's review comments

Update to use PSR HTTP Message Response Interface as per @totten

Move content of returning response into CRM_Utils_System and have user_system based method to set HTTP Status code

Set version to 5.16.alpha1

Set version to 5.15.beta1

Merge pull request #14472 from eileenmcnaughton/from_to_test

[nfc] [test] Skip testGetFromTo on date transition

Merge pull request #14470 from eileenmcnaughton/alike

[NFC][test-fix] centralise use of assertLike for comparing sql

Merge pull request #14448 from eileenmcnaughton/exception2

Fix Exception api to save lower id number as contact 1

Merge pull request #14473 from eileenmcnaughton/custom_filter

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fix Exception api to save lower id number as contact 1

This is already actioned (& tested) via ajax fn markNonDuplicates - this moves the functionality from
the ajax layer to the BAO layer & stdises it under unit tests

Merge pull request #14447 from eileenmcnaughton/exception

Fix Exception to remove result from duplicates

Move sort_name definition to searchFieldMetadata

Merge pull request #14475 from eileenmcnaughton/contact_def

Permit sort_name as a url parameter on advanced search

Convert sort_name to use search form methodology to pass by url

Merge pull request #14476 from eileenmcnaughton/basic_search

[REF] Move retrieval of basicSearchFields to 'get' fn

Move retrieval of basicSearchFields to 'get' fn

This is a partial of https://github.com/civicrm/civicrm-core/pull/14418

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fixes a bug where contact.get does not retrieve results if a custom field is requested and an activity custom field exists

This bug came about because a recent fix made metadata more consistent but it turns out this class was filtering the
metadata by the passed in  - and then iterating through the metadata. Now it iterates the ids instead.

Merge pull request #14469 from eileenmcnaughton/pledge_msg

Fix version for pledge message update, also remove 'please print'

Skip testGetFromTo on date transition

This avoids fails when the expectations were calculated right at the end of a day but then are compared with the new day's values.

Results in this case look like

CRM_Report_FormTest::testGetFromTo with data set #1 ('20190606000000', '20190606235959', 'this.day', '', '')
fail on data set [ this.day ,  ,  ]. Local php time is 2019-06-07 00:00:13 and mysql time is 2019-06-07 00:00:13
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => '20190606000000'
-    1 => '20190606235959'
+    0 => '20190607000000'
+    1 => '20190607235959'

[NFC][test-fix] centralise use of assertLike for comparing sql

This function makes the tests cope with minor changes to whitespace in sql strings being compared

Merge pull request #14461 from colemanw/displayPrefsStyle

Use crm-checkbox-list styling in display prefs tpl