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

The processAmount function is really problematic because it's trying to do several disparate things & is called from
all over the place with unclear intent. From looking at it I see it does a few things. It reformats line items,
it does some obscure & likely flawed filtering, it generates a total cost & a tax cost and it generates a very
specifically formatted array of autoreneal properties,

The way I see this going is
1) Move the formatting of the autorenewal back to the calling form (this PR) & simplify it
2) Split out the foreach so it goes through once & formats - this can be shoved out to a separate function
- and then it goes through the formatted array & calculates total_amount & tax_amount - we should
have a wrapper function that just returns these & we might see that is most of what is needed
3) Move all that awful partial_payment_total stuff back to the event form. Note that we are working
to entirely remove that from here are it makes so much less than no sense.
4) Calculates amount_level text - that should have it's own function.

It worth noting all of this does very little intensive work - a DB lookup or 2 that  could be cached & an iteration
through a very small array so it would be fine to have 3 functions -
- getAmountTotal
- getTaxTotal
- getAmountText

that each go through the same process of generating a formatted array from price_x => 5 etc rather than trying
to pass the  array around for 'performance' or to 'save work'.

From previous refactorings I would suggest we add an Order class where by you set the price fields
& then you can call 'getLineItems' - but that is a few steps after this....

Merge pull request #15819 from seamuslee001/dev_core_183_price_set

dev/core#183 Event Price Set Custom Search to using CRM_Utils_SQL_Tem…

Merge pull request #15793 from seamuslee001/depreacete_dao_temp_table_name

dev/core#183 Remove references to and noisly deprecated CRM_Core_DAO::createTempTableName

Merge pull request #15792 from seamuslee001/convert_temp_table_creation_campaign_upgrade

dev/core#183 Use Standard CRM_Utils_SQL_TempTable builder to create temporary tabl…

Merge pull request #15876 from seamuslee001/dev_core_183_random_segement

dev/core#183 Convert the generating of temporary tables within the ra…

Add in unit test of Campaign BAO query voter clause

Add in unit test of the priceset custom search

dev/core#183 Event Price Set Custom Search to using CRM_Utils_SQL_TempTable interface

Remove references to and noisly deprecated CRM_Core_DAO::createTempTableName

Update unit test to match new temp table format

Use Standard CRM_Utils_SQL_TempTable builder to create temporary table in Campaign and upgrade

Merge pull request #15785 from eileenmcnaughton/contribution_url_params

Contribution url params - add contribution_page_id as a supported url parameter

Merge pull request #15902 from eileenmcnaughton/transaction_sillyness

[REF] remove another unnecessary pass-by-reference

Merge pull request #15903 from eileenmcnaughton/except

Convert a few fatals to exceptions.

[REF] remove another unnecessary pass-by-reference

the recur function is only called once. Immediately after it is called there is a return &
the calling function has not passed in any variables by reference - ergo we gain nothing but confusion by using pass-by-ref
here

Merge in 5.20

Merge pull request #15841 from mattwire/participant_cleanup_removeparticipantfrominput

Remove unused variables in completeOrder()

Merge branch '5.20-security_release_release_notes' into '5.20-security'

Release notes for 5.19.2 Release

See merge request security/core!90

Update 5.19.2.md

Update 5.19.2.md

Update release notes for Public MRs being merged into 5.19

Update 5.19.2.md

Update 5.19.2.md

Update 5.19.2.md

Minor update to release notes

Update release notes as per Tim's copy editing of the SA

Release notes for 5.19.2 Release

Update to latest versions of polyfill-ctype and polyfill-iconv

securiry/core#70 Upgrade Symfony version to 2.8.50

ContactApiKeyTest - Add additional test-case

This case was failing before but now passes with the update

Ensure that APIv4 Properly filters out permissable fields when no permission to view field exists

Update unit test to match the output of the CRM_Utils_String function

SavedSearch API - Only accept safe inputs

CRM_Utils_String::unserialize() - Add wrapper for parsing safe exprs (int/string/array/etc)

[NFC] Update Doc block as per Patrick

Revert changes in CRM/Core/BAO{Cache|ConfigSetting}.php CRM/Extension/ClassLoader.php CRM/Queue and CRM/Utils/Cache

Return FALSE instead of throwing Exception if package throws Exception

Implement xKerman/restricted-unserialize package to guard against unsafe unserialize

ContactInterchangeTest - Add coverage for APIv4 joins/chains

