Merge pull request #16720 from agh1/5.24.0-release-notes

5.24.0 release notes: initial run

Merge pull request #16721 from seamuslee001/524_1637

dev/core#1637 Revert "Make $civicrm_paths less sensitive to trailing slashes. Add t…

Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests."

This is currently causing breakage on wordpress sites where clean urls are not enabled.

Compare the 2 urls below - the top one has an extra (breaking) slash added by this PR.

I propose a quick revert  & patch release followed by 'the right' fix at a slower pace
/wp-admin/admin.php/?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext
/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext

This reverts commit 232fdd3dfac4ea09749d7f178818139514e48e38.

5.24.0 release notes: added boilerplate

5.24.0 release notes: raw from script

Merge pull request #16707 from demeritcowboy/smartgroup-warning

dev/core#1636 - In system status check avoid E_NOTICEs for smart groups that don't have form_values[0]

skip smart groups that don't have a form_values[0]

Set version to 5.24.beta1

Merge pull request #16677 from eileenmcnaughton/mem

[REF] Change function signature to support moving this off the form layer

Merge pull request #16666 from colemanw/smarter

APIv4-based smart groups

Merge pull request #16649 from alifrumin/copyState3

dev/core#1605 State/province not copied on shared address

[REF] Change function signature to support moving this off the form layer

This is really a business logic function & belongs on the BAO. This change simply removes the
access to  such that it can be moved by making it an input parameter.

Next step would be o make it static & move  it to CRM_Member_BAO_Membership

Evenually this should not be called by the form as the CRM_Member_BAO_Membership
::create function will call it

Fix "no such field" failure in SmartGroup upgrade

Calling api get with no return fields specified is always risky during upgrades because upgrades can add/drop columns.

APIv4 Explorer - add docs for save & execute buttons

APIv4-based smart groups

APIv4 - Add SavedSearch entity

Add columns for api_entity & api_params to civicrm_saved_search

Merge pull request #16684 from civicrm/5.23

5.23

Merge pull request #16686 from demeritcowboy/install-seqcreditnotes

Fix parameter format for upgrade call to install/enable sequentialcreditnotes

Merge pull request #16687 from demeritcowboy/letter-h

[NFC] Add missing letter "h" in upgrade script for task description

Merge pull request #16685 from mlutfy/fixSettingsYesNo

Fix SettingTrait YesNo translation

Merge pull request #16680 from magnolia61/Contribution_Invoice_Enhancements

Style & layout clean up

Merge pull request #16683 from demeritcowboy/disable-xoauth2

dev/mail#59 - Don't release 5.23 with XOAUTH2 option enabled since it isn't ready yet

fix parameter format

add missing letter h

Merge pull request #16020 from mattwire/setentityid

Add setEntityId() to entityForm

Fix SettingTrait YesNo translation

Style & layout clean up

Merge pull request #16682 from colemanw/remove

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

Update unit test now that the xoauth2 option is disabled for the time being

Merge pull request #16681 from alifrumin/rn5.23secondpass

rn-5.23: adding late changes to the 5.23 release notes [nfc]

add test

xoauth2 isn't ready yet

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

adding late changes to the 5.23 release notes

Merge pull request #16675 from christianwach/lab-1628

Fix display of administrator permissions in WordPress Multisite

Fix display of administrator permissions in WordPress Multisite

Merge pull request #16676 from christianwach/lab-1629

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16679 from pradpnayak/priceSetSchema

added sudo constant for sms api type

added sudo constant for sms api type

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16672 from MegaphoneJon/core-1627

Activity Details report: fix filtering by is null/is not null

Activity Details report: fix filtering by is null/is not null

Merge pull request #16667 from eileenmcnaughton/acl

[REF] Only call getACLs when contact_id is present, remove handling

Merge pull request #16566 from eileenmcnaughton/setting_fix

Remove helper function now that contribution settings is not weirdly stored

Merge pull request #16647 from JMAConsulting/core-1623

core#1623: My Case dashlet doesn't sort by name but contact_id instead

Merge pull request #16669 from demeritcowboy/audit-tpl-2

dev/core#1366 - CRM_Case_XMLProcessor_Report::run() is never called anymore

Merge pull request #16668 from eileenmcnaughton/php

Increase php min recommended version

remove unused function

core#1623: My Case dashlet doesn't sort by name but contact_id instead

Increase php min recommended version

We seem to not be doing a good job of communicating about dropping php 7.0
but given php 7.1 is now out of support https://www.php.net/supported-versions.php
it seems pretty clear it meets no longer meets the criteria for 'minimum recommended'

https://www.php.net/supported-versions.php

[REF] Only call getACLs when contact_id is present, remove handling

getACLs is called from one place in the code. Within the function there are 2 places that
handle the possibility it might be null - this moves it to the calling function
& requires contact id to be an int

