Merge pull request #20536 from MegaphoneJon/mailing-96

mailing#96: Warn on invalid mailing options

Update CRM/Mailing/BAO/Mailing.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

Merge pull request #20615 from colemanw/afformCivix

[NFC] Afform - Regenerate civix files

Merge pull request #20614 from seamuslee001/fix_test_setup

[NFC] Fix test setup on a couple of tests to make them pass in php8

Merge pull request #20608 from seamuslee001/fix_core_form_array_to_string

[php8-compact] Fix Array to String in formButtons in CRM_Core_FormTest

Merge pull request #20613 from seamuslee001/fix_participant_templates

[php8-compact]  Fix issues with participant message templates causing…

Afform - Regenerate civix files

[NFC] Fix test setup on a couple of tests to make them pass in php8

Merge pull request #20570 from eileenmcnaughton/return

Test cleanup - specify return

[php8-compact]  Fix issues with participant message templates causing Event Tests to fail

Merge pull request #20612 from civicrm/5.39

5.39

Merge pull request #20609 from colemanw/readOnlyApis

APIv4 - Make LineItem, EntityFinancialTrxn and FinancialTrxn read-only

Merge pull request #20611 from eileenmcnaughton/539_o

Fix unreleased api change to more logical variant

Merge pull request #20573 from colemanw/entityTagFilter

APIv4 - Add filter for entity tags

[php8-compact] Fix Array to String in formButtons in CRM_Core_FormTest

test failure fix

Fix unreleased api change

An unreleased change adds the entity ids for membership to the return values.

However, on digging into further cleanup I realised that the entity ids are in
the line item array, along with other values which may or may not be useful,
and it's cleaner, more complete and more maintainable to return the line_items.

It requires an extra foreach loop in the calling code but
I don't see that as a downside.

We should change this in the rc so what is released is consistent

Test cover in testSubmitRecur & other tests in CRM_Member_Form_MembershipTest

APIv4 - Make LineItem, EntityFinancialTrxn and FinancialTrxn read-only

Adds a new ReadOnly trait which annotates write methods as @internal
and sets write permissions to ALWAYS_DENY.

This effectively hides the write actions from the API Explorer,
and restricts use of the write methods to when `checkPermissions = FALSE`.

Regenerate civicrm_generated.mysql

APIv4 - Add filter for entity tags

Merge pull request #20606 from totten/master-html-types

dev/translation#67 - Allow translation of fields which lack an explicit HTML type

Merge pull request #20605 from seamuslee001/fix_contact_test_mysql8

[NFC] Fix test failure on MySQL8 because of lack of order in API

[NFC] Fix test failure on MySQL8 because of lack of order in API

dev/translation#67 - Allow translation of fields which lack an explicit HTML type

_Overview_: The `Translation` entity can be used to translate certain fields. Currently, the field settings (`data_type`, `html_type`)
must satisfy some requirements before being translated. This change slightly relaxes those requirements.

_Before_: String fields can be flagged as translatable - but only if they a text-style HTML widget.

_After_: String fields can be flagged as translatable - if they have text-style HTML widget or no clear widget.

_Comments_: This is motivated to allow `civicrm_msg_template` (`msg_subject`, `msg_text`, `msg_html`) to marked as translatable.

The `data_type` and `html_type` constraints were introduced during a much earlier draft. At that time, the list of fields was
open-ended. Now, the list is much narrower (default=none) and requires an extra opt-in (`hook_translateFields`). Consequenty,
there's less concern about someone trying to translate an inappropriate field.

This patch seemed simplest/lowest-change, though I am open/ambivalent about any of these approaches:

