Merge pull request #20726 from colemanw/fixUndefinedTplVars

Fix undefined tpl vars on custom field page

Merge pull request #20716 from colemanw/emitApi4NoticesAboutDeprecatedJoinAliases

APIv4 - Emit notices about deprecated join aliases

Fix undefined tpl vars on custom field page

Merge pull request #20725 from colemanw/fixUndefinedRowClass

Fix often undefined variable $row.class in tpl tables

Fix often undefined variable $row.class in tpl tables

APIv4 - Use non-deprecated join syntax internally

Updates APIv4 calls in CiviCRM core to use e.g. `event_id.title` instead of `event.title`

Merge pull request #20723 from colemanw/api4ExplorerJoins

APIv4 Explorer - Render joins using addJoin function in OOP

Merge pull request #20707 from colemanw/getIdField

APIv4 - Don't assume the identifier field for a table is named 'id'

Merge pull request #20452 from jaapjansma/dev_financials_6_contactsummary

dev/financial#6 Template contributions on the contact summary

Merge pull request #20451 from jaapjansma/dev_financials_6_search

dev/financial#6 Added is template to search screen

Merge pull request #20722 from demeritcowboy/sms-translate

 dev/core#1711 - small translation improvement

APIv4 - Emit notices about deprecated join aliases

Merge pull request #20721 from colemanw/apiExplorerBridge

APIv4 Explorer - When joining via EntityBridge, make bridge fields selectable

APIv4 Explorer - Render joins using addJoin function in OOP

translation improvement - dev/core#1711

APIv4 Explorer - When joining via EntityBridge, make bridge fields selectable

Merge pull request #20720 from mattwire/extensionsaddnewdescription

Replace extension key with description in 'Add New' tab

Replace extension key with description in 'Add New' tab as already done in main list

Merge pull request #20220 from johntwyman/core-1711-show-sms-segment-count

dev/core#1711 - Display SMS segment count when constructing Mass SMS

Merge pull request #20711 from demeritcowboy/dashboard-err

Some php warnings on case dashboard

dev/core-1711 Show segment count for SMS construction

Use Standard methods for downloading third party js package

Update composer.lock

Merge pull request #20713 from totten/5.39-setup-url

dev/core#2663 - Setup - Consistently handle special characters per URL conventions

Merge pull request #20714 from demeritcowboy/fix-time

Hopefully fix time rollover problem in MessageTemplateTest

Merge pull request #20697 from eileenmcnaughton/prem

Remove deprecated use of format money

Merge pull request #20712 from demeritcowboy/newcase-1

Some php warnings on new case

dev/core#2663 - Setup - Consistently handle special characters per URL conventions

Overview
--------

This addresses a problem with the setup UI when installing CiviCRM with certain database credentials.
Specifically, it handles special characters more correctly.

https://lab.civicrm.org/dev/core/-/issues/2663

Before
------

If the database password requires any special characters (e.g. "#" or "&"), then they cannot correctly
entered.

