(NFC) Update CRM/Contact to match new coder style

Merge pull request #14015 from seamuslee001/nfc_code_style_fixes

(NFC) Minor code style fixes

(NFC) Minor code style fixes

Set version to 5.14.alpha1

Set version to 5.13.beta1

Merge pull request #13551 from eileenmcnaughton/ext_up

Load hooks during upgrade mode

Added myself

Load hooks during upgrade mode

For unknown, svn, reasons extension hooks are not loaded during upgrade
(this doesn't apply to drupal modules) - this causes some fairly serious problems
1) settings are re-loaded & cached with settings from extensions being lost
2) trigger alter hooks are lost this means
 - the summary fields triggers are frequently lost on upgrade
 - hooks that unset various tables to prevent them from being logged can fail, resulting in those log tables being created
 - hooks that specify the table should be innodb can fail to run, resulting in archive format.

I can't think WHY we do this? Presumably there was some problem that would have been better solved another
way but which was solved this way?

Fix "Load hooks during upgrade mode" (45312e1e64dd6af0281fe5fb7f96dbd8be39e524)

In my testing, the commit doesn't do what it says because the symbols are wrong.

Merge pull request #14005 from colemanw/shortSettings

[NFC] Short array syntax - auto convert settings dir

Merge pull request #13995 from eileenmcnaughton/loc_title

dev/financial#2 Update new payment_processor.title field to be localisable

Merge pull request #14007 from colemanw/colorTest

Improve test coverage for CRM_Utils_Color::getRgb()

Merge pull request #14009 from eileenmcnaughton/recur_flush

Improve flushing after creating a processor so it can be used for a recurring in the same run

Merge pull request #14008 from totten/master-hook

(REF) CRM_Core_Resources - Move hook declaration from addCoreResources() to Container.php

Merge pull request #14006 from eileenmcnaughton/payment_pay

Include lower level data when throwing an exception on payment processor.pay

Merge pull request #14002 from colemanw/menuConfig

Menu config screen improvements

Improve flushing after creating a processor so it can be used for a recurring in the same run

Mostly affects tests....

Merge pull request #14000 from eileenmcnaughton/recur_fn

Add unit testing for activity creation when cancelling a recurring, related cleanup

Short array syntax - auto convert settings dir

Update new payment_processor.title field to be localisable

Re-order upgrade to fix upgrade process and ensure there is the runSql step

colorTest

(REF) CRM_Core_Resources - Move hook declaration from addCoreResources() to Container.php

tldr: It's easier to declare `hook_civicrm_buildAsset` listeners at a high-level.

Asset building can use two modes -- production mode writes a static file to
disk when it's being reference.  Debug mode just generates a URL for a
web-service (which in turn dynamically renders the content in a separate
page-view).

If the only mode were production mode, then the code would be on pretty
solid ground.  We could even simplify things a lot by changing the
AssetBuilder contract to replace the hooks with callbacks, as in:

```php
Civi::service('asset_builder')->getUrl('crm-menu.css', function() {
  return '...the css code...';
});
```

Why have a hook?  Because hooks are generally context-free and
always-available.  If we use debug-mode (or if we add a feature to warm-up
the caches during deployment), then we'll want to fire that hook from a
different context (e.g.  web-service or CLI), and the hook-listener needs to
be available in those other contexts.

It would be nice if we could declare hooks generally without needing to edit
the `Container.php` mega-file (e.g.  maybe some kind of annotation).  But,
for the moment, I think this is the best spot that we have in `civicrm-core`
for ensuring that hook listeners are fully/consistently registered.

Merge pull request #13996 from colemanw/menuColor

Configurable menubar color

Include lower level data when throwing an exception on payment processor.pay

Pass menubar preference as a param. Simplify cache mechanics. (#8)

Ex: If an admin uses an API call (CLI/REST) to change the menubar color,
then they don't need to follow-up with a cache-clear.  The new setting just
goes live.

Ex: If a customization (via `civicrm.settings.php` or via extension) decides
on the color scheme programmatically (e.g.  per-domain or per-role or
per-user-preference), then they don't need to clear cache.  Multiple color
schemes can coexist.

Merge pull request #14003 from seamuslee001/sl_fix_4_7_31_upgrade

Fix 4.7.31 Upgrade in multilingual mode

Add unit testing for activity creation when cancelling a recurring, related cleanup

Fix 4.7.31 Upgrade in multilingual mode

Configurable menubar color

Navigation admin - live refresh and links to related settings

Navigation admin - fix icon picker & use select2

Merge pull request #13998 from demeritcowboy/fix-formRule

PHP warnings on Case Dashboard and Find Cases

Use asset-builder to render menubar css

Merge pull request #13999 from eileenmcnaughton/recur_fn

Add unit test for api ContributionRecur.cancel, add support for cancel_reason

Add unit test for api ContributionRecur.cancel, add support for cancel_reason

fix formRule signature

Merge pull request #13992 from seamuslee001/member_node_import_logging_price

(NFC) Update CRM/Member CRM/Note CRM/Logging CRM/Import and CRM/Price…

Merge pull request #13991 from eileenmcnaughton/noisy_guzzle

dev/core#855 Fix guzzle noisiness

Merge pull request #13990 from seamuslee001/admin_activity_batch_new_coder

(NFC) Update CRM/Activity CRM/Admin and CRM/Batch folders to be the f…

(NFC) Update CRM/Member CRM/Note CRM/Logging CRM/Import and CRM/Price folders to be a future coder standard

Fix guzzle noisiness

There is no reason to echo the failure here

(NFC) Update CRM/Activity CRM/Admin and CRM/Batch folders to be the future coder standard

Merge pull request #13987 from seamuslee001/new_coder_pcp_pledge_profile_queue_report

(NFC) Update coding style in PCP, Pledge, Profile, Queue, Report folders

Merge pull request #13986 from seamuslee001/coder_upgrade_uf_tag_sms

(NFC) Update CRM/SMS/ CRM/UF/ CRM/Upgrade/ CRM/Tag/ to be up to speed…

Merge pull request #13985 from seamuslee001/new_coder_crm_utils

(NFC) Bring CRM/Utils folder up to future coder standards

Merge pull request #13988 from seamuslee001/coder_upgrade_dao_files

(NFC) Set _log and _tableName variables to be public

(NFC) Update coding style in PCP, Pledge, Profile, Queue, Report folders

(NFC) Bring CRM/Utils folder up to future coder standards

Merge pull request #13983 from seamuslee001/new_coder_tests

Update Unit test styling to cover the future coder version

(NFC) tests/ - Cleanup phpcbf oddities

(NFC) Set _log and _table_name variables to be public

(NFC) Update CRM/SMS/ CRM/UF/ CRM/Upgrade/ CRM/Tag/ to be up to speed with the new coder standard

Update Unit test styling to cover the future coder version

Merge pull request #13984 from seamuslee001/nfc_comment_fix_ang

(NFC) Fix location of comment to match future coder version

Fix location of comment to match future coder version

Merge pull request #13980 from seamuslee001/new_coder_style_check_api

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

Merge pull request #13982 from colemanw/then

Encourage developers to use .then instead of .done

Merge pull request #13981 from seamuslee001/new_coder_civi

(NFC) Upgrade Civi Folder to the new coder version

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

Encourage developers to use .then instead of .done

jQuery promises have both methods which (if only passing one argument) are
interchangeable. But native js promises do not have a .done method, so let's
start getting our code more future-proof.

(NFC) Upgrade Civi Folder to the new coder version

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