Fix incorrect reference on extracted function

Per https://github.com/civicrm/civicrm-core/pull/15803#pullrequestreview-317543503

Merge pull request #15851 from demeritcowboy/document-customdata

[NFC] dev/core#1352 - Add comments why there's two custom blocks on new case form

Merge pull request #15830 from eileenmcnaughton/dedupe4

[REF] refactor on nasty Dedupe function

Merge pull request #15810 from eileenmcnaughton/mem_fix

[REF] calculate 'amount' on ContributionPage in a shared way in one scenario

document why there's two custom blocks

Merge pull request #15850 from civicrm/5.20

5.20 to master

Merge pull request #15844 from mlutfy/fixMsgTpl

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

Merge pull request #15838 from demeritcowboy/getcasereport-split

[REF] XMLProcessor/Report - Extract method to make it more testable

Merge pull request #15825 from seamuslee001/dev_core_183_logging

dev/core#183 Convert Logging report summary report to using CRM_Utils…

Merge pull request #15649 from JMAConsulting/core-1346

core#1346: Tagsets should display vertically in activity/new case forms

Merge pull request #15843 from totten/master-simplehead

(NFC) (dev/core#878) Simplify standard header. Remove year+version.

Merge pull request #15839 from demeritcowboy/should-it-be-static

CiviCase - Make static function `allActivityTypes()` static

Merge pull request #15771 from eileenmcnaughton/partially

dev/financial#100 Remove 'partially paid' as a contribution status option for 'record p…ayment'

fixup templates/CRM/Admin/Form/Setting/Smtp.hlp

(NFC) (dev/core#878) Simplify copyright header (CRM/*)

(NFC) (dev/core#878) Simplify copyright header (templates/*)

(NFC) (dev/core#878) Simplify copyright header (misc)

(NFC) (dev/core#878) Simplify copyright header (api/*)

(NFC) (dev/core#878) Simplify copyright header (tests/*)

(NFC) (dev/core#878) Simplify copyright header (Civi/*)

(NFC) (dev/core#878) Simplify copyright header template

https://lab.civicrm.org/dev/core/issues/878

(NFC) (dev/core#878) Simplify '@copyright' annotation

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

make static function static

pull out to make more testable

Merge pull request #15833 from yashodha/participant_edit

(dev/core#1065) Contact missing in membership renewal form

Merge pull request #15835 from demeritcowboy/unused-reporturl

[NFC] Remove unused variable not used since v4.4

unused variable since v4.5

Merge pull request #15756 from eileenmcnaughton/qfbug

[REF] very minor tidy up

(dev/core#1065) Contact missing in membership renewal form

[REF] calculate 'amount' on ContributionPage in a shared way in one scenario

I have discovered a lot of tests are creating invalid contributions - https://github.com/civicrm/civicrm-core/pull/15706

So far the issues have been in the test + us permitting something that doesn't work on the form - ie https://github.com/civicrm/civicrm-core/pull/15771

I'm trying to work through them all & then we can ideally validate payments in general. In this case
it turns out that because 'amount' is currently a 'required' parameter the tests have 'any value' stuck in there.
In a real submission it would be calculated so I'm trying to share the code that would do that with
the path used by the test (& in this case the api) and to move towards getting the tests valid

Merge pull request #15832 from eileenmcnaughton/mail_s

[NFC] minimal test for Mailing.update_email_resetdate

[NFC] minimal test for Mailing.update_email_resetdate

ALso removes a couple of unused variables, adds a few exception comments

Merge pull request #15823 from seamuslee001/dev_core_183_mailing

dev/core#183 Convert creating of temporary tables in CRM/Mailing file…

[REF] refactor on nasty Dedupe function

I found with other pieces of refactoring static functions that creating a class to support the refactor
made it much easier as that way I could leverage the fact thatt classes have properties and
get away from the crazy param passing that characterizes a nest of static functions.

This adds a class for that purpose and moves a small chunk of code handling into the class.

The goal is to move the handling is done purely for the form back onto the form....

dev/core#183 Convert creating of temporary tables in CRM/Mailing files to using CRM_Utils_SQL_TempTable

dev/core#183 Convert Logging report summary report to using CRM_Utils_SQL_TempTable

Merge pull request #15822 from eileenmcnaughton/label

Fix LabelFormat class to throw exceptions rather than fatals

Merge pull request #15803 from eileenmcnaughton/participant

Add unit test for searching by participant_status_id & switch to generic function

Fix LabelFormat class to throw exceptions rather than fatals

I also added some throws to the comment blocks & fixed on fn not to be called by ref

Add unit test for searching by participant_status_id & switch to generic function

Merge pull request #15791 from seamuslee001/event_search_url_variables

Fix Participant Search in force mode to support query parameters in URL

Merge pull request #15812 from eileenmcnaughton/fee

Convert fee selection test to use Order api

Merge pull request #15809 from eileenmcnaughton/except

[NFC] declare thrown exceptions

Convert fee selection test to use Order api

This switches the changeFeeSelection test to use Order api to set up the line items etc correctly

I had to remove a couple of checks that were no longer true but they were never 'true' in a
real working flow

Merge pull request #15699 from mattwire/participant_cleanup_completeOrderPBRef

Don't pass \$input to completeOrder by reference - return values are never used

Fix Participant Search in force mode to support query parameters in URL in Advanced Search.

[NFC] declare thrown exceptions

Merge pull request #15807 from eileenmcnaughton/part_status

Handling comma separated list of participant statuses in url

Merge pull request #15806 from eileenmcnaughton/member_urls

[REF] Add setSearchMetadata to each of the classes that extend CRM_Core_Form_Search

Merge pull request #15805 from eileenmcnaughton/cont_urls

[REF] extract setDeprecatedDefaults

Merge pull request #15802 from eileenmcnaughton/complete_order_to_calling

Test fixes - addresses more invalid setup data

Merge pull request #15799 from eileenmcnaughton/valid

FinancialTrxnTest - Fix unit test to  be 'legit'

Update url handling of select fields

Merge pull request #15808 from civicrm/5.20

5.20

Merge pull request #15804 from eileenmcnaughton/fns

dev/core#1381 fix fatal on case search (unreleased regression

Merge pull request #15795 from seamuslee001/dev_core_183_mailing_test

dev/core#183 Convert creating of temp table in mailing test to use st…

Add setSearchMetadata to each of the classes that extend CRM_Core_Form_Search

This doesn't do something in itself but makes it simpler to handled ensuring url_params work & stdises them

[REF] extract setDeprecatedDefaults

Remove 'partially paid' as a contribution status option for 'record payment'

Fixes a bug where it is possible to select contribution statuses that do not result in valid financial
transactions. Specifically the 'Partially Paid' option creates no payment transaction and any subsequent
financial_trxns get the wrong line item allocations as a result.

dev/core#1381 fix fatal on case search (unreleased regression

dev/core#183 Convert creating of temp table in mailing test to use standardised CRM_Utils_SQL_TempTable interface

[REF] very minor tidy up

This just brings the 3 lines that built this 'array within an array' together & makes them a little more
readable

Fix data to be valid for one of the a.net tests

The tests were munging the line items in set up - this fixes it - but only for one so far

Merge pull request #15801 from eileenmcnaughton/format

[NFC] reformat CiviUnitTestCase.php

Fix unit test to  be 'legit'

This unit test is one of a bunch that came up as not creating valid payments. The issue is
in the test set up methodology (the 'partial_payment_to_pay' key doesn't work).

On tidying up the test proved to be mostly cruft from an earlier iteration

Merge pull request #15798 from seamuslee001/dev_core_183_imports_2

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable …

[NFC] reformat CiviUnitTestCase.php

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable fuctionality and remove Incomplete Jobs handling as it is not called

Merge pull request #15790 from civicrm/5.20

5.20

Merge pull request #15787 from eileenmcnaughton/recur_ug

Remove unhelpful alert from contribution search

Merge pull request #15665 from MikeyMJCO/patch-1

Create a GitLab general issue template

Merge pull request #15789 from seamuslee001/5_19_1_version_notes

5.19.1 Version Release Notes

Merge pull request #15783 from eileenmcnaughton/sort_id

[REF] replace copy & paste with shared getSortID function.

Update 5.19.1.md

Add in release notes for 5.19.1 drop

Merge pull request #15786 from eileenmcnaughton/cont_type

Remove obsolete warning

Remove unhelpful alert from contribution search

[REF] replace copy & paste with shared getSortID function.

I've replaced this repetitive code chunk across the forms that inherit from CRM_Core_Form_Search

Merge pull request #15784 from civicrm/5.20

5.20

Remove obsolete warning

The time has come to let this one go.....

Merge pull request #15781 from eileenmcnaughton/cont_search

dev/core#1374 Fix search formValue handling on contribution search

Merge pull request #15782 from seamuslee001/dev_mailing_56_2

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix i…

dev/core#1374 Fix search formValue handling on contribution search

This fixes 2 issues
1) Search params being lost when editing a related entity per
https://lab.civicrm.org/dev/core/issues/1374
2) force=1&sort_name=p not working in contribution search url

In digging I concluded the problem is we have 3 underlying arrays which we keep jumbling together

1) formValues - the actual submitted values, augmented by any  url passed params
2) the default values - values to load by default on the form
3) our working query params - a copy of formValues that we have prepared for the query

We need to stop mangling them. I added subtle code comments

Merge pull request #15777 from eileenmcnaughton/format

[NFC] Reformat Contact_Form_Search_Advanced

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix issue where by mailing_name has been namespaced also in 5.20

Merge pull request #15779 from seamuslee001/dev_mailing_56

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namesp…

Merge pull request #15773 from civicrm/5.20

5.20

[NFC] Reformat Contact_Form_Search_Advanced

Fix array style

Merge pull request #15775 from seamuslee001/convert_fatal_to_error_system_cache

Converts using deprecated fatal function for exception in CRM/Utils/Sys…

Merge pull request #15776 from eileenmcnaughton/url_params

[NFC] comment fixes - exceptions & don't declare NULL

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namespaced fields when building the row array for mailing reports

Merge pull request #15774 from seamuslee001/eileen_ts_15770

Add in translation function around newly converted statusBounce messages

Converts using deprated fatal function for exception in CRM/Utils/System and in CRM/Utils/Cache and changes default for retrieve function in the Request class to throw exceptions

Update doc blocks

[NFC] comment fixes - exceptions & don't declare NULL

Add in translation function around newly converted statusBounce messages

Merge pull request #15770 from eileenmcnaughton/except

Swap some fatals on form & page classes for statusBounces

Merge pull request #15772 from agh1/5.20.0-releasenotes-initial

5.20.0 release notes initial run

5.20.0 release notes: added boilerplate

5.20.0 release notes: raw from script