Merge pull request #20729 from demeritcowboy/assign-cleanup

[REF] Remove unnecessary variable wrangling in case roles section

Merge pull request #20754 from colemanw/fixContactSummarySmartyUndefinedVars

Fix undefined tpl vars on contact summary page

Fix undefined tpl vars on contact summary page

Merge pull request #20750 from mattwire/viewrecur

Add missing "View Recurring Contribution" link to contribution view

View recurring contribution button when displaying lineitems

Merge pull request #20747 from seamuslee001/part_guard

[REF] Add in template guard for  not being always set as per html format

Merge pull request #20586 from colemanw/is_current

APIv4 - add is_current as a pseudo (calculated) field

[REF] Add in template guard for  not being always set as per html format

Regen SQL

APIv4 - add is_current as a pseudo (calculated) field

This adds the first calculated field to APIv4. It works by injecting a SQL function and aliasing it
with the name of the fake field. This approach allows it to work as part of the WHERE clause
as well as the SELECT clause, even across joins.

Marking the old "is_current" api param deprecated because the field works better.

Merge pull request #20648 from eileenmcnaughton/ret1

[NFC] Improve use of return parameters in ContributionPage test class

Merge pull request #20745 from demeritcowboy/test-20602

[NFC] Unit test for 20602

test for 20602

Merge pull request #20602 from ixiam/dev#PreserveActivityTabFilter

Fix activity_date_time_relative filter when `preserve_activity_tab_fi…

Merge pull request #20627 from demeritcowboy/currency-fixation

dev/financial#175 - Unable to remove a currency once added

Merge pull request #20744 from deb1990/RSESPRT-64-fix-profile-submit-api

Support saving more than 25 records in Profile.submit api

fix activity_date_time_relative filter when `preserve_activity_tab_filter` setting is on

Merge pull request #20396 from eileenmcnaughton/entry3

[REF] Remove isPayLater from membership batch screen

Merge pull request #20394 from eileenmcnaughton/entry

[REF] Extend testing on membership batch entry & minor cleanup

Merge pull request #20676 from eileenmcnaughton/recur

Fix test to use valid financials

Merge pull request #20742 from eileenmcnaughton/throw

Throw exception rather than fatal

Merge pull request #20734 from seamuslee001/fix_patch_php8

[REF] Fix trying to access array offset from boolean value

Merge pull request #20731 from colemanw/apiExplorerWriteJoins

APIv4 Explorer: show joins for write actions

RSESPRT-64: Support saving more than 25 records in Profile.submit api

Merge pull request #20741 from civicrm/5.39

5.39

Merge pull request #20737 from colemanw/fixUndefinedGroupTree

Fix possibly undefined custom groupTree tpl variable

Merge pull request #20735 from eileenmcnaughton/tpl

Tpl enotice fix

Merge pull request #20740 from seamuslee001/5.39

Add release-notes/5.38.1.md

Throw exception rather than fatal

This makes it easier to debug if the entityName is not
resolved - in general this is a dev site issue as it is
to do with load order

Add release-notes/5.38.1.md

Merge pull request #20717 from seamuslee001/use_civi_event_dispatcher_setup

[REF] Alter new Setup process to use CiviCRM's Event Dispatcher

Fix possibly undefined custom groupTree tpl variable

Tpl enotice fix

I searched for hideStepNumbers & believe it is never assigned

[REF] Fix trying to access array offset from boolean value

[REF] Alter new Setup process to use CiviCRM's Event Dispatcher

Apply patch from Tim to limit what events are dispatched

Ensure APIv3 EntityRef fields always return the value of the `id_field`.

Usually it's safe to assume the `id_field` === "id", which is automatically returned
by APIv3. However in the case of looking up records by name, etc. we must ensure that value
gets returned.

APIv4 Explorer - Show joins for write actions

It recently became possible in APIv4 to use joins during write operations;
now that feature is advertised in the Explorer.

APIv4 explorer - move metadata into .ang.php file

unnecessary wrangling

Merge pull request #19737 from sunilpawar/show_inactive_active_case_role

dev/core#542 dev/core#1947 - Show inactive active case role for closed cases

Merge pull request #20724 from colemanw/apiFieldPermissions

APIv4 - Silently ignore non-permissioned fields instead of throwing exceptions

Merge pull request #20662 from colemanw/contactSummaryImageRegion

Add contact-summary-image region

Merge pull request #20450 from jaapjansma/dev_financials_6_reports

dev/financial#6 Do not include template contributions in reports

dev/financial#6 do not include template contributions in reports

APIv4 - Silently ignore non-permissioned fields instead of throwing exceptions

Previously the api would throw an exception when the user did not have permission to use a field
in the WHERE or HAVING clause.

Merge pull request #20715 from colemanw/fixFkJoins

APIv4 - Fix same-table joins and remove unused code

Merge pull request #20654 from eileenmcnaughton/part_fix

Fix bug where only one participant registration is completed

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

show inactive role on closed case, and give option to active role on case

Fix php notice for case not having manager

Fix pagination count issue

Revert pagination code

revert additional pagination changes

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

APIv4 - Fix same-table joins and remove unused code

This fixes implicit joins between a table and itself, e.g. the Contact.employer_id which links to the Contact table
Also removes alpha code no longer used by APIv4.

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