[REF] Remove always false if

We recently added code to ensure line_item is always populated
before this function is called. So line items is never empty at this point

Merge pull request #20621 from eileenmcnaughton/mps

Add test validation for Membership & participant payments

Merge pull request #20623 from seamuslee001/fix_event_form_test_php8

[php8-compact] Fix Event Form test for php8 by setting totalAmount in…

Add test validation for Membership & participant payments

Merge pull request #20639 from colemanw/getReferencesToContactTable

dev/core#2656 Fix getReferencesToContactTable to ignore non-id references

Merge pull request #20647 from eileenmcnaughton/cl

[NFC] [Test] Remove duplicate cleanup

Merge pull request #20633 from eileenmcnaughton/ret3

[NFC] [TEST] cleanup, more use of return in ActivityTest

Merge pull request #20636 from eileenmcnaughton/mm

[NFC] Cleanup on membership test

Merge pull request #20637 from eileenmcnaughton/mem

[NFC] [Test] Remove duplicate cleanup

Merge pull request #20638 from eileenmcnaughton/ret2

[NFC] Cleanup on contribution v3 api test

Merge pull request #20644 from lcdservices/dev-core-2657

dev/core#2657 trigger pre hook on LineItem delete

[NFC] [Test] Remove duplicate cleanup

Merge pull request #20646 from civicrm/5.39

5.39

Merge pull request #20554 from colemanw/noteUsedFor

Convert Notes to use option_group instead of hardcoded list

Merge pull request #20645 from agh1/5.39.0-releasenotes-initial

5.39.0 release notes initial run

Regenerate sql with note_used_for option group

Convert Notes to use option_group instead of hardcoded list

5.39.0 release notes: added boilerplate

5.39.0 release notes: raw from script

Merge pull request #20642 from colemanw/admin

Remove unreachable code from admin page

Merge pull request #20643 from colemanw/moveUpgradeCode

Move upgrade code to correct version

dev/core#2657 trigger pre hook on LineItem delete

Merge pull request #20641 from pradpnayak/urlfix

Fixed url

Merge pull request #20640 from colemanw/smartyNotices

Show smarty E-notices when in debug mode

Move upgrade code to correct version

See https://github.com/civicrm/civicrm-core/pull/20553

Remove unreachable code from admin page

Merge pull request #20553 from JMAConsulting/add_product_id_fk

dev/core#2486 - Add foreign key on civicrm_contribution_product.product_id

Fixed url

Show smarty E-notices when in debug mode

Fix getReferencesToContactTable to ignore non-id references

Fixes dev/core#2656
https://lab.civicrm.org/dev/core/-/issues/2656

Merge pull request #20635 from eileenmcnaughton/reta

[NFC] [TEST] cleanup, more use of return in AddressTest

Add product_id FK

[NFC] Cleanup on contribution v3 api test

[NFC] [TEST] cleanup, more use of return in ActivityTest

[NFC] [Test] Remove duplicate cleanup

Cleanup on membership test

Merge pull request #20607 from colemanw/searchDisplayAccessBypass

SearchKit - Allow super admins to disable Search Display access checks

Merge pull request #20634 from highfalutin/patch-7

replace unnecessary function call with string literal

Merge pull request #20630 from eileenmcnaughton/trigg_mailing

Use mysql to set mailing.created_date

[NFC] [TEST] cleanup, more use of return in AddressTest

Merge pull request #20632 from eileenmcnaughton/ret

[NFC] [Test] Specify return values on get calls

replace unnecessary function call with string literal

Merge pull request #20628 from colemanw/deprecateAngular

AngularLoader - Emit warning when setModules method is used.

Use mysql to set mailing.created_date

On digging I determined this trigger hokiness was all about mysql
5.5 and we can ditch it now

https://dev.mysql.com/doc/relnotes/mysql/5.6/en/news-5-6-5.html#mysqld-5-6-5-data-types

and https://github.com/civicrm/civicrm-core/pull/10953#issuecomment-327613771

Note that we still support mysql 5.6 and the current version is 5.6.51

I think it's fine to require that the 5.6 minimum is 5.6.5 as released in 2012

I'll do the upgrade script as a follow up

Merge pull request #20622 from eileenmcnaughton/act2

dev/core#1191 [TEST][NFC] Increase setting of return in tests

[NFC] [Test] Specify return values on get calls

Merge pull request #20598 from colemanw/entityDescriptions

SearchKit - Add translatable descriptions to entities

Merge pull request #20617 from eileenmcnaughton/cust_g

Improve CustomGroupTest

Improve CustomGroupTest

Use api, various cleanups

Merge pull request #20629 from civicrm/5.39

5.39

SearchKit - Add translatable descriptions to entities

Before: Entities had descriptions derived from code comments, which were written for developers and English-only
After: Entities can have a user-friendly description which is translated

Merge pull request #20618 from eileenmcnaughton/return2

dev/core#1191 Try adding a strict return function to getAndCheck

Merge pull request #20624 from seamuslee001/query_test_php8

[php8-compact][NFC] Update testGroupClause unit test to work on php8

Merge pull request #20610 from colemanw/fixApiOrder

dev/report#68 Fix smart group crash when ordering by aggregated column

Merge pull request #20536 from MegaphoneJon/mailing-96

mailing#96: Warn on invalid mailing options

AngularLoader - Emit warning when setModules method is used.

It can cause conflicts with other extensions who have added modules.

SearchKit - Fix smart group crash when ordering by aggregated column

Fixes dev/report#68

Update CRM/Mailing/BAO/Mailing.php

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

[php8-compact][NFC] Update testGroupClause unit test to work on php8

[php8-compact] Fix Event Form test for php8 by setting totalAmount in the form params

[TEST][NFC] Increase setting of return in tests

Try adding a strict return function to getAndCheck

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

SearchKit - Enable super-admins to disable Search Display access checks

This allows users with 'all CiviCRM permissions and ACLs' to configure a search display
to bypass permission checks and display all records to the user.

Once a display is set to bypass ACLs, it can only be edited by a super-admin,
ordinary admin users will not be able to edit the display or the saved search.

Such a display will not automatically appear on its own page; it must be
embedded in an Afform, and the Afform will act as gatekeeper for users
to view the display.

APIv4 - Preserve field data type when aggregating into an array using GROUP_CONCAT

Ensures that e.g. an array of integer fields will be returned as integers and not an array of strings

SearchKit - Add acl_bypass column to civicrm_search_display table

The new column determines whether permissions will be checked when running a display

Change CRUD permissions for SavedSearch and SearchDisplay to 'administer CiviCRM data'

By default, the permission was previously 'administer CiviCRM'.
The new permission is a subset for data administrators.

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

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

SearchKit - Regenerate civix file

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