(security/core#67) Port APIv3's escaping scheme to APIv4

HTMLInputCoder - Add more variants for encoding arrays

Fix unit test for APIv4 format

Fix api_key handling from joins

security/core#65 - Fix XSS in Dashboard Report Title

This fixes an XSS in report titles displayed on the dashboard page
by escaping the title on output.

security/core#65 Fix Display of Report title that contains XSS in it when shown on dashboard

security/core#59 - Fix SQL injection in civicrm/ajax/dedupefind

This fixes an SQL injection in civicrm/ajax/dedupefind by validating
the order direction parameter.

Fixes #59

Merge pull request #15890 from civicrm/5.20

5.20

Merge pull request #15892 from totten/master-extgettest

(NFC) api_v3_ExtensionTest - Fix when executing on build with 25+ extensions

Merge pull request #15889 from eileenmcnaughton/refund

dev/core#1409 Remove net_amount from Addtional Payment form

dev/core#1409 Remove net_amount from Addtional Payment form

This is causing a validation problem. We used to have an issue on the contribution form which we eventually
resolved by removing net_amount as it's best calculated anyway

In order to make this changed I had to ensure Payment.create adds the net_amount & had
to do a couple of tweaks for the test to pass

(NFC) api_v3_ExtensionTest - Fix when executing on build with 25+ extensions

Overview
--------

This fixes a small issue with running the test locally.

Before
------

If the system has code for 25+ extensions (whether enabled or not), `api_v3_ExtensionTest` fails.

After
-----

The number of extensions doesn't matter.

Merge pull request #15326 from totten/master-headfoot-2

CiviMail - Only display "Headers and Footers" if there are headers and footers

Merge pull request #15863 from seamuslee001/status_bounce_extension

[REF] Convert CRM_Core_Error::fatal to CRM_Core_Error::statusBounce i…

Merge pull request #15888 from seamuslee001/lab_core_1406

dev/core#1406 Ensure that we use the CRM_Member_DAO_Membership as dao…

[REF] Convert CRM_Core_Error::fatal to CRM_Core_Exception in Extension folder

dev/core#1406 Ensure that we use the CRM_Member_DAO_Membership as dao for the civicrm_membership table

Merge pull request #15821 from seamuslee001/dev_core_183_custom_group

dev/core#183 Convert Include group / exclude group and include tags /…

Merge pull request #15886 from seamuslee001/nfc_composer_lock_update

[NFC] Update composer lockfile to remove warning

Merge pull request #15864 from mfb/string-type-declaration

Add string type declarations to global API functions.

dev/core#183 Convert the generating of temporary tables within the random segment custom search to using standard CRM_Utils_SQL_TemporaryTable method

[NFC] Update composer lockfile to remove warning

Various tests assume civicrm_api() $entity param is nullable.

Add string type declarations to global API functions.

Remove unused variables in completeOrder()

Merge pull request #15880 from mlutfy/fixUserDashboardEvent

Remove nbsp from UserDashboard Event section

Merge pull request #15817 from colemanw/Fix

Fix api explorer formatting IS NULL clauses

Merge pull request #15881 from civicrm/5.20

5.20

Merge pull request #15879 from alifrumin/rn5.20

First pass at sorting and annotating 5.20 release notes

Remove nbsp from UserDashboard Event section

Merge pull request #15878 from civicrm/5.20

5.20

release notes 5.20 - first pass at editing, sorting and annotating

Merge pull request #15846 from greenpeace-cee/fix-cancel-date

Fix contribution cancel_date not being loaded

crmMailingAB - Only display "Headers and Footers" if there are headers and footers

crmMailing - Only display "Headers and Footers" if there are headers and footers

dev/core#1391 - Fix contribution cancel_date not being loaded

This fixes an issue where the contribution cancel_date is not loaded
in the contribution search, causing cancelled contributions not to be
greyed out and the cancel date not to be shown.

Merge pull request #15800 from eileenmcnaughton/anet_valid

Remove unused  parameter recur from CompleteOrder signature

Merge pull request #15871 from eileenmcnaughton/init_fatal

[REF] Move CRM_Event_Form_EventFees::buildQuickForm to Participant_Form class

Merge pull request #15866 from eileenmcnaughton/test_dumb

[NFC] [Test-only] Remove partially paid from statuses tested for RenewContribution

Merge pull request #15870 from eileenmcnaughton/init_set

[REF] Minor simplifcation on initSet

Merge pull request #15854 from eileenmcnaughton/test_fix

[NFC] Test fix - incorrect records created due to test set up using PartiallyPaid when no payment is made

Merge pull request #15869 from eileenmcnaughton/add_pay

[REF] remove unused lines of code

Convert a few fatals to exceptions.

This is a general thing we are doing. In this case they are all ones that really should
never be hit

Merge pull request #15826 from seamuslee001/dev_core_183_dedupe

dev/core#183 CRM/Dedupe/BAO/QueryBuilder/IndividualUnsupervised.php report to using CRM_Utils…

Merge pull request #15853 from demeritcowboy/civicasetestcase-duplicate-option-values

[TEST] dev/core#1394 - Avoid duplicate activity types in tests that use CiviCaseTestCase

Merge pull request #15849 from seamuslee001/dev_core_183_dedupe_test

[NFC] Add in unit test of code being altered by #15826

Merge pull request #15874 from seamuslee001/dev_core_183_env_check

dev/core#183 Use standard temporary table name format when creating t…

Add in unit test of code being altered by #15826

Merge pull request #15875 from civicrm/5.20

5.20

Merge pull request #15872 from eileenmcnaughton/sybunt

Fix Sybunt to select mark rows as selected

dev/core#183 Use standard temporary table name format when creating temporary utf8mb4 temporary table

Merge pull request #15873 from eileenmcnaughton/cont_ag

dev/core#1377 Fix bug where search action doesn't work.

Merge pull request #15865 from eileenmcnaughton/query

Fix incorrect reference on extracted function

dev/core#1377 Fix bug where search action doesn't work.

Fixes a regression where the search actions are not working on the contributionAggregate custom search

Merge pull request #15867 from eileenmcnaughton/test_dumber

[NFC] fix test to correctly set up partially paid contribution

Fix Sybunt to select mark rows as selected

Merge pull request #15820 from seamuslee001/dev_core_183_custom_contribsybnt

dev/core#183 Finish converting contrib sybnt custom search to using t…

Merge pull request #15868 from colemanw/sortable

Upgrade ui-sortable angular library to v0.19.0

[REF] Minor simplifcation on initSet

Instead of passing the information to calculate the priceSetID into this function it's more helpful
to calculate it first & pass it in - this determination makes more sense on the calling form

[REF] Move CRM_Event_Form_EventFees::buildQuickForm to Participant_Form class

It turns out this function is only ever used from the ParticipantForm class so moving it back there simplifies
that for people looking at the code

[REF] remove unused lines of code

On checking this out this variable is never used ( is)

Merge pull request #15815 from artfulrobot/issue-1108-fix-unsubscribe

dev/core#1108 Fix unsubscribe bug