some typo fixes

Merge pull request #20869 from eileenmcnaughton/acl3

[REF] Fix functions to be protected

Merge pull request #20880 from colemanw/searchKitTokenSelect

SearchKit - Support all fields as tokens

Merge pull request #20893 from colemanw/fixAPIComparisonFunctions

APIv4 - Fix and add tests for comparison SQL functions

Merge pull request #20896 from colemanw/groupConcatOutputValue

APIv4 - Better handling of output value using GROUP_CONCAT

SearchKit - Allow tokens in link text

SearchKit - Support all fields as tokens

Previously, only fields present in the SELECT clause could be tokens.
Now the SearchDisplay::Run api will add any fields used as tokens to the SELECT automatically.

APIv4 Explorer - Fix display of bridge entity fields for symmetric bridge

Symmetric bridge entities such as RelationshipCache which joins Contact to Contact
need to have the second entity id availble to join on.

APIv4 - Validate number of args passed to SQL functions

APIv4 - Better handling of output value using GROUP_CONCAT

Merge pull request #20894 from civicrm/5.40

5.40

Merge pull request #20891 from demeritcowboy/timestamp-update

[NFC/Unit test] - Failing test demonstrating blank timestamp updates

APIv4 - Fix and add tests for comparison SQL functions

Merge pull request #20890 from demeritcowboy/ignore-log-upgrade-alt

dev/core#2666 - (Alternate) Don't repeatedly log about crm-l10n.js during upgrade

Merge pull request #20865 from totten/master-getset-trait

(REF) Civi/Schema - Extract MagicGetterSetterTrait. Add test coverage.

(NFC) AbstractAction - Pithiness

failing test for timestamp updates

don't log about missing l10n for every upgrade task

Merge pull request #20850 from eileenmcnaughton/int

Use type hinting for id

Merge pull request #20848 from eileenmcnaughton/dep

Remove deprecated code

Merge pull request #20853 from eileenmcnaughton/auth

Remove never-true-if

Civi/Schema - Extract MagicGetterSetterTrait. Add test coverage.

Merge pull request #20881 from eileenmcnaughton/batch2

Remove non-variable variables

Merge pull request #20879 from colemanw/fixOnCrmUiSelect

Fix onCrmUiSelect to use current scope and apply to the digest cycle

Merge pull request #20878 from colemanw/fixSearchKitCount

SearchKit - Fix pager count return value

Remove non-variable variables

Fix onCrmUiSelect to use current scope and apply to the digest cycle

Evaluates expression in current scope rather than parent scope,
and uses $scope.$apply.

Merge pull request #20871 from eileenmcnaughton/val

[Ref] Remove never-passed parameter

SearchKit - Fix pager count return value

Merge pull request #20874 from eileenmcnaughton/batch

[REF] Remove some non-variable variables

Merge pull request #20864 from eileenmcnaughton/tok

dev/core#2650 Add support for contribution_status_id to the token processor

Merge pull request #20801 from eileenmcnaughton/upg

Enotice fix on extensions page

Merge pull request #20873 from eileenmcnaughton/remove_dep

Remove deprecated lines of code

dev/core#2650 Add support for contribution_status_id to the processor

Per https://lab.civicrm.org/dev/core/-/issues/2650 the goal is to get the
token processor (for each entity, in this case contributions) to be processing fields
in the same way as the legacy processor does and for this to be thoroughly tested.

This allows us to expose the token processor classes to message template processing
and use them interchangeably, with an eventual goal of migrating over.

It turns out that althought 'Contribution Status ID' is exposed in the UI
for scheduled reminders it did not work. This left us open to simply add the
token contribution.contribution_status_id and add tests.

Per 2650 the handling of pseudoconstants is inconsistent but our preferred goal
is that they would follow apiv4 style syntax and that the actual field name
would hold the actual field value.

Although the token processor exposes a value 'status' - it seems this is likely never
actually exposed to the user and is probably unused.

We can come back to how to deprecate & remove

Merge pull request #20876 from eileenmcnaughton/cust

Api test cleanup fixes

Api test cleanup fixes

[REF] Remove some non-variable variables

This previously shared function has parameters that are hard set to a specific value.
Most notably pending is set to FALSE - this removes all references to pending from the function

Merge pull request #20791 from eileenmcnaughton/batch_renew

[REF] Simplify isRenew handling on batch for membership

Remove deprecated lines of code

[Ref] Remove never-passed parameter

this function is called from only one place & it does not pass in financialTrxnValues

Merge pull request #20820 from eileenmcnaughton/tpl

dev/core#2684 Fix filters tpl in civi reports to permit more than one  table in a grouping

[REF] Fix functions to be protected

I did some poking around & these function are not called outside the class

Once we can see that we can consider updating to save 0 to the table, allowing
a lot of simplification

Merge pull request #20829 from eileenmcnaughton/order

[REF] Further order api cleanup

Merge pull request #20863 from eileenmcnaughton/api4

Fix v4 api to do custom field delete in tearDown

Fix v4 api to do custom field delete in tearDown

This existing efforts are wrong because
1) the delete has to go in the tearDown not the clean up
2) it is rigid about the custom group table name - which
will get us into problems as the name can be set and we
should be ensuring that it can in our tests

Merge pull request #20578 from eileenmcnaughton/admin_data

Give administer CiviCRM data access to administer custom groups

dev/core#2684 Fix filters tpl in civi reports to permit more than one table in a grouping

Merge pull request #20860 from civicrm/5.40

