Merge pull request #16749 from civicrm/5.24

5.24

Merge pull request #16750 from eileenmcnaughton/sgroups

Fix smart group custom field  check to cope  with api error

Merge pull request #16741 from mattwire/addrecurid2

Add recurring contribution ID to doCancelRecurring

Merge pull request #16678 from seamuslee001/php_7_1_bump

Bump Minimum Install version to be 7.1 (Do not merge until 5.24 RC is cut)

Merge pull request #16651 from kartik1000/core-1613

dev/core#1613/Updated Misleading labels

Rebased

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

Add recur ID to doCancelRecurring

Fix smart group custom field  check to cope  with api error

As we have added new fields to the saved_search table this will lead to a fatal if it runs before
the database update. This catches that error and handles it

Merge pull request #16748 from seamuslee001/5_23_1_release_notes

5 23 1 release notes

Update 5.23.1.md

Add release-notes/5.23.1.md

Merge pull request #16744 from civicrm/5.24

5.24

Merge pull request #16740 from mattwire/addrecurid

REF Switch to getter on cancelSubscription form

Merge pull request #16743 from mlutfy/dev547c

dev/core#547 Fix Event financial_type_id getOptions

Merge pull request #16735 from totten/5.24-paths

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

dev/core#547 Fix Event financial_type_id getOptions

Merge pull request #16734 from eileenmcnaughton/array2

[REF] Minor code simplification

Switch to getter function for subscriptionDetails

E2E_Extern_CliRunnerTest - Update to match change in `./`

Per 16735, the interpretation of `getPath('[foo]/.`)` changed
slightly - from:

* 5.22: Inconsistent/undefined tail end (may or may not have trailing `/`)
* 5.23.0: Defined to always end with `/`
* #16735: Defined to never end with `/`

Merge pull request #16501 from eileenmcnaughton/cancel_recur

Update cancelSubscription  form to use updated methodology

Merge pull request #16732 from demeritcowboy/conformance

[REF] Hopefully fix regularly failing conformance test

Merge pull request #16718 from colemanw/respond

Improve conditional in api3 ChainSubscriber

[REF] Minor code simplification

Move the return earlier, simplify the IF

Merge pull request #16737 from artfulrobot/artfulrobot-array-values-followup

CRM_Utils_Array::values follow up from #16699

Fix array value accessing non existent properties on an ArrayAccess object

add failing test

Merge pull request #16699 from eileenmcnaughton/array

Support PropertyBag in CRM_Utils_Array

CRM_Core_I18n - Fix usage of `getPath('[foo]/.')`

The interpretation of `/.` is evolving per #16735:

* When this code was first written, it was unspecified/variable whether the value `[foo]/.` would end in `/`
* During most of the testing of 5.23.beta1, this was defined to always return `/`
* During a regression fix in 5.23.1, we're flipping it back the other way so that `[foo]/.` never ends in `/`.

Support PropertyBag in CRM_Utils_Array

We now use the PropertyBag in payment processors - but as @mattwire discovered the CRM_Utils_Array::value
function is commonly used to access values now potentially 'in the bag' and the 'is_array'
filtering here means it is not being returned

Merge pull request #16729 from colemanw/arrayValue4

[REF] Use empty() instead of CRM_Utils_Array::value() in inline conditionals

Merge pull request #16728 from civicrm/5.24

5.24

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

For a full write-up, see PR

dev/core#1637 - PathsTest - Redefine "[foo]/", "[foo]/0", "[foo]/."

The rationale will be discussed more via PR description.

(NFC) PathsTest - Make it easier to identify/debug test cases

When browsing the list of outputs from this test class, each of the test
cases was identified by its numerical position in the list of `$exs`.  This
makes it hard to keep track of the failures.

1. Add a symbolic name to each case (that's easier to search on)
2. Add more verbose output for failed assertions

(NFC) dev/core#1637 - Add E2E test for generation of Civi-WP backend URLs without trailing /

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

This reverts commit cbcbfd69d34c9895342e3d813a42f97d93dbb76d.

hopefully fix failing test

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

Use empty() instead of CRM_Utils_Array::value() in inline conditionals

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

Improve conditional in api3 ChainSubscriber

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

Bump Minimum Install version to be 7.1

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