Issue #1640 including a test

Merge pull request #16862 from eileenmcnaughton/build

[REF] Remove redundant call to build permissions

Merge pull request #15709 from seamuslee001/report_jcalender_upgrade

dev/core#561 Convert Reports from jcalender to datepicker format

Merge pull request #16846 from jitendrapurohit/core-1664

dev/core#1664 - Fix errors recorded in log message on Manage Case scr…

[REF] Remove redundant call to build permissions

Per the comments I added to the function the output of this permissions clause is overwritten. I stepped through
this in the process of reviewing the datepicker PR & confirmed it

Convert Reports from jcalender to datepicker format

Add in unit test of report upgrade process

Fix time handling as per review from Eileen and fix query for time handling

dev/core#1664 - Fix errors recorded in log message on Manage Case screen and case dashboard page

Merge pull request #16850 from colemanw/bool5

Cleanup reverse boolean expressions

Merge pull request #16852 from colemanw/bool1

CRM/Core - Cleanup boolean expressions

Merge pull request #16859 from civicrm/5.24

5.24 to master

Merge pull request #16858 from eileenmcnaughton/5.24

dev/core#1662 - Saved mappings doesn't work for all use cases

Merge pull request #16851 from colemanw/bool2

CRM/Utils - Cleanup boolean expressions

Merge pull request #16854 from colemanw/bool4

CRM - Cleanup boolean expressions

Merge pull request #16836 from tunbola/modify-form-values

dev/core#1569: Increase Data Size for `form_values` column on civicrm_report_instance table

Merge pull request #16856 from colemanw/writeRecord

Add DAO::writeRecord and DAO::deleteRecord methods

dev/core#1662 - Saved mappings doesn't work for all use cases

Merge pull request #16848 from colemanw/profilePerms

Fix checking permissions in api3 profile get

Merge pull request #16715 from mattwire/cancelsubscriptiongeneratetext

Generate text for cancelSubscription form in payment class

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

Add DAO::writeRecord and DAO::deleteRecord methods

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.

Cleanup reverse boolean expressions

CRM - Cleanup boolean expressions

CRM/Utils - Cleanup boolean expressions

CRM/Core - Cleanup boolean expressions

UFMatch - deprecate unused functions

Fix checking permissions in api3 profile get

The boolean being passed to the $skipPermission param was reversed.

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.

dev/core#1569: Add SQL code for modifying column in upgrader.

dev/core#1569: Update XML schema and re-generate Report Instance DAO

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`