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

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

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

This is a partial of https://github.com/civicrm/civicrm-core/pull/13330 - trying
to separate out the main code move from the changes. My only change was to change the name to
addPayment since that is what we are doing but here are some additionaly thoughts.

1) I'm inclined to put the class onto a new pseudo-bao CRM_Finanial_BAO_Payment. We already have this
pseudo-entity on the api level & it gives us a way to build up a place for the functions we are migrating
towards.

2) I think we should consider changing the input parameters to this function to be
- contributionID, total_amount & line_items or in some way limiting the parameters tha
t get passed through. We have historically gotten ourselves in trouble by
having people hacking in another parameter to go through whereas the strength of the api
is clear inputs & outputs. I note the api will need to support
'is_email_receipt' as will this fn & the CRM_Contribute_Form_AdditionalPayment::emailReceipt
should be moved off the form layer - possibly to our new BAO

3) I think the exceptions need to be CRM_Core_Exceptions if thrown in the BAO although this is kinda cosmetic

Change-Id: Ib5caf0d878e9c5cc735cb1a20c640f61da2dbfc6

Merge pull request #13371 from eileenmcnaughton/activity_extract

[REF] simplify CRM_Activity_BAO_Activity function by using early returns

[REF] simplify CRM_Activity_BAO_Activity function by using early returns.

This function returns a bool. In the process it sets the  variable, however, once set to false in two places
it can never be set to true, so I am simply replacing a big if block with an early return to simplify the code.

This is a further block that can go but it is a slightly different patter & would remove 2 levels of 'if'
so it feels like an easier review if I leave that out of scope

Merge pull request #13367 from mattwire/nfc_commentscustomfield

NFC tidy up comments, remove unused variables in customfield classes

Merge pull request #13368 from mattwire/case_cachecount

Cache case count when getting list of cases so query is executed once instead of three times

Merge pull request #13359 from chamilwijesooriya/issue-625

(dev/core#625) DB error on Case Summary report

Merge pull request #13328 from eileenmcnaughton/phone_handle

[REF] genericise function to add pseudoconstant data to fields to process

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

Revert to showing overdue activities on a case first

Cache case count when getting list of cases so query is executed once instead of three times

NFC tidy up comments, remove unused variables in customfield classes

Merge pull request #13366 from seamuslee001/refinement_623

dev/core#623 Refine the upgrade statement to only those of the old st…

dev/core#623 Refine the upgrade statement to only those of the old standard value

GitLab issue 625 DB error on Case Summary report

Merge pull request #13299 from colemanw/activityDate

Activity date fields -> datepicker

Merge pull request #13336 from mattwire/missingcaseaddaction

Don't crash with missing class if action is not defined when opening new case

Merge pull request #13363 from civicrm/5.9

5.9

Merge pull request #13361 from seamuslee001/lab_core_594

(dev/core#594) Fix key-value mappings for WYSIWYG setting (editor_id). Accept keyColumn option.

Merge pull request #13362 from seamuslee001/lab_core_623

dev/core#623 add in upgrade step to update Away type hold threshold t…

FiveNine.php - Copy-edit post-upgrade message

dev/core#594 Fix issue where editor_id is expected to be keyed on name not on value

Add in a post upgrade message encourging people to check their editor setting

dev/core#623 add in upgrade step to update Away type hold threshold to be 30 as agreed to in CRM-6405 version 3.2. It appears there may have been no upgrade made at the type so catching up now

Merge pull request #13215 from mattwire/recur_invoice_selfservice_links

Display self-service links in email receipts based on payment processor capabilities

dev/core#561 - case_start_date field -> datepicker

activity_duration field -> number

dev/core#561 - activity_date_time field -> datepicker

To revert to showing overdue activities first

Based on this issue, I'd like to remove the data-order, as the BAO Case file already sorts.

https://lab.civicrm.org/dev/core/issues/626

Don't crash with missing class if action is not defined when opening new case

Display self-service links in email receipts based on payment processor capabilities

Merge pull request #13350 from francescbassas/patch-16

#620 - Unable to show custom fields on Repeat Contributions Report

(dev/core#491) campaign cleanup for 'pledged but not paid' report and show disabled campaigns as well

Merge pull request #13354 from seamuslee001/lab_core_35

dev/mail#35 Ensure that Public View link is visible in Scheduled and …

Merge pull request #13356 from yashodha/campaign_reports

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

Merge pull request #13355 from seamuslee001/lab_core_621

dev/core#621 Ensure that only 2 decimal places are dispalyed for opti…

Merge pull request #13316 from alifrumin/amountlink

On the Contribution Detail Report link the Amount field to the contribution

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

dev/core#621 Ensure that only 2 decimal places are dispalyed for option amount and also ensure that non deductable amount is cleaned before saving to db

Merge pull request #13351 from yashodha/campaign_reports

(dev/core#491) campaign cleanup for soft credit report and show disab…

Merge pull request #13352 from yashodha/fix-verbiage

fix verbiage for Edit Contact Information title

dev/mail#35 Ensure that Public View link is visible in Scheduled and Sent Screens

Merge pull request #13339 from yashodha/dev-610

(dev/core#610) Page title displays twice on Activity form

Merge pull request #13347 from mfb/ellipsify-optimize

optimize CRM_Utils_String::ellipsify() method

fix verbiage for Edit Contact Information title

Merge pull request #13349 from yashodha/fix-verbiage

fix verbiage for Dashboard title

(dev/core#491) campaign cleanup for soft credit report and show disabled campaigns as well

Merge pull request #13348 from yashodha/campaign_reports

(dev/core#491) campaign cleanup for contribution (org) report and sho…

Update Repeat.php

#620 - Unable to show custom fields on Repeat Contributions Report

https://lab.civicrm.org/dev/core/issues/620

fix verbiage for Dashboard title

(dev/core#491) campaign cleanup for contribution (org) report and show disabled campaigns as well

optimize CRM_Utils_String::ellipsify() method

Merge pull request #13341 from yashodha/campaign_reports

add suport for order by clause for campaign and clean up

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

Improved support for case variability in UF* API calls.

Merge pull request #13291 from eileenmcnaughton/process_default_active

Add default of TRUE for is_active on PaymentProcessor.create api.

Merge pull request #13329 from eileenmcnaughton/activity_extract

[REF] Add caching to function to determine if activity type is permitted.

Merge pull request #13250 from francescbassas/patch-16

#469 - Error on action "Email - schedule/send via CiviMail" with mult…

add suport for order by clause for campaign and clean up

Merge pull request #13241 from seamuslee001/561_schedule_reminders

dev/core#561 Convert Scheduled Reminders Form to use datepicker and e…

Merge pull request #13281 from yashodha/campaign

campaign cleanup for household summary

(dev/core#610) Page title displays twice on Activity form

Merge pull request #13311 from mattwire/entityform_customsubtype

Support customdata subtype on entityform (for activity/relationship types)

Merge pull request #13337 from GinkgoFJG/crmPageTitle

NFC: Improved documentation of crmPageTitle directive.

Improved documentation of crmPageTitle directive.

Documented assumption that AngularJS base page be named "CiviCRM."