[NFC] Minor IDE cleanup

Merge pull request #19779 from artfulrobot/artfulrobot-improve-quicksearch

Improve quicksearch result presentation to enable browser link functions

Merge pull request #19803 from eileenmcnaughton/little_bit_super

[REF] Minor extraction to help cleanup permission code

Merge pull request #19810 from jaapjansma/dev_2447

Fix for dev/core#2447: wrong event fee stored

Merge pull request #19805 from MegaphoneJon/escape-boolean-booleans

core#2461 - allow booleans to be of type boolean

Merge pull request #19807 from jaapjansma/dev_2449

Fixed issue dev/core#2449: remove an unnecessary call to cleanMoney

Fix for dev/core#2447: wrong event fee stored

Fixed issue dev/core#2449

Fixed issue dev/core#2449

Merge pull request #19799 from eileenmcnaughton/mem_recur

[REF] Minor parameter cleanup

Improve quicksearch result presentation to enable browser link functions

core#2461 - allow booleans to be of type boolean

Merge pull request #19802 from demeritcowboy/carmarthenshire

dev/core#2027 - Fix state/province abbreviation for Carmarthenshire

Merge pull request #19784 from eileenmcnaughton/member

[REF] Extract code to get generic membership parameters

[REF] Minor extraction to help cleanup permission code

Merge pull request #19796 from eileenmcnaughton/desc

[REF] Cleanup on permission code

fix abbreviation

[REF] Minor parameter cleanup

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

[REF] Extract code to get generic membership parameters

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)

Merge pull request #19795 from MJCO/resetPathsAPI4

Add `resetPaths` as per issue #2458 #1

Fix docblock

Merge pull request #19793 from eileenmcnaughton/pay_mem

[REF] Extract getPaymentInstrumentID

[REF] Cleanup on permission code

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

[REF] Extract getPaymentInstrumentID

This allows it to be consistently accessed

Merge pull request #19786 from eileenmcnaughton/mem4

[Ref] Remove copy and paste overkill

Add ResetPaths API4 endpoint #2458

Merge pull request #19697 from eileenmcnaughton/setting

Enable & do single field metadata conversion for defaultContactCountry

Merge pull request #19790 from eileenmcnaughton/token

Add further test for token custom fields

Merge pull request #19789 from eileenmcnaughton/permc

[REF]  getCorePermissions cleanup

Merge pull request #19792 from civicrm/5.36

5.36

[Ref] Remove copy and paste overkill

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

Merge pull request #19785 from eileenmcnaughton/mem2

[REF] Very minor simplification

Merge pull request #19791 from eileenmcnaughton/memberjs

Fix rc regression - js error on autorenew checkbox

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

Add further test for token custom fields

Standardise handling of administer Multiple Organizations

[REF] Stop passing paramter not recognised by getCorePermissions

getCorePermissions does not accept any parameters so this stops passing them

Merge pull request #19788 from eileenmcnaughton/per_fin_types

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

dev/core#2454 Extend financial acls view limitations to ContributionRecur

[REF] Move financial acl permissions declaration to extension

Just some minor cleanup to use the hook

Merge pull request #19783 from colemanw/searchActionsSpinner

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

[REF] Very minor simplification

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)

SearchKit - Refactor tasks list to use API callback

Merge pull request #19782 from eileenmcnaughton/token

Remove addressee, email_greeting_id, postal_greeting_id from exposed tokens

SearchKit: Support custom fields when saving smart group

SearchKit - Show spinner while actions are loading

Merge pull request #19756 from eileenmcnaughton/mem2

[REF] Cleanup code to determine financial_type_id

Remove addressee, email_greeting_id, postal_greeting_id from exposed tokens

Per discussion on https://github.com/civicrm/civicrm-core/pull/19550#issuecomment-781597319
there appears to be agreement that supporting tokens like
addressee_id (which resolves to '{contact.individual_prefix}{ }.....')
should not be exposed / supported as they seem both unuseful and likely
to be breaky.

These were exposed unintentionally as part of a change to make them
available as WHERE filters on apiv3
https://github.com/civicrm/civicrm-core/commit/54e389ac6565ff534b87e5e46137a57d48d6c5c8

The discussion suggests that by contrast we should
add support to hash in the token compat subscriber

Merge pull request #19474 from mattwire/testexpiredpriceset

Make testExpiredPriceSet more realistic

Merge pull request #19778 from colemanw/readonly3

Schema - add `readonly` tag to more fields

Merge pull request #19761 from MegaphoneJon/financial-169

Fix batch transaction export on D8/9

Merge pull request #19775 from mattwire/buildmembershipblockunused

Remove unused variables/assigns from buildMembershipBlock function

Merge pull request #19741 from demeritcowboy/case-custom-nochange

dev/core#2019 - Don't create an empty activity if nothing changed when changing case custom data

Merge pull request #19781 from civicrm/5.36

5.36

Merge pull request #19780 from colemanw/searchTokenPopup

