Merge pull request #13417 from civicrm/5.10

5.10

Merge pull request #13414 from eileenmcnaughton/count

Fix php 7.2 countable warning

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…

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

[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

dev/core#527 Add in abiity to set translated text for Submit and Cancel button text

Convert to using entity fields style adding to form

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