Merge pull request #16768 from colemanw/arrayValue5

[REF] Replace CRM_Utils_Array::value with ?? in variable assignments

Replace CRM_Utils_Array::value with ?? in variable assignments

Merge pull request #16802 from colemanw/coalesce

[REF] Replace coalesce pattern with coalesce operator

Merge pull request #16436 from seamuslee001/phpmimetype_library_upgrade

[REF] Switch to using a non abandoned library for getting a mapping of mimetypes to extensions

Merge pull request #16800 from pradpnayak/notifyEmail

Respect 'Donot notify activity type' setting

Merge pull request #16825 from eileenmcnaughton/del_test

[NFC] Use helper / api to delete contacts in tests

Replace coalesce pattern with coalesce operator

Merge pull request #16824 from eileenmcnaughton/cont_restore

Deprecate calling contactTrashRestore function

Merge pull request #16810 from eileenmcnaughton/xml

(NFC) Minor typo: receipients > recipients

Merge pull request #16813 from eileenmcnaughton/deprecated

Remove deprecated parameter

Merge pull request #16823 from eileenmcnaughton/cont_api

[NFC] Minor code cleanup

Fixed warnings

use activity type id from form object

Respect 'Donot notify activity type' setting

Complete deprecation of trashRestore

Deprecate calling deleteContact in order to undelete

This pattern really is very silly. We have one function doing both delete & an update that is
no different to the update done by a normal create call

[NFC] Use helper / api to delete contacts in tests

CRM_Contact_BAO_Contact::deleteContact has a silly signature - let's use it less & improve it so
it doesn't have the silly idea of supporting restore in the same function as delete, with little shared
code

Merge pull request #16822 from eileenmcnaughton/arraa4

[REF] Simplify obtuse boolean expressions

[NFC] Minor code cleanup

Fix some exception declaration, comparisons, handling for non-present param

Merge pull request #16818 from eileenmcnaughton/del

[NFC] Deprecate passing in silly data

Merge pull request #16817 from eileenmcnaughton/deltest

[NFC] Test cleanup

[REF] Simplify obtuse boolean expressions

Partial reviewer commit of https://github.com/civicrm/civicrm-core/pull/16814

Merge pull request #16790 from eileenmcnaughton/pcp

[REF] Fix PCP getPcpDashboardInfo to be tested & use sensible functions

Merge pull request #16820 from eileenmcnaughton/array2

[REF] Replace CRM_Utils_Array::value with ?? in variable assignments

Merge pull request #16821 from eileenmcnaughton/array3

[REF] cleanups on array operator

Merge pull request #16819 from eileenmcnaughton/array

[REF] Simplify obtuse boolean expressions

[REF] Replace coalesce pattern with coalesce operator

This replaces the innstances in the test folder.

[REF] Replace CRM_Utils_Array::value with ?? in variable assignments

Partial of  #16768 - only contains test lines which I don't think need individual review

[REF] Replace CRM_Utils_Array::value with ?? in variable assignments

This is a partial of https://github.com/civicrm/civicrm-core/pull/16768 & consists of the lines I have reviewed from it

[REF] Simplify obtuse boolean expressions

Partial of https://github.com/civicrm/civicrm-core/pull/16814

I've reviewed this handful  & will merge on pass

Deprecate passing in silly data

Later we can move to requiring an integer

Merge pull request #16816 from eileenmcnaughton/ex1

[REF] Remove titleHeader var

[NFC] Test cleanup

Fix some comment blocks, remove some unused variables

Fix PCP getPcpDashboardInfo to be tested & use sensible functions

Merge pull request #16812 from eileenmcnaughton/deprec_gone

Remove unused deprecated function

Remove titleHeader var

This is never passed in, and, logically, it would break the csv if it was

Merge pull request #16731 from eileenmcnaughton/regress

Skip expensive smarty Processing when nothing to see here

Remove deprecated parameter

This parameter was deprecated a while back - we obviously used caution at the time
but we can remove it now

Merge pull request #16774 from mattwire/settingssmtpfreezeoutboundoption

Fix setting outBound_option readonly in UI

Remove unused deprecated function

copyLocBlock was marked as deprecated & unused around 9 months ago - not much has changed - remove

Merge pull request #16671 from eileenmcnaughton/acl

[REF] simplify references to civicrm_acl

(NFC) Minor typo: receipients > recipients

Merge pull request #16804 from UshaMakoa/patch-9

(NFC) Minor typo: contacts(s) -> contact(s)

Merge pull request #16803 from UshaMakoa/patch-8

(NFC) Query.php / Minor typo: Genrated -> Generated

Merge pull request #16474 from sushantpaste/test-for-pr-13809

Test for PR 13809

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

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

(NFC) Minor typo: contacts(s) -> contact(s)

Update Query.php

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

Fix setting outBound_option readonly in UI

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