Merge pull request #16728 from civicrm/5.24

5.24

Merge pull request #16727 from pradpnayak/priceSetSchema

pseudoconstant for bounce type

Merge pull request #16719 from colemanw/arrayValue3

[REF] Use ?? operator instead of CRM_Utils_Array::value() in return statements

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

5.24.0 release notes: initial run

pseudoconstant for bounce type

Merge pull request #16711 from eileenmcnaughton/deprecated

Fix another instance of NULL being passed to CRM_Utils_Array::value

Merge pull request #16725 from civicrm/5.24

5.24

Merge pull request #16721 from seamuslee001/524_1637

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

Merge pull request #16692 from MegaphoneJon/financial-119

Fix PayPal Express on events

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

Use ?? operator instead of CRM_Utils_Array::value() in return statements

Merge pull request #16713 from eileenmcnaughton/regress

dev/core#1637 regression fix on WP urls

Merge pull request #16712 from eileenmcnaughton/dep2

Fix another instance of silly use of CRM_Utils_Array::value

Merge pull request #16693 from eileenmcnaughton/prior

[NFC] Fix comments, use single quotes

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.

Fix another instance of silly use of CRM_Utils_Array::value

Merge pull request #16703 from pradpnayak/priceSetSchema

Added pseudoconstant for civicrm_tag.parent_id

Fix another instance of NULL being passed to CRM_Utils_Array::value

Note that the test has only cosmetic changes picked up while running the test to identify the bug

Merge pull request #16709 from civicrm/5.24

5.24

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]

Merge pull request #16706 from eileenmcnaughton/array2

Remove an instance of passing non-arraay to CRM_Utils_Array::value(

Merge pull request #16704 from colemanw/arrayValue1

[REF] CRM_Utils_Array::value() -> empty()

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

Remove an instance of passing non-arraay to CRM_Utils_Array::value(

Merge pull request #16696 from eileenmcnaughton/remove

Remove calls & deprecate CRM_Core_BAO_PrevNextCache::setItem

Merge pull request #16705 from mlutfy/api4-ajax-perm

Api4 AJAX endpoint: change required permission to access AJAX API.

CRM_Utils_Array::value() -> empty()

Replaces CRM_Utils_Array::value() with empty() when the value is immediately cast to boolean.
The two statements have identical meaning but the latter is faster & easier to read.

Api4 AJAX endpoint: change required permission to access AJAX API.

Flush pseudoconstant cache when saving a tag

Merge pull request #16701 from eileenmcnaughton/array2

Fix passing a non-array to CRM_Utils_Array::value

Fix passing a non-array to CRM_Utils_Array::value

As surfaced in https://test.civicrm.org/job/CiviCRM-Core-PR/32543/console when we tested deprecating
passing in a non-array

Added pseudoconstant for civicrm_tag.parent_id

Merge pull request #16690 from kartik1000/Shift_Update_Membership

Shifted UpdateMemberhsip to BAO Layer

Merge pull request #16689 from pradpnayak/priceSetSchema

Use dbAlias to generate where clause for date field in activity report

Merge pull request #16698 from eileenmcnaughton/note

[NFC] Mark unused function deprecated, fix some comments & formatting

[NFC] Mark unused function deprecated, fix some comments

Remove calls & deprecate CRM_Core_BAO_PrevNextCache::setItem

We have this function which

1) is mildly misleading - it appears to be generic to the prevnext table but is actually
only relevant to deduping as searches no longer use it
2) is about 50% deprecated and
3) the remainder is a single insert
4) is called from 2 places which use it a little differently

I think it's not really adding much value - I was going to remove the deprecated code
but I think in fact the goal should be to remove the whole function.

In general I think code on CRM_Core_BAO_PrevNextCache that is really dedupe-only code
should be on the dedupe classes. The history is that prevnext was created for searches
and kindof twisted to support dedupe as well but now search doesn't use much of what
is in the BAO class (if any)

Fix PayPal Express on events

[NFC] Fix comments, use single quotes

added empty line

Signed-off-by: Kartik Kathuria <kathuriakartik0@gmail.com>

Shifted UpdateMemberhsip to BAO Layer

Signed-off-by: Kartik Kathuria <kathuriakartik0@gmail.com>

Merge pull request #16550 from eileenmcnaughton/display

Remove additional display of buttons for events

Merge pull request #16625 from seamuslee001/jquery_validation_upgrade

Upgrade jQuery validation version to v1.19.1

Merge pull request #16688 from eileenmcnaughton/prior

Mark unused function as deprecated

Merge pull request #14667 from hoegrammer/master

CRM-20553 Stop contact being overwritten with paypal account name

Use dbAlias to generate where clause for date field in activity report

Mark unused function as deprecated

Set version to 5.25.alpha1

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.