Merge pull request #16665 from pradpnayak/priceSetSchema

Add pseudoconstant for priceset

Merge pull request #16664 from totten/master-ext-hidden

Extensions - Change new tag "hidden" to "mgmt:hidden"

Merge pull request #16624 from mattwire/tokencompatsubscriber

TokenProcessor - fix greetings tokens

Add pseudoconstant for priceset

Extensions - Change new tag "hidden" to "mgmt:hidden"

Overview
--------

This is follow-up from the discussion about how to name extension tags

https://github.com/civicrm/civicrm-dev-docs/pull/761

It's a follow-up #16531 (dev/financial#84) within the same 5.24.alpha1 dev cycle.

Before
------

The `sequentialcreditnotes` extension is tagged `hidden` and omitted
from display in the web UI.

After
-----

The `sequentialcreditnotes` extension is tagged `mgmt:hidden` and omitted
from display in the web UI.

Merge pull request #16662 from seamuslee001/tcpdf_depreaction

[REF] Deprecate _html2pdf_tcpdf function in favour of _html2pdf_dompdf

Merge pull request #16663 from eileenmcnaughton/acl

[REF] Removed unused function

Merge pull request #16655 from seamuslee001/hook_uf_fields

[Feature] Add in new hook alterUFFields to allow extensions to modify…

Add in code doucmentation of the format of the fields array and add in a unit test

[REF] Removed unused function

I did some searches - this isn't called

Merge pull request #16660 from demeritcowboy/audit-tpl

dev/core#1366 - Case Audit.tpl file no longer used

[REF] Deprecate _html2pdf_tcpdf function in favour of _html2pdf_dompdf

Merge pull request #15964 from mlutfy/financial109

dev/financial#109 Fix country/province assignation in the contribution invoice

Merge pull request #16648 from pradpnayak/priceSetSchema

Add pseudoconstant for priceset

Merge pull request #16661 from seamuslee001/typo_fix

Fix unreleaesd typo error

Fix unreleaesd typo error

Merge pull request #16658 from mattwire/fixtypo

Fix typo (unrelease regression)

Add pseudoconstant for priceset

dev/financial#109 Fix country/province assignation in the contribution invoice

remove files no longer used

Fix typo (unrelease regression)

Merge pull request #16657 from mattwire/fixcomment

Fix mistake in comment

Fix mistake in comment

[Feature] Add in new hook alterUFFields to allow extensions to modify which fields can be added to a profile

Merge pull request #16652 from UshaMakoa/patch-2

Update contributor-key.yml

Merge pull request #16653 from seamuslee001/hook_pre_post_uf_field

[REF] Add in pre and post hooks to UFField Entity

Update TokenTest.php

Merge pull request #16644 from seamuslee001/extension_notices

[NFC] Use insert ignore for inserts into civicrm_extension to stop warnings on duplicate entry for sequential credit notes extension

[REF] Add in pre and post hooks to UFField Entity

Update contributor-key.yml

dev/core#1605 State/province not copied on shared address

Merge pull request #16632 from colemanw/dash

News dashboard - Code cleanup to update js & css

Merge pull request #16645 from colemanw/null

APIv4 - Correctly return null values from DAO save actions

Add basic test for TokenCompatSubscriber

APIv4 - Correctly return null values from DAO save actions

Make sure the value "null" is not returned as a string
Don't return null values from saved DAO objects unless it is in params or reload is set to true

[NFC] Use insert ignore for inserts into civicrm_extension to stop warnings on duplicate entry for sequential credit notes extension

Merge pull request #16615 from MegaphoneJon/selfsfc-refactor

[REF] Extract self-service eligibility code into its own function

Merge pull request #16627 from seamuslee001/aug_custom_fields_fix

[REF] Apply fix for CRM-607 for when building the select section of t…

Merge pull request #16638 from UshaMakoa/patch-1

Contact Type Values with Cap in order to be well translated with ts()

Merge pull request #16643 from demeritcowboy/activity-summary-groupby

dev/core#1619 - Fatal error when grouping Activity Summary report by activity date

Merge pull request #16642 from eileenmcnaughton/ss2

[REF] simple function extraction

Merge pull request #16595 from seamuslee001/refactor_payment_processor_radio

[REF] Refactor adding payment processor radio section onto register a…

Merge pull request #16637 from eileenmcnaughton/ss

Remove unused columns from civicrm_saved_search

Merge pull request #16639 from seamuslee001/dev_core_1621

[NFC] dev/core#1621 Extend unit tests to ensure that entity financial account is co…

Merge pull request #16640 from JMAConsulting/dev-core-1158

core#1158: Unit Test to ensure primary addresses are printed on mailing label if searchPrimaryDetailsOnly is disabled

Merge pull request #16641 from colemanw/apiError

Api4 - Display sql errors in explorer

[REF] simple funnction extraction

[REF] Extract self-service eligibility code into its own function