Merge pull request #16777 from eileenmcnaughton/ftype_form

[REF]  cleanup financial type form - remove complex inheritence

[REF]  cleanup financial type form - remove complex inheritence

Inheriting from 2 intermediary forms adds complexity  with little benefit. Since  we have the entityFormTrait
this increases use of that

Scheduled jobs belonging to extensions cannot be disabled (#16430)

Authored-by: Rich Lott / Artful Robot <forums@artfulrobot.uk>

Merge pull request #16695 from mattwire/pcpshortcodes

Add shortcodes for Personal Campaign Pages

Merge pull request #16767 from civicrm/5.24

5.24

Merge pull request #16766 from seamuslee001/5_23_2_release_notes

release-notes/5.23.2.md

release-notes/5.23.2.md

Merge pull request #16761 from eileenmcnaughton/joomla

dev/financial#120 - Fix CiviCRM base url on Joomla frontend

Merge pull request #16759 from eileenmcnaughton/fatal2

Remove instances of fatal

Merge pull request #16764 from eileenmcnaughton/mem_tax

[NFC] cleanup in test class

[NFC] cleanup in test class

Merge pull request #16710 from colemanw/arrayValue2

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

Alternate Joomla! fix - move to CMS override

Merge pull request #16758 from pradpnayak/priceSetSchema

add pseudoconstant for location type

Remove instances of fatal

add pseudoconstant for location type

Merge pull request #16730 from demeritcowboy/unused-activitytypename

[REF] dev/core#1116 - Remove unused misnamed activityTypeName variable

Merge pull request #16754 from pradpnayak/priceSetSchema

Added pseudoconstant for priceset

iAdded pseudoconstant for priceset

Added pseudoconstant for priceset

Merge pull request #16694 from eileenmcnaughton/dedupebug

Fix bug where a % in a serialized array can lead to the data being broken

Fix bug where a % in a serialized array can lead to the data being broken

It turns out that is a field in a serialized array has a %2 (for example) this gets swapped in executeQuery for the
%2 value (in this case srcID - rendering the serialized array invalid. This proposes that we
explicitly handle arrays as  a data type in compose query.

Some thoughts
1) we could make serialized arrays valid types in validate (not done here)
2) we could iterate through the  array keys  & values escaping them -
at this stage it's left in the calling function
3) there are whole bikeshed factories to keep in business on discussion of whether
'Array-1', 'Array-2' etc are the right format

Use ?? operator instead of CRM_Utils_Array::value() in array assignments with null-ish defaults

Use ?? operator instead of CRM_Utils_Array::value() in array assignments

Use ?? operator instead of CRM_Utils_Array::value() when fetching ids

Merge pull request #16691 from mattwire/eventcart_cleanup

Event Cart cleanup

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

remove unused misnamed variable

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