Remove redundant function

Merge pull request #13416 from seamuslee001/647_selector_test

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

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 …

(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…

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

Merge pull request #13185 from seamuslee001/lab_core_572

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

(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.

Merge pull request #13345 from GinkgoFJG/generic-settings-form

Added support to generic settings form for sorting settings by weight.

Merge pull request #13392 from PeaceWorksTechnologySolutions/master

dev/mail#32 - Making mailing test email non-case-sensitive

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

Per gitlab

The DAO object since ? 4.7.x? has been freed on _destruct. Using the ->free action has been demonstrated to create some rare bugs - ie. because query sets from the outer loop can be lost. There is no benefit in calling it any more and some harm

https://github.com/civicrm/civicrm-core/pull/13192 was the last one in this series

(dev/core#491) Campaign Cleanup in Activity, Member Contribution Detail and Member Lapse reports

Making mailing test email non-case-sensitive
https://lab.civicrm.org/dev/mail/issues/32

Merge pull request #13280 from aydun/tokenrow_fix

Fix  \Civi\Token\TokenRow::customToken() failure if field is not set

Merge pull request #13284 from aydun/evaluate_fix

Prevent \Civi\Token\TokenCompatSubscriber::evaluate() erroring when no contactId is given.

Merge pull request #13282 from aydun/custom_value_zero

Format token custom fields with value of 0 correctly

Merge pull request #13286 from aydun/tokenprocessor_schema

(NFC) Add 'schema' to \Civi\Token\TokenProcessor()

Merge pull request #13279 from aydun/list_tokens

(NFC) Add listTokens() function to return formatted list of tokens for forms

Merge pull request #13278 from aydun/abstract_token_class_prop

(REF) Make activeTokens a class property of \Civi\Token\AbstractTokenSubscriber

Set version to 5.11.alpha1

Set version to 5.10.beta1

Merge pull request #13389 from civicrm/5.9

5.9

Merge pull request #13388 from totten/5.9-msg-tweak

Five Nine - Revise upgrade notice about Money/Int/Float fields

FineNine - Revise upgrade notice to be less alarming

Based on discussion at https://github.com/civicrm/civicrm-core/pull/13384/files#r244890998

(NFC) FiveNine.php - Whitespace

Merge pull request #13386 from civicrm/5.9

5.9

Merge pull request #13387 from seamuslee001/release_notes_update

Update 5.9 Release notes for final fixes

Merge pull request #13385 from eileenmcnaughton/5.9

dev/core#619 fix regression on Address contact-reference fields not rendering results

Update 5.9 Release notes for final fixes

Merge pull request #13384 from seamuslee001/dev_core_337

dev/core#337 Add in upgrade script to set search by range to be false…

Merge pull request #13189 from JMAConsulting/core-564

dev-core#564: Differentiate case relationships in Contact Summary page

dev/core#619 fix regression on Address contact-reference fields not rendering results.

Fixing results not rendering on onbehalf profiles broke addresses - this
limits the handling added for onbehalf to only apply to onbehalf.

Merge pull request #13314 from colemanw/dev/core#337

dev/core#337 - Fix search range for select/radio custom fields

dev/core#337 Add in upgrade script to set search by range to be false for Fields of money, float, int with widget types radio and select which were how they were effectively implementeted originally

Merge pull request #13381 from civicrm/5.9

5.9.0 release notes (#13244) to master

5.9.0 release notes (#13244)

Merge pull request #13380 from yashodha/campaign_report_clean

(dev/core#491) campaign cleanup for member summary report and show di…

Merge pull request #13338 from eileenmcnaughton/export

Export merge to household - fix DB error relating to fields too long for table.

(dev/core#491) campaign cleanup for member summary report and show disabled campaigns as well.

Merge pull request #13358 from yashodha/campaign_reports

(dev/core#491) campaign cleanup for 'pledged but not paid' report and…

Merge pull request #13379 from yashodha/campaign_report_clean

(dev/core#491) campaign cleanup for member detail report and show dis…

dev-core#564: Differentiate case relationships in Contact Summary page

(dev/core#491) campaign cleanup for member detail report and show disabled campaigns as well.

Merge pull request #13378 from eileenmcnaughton/activity_notice

Fix notice on first loading activity search form

Fix notice on first loading activity search form

Merge pull request #13372 from eileenmcnaughton/activity_extract

Simplify handling for case checking.

Merge pull request #13376 from yashodha/campaign_report_clean

(dev/core#491) campaign cleanup for participant listing report and sh…

Merge pull request #13375 from yashodha/dev_622

(dev/core#622) User Dashboard doesn't show Edit Contact Information link

(dev/core#491) campaign cleanup for participant listing report and show disabled campaigns as well.

(dev/core#622) User Dashboard doesn't show Edit Contact Information link

Merge pull request #13344 from seamuslee001/unit_test_13343

Add in unit test to support changes in UF API in PR 13343

Merge pull request #13343 from GinkgoFJG/case-issues-w-ufgroup-api

Improved support for case variability in UF* API calls.

Export merge to household - fix DB error relating to fields too long for table.

The key fix here is that the fields for merging to household are no longer added to the temporary table.

They are fully loaded in php anyway so there is no performance gain using a temp table.
Instead we iterate them in php as we process each row

remove unneccessary code

Merge pull request #13370 from eileenmcnaughton/payment

[REF] extract internals of Payment.create into function on BAO class.

Simplify handling for case checking.

We already check if the contact has generic case permissions in the component checking section.

We can remove that check from the case check & also early return from there since a NO
at that point can't be overriden