[REF] Remove loading contribution page id from passed in object

I believe this became obsolete with this PR
https://github.com/civicrm/civicrm-core/commit/6c3e774c322267c8e4449006db9cc14202b7028f

The other 2 (paypal) IPN classes no longer set contribution_page_id deliberately so
would be relying on the same data used for getTemplateContribution

Merge pull request #19535 from colemanw/escapeRows

dev/core#2360 - Escape the word `rows` in sql query

Merge pull request #19508 from mattwire/loadmemlineitems

Fix display of membership dates on receipts when lineitems do not have a price_set_id

Merge pull request #19534 from civicrm/5.35

5.35

dev/core#2360 - Escape the word `rows` in sql query

Merge pull request #19533 from totten/master-api4-status-code

APIv4 - Fix HTTP status code selection

Merge pull request #19529 from agh1/5.35.0-releasenotes-initial

5.35.0 release notes - initial run

APIv4 - Fix HTTP status code selection

This is a follow-up to #19526 which addresses a typo that causes a misbehavior in reporting the HTTP status code.

Before
------

If an API request encounters an exception, then it always returns HTTP 403.

After
-----

If an API request encounters an exception, then:

* It may return HTTP 403 (for an authorization exception)
* It may return HTTP 500 (for any other/unrecognized exception)

Merge pull request #19463 from colemanw/removeCampaignPseudoconstant

Remove campaign_id pseudoconstants

Remove campaign pseudoconstants from tests

Remove campaign_id pseudoconstants

Pseudoconstants are intended for option lists small enough to be loaded into memory,
whereas FKs are for options of any size.
It is possible to have thousands of campaigns, and some sites do,
which slows or breaks some UIs (first noticed in the Search Kit bulk update action).

All these fields have both an FK and a pseudoconstant declared, which is redundant;
both are not needed, and the pseudoconstant was unnecessarily hogging memory for sites with many campaigns.

5.35.0 release notes: added boilerplate

5.35.0 release notes: raw from script

Merge pull request #19527 from eileenmcnaughton/complete

[REF] Complete order Unset contribution once we have finished with it.

Merge pull request #19522 from eileenmcnaughton/ref_template

[REF] Further extract the portion of sendTemplate that relates to rendering

Merge pull request #19494 from seamuslee001/apiv4_annon_access

(REF) APIv4 - Add hook for whitelisting select AJAX API's for anonymous invocation

Merge pull request #19526 from totten/master-api4-error

APIv4 - AJAX errors should say *something* useful

Set version to 5.36.alpha1

Set version to 5.35.beta1

[REF] Complete order

Unset contribution once we have finished with it.

This patch does very little but it takes a bit of searching to check the handful of places
that call completeOrder and check that none of them use the contribution
object after this function call. With this merged it will be possible to
cleanup the references to the contribution in the repeattransaction
code safe in the knowledge that the object is not needed after that
function (or within it from my reading). It seems likely we can switch to
passing in contributionID rather than the whole object.

APIv4 - AJAX errors should say *something* useful

When calling APIv4 via AJAX, you may sometimes encounter an error. What response do you get?

Before
------

You are likely to get a completely blank response (`status=500, body=[]`).
There is no information in any of the logs (Apache, PHP, CiviCRM, etc).  You
have no way to tell what's gone wrong.

Of course, if you're logged in as a full administrator, then you may have
permission `view debug output`, in which case there might be something
useful.  But this won't help if you're using a less privileged user.

After
-----

For the administrator (`view debug output`), you still get a detailed error response.

For less privileged users, the error is logged. The response provides a generic message along with an
"Error ID". You can use the "Error ID" to locate information in the log.

Also, if the error is an `UnauthorizedException`, then the response code will be a semantic 403 instead of a generic 500.

Afform.prefill - Allow anonymous requests

This API has its own security checks

(NFC) hook_civicrm_alterApiRoutePermissions - Expand on docblock