SearchKit - Fix token menu positioning

SearchKit - Fix token menu positioning

Merge pull request #19777 from lcdservices/dev-core-2453

dev/core#2453 cast balance due to float

Schema - add `readonly` tag to more fields

dev/core#2453 cast balance due to float

Remove unused variables/assigns from buildMembershipBlock function

Merge pull request #19745 from eileenmcnaughton/member

[REF] Remove now redundant code to set line item financial type

Merge pull request #19774 from civicrm/5.36

5.36

Merge pull request #19773 from demeritcowboy/safe-foreignkey

dev/core#2452 - Make upgrade to 5.36 more robust when creating foreign key

make safer

Merge pull request #19771 from francescbassas/patch-17

Enable translation for Contribution Ordinality options on Contribution Detail Report

Merge pull request #19755 from eileenmcnaughton/sum_api

dev/core#2415 Add api support for aggregate without group by

Merge pull request #19772 from francescbassas/patch-18

Enable translation for Contribution Ordinality options on Member Contribution Detail Report

Merge pull request #19768 from demeritcowboy/grant-placeholder

dev/core#2411 - Fix placeholder text for grants.

Merge pull request #19767 from colemanw/campaignPlaceholders

Fix incorrect placeholder text on 'Create Campaign' form

Merge pull request #19769 from MegaphoneJon/remove-unused

[REF] remove unused code in recordFinancialAccounts

Enable translation for Contribution Ordinality options on Member Contribution Detail Report

Enable translation for Contribution Ordinality options on Contribution Detail Report

Merge pull request #19770 from agileware/CIVICRM-1679

Include the timezone for date values in the Activity iCal

Merge pull request #19762 from agileware/CIVICRM-1676

Include the timezone for date values in the Event iCal

Fix test to teardown it's own mess

This business of doing cleanup in the setup is an anti-pattern and
it actually adds some internal dependency - if you
run JUST the one test (conditions probably apply) it
falls over with a 'cannot find DSN'. Rather than track that
down that weirdness we should fix the tests to
tearDown in the teardown

CIVICRM-1679 Include the timezone for date values in the Activity iCal

[REF] remove unused code in recordFinancialAccounts

fix placeholder

Fix incorrect placeholder text on 'Create Campaign' form

Merge pull request #19732 from mlutfy/fixWpAnon

WordPress: avoid PHP notice if the anonymous user does not have any capabilities

Merge pull request #19733 from colemanw/exportUiCheckPerm

Export UI - Only present "save mapping" button to permissioned users

Add api support for aggregate without group by

For example

SELECT SUM(amount) FROM civicrm_pledge

Should be doable with no group by if none is desired

Merge pull request #19763 from civicrm/5.36

5.36 to master

Merge pull request #19736 from colemanw/empty

Add is empty filter to search / api

CIVICRM-1676 Include the timezone for date values in the Event iCal

Fix batch transaction export on D8/9

Add is empty filter to search / api

This is already offered in Query
https://github.com/civicrm/civicrm-core/blob/5db2212e2d408f4611439734db1a31ab32dced2f/CRM/Contact/BAO/Query.php#L3420-L3428

And in Report
https://github.com/civicrm/civicrm-core/blob/c3fffe27cb8203634c7a2c047686ba3d12cc38bd/CRM/Report/Form.php#L2105-L2124

(the latter munges it in with NULL but as we often save empty strings NULL
does not alwasy work for strings)

Merge pull request #19760 from civicrm/5.36

5.36

Merge pull request #19758 from colemanw/searchKitFix

SearchKit - Fix broken update dialog and add input type float

Merge pull request #19729 from colemanw/coreExtCleanup

Fix ts() namespace in searchKit & afform extensions

SearchKit - Fix broken update dialog and add input type float

Fix ts() namespace in searchKit & afform extensions

[REF] Cleanup code to determine financial_type_id

Financial type id is really simple on this form - it's either required or
it can be determined form the price set.

However, the code passes the financial_type_id from array to array, calculating
it in more than one place which rather hides the underlying simplicity of it.

This retrieves it using a function that does the same from anywhere in the code.

Note that if someone tried to add it before this->order is built it would
hard fail & kill a bunch of tests. this->order is built at the start of the
submit routine

Merge pull request #19746 from eileenmcnaughton/mem_2

[REF] Clean up code around is_email_receipt

Merge pull request #19486 from eileenmcnaughton/pddemail

dev/financial#162 Simplify isPdf code

Merge pull request #19754 from eileenmcnaughton/api

Fix apiv4 test sloppiness

Merge pull request #19751 from colemanw/readonly2

Add more 'readonly' tags to the schema for fields not intended to be editable by users

Fix apiv4 test sloppiness

Why would we exclude e-notices from causing test fails - I suspect this
canary-killer got added a long time back & no-one noticed

Merge pull request #19752 from colemanw/searchKitSmartGroupCodeByeBye

SearchKit - remove unused smartGroup code

SearchKit - remove unused smartGroup code