There are two ways you might try to enter a password (e.g.  plain
`mysql://myuser:top#secret...`or URL-encoded `mysql://myuser:top%23secret...`). Neither
of these work. (The plain notation might pass the validator, but it won't be written correctly to disk.
The URL-encoded notation won't pass the validator.)

After
-----

You may use special characters, as long as the URL is properly encoded, e.g. `mysql://myuser:top%23secret...`

fix time rollover

APIv4 - add test for id_field

APIv4 - Don't assume the identifier field for a table is named 'id'

APIv4 getInfo() now returns the name of the "id_field" for any entity.
This modifies the selectQuery so that `get` actions rely on that metadata.

php errors

php warnings

Merge pull request #20694 from colemanw/validateCustomGroupName

Ensure custom group name does not conflict with existing field

Merge pull request #20708 from colemanw/cacheApi4Info

[REF] - Improve efficiency of APIv4 GetInfo

Merge pull request #20696 from eileenmcnaughton/except

Throw exception when baoName is not found

Merge pull request #20709 from colemanw/api4CleanupBridgeJoinable

APIv4 - Remove POC code leftover from early alpha development

Throw exception when baoName is not found

It the BAO name is not found at this point we currently get a fatal error. This makes it more
recoverable. The specific case I'm hitting at the moment relates to monolog - the
caching of the symfony container means it persists between builds and is
accessed before the extension is installed. However the entity is not
yet registered. Historically I have hit this fatal in other
scenarios but I can't recall the details.

APIv4 - Refactor getInfo calls to use new utility function

APIv4 - Store entity info in metadata cache

Note that this caches the info returned by the generic function in
`AbstractEntity::getInfo()` and some entities override that function.
However the overrides generally call the parent and do not perform any expensive
computations of their own, so this seems like the best spot to add caching.

APIv4 - Remove POC code from early alpha

During early APIv4 development there was an idea to create a BridgeJoinable class and
connect things like activity->assignees.
It was experimentally implemented for that field one field, but never documented or tested.

Merge pull request #20631 from eileenmcnaughton/custom_ref

dev/core#2656 Add test to demonstrate issue / bug

Merge pull request #20673 from colemanw/contactTabsCleanup

REF - Get Contact Summary tab counts after calling hook to avoid wasting time

Merge pull request #20705 from eileenmcnaughton/prim

Fix schema support for composite primary keys

Merge pull request #20698 from colemanw/limit25ForGet

APIv4 Explorer - Only apply default limit 25 for "get" action

Fix support for composite primary keys

Fixes what looks like an oversight
https://github.com/civicrm/civicrm-core/commit/022785d8d5d4876aa1211b7adc0d7c31e2995fc5#diff-08ba0d294c1f2e5b9d42e4768aa3b30a10e5128809c590f155c5ac0cc9b2f69cL480-L481

An 'if' was added but the uncondition line left in

The result is that if a composite primary field IS defined
per below an entry for it is incorrectly added to the field
array with just autoincrement set.
`
    <primaryKey>
        <name>contact_timestamp_type</name>
        <field>contact_identifier</field>
        <field>recipient_action_datetime</field>
        <field>event_type</field>
    </primaryKey>
`

Merge pull request #20700 from colemanw/removeApi4BoilerplateComment

[NFC] Remove boilerplate comment block from APIv4 classes

Merge pull request #20701 from colemanw/apiEntitySince

APIv4 - Add `@since` annotation to each entity

Merge pull request #20703 from civicrm/5.39

5.39 to master

Merge pull request #20699 from demeritcowboy/urlfix-539

Backport of 20641 - url fix on payment edit form

APIv4 - Add `@since` annotation to each entity

This will be output by `Entity:get` to show which version each entity was added

Remove boilerplate code block from APIv4 classes

backport 20641

Merge pull request #20693 from eileenmcnaughton/trigg

Fix sorting for drop triggers

dev/financial#6 Added is template to search screen

APIv4 Explorer - Only apply default limit 25 for "get" action

The API Explorer sets a default limit of 25 so you don't get an overwhelming result set
from a `get` action.

However, for other actions like `getFields` or `getActions`, there is no need for
a limit and the default is annoying.

Remove deprecated use of format money

This is logging deprecation errors. It doesn't work quite right before or after but
I suspect no-one much cares as it hasn't been raised before now

Ensure custom group name does not conflict with existing field

Fix sorting for drop triggers

I added a ksort but it should have been an asort - the createTriggers has an array like
['civicrm_acl'] => [.....]

which suits asort but in this location the array is

[0 => 'civicrm_acl', 1 =>.....

so asort is right

Merge pull request #20691 from demeritcowboy/statlinks

dev/core#2647 - Deprecated call to non-static CRM_Financial_Form_BatchTransaction::links()

dev/core#2647 - deprecated call to non-static

Merge pull request #20677 from colemanw/resolveFKValues

APIv4 - Allow FK dot notation in create/update/save actions

Merge pull request #20690 from civicrm/5.39

5.39

Merge pull request #20689 from agileware/CIVICRM-1784

Reinstate invoiceID param to membership renewal payment

Merge pull request #20686 from demeritcowboy/newtag

dev/core#2660 - Remove old menu entry for "New Tag"

Merge pull request #20688 from demeritcowboy/coincidence

Force the right path in github action script

CIVICRM-1784 Reinstate invoiceID param to membership renewal payment

Fixes a compatibility regresson introduced for 5.36

force the right path

remove old menu entry

Merge pull request #20625 from seamuslee001/allow_civicrm_install_php8

[php8-compact] Allow for CiviCRM to be installed by 3rd parties using…

Merge pull request #20626 from akosgarai/issue/2645

dev/core#2645 Proposed fix for exporting preferred_mail_format field

Merge pull request #20620 from eileenmcnaughton/539_test

Add file_exists check on loading services xml

Merge pull request #20684 from colemanw/tplVars

Fix undefined tpl vars

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

Fix undefined tpl vars

Merge pull request #20664 from eileenmcnaughton/notice

Always assign mode to smarty

dev/financial#6 Added is template to search screen

dev/financial#6 Added is template to search screen

Merge pull request #20683 from civicrm/5.39

5.39

Merge pull request #20675 from eileenmcnaughton/ev

dev/event#58 Ensure custom data is not saved to template's row

Merge pull request #20679 from eileenmcnaughton/smarty1

Extract feeBlock to it's own tpl

Merge pull request #20680 from demeritcowboy/regen-github-action

[NFC] Github action script to run regen.sh for you

APIv4 - Support FK field lookups in create/update/save actions

Test cleanup - use new APIv4 syntax for checkPermissions

script to run regen.sh for you

Merge pull request #20268 from agileware/CIVICRM-1725

User experience improvement - Remove the confusing mixed terminology used to describe Tags: "Tags (Categories)" and "Tag (Category)"

Extract feeBlock to it's own tpl

This is a step towards moving the FeeBlock rendering to it's own form instead of the current 'overload'
approach

CIVICRM-1725 User experience improvement - Remove the confusing mixed terminology used to describe Tags: "Tags (Categories)" and "Tag (Category)"

Start to tighten open-ended 'return' in tests

dev/core#2656 Add test to demonstrate issue / bug

Fix enotice in test

Why didn't this cause it to fail before

Merge pull request #20674 from seamuslee001/regen_dao

[NFC] Regenerate ContributeProduct DAO and also fix search_kit instal…

Add unit test for event copy

dev/event#58 Ensure custom data is not saved to template's row

This ensures than when we are copying (implicity a create-new-action) we do not
update an existing row in the custom data table by explicity unsetting the
id in the copy function. This has been bouncey behaviour because
the formatting for form purposes relies on form type formatting and yet
for save purposes it's not really fit for purpose. Although a bit
hacky this approach should be robust & avoids a re-write on the form

Merge pull request #20667 from eileenmcnaughton/prt

Fix up on invalid test data

Merge pull request #20669 from eileenmcnaughton/member

Move determination of priceSetID to the internal order class

Merge pull request #20668 from eileenmcnaughton/tt

Fix test to use valid amounts

[NFC] Regenerate ContributeProduct DAO and also fix search_kit install on MySQL 5.6 on test boxes

[php8-compact] Allow for CiviCRM to be installed by 3rd parties using composer on php8 systems

Fix unit test

REF - Get Contact Summary tab counts after calling hook to avoid wasting time

If a tab was removed by hook, the count isn't needed.

Merge pull request #20671 from eileenmcnaughton/tax_null

Deprecate passing in 'null' for tax