Merge pull request #19521 from civicrm/5.34

5.34

Merge pull request #19519 from colemanw/afformGuiFixBlocks

Afform GUI - Fix errors when creating & saving blocks

[REF] Further extract the portion of sendTemplate that relates to rendering

This is primary for the purposes of simplification at this stage

Merge pull request #19520 from agh1/5.34.0-releasenotes-final

5.34.0 release notes: final edits and late changes

5.34.0 release notes: final edits and late changes

Afform Gui - Exclude self from list of available blocks when editing a block

Merge pull request #19464 from eileenmcnaughton/menu

dev/core#2340 Skip rather than fail on bad menu item

Merge pull request #19514 from colemanw/apiRelativeDateFix

APIv4 - Fix bug when using relative date filters in ON clause of a join

Afform GUI - Fix errors when creating & saving blocks

Merge pull request #19517 from seamuslee001/fix_user_load_deprecation

Override the DrupalBase getUserObject function with a Drupal8/9 compa…

Merge pull request #19516 from colemanw/apiGetFkFields

APIv4 - Enable getFields to find fields across implicit FK joins

Merge pull request #19510 from eileenmcnaughton/msg_tpl

[REF] Extract logical functions from sendTemplate

Override the DrupalBase getUserObject function with a Drupal8/9 compatible version

Merge pull request #19443 from eileenmcnaughton/ipn

[REF] Move sendNotification out of recur, remove unused related_contact

Merge pull request #19511 from eileenmcnaughton/mem_clean

[REF] Remove invalid attempt to load contriution id from invoiceID

Merge pull request #19498 from eileenmcnaughton/complete_order_test

[REF] Stop passing contributionPageID to isEmailReceipt

Merge pull request #19515 from demeritcowboy/amtg-notice

dev/core#2355 - E_NOTICE on Manage Contribution and Manage Event listings

APIv4 - Enable getFields to find fields across implicit FK joins

Now it is possible to retrieve field metadata for a joined entity

E_NOTICE on contribution pages

APIv4 - Fix bug when using relative date filters in ON clause of a join

Merge pull request #19496 from colemanw/afformBugFixes

Afform - Misc tweaks, validation & bug fixes

Merge pull request #19513 from eileenmcnaughton/kernel_error

Move require once in api kernel to support edge cases

Merge pull request #19492 from demeritcowboy/fatalerrorhandler

dev/core#2350 - Oauth extension - Setting a fatalErrorHandler might override the redirect url because typo

config-form.html - Fix rendering of client-side validation error for "server_route"

Move require once in api kernel to support edge cases

The function 'createError' on the kernel class calls 3 functions that are in the api v3 utils file.
It is possible to reach this from apiv4 using drush (I realise we
don't really know much about how to support api v4 with drush but it DOES work with a simple
api with this patch in my testing). We could go with only including utils.php in
v4 api AND the createError function. I think it's more readable this way.

To test this try

 drush  cvapi  Contribution.get version=4

it will legitimately fail without checkPermissions=0 but it will fail with a
require_once error without this patch

Search Kit - Improve display of afforms in search list

Remove defunct reference to skilCleanMoney

