Merge pull request #16797 from civicrm/5.24

5.24

Merge pull request #16796 from seamuslee001/notes_5_23_3

release-notes/5.23.3.md

release-notes/5.23.3.md

Merge pull request #16792 from colemanw/dashboardContact

[REF] Cleanup api3 DashboardContact

Merge pull request #16788 from civicrm/msg_template_no_id

[REF] Fixes a bug in Message Template create API where by user permissions checks were being done on system workflow messages

Merge pull request #16794 from civicrm/5.24

5.24

Merge pull request #16791 from totten/5.24-dot

(REF) dev/core#1637, dev/core#1651 - Restore format of packagesBase

Cleanup api3 DashboardContact

(NFC) Document format of packagesBase, resourceBase.

Try to prevent future bounciness in changing these variables.

dev/core#1637 - Restore format of packagesBase

These use-cases had been tested during PR dev for 5.23.alpha, but they
regressed in 5.23.1.  In 5.23.1's #16735, note item (5) and the flip-flop on
`/.` Item (5) references some greps to find references `/.` For obscure
reasons, the  file `l10n.js.tpl` didn't match the greps.

Revert "inline editing not working and other js packages issues"

This reverts commit c68b3313741a367e2510ea6be54895405e4438ca.

[REF] Fix permission check for system workflow message created

[NFC] Add in unit test for creating a new system work flow message template with only edit system workflow message templates permission

Merge pull request #16746 from eileenmcnaughton/fatal

[REF] Remove calls to fatal()

Merge pull request #16778 from colemanw/arrayValue6

[REF] Cleanup uses of CRM_Utils_Array::value related to numbers

Merge pull request #16786 from civicrm/5.24

5.24

Merge pull request #16724 from kartik1000/core-1599

dev/core#1599/Deceased Contact's Membership Marked to Deceased via Inline

Merge pull request #16763 from eileenmcnaughton/mem_dash

dev/user-interface#14 Fix regression whereby links from membership dashboard don't work for export

Merge pull request #16783 from civicrm/5.24

5.24 to master

Merge pull request #16775 from demeritcowboy/fin-fatals

dev/core#1645 - (Test for) Fatal error when assigning account to financial type

Merge pull request #16782 from civicrm/5.24

5.24

Merge pull request #16776 from eileenmcnaughton/fform

dev/core#1645 fix  regression  by removing form inheritance

Merge pull request #16779 from demeritcowboy/inline-edit

dev/core#1651 and dev/core#1637 - Inline editing not working on many admin screens and other js packages issues

inline editing not working and other js packages issues

Merge pull request #16777 from eileenmcnaughton/ftype_form

[REF]  cleanup financial type form - remove complex inheritence

Cleanup uses of CRM_Utils_Array::value related to numbers

Some manual cleanup around the areas of code where CRM_Utils_Array::value is used for a number defaulting to 0

dev/core#1645 fix  regression  by removing form inheritence

The Assign  Account form is failing to load because it inherits indirectly from CRM_Admin_Form which
now has a different visibility on the _id property.

I  took a look and there really is no reason for this complex  inheritence - this is a standalone form
and the setDefaults of  the parent seems of no use, let alone the parent's parent.

Assign, edit & browse seem to still work fine

fatal error when assigning account to financial type

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

Deceased's Contact Membership's Status updated to Deceased via Inline

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

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

dev/user-interface#14 Fix regression whereby links from membership dashboard don't work for export

Note it  also applies to membership_status_id but the lack of a uniquename makes the fix trickier  so I'm reluctant to target the
rc

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

[REF] Remove calls to fatal()

Also other minor cleanup on
- comment blocks
- using strict comparison
- don't use elseif where the prior if threw an exceptionn

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