[NFC] cleanup to declared exceptions, quotes, strict mode.

Merge pull request #16103 from civicrm/5.21

5.21 to master

Merge pull request #16052 from eileenmcnaughton/desc

Fix unreleased regression where processors using getPaymentDescription get a fatal

Merge pull request #16096 from eileenmcnaughton/pf

E notice fix on Price set form

Merge pull request #16078 from colemanw/snippets

Allow Angular modules to provide dynamic HTML snippets

hook_angularModules - Accept 'partialsCallback' for dynamic partials

This is similar to the antecedent commit which used a field 'snippets' that
contained a list of HTML files and their content. However, this
uses a callback function instead of providing the full HTML content.

This makes it safer to work with the Angular metadata without needing to
load the content of all partials into memory.

E notice fix on Price set form

Merge pull request #16088 from eileenmcnaughton/paypal_clean

Paypal class cleanup

Merge pull request #16094 from pradpnayak/ActivityTypeName-Case

dev/core/issues/1470, show labels instead name for activity type

dev/core/issues/1470, show labels instead name for activity type

Merge pull request #16093 from civicrm/5.21

5.21

Merge pull request #16092 from seamuslee001/5.21

Fix Release note header

Fix Release note header

Merge pull request #16091 from civicrm/5.21

5.21

Merge pull request #16057 from eileenmcnaughton/act_perm521

dev/core#1444 Permit activities with campaigns on contact tab for con…

Merge pull request #16090 from seamuslee001/5.21

Add release-notes/5.20.2

Add release-notes/5.20.2

Merge pull request #16080 from eileenmcnaughton/org_up

Remove unnecessary query when updating org, if no name update.

Remove unnecessary query when updating org, if no name udpate.

This removes an unnecessary query that can be locking at high volume.

The updateCurrentEmployer function sets the organization_name for any related contacts. We don't need to
do this if we are creating the organization for the first time, or updating the organization but not setting (passing in)
organization_name. We can check & avoid in php.

Also we don't need to check for an existing employer_id unless we are dealing with an Individual record. We can
skip that query for Orgs & Households.

These are cheap queries but at high volume can contribute to locks

Fix unreleased regression where processors using getPaymentDescription get a fatal

We added the new property bag in https://github.com/civicrm/civicrm-core/pull/15697 but in the back & forth the
call to the function getDescription was retained but the function was removed. This would leave the
core processor that calls it (Paypal std) with a fatal. The known external processor using it (Omnipay)
also hits a notice because the signature changed ($params became optional). From the Omnipay
pov I think it makes sense just to stop overriding the function - hence I am sneaking a tiny
change into this to make that possible.

Note the doPayment function ensures that propertyBag is set. There is a small chance that an external processor
is fully overriding doPayment AND calling this - that would be the case for Omnipay except I'll remove
the override in conjunction with this - so I've added a few lines to ensure it is set.

Note the test would fail with the changes to Payment_Dummmy & not the changes to Core_Payment so it provides
test cover

Merge pull request #16086 from civicrm/5.21

5.21

Merge pull request #16087 from seamuslee001/deb_core_1467

dev/core#1467 - Multiple participant registration sends only p1 amoun…

Paypal class cleanup

- remove handling for returned error - this is never returned now
- remove url parameter on invokeApi - this is never passed to it
- throw an exception rather than use a fatal if curl not installed

dev/core#1467 - Multiple participant registration sends only p1 amount to the processor

Merge pull request #16072 from eileenmcnaughton/paypal_extract

Code cleanup Update Paypal invokeApi to only throw exceptions.

Merge pull request #16081 from artfulrobot/payment-property-bag-add-getters

Payment PropertyBag: Add generic getter, setter methods for convenience

Merge pull request #16083 from artfulrobot/advanced-search-contrib-source-search-dev-core-1468

Fix searching for contribution source in advanced search

Fix searching for contribution source in advanced search

Update Paypal invokeApi to only throw exceptions.

Currently invokeAPI will return an error for a curl function and throw a processorException for an outcome other than success.

In just about every instance where it is called the Core_Error is converted into an exception anyway - this
can all be cleaned up as a follow up. The exceptions are

