(NFC) Bring up API folder to style of future coder checker

Merge pull request #13979 from totten/master-phpcbf-2

(NFC) Update various files to pass future civicrm/coder ruleset

Merge pull request #13978 from totten/master-phpcbf

(NFC) Update various files to pass current phpcs

(NFC) Apply upcoming civicrm/coder policies (batch 2)

Method:

* Checkout latest merged branch of civicrm/coder (`8.x-2.x-civi`)
* Run this command to autoclean a batch of 100 files
  `PG=2 SIZE=100 ; find Civi/ CRM/ api/ bin/ extern/ tests/ -name '*.php' | grep -v /examples/ | grep -v /DAO/ | sort | head -n $(( $PG  * $SIZE )) | tail -n $SIZE | xargs phpcbf-civi`
* Go through the diff. For anything that looks wonky, open in an editor and find a better solution.

Note: The automated checker makes good points about awkward indentation, but the automated cleanup often makes it
worse.  So that's why I have to open it up.

Merge pull request #13973 from eileenmcnaughton/array_format6

[NFC] Reformat tricksy file CRM_Mailing_BAO_Mailing

(NFC) Apply upcoming civicrm/coder policies (batch 1)

Method:

* Checkout latest merged branch of civicrm/coder (`8.x-2.x-civi`)
* Run this command to autoclean a batch of 100 files
 `PG=1 SIZE=100 ; find Civi/ CRM/ api/ bin/ extern/ tests/ -name '*.php' | grep -v /examples/ | grep -v /DAO/ | sort | head -n $(( $PG  * $SIZE )) | tail -n $SIZE | xargs phpcbf-civi`
* Go through the diff. For anything that looks wonky, open in an editor and find a better solution

Note: The automated checker makes good points about awkward indentation, but the automated cleanup often makes it
worse.  So that's why I have to open it up.

(NFC) Various updates for future version of civicrm/coder

(NFC) Update to pass new phpcs

(NFC) tests/ - Update to pass new phpcs

(NFC) api/v3 - Update to pass new phpcs

(NFC) CRM/ - Update to pass new phpcs

(NFC) Civi/ - Update to pass new phpcs

[NFC] Reformat tricksy file CRM_Mailing_BAO_Mailing

Merge pull request #13974 from eileenmcnaughton/array_format7

[NFC] Reformat tricksy file CRM/Contribute/Import/Form/MapField

[NFC] Reformat tricksy file CRM/Contribute/Import/Form/MapField

Merge pull request #13969 from eileenmcnaughton/array_format2

[NFC] short array syntax Autoformat - just  the tricksy bits of CRM/Activity

Merge pull request #13970 from eileenmcnaughton/array_format_3

[NFC] array format tricksie file  CRM/Admin/Form/MessageTemplates

Merge pull request #13971 from eileenmcnaughton/array_format_4

[NFC] array format tricksie file CRM_Utils_Rest

Merge pull request #13972 from eileenmcnaughton/array_format_5

[NFC] array formatting tricksy tricksie file (another CRM/UF/Form/Group.php)

Merge pull request #13968 from eileenmcnaughton/array_format

[NFC] short array syntax Autoformat - just CRM/ACL dir

Merge pull request #13915 from colemanw/shortCRM

[NFC] Short array syntax - auto-format CRM directory

Short array syntax - auto-format CRM directory

[NFC] array formatting tricksy tricksie file (another)

[NFC] array formatting tricksy tricksie file

[NFC] array format tricksie file

Array syntax reformat on activity files

Autoformat - just CRM/ACL dir

Merge pull request #13967 from eileenmcnaughton/activity_token

[REF] extract token functions

Merge pull request #13964 from eileenmcnaughton/test

Improve data when known time-dependent-failing test fails

Merge pull request #13965 from eileenmcnaughton/member_test

Remove reference to non-iso date format from membership form tests

[REF] extract token functions