The BAO used to handle this param but it no longer does. The api does respond to it - but defaults to
true (we don't call the api here but anything other than setting it to false will always do nothing)

Remove attempt to load contriution id from invoiceID

This might make sense (maybe) on the front end code this was previously shared with but on the
backoffice form id would be specifically set in the url

[REF] Extract logical functions from sendTemplate

This simply makes it clear what the main chunks of code are doing

[REF] Stop passing contributionPageID to isEmailReceipt

By switching to passing in contributionID instead we can significantly simplify this.

It could cause an extra query in some cases but the query would be very quick based
on my previous query reduction efforts and this will help us
remove a bunch of other code

Merge pull request #19417 from eileenmcnaughton/test

[REF] Standardise methods of determining isTest

Merge pull request #19505 from mattwire/remotememrelated

Remove another use of loadRelatedObjects() function

Merge pull request #19500 from eileenmcnaughton/cancel

dev/core#2206 Unhide contributioncancelactions core extension

Merge pull request #19507 from eileenmcnaughton/trans

Comment intent to remove call to transition components

Fix display of membership dates on receipts when lineitems do not have a price_set_id

Comment intent to remove call to transition components

The other usages have been migrated to contributioncancelactions so only update-to-completed remains

Merge pull request #19502 from mattwire/sendReminder

CRM_Core_BAO_MessageTemplate::sendReminder() is not used anywhere

Merge pull request #19504 from colemanw/labelField

Add "labelField" metadata to entities

Unhide contributioncancelactions

With https://github.com/civicrm/civicrm-core/pull/19289 merged we
can now unhide this extension & people can disable it if they wish (as a
supported option)

Remove another use of loadRelatedObjects() as we call it again before using

Merge pull request #19503 from mattwire/casecomponentignoreexception

Use exceptions when enabling case component / checking for 'CREATE VIEW' permissions

Add "labelField" metadata to entities

This determines which field will be shown when displaying a single record
e.g. when viewing a Contact, show the display_name.

Afform - Fix repeatable blocks in GUI

Search kit: Fix bug when showing functions in HAVING clause

Afform Admin - Pluralize tabs & rename variable with same name as another

Afform - Fix filling preset values during submit

At some point the `af-values` attribute got renamed `data` but the Submit api wasn't updated.

Afform Gui - Reset server_route and is_dashlet when cloning

Afform Gui - Basic validation & defaults for server_route

Use exceptions when enabling case component / checking for 'CREATE VIEW' permissions

CRM_Core_BAO_MessageTemplate::sendReminder() is not used anywhere

Merge pull request #19499 from eileenmcnaughton/silly_if

Remove always true if

Merge pull request #19289 from eileenmcnaughton/cancel

dev/core#2206 Move cancel pledge into Contribution.create and fix

Merge pull request #19493 from civicrm/5.34

5.34

Merge pull request #19497 from eileenmcnaughton/534

Fix for ReportTest when launching a pdf

Remove always true if

This if is logically always true based on the if that precedes it

Fix for ReportTest when launching a pdf

REF Open APIv4 AJAX route up to potentially Annonymous Access if Extensions determine it appropriate

Afform - Rename 'Forms' menu item to 'Form Builder'

Merge pull request #19468 from mattwire/cpcleanup

NFC Minor code cleanup to contribution pages

Merge pull request #19481 from eileenmcnaughton/pdffix

dev/core#2344 unit test for fix regression affecting sending thank you letters when grouped

Merge pull request #19490 from mlutfy/fixTokenProcessorEscape

Token Processor: fix PHP syntax for purifyHTML

Minor code cleanup to contribution pages

dev/core#2340 Skip rather than fail on bad menu item

Merge pull request #19489 from colemanw/searchKitBugs

Search Kit: Misc bug fixes

Merge pull request #19487 from civicrm/5.34

5.34

Merge pull request #19491 from herbdool/issue-2352

Issue #2352: fix Views integration suggestion

Merge pull request #19461 from colemanw/searchCancel

Greenwich: Fix BS3 conflict with html5 search clear button

Merge pull request #19485 from seamuslee001/composer_update

[NFC] Update composer.lock

seems like a typo

Afform Gui - Align buttons to right on admin list

Issue #2352: fix Views integration suggestion

Token Processor: fix PHP syntax for purifyHTML

dev/core#2349 Fix Search Kit display of pseudoconstant fields

Merge pull request #19488 from eileenmcnaughton/act_map_form

[REF] Remove copy & paste code from Activity mapping form

Search Kit: Fix rowCount to run only if results exceed limit

dev/core#2347 Search Kit: Fix 'Select All' in table display

Merge pull request #19479 from colemanw/afformSearchBugFixes

Search Kit - Fix display pager and improve error handling