Merge pull request #15338 from totten/master-poc-postcommit

(POC) Add `hook_civicrm_postCommit`, a less foot-gunny variant of `hook_civicrm_post`

Merge pull request #16798 from systopia/CORE-1646

dev/core#1646 Merge Case Modal not showing relevant cases

Merge pull request #16799 from pradpnayak/priceSetSchema

Added pseudoconstant for state province

Merge pull request #16801 from civicrm/5.24

5.24

Merge pull request #16752 from eileenmcnaughton/ext24

Fix fatal error on loading extension page when an extension has been deleted

Merge pull request #16496 from mattwire/billingsamecheckboxchange

Make the 'billing address is the same' checkbox respond to the change event

Merge pull request #16773 from UshaMakoa/patch-7

(NFC) Minor string error : for for > for

Merge pull request #13809 from sushantpaste/auto-complete-search

dev/core#787 Auto-complete search results fixes

Merge pull request #16479 from twomice/1579_avoid_needless_getPayPalPaymentProcessorID_call

dev/core#1579 CRM_Core_Payment_PayPalProIPN should not call getPayPalPaymentProcessorID() if processor_id is clearly provided in URL

Added pseudoconstant for state province

Merge pull request #16755 from eileenmcnaughton/et

[NFC] Test cleanup, remove duplicate test

Minor string error : for for > for

Merge pull request #16546 from eileenmcnaughton/part_end2

Fix Bug where Payment Balance is sometimes miscalculated

Merge pull request #16797 from civicrm/5.24

5.24

would this fix CORE-1646?

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

Fix Bug where Payment Balance is sometimes miscalculated

I observed that when this function is called without the mystical  the paymentBalance is miscalculated

The payment balance is calculated as the Contribution Total less the amount paid (for less queries the
total is passed into getContributionBalance & used if passed in). We were passing in the Balance
as the total causing the balance to be calculated as the Balance less any amount paid.

From what I can tell this function has been honed & cleaned up but because the parameter never made much sense
the impact of different variants was not really tested. This removes & fixes

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

[NFC] Test cleanup, remove duplicate test

It turns out the removed test is identical to testEntityTagDeleteINDHH earlier in the class
- probably consolidation of the functions called led to this

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

Fix fatal error on loading extension page when an extension has been deleted

Since we added the concept of 'hidden' tags we are loading them in a way where the exceptions are not caught. This means
that the whole page cannot load - which is needed to disable the missing extension

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