This is an reviewer's commit from https://github.com/civicrm/civicrm-core/pull/12012/files

We are about to merge array formatting changes that will make lots of PRs go stale.

I couldn't get this reviewed & merged before the change but I thought if I could do it
through a sub-commit it would be better than just making it go stale.

This is a simple extraction & I will add merge on pass as it is a reviewer's commit

Remove reference to non-iso date format from membership form tests

The date format wrangling is not handled by datepicker on the js layer so the php
layer no longer needs to test for this

Merge pull request #13961 from eileenmcnaughton/validate

Allow extensions to enable validate.tpl

Merge pull request #13962 from eileenmcnaughton/test_fix_recur

Flush ContributionRecur static cache when flushing processors

Improve data when known time-dependent-failing test fails

Flush ContributionRecur static cache when flushing processors

When creating a processor in a unit test it may not be available to use when creating
a recurring in the same test without flushing the static cache here

Merge pull request #13935 from eileenmcnaughton/recur_cancel_load

dev/core#704 Fix loss of links for recurrings with no payment_processor_id

Allow payment processors to enable validate.tpl

From https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/issues/106
I found that I couldn't enable our validation from the payment processor currently.

It's a bit immature in that our front end presentation is poor,but this allows us to
start improving that

Merge pull request #13699 from mattwire/contributionrecur_report_by_payment_processor_id

Add payment_processor column/filter to recurring contribution report

Merge pull request #13960 from eileenmcnaughton/master

Grammar fixes

Merge pull request #13957 from mattwire/extract_assignpaymentfields

Extract assignPaymentFields

Merge pull request #13899 from eileenmcnaughton/line_item

Move code to assign tax information into shared parent

Move code to assign tax information into shared parent

Grammar fixes

Merge pull request #13944 from agileware/CIVICRM-1140

Added a DB check to prevent deleting exisiting CiviCRM data from database.

Merge pull request #13950 from seamuslee001/convert_receive_renewal_date_membership

Convert receive_date and renewal_date on membership forms to use date…

Merge pull request #13955 from colemanw/Promise

Promise Polyfill for older browsers

Merge pull request #13906 from agh1/whygroupcontribamt

dev/core#655 Don't sum on the contribution total_amount in contribution detail report

Merge pull request #13762 from mattwire/event_register_thankyou_region

Add billingblock region to event registration thankyou to match contribution thankyou

Promise Polyfill for older browsers

Merge pull request #13953 from eileenmcnaughton/pay_api

Add minimal PaymentProcessor.pay api

Merge pull request #13946 from agileware/CIVICRM-1163

Replaced get_headers functions call with Guzzle HTTP request.

Extract assignPaymentFields

Updated the timeout to 500ms.

Fix tests and ensure that receive_date is always set

Merge pull request #13952 from eileenmcnaughton/refund

dev/financial#38 : Support refund payment using payment processor

Merge pull request #13954 from eileenmcnaughton/payment_processor_title

dev/financial#2 add PaymentProcessor.title field

Convert recieve_date and renewal_date on membership forms to use datepicker rather than jcalendar

CIVICRM-1163: Replaced get_headers functions call with Guzzle HTTP request.

Add basic Pay api

dev/financial#2 add PaymentProcessor.title field

There was a long discussion a while back about adding a label or title field to the
civicrm_payment_processor table. It kinda died but since 5.13 adds another field
(contribution_recur.cancel_reason) which is kinda rare I thought we should probably
add this field in the same release as we can better keep most releases to no
schema changes that way.

At this stage the field is not in use - but I figure getting it added will make the
next steps of exposing it easier & there is general agreement we need
something of this nature.

(the cancel reason field will also take further commits to expose in the UI)

dev/financial#38 : Support refund payment using payment processor

Merge pull request #13940 from eileenmcnaughton/recur_form

Rationalise url variables onto shared parent for recurring contribution forms

