Merge pull request #19556 from mattwire/renamevar

[REF] Rename variable to clarify it's usage

Rename variable to clarify it's usage

Merge pull request #19551 from eileenmcnaughton/msg_tpl_convert_test

Add unit test cover for the MessageTemplate::renderMessageTemplate function

Merge pull request #19509 from jmcclelland/correctSmsNumbers

Correct sms numbers

Merge pull request #19553 from demeritcowboy/activity-import-test

dev/core#2308 - Fix Activity Import Parser test and convert to civi statics

fix test and convert to civi statics

Add unit test cover for the MessageTemplate::renderMessageTemplate function

This provides cover for the 4 methods
- domain tokens
- contact tokens
- smarty assigns within the function
- smarty assigns before the function

Merge pull request #19552 from demeritcowboy/php74-notice

php 7.4 E_NOTICE in Contact/BAO/Query

php 7.4 notice

Merge pull request #19531 from ahed-compucorp/master

Improve consistency of field titles, labels, types across the schema

Fix failing tests after regenerating DAO files

Regenerate DAO files

Merge pull request #19524 from eileenmcnaughton/member

[REF] Simplify is_email_receipt handling

Merge pull request #19545 from demeritcowboy/membership-notice-php74

php 7.4 - E_NOTICE on membership dashboard

Merge pull request #19548 from civicrm/5.35

5.35 to master

Merge pull request #19546 from colemanw/labelField

APIv4 - Fix label_field to use underscore instead of camelCase

Fix any title or label that contain name of own entity

Ensure field's title and html's lable attributes populated properly

Add html tag to all files in xml/schema

Merge pull request #18830 from adixon/asset-building-error-2137

put asset rendering call into a try/catch - dev/core #2137

[REF] Simplify is_email_receipt handling

This removes a chunk of confusing handling from the previously shared code.

Philosophically we have a lot of arrays of data being passed around. The difference
between these arrays is unclear & the array that appears to exist specifically to
how what was actually submitted (formValues) is edited rather than left intact.

Where we are not dealing with a calcuated value, but rather referring to
what was submitted the quickform function getSubmittedValue() seems
like a good pick. It's clear we are looking up the submitted value not
some calculated value & when code is moved around we know this
value is not dependent on where it is sitting in the code.

The function returns 1 or NULL - hence I cast to boolean where I'm not
sure NULL is good enough

php 7.4 is a bit stricter

APIv4 - Fix label_field to use underscore instead of camelCase

Log unexpected missing asset during AssetBuilder::build

put asset rendering call into a try/catch

Merge pull request #19538 from eileenmcnaughton/repeat

[REF] [Repeattransaction] Stop setting contribution->id

Merge pull request #19539 from eileenmcnaughton/repeat_fin

[REF] [Repeattransaction] use financial_type_id from getTemplateContribution (which already has handling)

Merge pull request #19540 from eileenmcnaughton/cont_page_input

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

Merge pull request #19530 from colemanw/searchDisplayBlocks

Afform Gui - Block editing enhancements

Merge pull request #19542 from seamuslee001/afform_fix_array_access

[REF] Fix array access on NULL error when running afform unit tests o…

Merge pull request #19541 from totten/master-aff-mock

afform_mock - Hide extension. Clearly indicate as development-only

[REF] Fix array access on NULL error when running afform unit tests on PHP 7.4

(NFC) afform_mock - Hide extension. Clearly indicate as development-only.

Merge pull request #19537 from colemanw/aliasLength

APIv4 - Increase alias max length to 256

[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

[REF] use financial_type_id from getTemplateContribution (which already has handling)

The removed code is already incorporated into getTemplateContribution
and has test cover in testRepeatTransactionPassedInFinancialType

[REF] Stop setting contribution->id

This is not needed as we can otherwise get it from the contributionResult directly

APIv4 - Increase alias max length to 256

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.

Afform Gui - Support editing entity-agnostic Content Blocks

Afform Gui - Improve speed & accuracy of rendering fields/blocks palette

Ensure the palette gets refreshed when saving a new block, and use ng-change instead of $watch

5.35.0 release notes: added boilerplate

5.35.0 release notes: raw from script

Afform Gui - Move elements to top of palette (block list could be very long)

Afform GUI - Fix search display block saving

Afform GUI - Enable blocks on search display afforms & optimize loading

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

refactor to improve readability and expand test coverage.

Merge pull request #19507 from eileenmcnaughton/trans

Comment intent to remove call to transition components