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 #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

backport 20641

Merge pull request #20693 from eileenmcnaughton/trigg

Fix sorting for drop triggers

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

Fix undefined tpl vars

Merge pull request #20664 from eileenmcnaughton/notice

Always assign mode to smarty

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

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

Merge pull request #20671 from eileenmcnaughton/tax_null

Deprecate passing in 'null' for tax

Move determination of priceSetID to the internal order class

This moves some functionality from the form to where we can start to use if from the order api.

I also added more 'internal' commenting. This is implicitly the case but
I think it would be tempting to use this class and the comments at the top
of the class saying not to could be missed

Deprecate passing in 'null' for tax

This is the source of the php8 test fails -and it's wrong - it should be 0 not null

Fix test to use valid amounts

This fixes the test so the rates match the appropriate rate for the tax

Merge pull request #20663 from eileenmcnaughton/prt_form

Minor tpl simplificaiton, notice reduction

Fix up on invalid test data

Merge pull request #20666 from civicrm/5.39

5.39 to master

Merge pull request #20661 from seamuslee001/dev_core_2604

dev/core#2604 Apply upstream patch to allow single quotes in return p…

Merge pull request #20659 from colemanw/fixUndefinedTplVars

Fix undefined tpl variables on contact summary

Merge pull request #20655 from seamuslee001/fix_apiv4_group_concat_php8

[php8-compact] Fix APIv4 Group Concat Test by setting dataType to be …

Always assign mode to smarty

We should assign regardless of nullity

Minor tpl simplificaiton, notice reduction

This registerMode stuff turns out to be just a long winded way of upper casing the variable

dev/core#2604 Apply upstream patch to allow single quotes in return patch emails

Fix undefined tpl variables on contact summary

Merge pull request #20652 from seamuslee001/fix_array_access_on_number

[NFC] Fixes an issue in this unit test where we are trying to do an a…

Merge pull request #20472 from eileenmcnaughton/trigger_clean

Improve output triggers when logged to file

Merge pull request #20471 from eileenmcnaughton/schema

Fix Schema calculation of usePrefix to cope with rpow:

Merge pull request #20649 from eileenmcnaughton/mem_clean

[NFC] [Test] Cleanup on membership type test

[php8-compact] Fix APIv4 Group Concat Test by setting dataType to be NULL if the first expression is a SQL Function

Merge pull request #20657 from demeritcowboy/these-go-to-11

[NFC/Unit test] - Add a test for getCaseRoles when there's a lot of roles assigned

test lots of roles on cases

[NFC] Fix around returns, cleanup

Merge pull request #20653 from eileenmcnaughton/you_have_lines

[REF] Remove always false if

Merge pull request #20616 from seamuslee001/fix_custom_group_Test

[php8-compact][REF] Fix failing custom group test on php8 by better h…

[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

[NFC] Fixes an issue in this unit test where we are trying to do an array acess on an integer

Fix bug found by Eileen on creating Custom Fields of Paritipant event type and add unit tests to lock in fix

[REF] Shift special handling of extends param of the customGroup create into apiv3 and change Custom Group form to use apiv3

[php8-compact][REF] Fix failing custom group test on php8 by better handling strings in 2nd key of the extends array and also validating the child and main entity work

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