Merge pull request #13435 from agileware/GREENS-199

Optimise the loading of the CiviCRM Deduplication Exception page

Merge pull request #13934 from seamuslee001/unfork_zeta_components

Unfork Zetacomponents mail and use patch to apply differences

Merge pull request #12998 from agileware/CIVICRM-990

Quote fee levels for regular expression in Participant search.

Merge pull request #13947 from eileenmcnaughton/no_test

[unreleased regression] Add error handling for when no test procesor row exists.

Merge pull request #13949 from eileenmcnaughton/sleep

Test fix

Merge pull request #13948 from civicrm/5.12

5.12

Test fix

Merge pull request #13853 from agh1/5.12.0-releasenotes

5.12.0 release notes

Add error handling for when no test procesor row exists.

This is not really a valid config but for sites that have processors without test rows recent changes
will make that problematic. We can be nice

Update 5.12.0.md

5.12.0 release notes: final edits

5.12.0 release notes: late changes

5.12.0 release notes: bulk of work

5.12.0 release notes: contributors

5.12.0 release notes: reorganizing

Merge pull request #13923 from seamuslee001/update_phpword_patches

Update PHPWord Patches to match the latest versions of their code

Merge pull request #13945 from pradpnayak/subtypeDefaul

dev/core/issues/823, fixed code to set contact type correctly

Add payment_processor_id to recurring contribution report

dev/core/issues/823, fixed code to set contact type correctly

Merge pull request #13938 from jackrabbithanna/core-725

Core/725 - Get state_province_id given non numeric country_id or country parameter

Merge pull request #13943 from mlutfy/setMessageError

Fix status type (error, not fail) for CRM_Core_Session::setStatus

get state_province_id given non numeric country_id or country parameter for APIv3 Address

Merge pull request #13927 from mlutfy/cart-php72

Event Cart: Fix PHP 7.2 fatal error (pass by ref)

Fix status type (error, not fail) for CRM_Core_Session::setStatus

Event Cart: Fix PHP 7.2 fatal error (pass by ref).

Merge pull request #13942 from civicrm/5.12

5.12 to master

Merge pull request #13939 from eileenmcnaughton/website

dev/core/issues/842, Fixed code to update website rather delete

Rationalise url variables onto shared parent for recurring contribution forms

Merge pull request #13051 from mlutfy/customvalue-checkbox-display

dev/core#499 Fix CustomField Checkbox display when values are non-sorted

dev/core/issues/842, Fixed code to update website rather delete

Merge pull request #13936 from yashodha/dev-835

(dev/core#835) Expose Registered by Participant Name field to partici…

Merge pull request #13931 from eileenmcnaughton/recur_cancel_reason

Add shared parent for ContributionRecur forms

minor fix

(dev/core#835) Expose Registered by Participant Name field to participant report

dev/core#704 Fix loss of links for recurrings with no payment_processor_id

In 5.8 changes were made that remove the cancel links from recurring payments where the payment processor object
doesn't load. This is appropriate for cases where there IS a processor but it's disabled. However, it is not
unknown for sites to import contribution_recur records from elsewhere as data records rather than 'functional
records' & it is appropriate to be able to edit those.

We already have a relevant patter - loading payment processor 0 loads the manual processor
(class is CRM_Core_Manual) which has functionality appropriate to non-automated flows
(also known as the paylater processor).

This PR switches to the function CRM_Contribute_BAO_ContributionRecur::getPaymentProcessorObject
which is a skinny wrapper on CRM_Contribute_BAO_ContributionRecur::getPaymentProcessor - which itself
was not actually called from core prior to this change (we didn't remove it as it was better than
functions in play & hence intended to start using it again). No processor is loaded
for an inactive processor so links do not appear there.

dev/core#499 Fix CustomField Checkbox display when values are non-sorted.

Merge pull request #13933 from seamuslee001/master

(NFC) Fix mode on files