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

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

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

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

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