Fix for #2447: wrong event fee stored

#2461 - allow booleans to be of type boolean

Fixed issue #2449: remove an unnecessary call to cleanMoney

[REF] Minor parameter cleanup

#2027 - Fix state/province abbreviation for Carmarthenshire

[REF] Extract code to get generic membership parameters

[REF] Cleanup on permission code

Removes 2 params that are set in order to be passed into
legacyProcessRecurringContribution - however that function does not use them

The form has 2 types of membership parameters
1) submitted parameters
2) calculated parameters that are per membership type

This extracts the portion that are submitted to be fetched
from a single place that will return the same result whereever
it is called from (allows us to move
code around without breaking the fragile parameter chain)

Add `resetPaths` as per issue #2458 #1

[REF] Extract getPaymentInstrumentID

The function returns either an array of arrays or an array of the
first key in the array, depending on the 'desriptions' parameter.

This cleans it up so that instead of handling the descriptions
parameter in multiple places we get the array
in all cases and do the formatting at the end

This allows it to be consistently accessed

[Ref] Remove copy and paste overkill

Enable & do single field metadata conversion for defaultContactCountry

Add further test for token custom fields

[REF]  getCorePermissions cleanup

5.36

This is a previously shared function that was copied and pasted back. However in the
context of this form they don't make sense because

1) It doesn't make sense to try to maintain the integrity (via the transaction)
of the contribution & recurring contribution over & above the other
items on the form - without the memberships they don't actually have integrity
and this might have made a little more sense front end in terms
of ensuring they were committed if a payment is made but
we can expect a back office user to observe an error

2) setting the contribution id on the form here doesn't make sense as
this line is only hit for credit-card recurring contributions and
either we are doing it elsewhere anyway or we should do it nowhere - there
is no logic for 'only do this in this one obscure flow' that makes
sense back office. Front office this was likely added for an otherwise missed
scenario

[REF] Very minor simplification

Fix rc regression - js error on autorenew checkbox

This commit
https://github.com/civicrm/civicrm-core/pull/19647/commits/cfab7c9d1dc5d1e8438131354685928cd9da3877 renamed membershipType to membershipTypeID but missed this spot,
resulting in js errors on the new membership by credit card form

getCorePermissions does not accept any parameters so this stops passing them

Financial type hook clean up and fix towards dev/core#2454 Extend financial acls view limitations to ContributionR…

Just some minor cleanup to use the hook

SearchKit - Refactor loading actions list & support custom fields for groups

ContributionParams is mostly passed in to processContribution
but a few parametrs are added to it within that function

This adds 2 of them to the passed in values for a very minor simplification.

Note that params['total_amount'] was set using order->getTotalAmount
and this will return a consistent result from anywhere in the postProcess
function (after line 1002)

Remove addressee, email_greeting_id, postal_greeting_id from exposed tokens