doQuery -  called by doPayment
doDirectPayment - this is called by doPayment which converts it into an exception. Calling doDirectPayment directly was deprecated in 4.6
createRecurringPayments - called by doExpressCheckout which in already throws exceptions in other scenarios,
updateSubscriptionBillingInfo - we can update the one place that calls this in core as a follow up like https://github.com/civicrm/civicrm-core/pull/15676

However, note that an exception was already thrown in all these places in the more common error scenario of the action not succeeding as opposed
to the rare curl error scenario

Follow ons to consider
1) introduce more nuance into PaymentProcessorExceptions - extend the exception class with PaymentProcessorConfigException, PaymentProcessingException or similar to differentiate
2) remove all the handling for invokeApi to have returned an error object.
3) fix the code that calls updateSubscriptionBillingInfo
4) ensure docs reflect that doPayment should throw an exception. AFAIK we have not yet documented other 'do' functions & I'd prefer to stdise them first

Payment PropertyBag: Add generic getter, setter methods for convenience

Allow angular modules to provide dynamic html snippets

Merge pull request #16077 from civicrm/5.21

5.21

Merge pull request #16076 from alifrumin/rn5.21

5.21 Release notes: First Pass

Merge pull request #16075 from seamuslee001/sig_status_bounce

[REF] Convert more petition signature fatals to status bounce

Release Notes 5.21 - Sorting and Annotating

Merge pull request #16074 from civicrm/5.21

5.21

[REF] Convert more petition signature fatals to status bounce

Merge pull request #16067 from eileenmcnaughton/fatal

Swap out fatal for statusBounce for disabled or unpermitted forms

Merge pull request #16071 from jitendrapurohit/521rc

dev/core#1463 Fix record payment form errors

Fix record payment form errors

Merge pull request #15676 from artfulrobot/handle-exceptions-from-payment-processors

We should give user feedback in case of a Payment Processor Exception

Swap out fatal for statusBounce for disabled or unpermittted forms

I have deliberately left copy editing & translate out of scope & focussed on a specific narrow pattern

Merge pull request #16070 from civicrm/5.21

5.21

Merge pull request #16069 from seamuslee001/5.21

Add release-notes/5.20.1

Add release-notes/5.20.1

Merge pull request #16063 from seamuslee001/unit_test_1460

[NFC] Add in unit test for dev/core#1460

Merge pull request #16066 from civicrm/5.21

5.21

[NFC] Add in unit test for dev/core#1460

Merge pull request #16060 from artfulrobot/payment-property-bag-add-props

Add extra Payment PropertyBag properties

Merge pull request #16064 from civicrm/5.21

5.21

Merge pull request #16061 from eileenmcnaughton/cont_hook

dev/core#1460 move post hook to avoid errors with custom fields created in the hook

dev/core#1460 move post hook to avoid errors with custom fields created in the hook

Per dev/core#1460 a fatal error can occur if custom data is created in the post hook. In addition I note it makes sense to
include custom data before the rollback

Merge pull request #16058 from eileenmcnaughton/mem_js

Fix js error on choosing membership type

Merge pull request #16056 from eileenmcnaughton/add_part

[REF] Minor simplification on additional_participants checks

Add extra Payment PropertyBag properties

Fix js error on choosing membership type

I'm seeing a js error when I select a membership type in the back end form. It seems I have recurProcessors assigned
& they have an onChange that calls
buildAutoRenew - which is non existant if the action is not 2. It seems to me that the code is simplest when functions
can be relied on to exist and aree called conditionally - this makes that change, also ensuring 'something' is
assigned for recurProcessors to prevent the error when it's not defined that probably let to the fn being encased in an if

I can't spot any obvious regression cause & perhaps there is something in my config but this might be a regression

dev/core#1444 Permit activities with campaigns on contact tab for contacts without 'administer CiviCampaign'

It seems that if an activity has a campaign it is suppressed from the activity table unless the user has 'administer CiviCampaign'.
This is inconsistent with the code to view the activity - which permits the activity to be viewed but the campaign cannot be edited
It is also inconsistent with activity search behaviour