5.40 to master

Merge pull request #20859 from agh1/5.40.0-releasenotes-initial

5.40.0 release notes initial run

5.40.0 release notes: added boilerplate

5.40.0 release notes: raw from script

Merge pull request #20840 from colemanw/membershipTypeAutoRenewFix

APIv4 - Fix output of membershipType auto_renew field

Merge pull request #20852 from eileenmcnaughton/paypal

[REF] [towards dev/core#2693] Use getter rather than passing variable

Merge pull request #20849 from civicrm/5.40

5.40

Remove never-true-if

This code is in the recur() function. the lines immediately before this function ensure contribution->contribution_status_id
is unset if it is 1 - meaning this if can never be true

Merge pull request #20847 from eileenmcnaughton/ref

Minor simplification - use casting

Merge pull request #20718 from eileenmcnaughton/ipn

[REF] Minor auth.net ipn simplification

Merge pull request #20844 from demeritcowboy/report-rectype

dev/core#2691 - On logging detail civireport show words instead of numbers

[REF] Use getter rather than passing variable

txnType is a required parameter for recurring & unused for single. This switches to retrieving
it as needed rather than passing it around. It removes one check for whether it is
set because the abort will exit if not

Use type hinting for id

This clarifies that the id must be an integer (0 for anonymous)

Function is only called from one place

Merge pull request #20846 from colemanw/fixCkEditorExtension

CKEditor - Fix .tpl insertion on every form

Remove deprecated code

We deprecated NOT passing an id into the function in 2018 - this removes

Note I double checked - if you pass in '2' to a strict casting of int
it accepts it and casts to int

Minor simplification

Merge pull request #20843 from demeritcowboy/smartyreptable

Smarty notices - Missing type on logging civireport

Merge pull request #20768 from eileenmcnaughton/mem_type_acl

dev/core#2115 Switch membership type browse page to use v4api for get, thus using acls properly

CKEditor - Fix .tpl insertion on every form

Merge pull request #20842 from in2part/add-to-contributors-file

Add to contributors file

Merge pull request #20821 from colemanw/afformBeta

Mark afform extensions as beta

APIv4 - Fix output of membershipType auto_renew field.

The field had been declared a "boolean" in the schema, which at a SQL level
is exactly the same as "tinyint" but it was resulting in incorrect API output.

show words instead of numbers

missing type

Add to contributors file

Merge pull request #20839 from in2part/fix-typo-in-social-buttons

Fix a typo in social networking buttons

Merge pull request #20833 from colemanw/customPerm

CustomGroup - change admin permission to 'administer CiviCRM data'

Merge pull request #20834 from colemanw/fixUnusedVars

[REF] Remove unused variables

Remove unused variables

Fix a typo in social networking buttons

Merge pull request #20831 from eileenmcnaughton/order_part

Remove a couple of instances of 'contribution_mode' from tests

Merge pull request #20775 from eileenmcnaughton/loadlines

Deprecate guess work in line item handling

[REF] Further order api cleanup

Deprecate guess work in processPriceSet

Testing to see how much of this we can deprecate - line items should
be 'complete' before reaching the processing function

Merge pull request #20784 from eileenmcnaughton/pay_fix

Fixes getTemplateContribution to use a more reliable way to load line items

Merge pull request #20838 from civicrm/5.40

5.40

Merge pull request #20836 from seamuslee001/dev_rc_13

Resolve dev/rc#13 by permitting civi.setupui events to be dispatched …

Resolve dev/rc#13 by permitting civi.setupui events to be dispatched during install

Merge pull request #20587 from eileenmcnaughton/rr

dev/core#2648 update redis class off deprecated functions

CustomGroup - change admin permission to 'administer CiviCRM data'

This permission is a subset of 'administer CiviCRM' and is more precise,
allowing finer-grained admin permissions.

Remove a couple of instances of 'contribution_mdoe' from tests

This should not be required - we should pass in correct line items....

Switch membership type to use apiv4

url is civicrm/admin/member/membershipType?reset=1
test cover in https://github.com/civicrm/civicrm-core/pull/20761

Merge pull request #20830 from eileenmcnaughton/ttp

E-notice fix

Merge pull request #20650 from eileenmcnaughton/partorder

Enable check on created participant payment rows

Merge pull request #20799 from colemanw/filterByPrefix

APIv4 - Fix pseudoconstant matching reliability across joins

Merge pull request #20828 from colemanw/tplFix

Fix undefined tpl vars

E-notice fix

Enable check on created participant payment rows

Extend participant check

Merge pull request #20670 from eileenmcnaughton/reg

Fix transfer registration to transfer participant_payment row

Fixes getTemplateContribution to use a more reliable way to load line items

My efforts to add testing a 'deprecate weird stuff' have identified an odd and fragile
flow for the line items in getTemplateContribution. It calls
getLineItemsByContributionID which, as it turns out, substitues the
actual entity table with 'civicrm_contribution'.

Then this line of weird handling swoops in and saves the day.

https://github.com/civicrm/civicrm-core/pull/20775/files#diff-a16d4d7449cf5f3a0616d1d282a32f27ab6d3f7d2726d076c02ad1d4d655af41R393

This switches us to something cleaner than just loads the line items (with v4 LineItem.get) and
no weird handling

Merge pull request #20826 from civicrm/5.40

Merge 5.40 to master

Fix undefined tpl vars

Merge pull request #20827 from eileenmcnaughton/tpl3

tpl enotice fix