Merge pull request #19559 from demeritcowboy/asset-more-info

dev/core#2137 - Enhance log message to include the actual error

include the actual error in log message

Merge pull request #19421 from eileenmcnaughton/act2

Cleanup towards dev/core#2308

Merge pull request #19547 from eileenmcnaughton/cont_obj

[REF] Fully remove contribution object from repeattransaction function

Merge pull request #19525 from eileenmcnaughton/member_soft

[REF] Remove unused pcp & soft credit handling from previously shared code

Merge pull request #19536 from totten/master-use-permlist

Afform (et al) - Fetch more complete list of permissions via APIv4

[REF] Fully remove contribution object from repeattransaction function

We can now see it is no longer used & remove it

Merge pull request #19554 from colemanw/searchFields

SearchKit - Add implicit joins for displaying labels of related entities

Merge pull request #19555 from demeritcowboy/activity-import-validate

[NFC/Unit Test] dev/core#2308 - Tests for activity import

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

tests for activity import

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

SearchKit - Use label_field to determine column to output as link

SearchKit - Use label_field to determine default columns

SearchKit - Auto-add label_field when adding a join

SearchKit - Add in FK fields for implicit joins

This adds pseudo-fields to each entity that actually belong to other entities,
the API is able to automatically fetch them using implicit joins.

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

Sort permission list by title. Make it easier to skim.

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

CRM_Core_Permission_List - Tweak labels for `*always allow*` and `*always deny*`

Nav Menu Editor - Use API for listing/searching available permissions

CiviReport - Use API for listing/searching available permissions. Use select2.

Afform - Use API for listing/searching available permissions

Merge pull request #19535 from colemanw/escapeRows

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

Permission.get API - Fix warning on Drupal/Backdrop

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

[REF] Remove unused pcp & soft credit handling from previously shared code

the ProcessContribution function has been split out from a shared function used by front end forms
and I'm pretty sure it was only relevant for those other forms. The only code that calls this function
is when a payment processor is used in recurring mode to pay for a membership.

I tested doing this and selected 'record payment from a different contact' without these
lines & it created the soft credit (using the code elsewhere in the function that
would be used if it were not recurring or not credit card payment)

(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