Minor simplification on additional_participants checks

Merge pull request #16054 from seamuslee001/sort_contact

[REF] Refactor Contact Search form to use standardised getSortID func…

Merge pull request #16053 from seamuslee001/activity_sort

[REF] Refactor Activity Search to use parent getSortID function

Merge pull request #16055 from seamuslee001/campaign_sort

[REF] Refactor Campaign Search form to use standardised getSortID fun…

[REF] Refactor Campaign Search form to use standardised getSortID function

[REF] Refactor Contact Search form to use standardised getSortID function

Merge pull request #16050 from eileenmcnaughton/qf

[NFC] code cleanup

[REF] Refactor Activity Search to use parent getSortID function

Merge pull request #15942 from seamuslee001/contact_fields_metadata

Add in searchFieldMetadata for contact_tags, contact_type, group and …

Merge pull request #16051 from civicrm/5.21

5.21

Merge pull request #15927 from eileenmcnaughton/event_form

[REF] Move wrangling of Front end form contribution param for autoRenew back to form

Only add in the additional metadata if we are also adding them to the form

Add in searchFieldMetadata for contact_tags, contact_type, group and deleted_contacts fields for contribute, pledge searches

Merge pull request #16047 from eileenmcnaughton/521

dev/membership#21 Fix regression causing transfer of membership to another contact if they paid

[NFC] code cleanup

Add some throws statements, fix some double quotes to single, fix some comparisons to strict

Merge pull request #16049 from civicrm/5.21

5.21

Merge pull request #16038 from seamuslee001/proper_event_badge_fix

Proper Fix for dev/core#1422

Merge pull request #16034 from seamuslee001/dev_core_1449

dev/core#1449 Do not dispatch pre/post hooks during upgrade

Merge pull request #16045 from seamuslee001/nfc_settings_array

[NFC] Add in a unit test of passing an array into the Civi::settings …

dev/membership#21 Fix regression on transferring member to another contact if they paid

dev/core#1449 Do not dispatch pre and post hooks whilst we are in an upgrade process

Add in code comments to point to new gitlab and reasoning for the change

Merge pull request #16044 from seamuslee001/remove_deprecated_function

[REF] Replace Deprecated function CRM_Core_BAO_Setting::setItem with …

[NFC] Add in a unit test of passing an array into the Civi::settings set facade and getting it back

Merge pull request #16042 from agileware/CIVICRM-1392

Update Smart Group form uses different labels for Group Title and Group Description

Merge pull request #16043 from agileware/CIVICRM-1391

Update Smart Group form unsets the existing Group Types

[REF] Replace Deprecated function CRM_Core_BAO_Setting::setItem with the relevant Civi::contactSettings function

CIVICRM-1391 Update Smart Group form unsets the existing Group Types for the Group

CIVICRM-1392 - Group, Update Smart Group uses different labels for Group Title and Group Description

Merge pull request #16040 from civicrm/5.21

5.21

Merge pull request #16033 from seamuslee001/dev_core_1447

dev/core#1447 Fix hard fail on upgrade due to duplicate names in the …

Add in unit test locking in the fix

Proper Fix for dev/core#1422

Merge pull request #15975 from eileenmcnaughton/setting

dev/core#1425 More aggressively deprecate old setItem & setOptionValue functions on BAO_Setting

Merge pull request #16037 from civicrm/5.21

5.21

Merge pull request #16036 from agh1/5.21.0-releasenotes

5.21.0 release notes: initial run

5.20.0 release notes: added boilerplate

5.20.0 release notes: raw from script

Throw and catch PaymentProcessorExceptions and give user feedback

dev/core#1447 Fix hard fail on upgrade due to duplicate names in the database when trying to fix the option group name

Merge pull request #16031 from agileware/CIVICRM-1389

Fix typo, Mailign should be Mailing

CIVICRM-1389 Fix typo, Mailign should be Mailing

Merge pull request #16006 from demeritcowboy/case-role-manager-api

dev/core#1426 Return correct manager role info during Case api get

Set version to 5.22.alpha1