* Completely remove the HTML-type constraint.
* Change the `html_type` data for `MessageTemplate`. (But I don't know if this will have other side-effects.)

Merge pull request #20603 from demeritcowboy/form-opening-tests

[php8-compat] Alternate to the alternate for testOpeningForms

improve flawed test

Merge pull request #20594 from seamuslee001/api_v4_php8_fix

[php8-compact] Fix apiv4 tests in php8 by not incrementing an array

Merge pull request #20599 from demeritcowboy/not-deprecated

[NFC] - update code comment

fix comment

Merge pull request #20597 from seamuslee001/php8_test_fixes

[php8-compact] Further test fixes for php8

Merge pull request #20523 from eileenmcnaughton/pcp2

Send pcp notification only when the contribution is completed

Merge pull request #20596 from seamuslee001/fix_authorizenet_test

[php8-compact] Fix failing AuthorizeNetIPN tests on php8

[php8-compact] Further test fixes for php8

Merge pull request #20595 from seamuslee001/fix_invoke_test

[php8-compact] Fix failing CRM_Core_InvokeTest on php8

Merge pull request #20593 from seamuslee001/session_test_fix

[php8-compact] Fix Session Test errors in php8

[php8-compact] Fix failing AuthorizeNetIPN tests on php8

[php8-compact] Fix failing CRM_Core_InvokeTest on php8

[php8-compact] Fix apiv4 tests in php8 by not incrementing an array

[php8-compact] Fix Session Test errors in php8

Merge pull request #20577 from eileenmcnaughton/smart

Fix a few more undefined template vars

Merge pull request #20592 from seamuslee001/fix_case_message_template

[php8-compact] Fix Case Activity Message Template to fix test failure…

Fix a few more undefined template vars

Merge pull request #20574 from eileenmcnaughton/return2

Reduce test api calls with no 'return'

Merge pull request #20591 from seamuslee001/fix_pdf_letter_test

[php8-compact]  Fix PDF Letter Test by ensuring that all actual DB fi…

Test cleanup - specify return

[php8-compact] Fix Case Activity Message Template to fix test failure in php8

Merge pull request #20589 from colemanw/pledgeApi

SearchKit - Show Pledge as a primary entity

[php8-compact]  Fix PDF Letter Test by ensuring that all actual DB fields are in the tokens array and also by resetting the static cache of tokens

Remove uncessary function call

Reduce test api calls with no 'return'

SearchKit - Show Pledge as a primary entity

Merge pull request #20584 from totten/master-translation-api

dev/translation#67 - Add APIv4 support for "Translation" entity.

Merge pull request #20585 from civicrm/5.39

5.39

dev/translation#67 - Add APIv4 support for "Translation" entity. Expand tests.

Merge pull request #20580 from demeritcowboy/upgrade-table

dev/core#2649 - Make 5.39 upgrade more robust

DynamicFKAccessTrait - Emit explicit error-message for invalid table references

Use-Case: You create a dynamic FK with a reference to an invalid table.

Before: Returns an API error/exception about type-matching (string vs null)

After: Returns an API error/exception about the invalid reference

Merge pull request #20579 from seamuslee001/fix_CRM_Contact_Form_IndividualTest

[php8-compact] Add in guards into various templates to fix the CRM_Co…

Merge pull request #20581 from seamuslee001/fix_contribution_form_tests

[php8-compact] Fixes test failures in the CRM_Contribute_Form_Contrib…

be re-runnable and don't force utf8mb4

[php8-compact] Fixes test failures in the CRM_Contribute_Form_ContributionTest class due to template issues

[php8-compact] Add in guards into various templates to fix the CRM_Contact_Form_IndividualTest suite

Merge pull request #20575 from seamuslee001/dashboard_test_fix

[php8-compact] Fix Dashboard Tests by adding in if guards into smarty…

Merge pull request #20544 from seamuslee001/even_more_guards

[php8-compact] Add in guards into various templates to fix issues in …

[php8-compact] Add in guards into various templates to fix issues in test runs on php8

Merge pull request #20546 from seamuslee001/even_more_guards2

[php8-compact] Add in more protection for unset variables in various …

Merge pull request #20547 from seamuslee001/admin_guards

[php8-compact] Add in protection for unset variables in admin smarty …

[php8-compact] Fix Dashboard Tests by adding in if guards into smarty templates

Merge pull request #20576 from civicrm/5.39

5.39 to master

Merge pull request #20571 from demeritcowboy/regen-539

Update civicrm_generated (5.39 version)

Merge pull request #20543 from seamuslee001/guards_common

[php8-compact] Add in guards into common templates to assit with fixi…

[php8-compact] Add in more protection for unset variables in various smarty templates

[php8-compact] Add in protection for unset variables in admin smarty templates

update civicrm_generated

Merge pull request #20568 from civicrm/5.39

5.39

Merge pull request #20567 from seamuslee001/fix_unittest_warning

[NFC] Fix phpunit9 deprecation issues on using assertType instead of …

[NFC] Fix phpunit9 deprecation issues on using assertType instead of more explicit assertIsArray or AssertIsInt

Merge pull request #20566 from eileenmcnaughton/role

Add pseudoconstant to acl_entity_role

Merge pull request #20565 from eileenmcnaughton/val_err

Throw exception rather than allow php fail

Add pseudoconstant to acl_entity_role

Not sure this will help with dev/core#2615 but spotted it while trying to see if I can replicate
(still doing that)

Set version to 5.40.alpha1

Merge pull request #20432 from JMAConsulting/core_365_2

(dev/core#365) Scheduled Reminders - Add effective start and end date to admin UI

Merge pull request #19973 from JMAConsulting/core_365_1

(dev/core#365) Scheduled Reminders - Add effective end and start date, extend unit test

ext/*/info.xml - Update version

Set version to 5.39.beta1

Throw exception rather than allow php fail

Merge pull request #20532 from eileenmcnaughton/f_add

Fix Financial item test to validate Financials

Merge pull request #20552 from eileenmcnaughton/fin

Mark DetailTest as having invalid financials

Merge pull request #20533 from totten/master-api4-omnivent

Introduce civi.api4.authorizeRecord and civi.api4.validate

Merge pull request #20564 from eileenmcnaughton/report

dev/core#2646 juice up the error a little

Merge pull request #20493 from jaapjansma/dev_membership_37

Fix for dev/membership#37

Merge pull request #20563 from colemanw/array_key

REF - Cleanup array key checking to use array_key_exists

dev/core#2646 juice up the error a little

On explaining the issue in https://lab.civicrm.org/dev/core/-/issues/2646 not
having the name of the failed entity in the error message seemed like a pain point

Expand CustomValue::_checkAccess()

Before: Reports that access is available based one delegated-check for `Contact.update`

After: Reports that access is available based on multiple checks:

  1. The user must have access to the relevant CustomGroup (by way of ACL or perms)
  2. The user must have acces to the underlying entity (by way of checkAccessDelgated)

Comments: I did a bit of testing with `Custom_*.get`, and it does seem to
give access to single-value CustomGroups. So I removed a comment about
multi-value CustomGroups and expanded to a larger list of entities.

Merge pull request #20562 from colemanw/shortArrayContactTask

[REF] CRM_Contact_Task - Use short array syntax

Merge pull request #20059 from samuelsov/dev/core#2479

dev/core#2479 Adjust generic copy for localizable fields

REF - Cleanup array key checking to use array_key_exists

Before: in_array($foo, array_keys($bar))
After: array_key_exists($foo, $bar)

CRM_Contact_Task - Use short array syntax

Partially rollback changes to `$userID`. Merely lay groundwork for future update.

Context: AuthorizeEvent did not allow tracking userID.  AuthorizeRecordEvent
is spec'd to track userID.  This is a step toward supporting checks when the
target user is non-present (ie not the user in the browser/session).
However, this step is not *sufficient* - additional work is also needed to
support non-present users.

Original: AuthorizeEvent and AbstractAction::isAuthorized did not report
current userID.  However, the wiring for AuthorizeRecordEvent is spec'd
to allow userID.

Previous: Made a breaking change in the signature of
AuthorizeEvent/AbstractAction::isAuthorized() to report userID.  However,
even with the break, it's not clear if this is the best approach.

Revised:
* Both AuthorizeEvent and AuthorizeRecordEvent report `userID`. This allows consumers to start using
  this information -- laying the groundwork for future changes.
* If an existing event-consumer ignores the `userID`, it will still work as correctly as before. This is
  because we guarantee that the userID matches the session-user.
* The signature of `AbstractAction::isAuthorized()` matches its original. No BC break. However, the method
  is flagged `@internal` to warn about the prospect of future changes.
* In the future, after we do more legwork on to ensure that the overall
  system makes sense, we may flip this and start doing non-present users.

Merge pull request #20560 from colemanw/searchable

APIv4 - Make the @searchable flag explicit for OptionList type entities.

Merge pull request #20545 from seamuslee001/hook_test_fix_part1

[php8-compat] Partial Fix of hook tests for php8

Merge pull request #20548 from eileenmcnaughton/group

Fix the populateTempTable to be more direct