Merge pull request #16855 from ray-wright/patch-2

Fix batch contact/activity update with radio options

Merge pull request #16762 from eileenmcnaughton/mem_allan

dev/core#1630 Partial fix on renew membership missing

Merge pull request #16849 from colemanw/ufMatch

UFMatch - deprecate unused functions

Wrap value in single quotes

Wrap radio value in single quotes to account for options with spaces.

UFMatch - deprecate unused functions

Merge pull request #16835 from colemanw/removeOldStuff

Remove old stuff

Merge pull request #16839 from mlutfy/fixCustomDataViewMemo

Fix CustomDataView.tpl plain empty memo field.

Merge pull request #16844 from eileenmcnaughton/coleman

CRM/Contact - Cleanup boolean expressions

CRM/Contact - Cleanup boolean expressions

Merge pull request #16843 from seamuslee001/master

5.24

Merge in 5.24

Merge pull request #16841 from seamuslee001/ckeditor_4_14_upgrade

Update CKEditor 4.14

Update CKEditor 4.14

Merge pull request #16771 from colemanw/pcp

Deprecate unused/unneeded pseudoconstant functions

Fix CustomDataView.tpl plain empty memo field.

Deprecated unused pseudoconstant functions

Annotate deprecated functions

Remove deprecated report functions

Remove deprecated CRM_Utils_Hook::alterMail

Merge pull request #16829 from demeritcowboy/blank-joindate

dev/report#28 - Join date field has no label on constituent detail report

Merge pull request #16831 from civicrm/5.24

5.24

Merge pull request #16830 from alifrumin/rn-5.24

[NFC] release-notes 5.24 first pass

Merge pull request #16828 from colemanw/api4Json

APIv4 - convert Result object to array when running through json_encode

release-notes 5.24 first pass

join date checkbox is blank

APIv4 - convert Result object to array when running through json_encode

Before: entire Result object was getting serialized leading to some weird output
After: only the array gets serialized

Merge pull request #16827 from colemanw/typos

Fix typos in xml/dao

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

Fix typos in xml/dao

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.