Fix trailing ampersands when empty array passed as query param

Merge pull request #13444 from eileenmcnaughton/payment_smarty

[REF] Replace ->assign with CRM_Core_Smarty::singleton()->assign in preparation for making function shareable

Merge pull request #13450 from civicrm/5.10

5.10 to master

Merge pull request #13448 from eileenmcnaughton/everyone_510

Fix unreleased regression on building GroupContact clause

Merge pull request #13449 from colemanw/caseDateSearch

Fix broken case activity date filter

Fix unreleased regression whereby acls are skipped for groups

Fix broken case activity date filter

Merge pull request #13438 from colemanw/timeIcon

Use icon for timepicker placeholder

Add unit test to cover granting everyone' group permission to access a group

Merge pull request #13447 from eileenmcnaughton/dumb_stuff

[tidy-up] remove a couple of useless bits of code

Merge pull request #13443 from eileenmcnaughton/move_payment

[REF] Create pseudo bao CRM_Financial_BAO_Payment & move create function to  it

Merge pull request #13446 from civicrm/5.10

5.10

Remove verify_mandatory that does nothing due to nothing to verify

Remove unused parameter

Merge pull request #13445 from seamuslee001/dev-656

(dev/core#656) unreleased regression - attendee report generates unde…

(dev/core#656) unreleased regression - attendee report generates undefined property notice

Replace ->assign with CRM_Core_Smarty::singleton()->assign in preparation for making function shareable

Per PR 1330 there is an intention to make this form receipting function a shared function. Part of making it sharable is making it static.

This is a preparatory refactor towards that end, getting rid of the 'this' in assigning variables to smarty

Create pseudo bao CRM_Financial_BAO_Payment & move create function to it.

This is per discussions coming off https://github.com/civicrm/civicrm-core/pull/13330

Note that this ONLY moves the code - that PR does other things as well that still need to be worked through

Use icon for timepicker placeholder

Merge pull request #13412 from mattwire/extract_customData

REF Convert forms to standard customData template

Don't call getEntitySubTypeId if a subType is not defined

Merge pull request #13134 from agileware/CIVICRM-1006

dev/core#500 Fix user-specific Case filtering on dashboard and searches to exclude cases from inactive relationships

Merge pull request #13433 from colemanw/actSub

Fix undefined index warning

Merge pull request #13377 from eileenmcnaughton/activity_extract2

Fix activity.getcount function to filter out unpermitted activities.

Merge pull request #13409 from yashodha/dev-646

(dev/core#646) Event date sorting doesn't work for ical listing

Fix undefined index warning

Merge pull request #13430 from civicrm/5.10

5.10

Merge pull request #13429 from seamuslee001/5.10

Port Test fixes to 5.10

(NFC) Ensure that when loading in the test data it is done with UTF8 encoding

(dev/core#647) Fix up Test Class names and file names to ensure all test classes are run by Jenkins

(dev/core#647) Fix Up Contact Selector Test by changing class name to get test to run and make changes following code changes that affect clauses and calling get rows

CRM_Contact_SelectorTest - Fix escaping

(dev/core#217) CRM_Contact_SelectorTest - Remove stale/unnecessary assertion

The table `civicrm_prevnext_cache` includes three columns (`entity_id1`,
`entity_id2`, `entity_table`).  These columns have some use-cases (e.g.  for
deduping) and some historical value, but the only one needed for
caching search-results is `entity_id1`.

dev/core#217 introduced an interface `CRM_Core_PrevNextCache_Interface` for
caching search-results.  This interface only passes through a list of CID's
(effectively `entity_id1`); because we don't need `entity_id2` and
`entity_table`, it makes no provisions for passing them through, and they're
not saved.  This simplifies the interface and makes it more realistic to
implement alternative prevnext storage mechanisms.

Merge pull request #13211 from eileenmcnaughton/search_grant

dev/core#561 Convert grant search fields to use datepicker

Merge pull request #13395 from eileenmcnaughton/datepicker-update

Add routine for updating smartgroups, currently handling datepicker conversion

Merge pull request #13426 from eileenmcnaughton/report_instance_domain

Add default for domain_id for report_instance.

Merge pull request #13428 from mattwire/nfc_membership_function

REF Remove redundant function

Remove redundant function

CIVICRM-1128: Added Unit Tests to validate the Case counts when case role is active/inactive.

Merge pull request #13416 from seamuslee001/647_selector_test

(dev/core#647) Fix Up Contact Selector Test by changing class name to…

Add default for domain_id for report_instance.

This is consistent with other api like Job, MembershipType, PaymentProcessor

Merge pull request #13424 from seamuslee001/dev-drupal-43

dev/drupal#43 Update composer requirement on PSR Log and civicrm-cxn-…

(dev/core#647) Fix Up Contact Selector Test by changing class name to get test to run and make changes following code changes that affect clauses and calling get rows

CRM_Contact_SelectorTest - Fix escaping

(dev/core#217) CRM_Contact_SelectorTest - Remove stale/unnecessary assertion

The table `civicrm_prevnext_cache` includes three columns (`entity_id1`,
`entity_id2`, `entity_table`).  These columns have some use-cases (e.g.  for
deduping) and some historical value, but the only one needed for
caching search-results is `entity_id1`.

dev/core#217 introduced an interface `CRM_Core_PrevNextCache_Interface` for
caching search-results.  This interface only passes through a list of CID's
(effectively `entity_id1`); because we don't need `entity_id2` and
`entity_table`, it makes no provisions for passing them through, and they're
not saved.  This simplifies the interface and makes it more realistic to
implement alternative prevnext storage mechanisms.

Merge pull request #13419 from tschuettler/37-dev-mail

dev/mail#37 - Replace non-supported 4-byte unicode characters in bounce processing

dev/mail#43 Update composer requirement on PSR Log and civicrm-cxn-rpc to fix compatability issue with psr/log from Drupal8

Merge pull request #13422 from colemanw/dev/core#561

dev/core#561 - Convert campaign form date fields

Merge pull request #13423 from colemanw/dev/core#561-followup

dev/core#561 - Convert activity followup form date field

Merge pull request #13420 from yashodha/dev-649

(dev/core#649) DB error on Find Activities with follow up criteria

dev/core#561 - Convert activity followup form date field

dev/core#561 - Convert campaign form date fields

(dev/core#649) DB error on Find Activities with follow up criteria

Merge pull request #13394 from totten/master-srchcache

(dev/core#635) Reduce unnecessary SQL writes

Replace 4-byte utf8 characters with unicode replacement character

Add unit test for utf8mb4 unicode character

Merge pull request #12923 from vinuvarshith/scheduled-reminders-error-fix

dev/core#434: Check If 'absolute_date' Is Used In 'prepareRepetitionEndFilter' To Prevent SQL Error

Merge pull request #13415 from seamuslee001/nfc_test_file_fixes

(dev/core#647) Fix up Test Class names and file names to ensure all t…

Merge pull request #13417 from civicrm/5.10

5.10

Merge pull request #13414 from eileenmcnaughton/count

Fix php 7.2 countable warning

(dev/core#647) Fix up Test Class names and file names to ensure all test classes are run by Jenkins

Fix php 7.2 countable warning

per https://lab.civicrm.org/dev/core/issues/406#note_12393
Warning: count(): Parameter must be an array or an object that implements Countable in CRM_Event_Form_Participant->submit() (line 1722 of .../all/modules/civicrm/CRM/Event/Form/Participant.php).

Merge pull request #13252 from jmcclelland/issue586

PCP report - fix number of donors and total committed.

Merge pull request #13413 from seamuslee001/nfc_test_utf8

(NFC) Ensure that when loading in the test data it is done with UTF8 …

Convert Membership/Membership Renewal forms to standard customData template

Convert Grant form to standard customData template

Convert Manage Event Info form to standard customData template

Convert UpdateSubscription form to standard customData template

Convert contribution form to standard customData template

Convert activity form to standard customData template

Convert case/case activity forms to standard customData template

Convert Survey/Petition to standard customData template

(NFC) Ensure that when loading in the test data it is done with UTF8 encoding

PCP report - fix number of donors and total committed.

ensure number of donors and committed are properly calculated in the PCP
report.

See: https://lab.civicrm.org/dev/core/issues/586

Merge pull request #13411 from mattwire/fix_buildCustomDataTwice

Membership custom fields sometimes don't display

Fix build custom data twice on edit forms

Merge pull request #13408 from seamuslee001/lab_core_644

(dev/core#644) Add in unit test showing backend able to handle user e…

(dev/core#646) Event date sorting doesn't work for ical listing

Merge pull request #13407 from MegaphoneJon/core-644

core#644 - extract function to return correct mailbox header

(dev/core#644) Add in unit test showing backend able to handle user email address as from email sensibly

Merge pull request #13393 from eileenmcnaughton/free

dev/core#562 remove calls to dao->free() from api folder

Merge pull request #13402 from pradpnayak/ReportDelete

Donot show delete task option on create new report

rename function and vars; move fix to common mail sending function

Merge pull request #13397 from pradpnayak/AddressCustomFieldFix

dev/core/issues/636 - Custom field for Address: The "No" value is not defaulted

core#644 - extract function to return correct mailbox header

Merge pull request #13406 from eileenmcnaughton/api_getsingle

Fix bug where getsingle calls chained actions twice

Merge pull request #13405 from davejenx/core-640-repeat-activity-permission

Don't require CiviEvent permission to create repeating activity

Merge pull request #13318 from eileenmcnaughton/api_perm

Do not check financial permissions on contribution.create if check_perrmissions is FALSE.

dev/core#643 fix for double delete attempt when chained with getsingle

dev/core#643 Add test for bug with double chaining on getsingle

Do not check financial permissions on contribution.create if check_permissions is FALSE.

Our standard is to only check permissions at the api layer if check_permissions is TRUE
(which is is by default for javascript calls but not in php calls)

Merge pull request #13396 from tschuettler/36-dev-mail

dev/mail#36 - Fix invalid unicode characters in bounce processing

dev/core/issues/640: don't require 'access CiviEvent' permission for civicrm/recurringentity/preview.

Donot show delete task option on create new report

Make class name match the filepath make tests run

dev/core/issues/636 - Check if value is isset rather empty

Merge pull request #13382 from seamuslee001/report_cleanup

(dev/core#491) Campaign Cleanup in Activity, Member Contribution Deta…

Replace invalid characters in bounce reason with unicode replacement character

Add unit test for invalid unicode character

Add routine for updating smartgroups, currently handling datepicker conversion

Merge pull request #13185 from seamuslee001/lab_core_572

dev/core#527 Add in abiity to set translated text for Submit and Canc…

Migrate to a datepicker range

(dev/core#635) Only write extra prevnext cache record if used

This line of code creates an extra record to facilitate clearing the
SQL-based prevnext cache.  However, with Redis-based prevnext cache, it's
not needed because the Redis implementation simply uses a TTL -- and (on
Redis deployments) this lines to gratuitous SQL writes.

(dev/core#635) CRM_Contact_Selector - Move count to 'long' cache

The selector was set to specifically store the size of the result set in the
SQL-based cache.  For epic:ro-db, we should be able to store these in a
non-SQL location.  The `long` cache suffices.