correct syntax "IS NOT NULL" to get pledge payment count

Merge pull request #13370 from eileenmcnaughton/payment

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

[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

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

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

Merge pull request #13335 from mattwire/opencase_comments

NFC Minor tweaks to docblocks on opencase form

Minor tweaks to docblocks on opencase form

Merge pull request #13321 from yashodha/campaign-clean

campaign cleanup for sybunt report and show disabled campaigns as well

Merge pull request #13323 from JMAConsulting/dedupe_zero

Unit test for PR#12699

Unit test for PR#12699

Merge pull request #12669 from eileenmcnaughton/dedupe_zero

Fix duplicate merge to not disregard zero values.

Merge pull request #13322 from JMAConsulting/dev-report-5

dev/report#5 - Fix mailing report unique count issue

Add caching to function to determine if activity type is permitted.

This adds caching & re-usability to the activity type checking

Merge pull request #13326 from mlutfy/intall-php72

installer: PHP 7.2 compatibility fixes

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

installer: PHP 7.2 compatibility fixes

Merge pull request #13313 from kainuk/issues-607

Search Builder fails with an error when searching for State if the location type differs from the display name.

Added test case

CiviLint

Test case for report

dev/report/#5 - Fix mailing report unique count issue

campaign cleanup for sybunt report and show disabled campaigns as well

Merge pull request #13315 from seamuslee001/5.9

Fix PHP7.2 Count error on test testImportParserWtihEmployeeOfRelation…

Merge pull request #12995 from eileenmcnaughton/activity_extract2

dev/core/454 rationalise activity case permissions & code

Link the Amount field to the contribution in the Contribution Detail Report

Fix PHP7.2 Count error on test testImportParserWtihEmployeeOfRelationship

Merge pull request #13170 from elisseck/dev/core/485

dev/core/485 Website type should be required as other type values are

Merge pull request #13312 from compucorp/fix-modal-cancel

dev/core#606: Don't redirect to the Option Groups page when page is displayed in a modal

Fix for issue 607

Don't redirect to the Option Groups page when page is displayed in a modal

Merge pull request #13309 from seamuslee001/min_php_version_bump

Bump Minimum Required PHP to be 5.6

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

Merge pull request #13296 from mathavanveda/603_case_summary_report

dev/core#603 not set default value yes/no for active relationship filters

Merge pull request #13308 from seamuslee001/5.9

Add release notes for 5.8.2 & 5.8.1

Bump Recommended PHP version to 7.2 and min recommend to be 7.1 as per Eileen

Bump Minimum Required PHP to be 5.6

release-notes/5.8.1.md - Add missing item

release-notes/5.8.1.md - Copy edit

Add release notes for 5.8.1

Add release notes for 5.8.2

Merge pull request #13293 from seamuslee001/nfc_additional_javascript_metadata

(NFC) Just some additional package metadata

Merge pull request #13306 from civicrm/5.9

5.9 to master

Merge pull request #13304 from mattwire/fixtargetcontactoncasecreate

dev/core#585 Fix setting target contact on activities when creating a case with activity sequence.