CRM-20420 - Add docblocks

CRM-20420 - Display `recordGeneratedLetters` setting in GUI

CRM-20420 - Generate combined or separated activities, per recordGeneratedLetters

When a user uses "Print/Merge Document" to generate a PDF or Word doc,
they may designate multiple contacts to include in the PDF/Doc. By
setting `recordGeneratedLetters`, they can specify whether to:

 * Create one activity for each contact. (This is the traditional behavior,
   but when you retain the fully-generated PDF documents, it wouldn't
   be a good idea to copy the PDF to all those activities.)
 * Create a single activity associated with all the contacts.

CRM-20420 - Define setting "recordGeneratedLetters"

CRM-20420 - PDFLetterCommon::createActivities() - Simplify. Address comment.

The comment asks why the code appears to use of `$contactIds` and
`$form->contactIds` interchangeably.  If you grep for calls to
`createActivities()`, all 3-ish callpaths do set `$contactIds` based on
`$form->contactIds` -- so they are the same.

```
// Look for calls to createActivities()
$ grep -r ::createActivities CRM/ Civi/ api/
CRM/Contact/Form/Task/PDFLetterCommon.php:      self::createActivities($form, $html_message, $form->_contactIds);
CRM/Member/Form/Task/PDFLetterCommon.php:    self::createActivities($form, $html_message, $contactIDs);
CRM/Contribute/Form/Task/PDFLetterCommon.php:    self::createActivities($form, $html_message, $form->_contactIds);

// Look for calls to createActivities via CRM/Member/Form/Task/PDFLetterCommon

$ rgrep -r postProcessMembers\( CRM/ Civi/ api/
CRM/Member/Form/Task/PDFLetter.php:    CRM_Member_Form_Task_PDFLetterCommon::postProcessMembers(
CRM/Member/Form/Task/PDFLetterCommon.php:  public static function postProcessMembers(&$form, $membershipIDs, $skipOnHold, $skipDeceased, $contactIDs) {
```

Merge pull request #10129 from JMAConsulting/CRM-20392

CRM-20392, Code cleanup

Merge pull request #10135 from seamuslee001/CRM-20410

CRM-20410 Reformat CRM_SMS_BAO_Provider to use standard create function style rather than the split

Merge pull request #10145 from eileenmcnaughton/test-fail

Fix test regression by correctly setting timestamp in test method

Fix test regression by correctly setting timestamp in test method

Try replacing static with DAO

Merge pull request #10139 from JMAConsulting/CRM-20391

CRM-20391 Added pan trucation to search

Merge pull request #10115 from jmcclelland/CRM-20388

CRM-20388 - ensure connection errors are properly reported to user.

Merge pull request #9848 from JMAConsulting/CRM-8526

CRM-8526 [ready for core team review] Added support for contribution tokens in emails sent via search results (Find contribution results and Advanced search when results are listed as contributions)

Merge pull request #10142 from JMAConsulting/IIDA-58

CRM-20022 Minor fix to change group by label

CRM-8526 Added contribution tokens to list

----------------------------------------
* CRM-8526: Support [contribution.custom_nn] tokens in templates
  https://issues.civicrm.org/jira/browse/CRM-8526

CRM-8526 Added support for contribution tokens in find contributions search results

CRM-20022 Minor fix to change group by label

----------------------------------------
* CRM-20022: Add Batch Title to Columns, Grouping tabs on Contribution Summary report
  https://issues.civicrm.org/jira/browse/CRM-20022

CRM-20392, Code cleanup

----------------------------------------
* CRM-20392: Credit card payment not recorded when contribution is partially paid
  https://issues.civicrm.org/jira/browse/CRM-20392

CRM-20391 Added card number to search

----------------------------------------
* CRM-20391: Add card number field on search form
  https://issues.civicrm.org/jira/browse/CRM-20391

Merge pull request #10138 from totten/master-hook-fix

CRM-19813 - Only dispatch through EventDispatcher after booting

Reformat hard code list as per suggestion from Tim

Merge pull request #10137 from eileenmcnaughton/iida2

Towards CRM-20392 fix receive_date to use date_picker

Switch to API calls and hard code execption list

Minor code cleanup

Minor fix

CRM-20410 Reformat CRM_SMS_BAO_Provider to use standard create function style rather than the split

Fix posthook

Merge pull request #10136 from eileenmcnaughton/iida

Towards CRM-20392 (IIDA-108), code cleanup, removed cruft code

CRM-19813 - Only dispatch through EventDispatcher after booting

After merging #9949, some screens (like the contact-result list or "View
Contact") reported warnings like:

```
Notice: Undefined index: in CRM_Core_BAO_Country::countryLimit() (line 90 of /home/foo/buildkit/build/dmaster/sites/all/modules/civicrm/CRM/Core/BAO/Country.php).
Notice: Undefined index: in CRM_Core_BAO_Country::provinceLimit() (line 62 of /home/foo/buildkit/build/dmaster/sites/all/modules/civicrm/CRM/Core/BAO/Country.php).
```

Bisecting the git history revealed that it stemmed from switching
`hook_civicrm_entityTypes` to go through EventDispatcher.

https://github.com/civicrm/civicrm-core/pull/9949/commits/fb1d9ad2f5116245ba3488a26f8e0b065f328006#diff-8869a8f3c6318eb0580ce2aa04b713bfL1835

This hook is apparently similar to `hook_civicrm_container` in that both
fires pre-boot.  If we attempt to dispatch it through the container in a
pre-boot environment, something initializes incorrectly.

This change proposes a general rule:
 * If you fire a hook before the container or EventDispatcher is available...
 * Then don't try to use the container or EventDispatcher.

Towards CRM-20392 (IIDA-108), code cleanup, removed cruft code

Towards CRM-20392 fix receive_date to use date_picker

This might not have been necessary in the end - https://github.com/civicrm/civicrm-packages/pull/178
seems like the fix for the breakage, but we should be making this change on
every form we touch in order to get rid of code complexity.

Merge pull request #9949 from totten/master-hook-ev2

CRM-19813 - Unify Symfony Events and hooks

Merge pull request #9796 from seamuslee001/CRM-19961

CRM-19961 Make civicrm_sms_provider able to be domain specific

Merge pull request #10132 from seamuslee001/CRM-20402

CRM-20402 Improve handling of spam bounces that get classified as syntax bounces

CRM-19961 Add in tests to prove logic and fix logic

remove changes to update function

Fix style

Add in Upgrade script for CRM-20402

improve "spam" pattern match for bounces which otherwise get classified as syntax bounces

Add in Domain id handling into SMS provider BAO

Add in domain id filtering to delete function

CRM-19961 Treat NULL domain_id values as ok when retriving SMS providers

Ensure domain id only gets set if its not already set

Treat NULL domain_id as valid and also allow for any domain to delete providers that have domain_id which is NULL

Add in API default for domain_id for sms provider

Use value from database if provided even if null otherwise fall back to current domain as default but use the value submitted first

Fix null test on update

Merge pull request #10080 from agileware/CRM-20360

CRM-20360: Remove superfluous showing of payment options header on pa…

Merge pull request #10114 from JMAConsulting/CRM-20385

CRM-20385 (IIDA-72), fixed deferred revenue account validation

Merge pull request #10128 from JMAConsulting/CRM-20400

CRM-20400, fixed payment receipt

Merge pull request #10131 from colemanw/CRM-20367

CRM-20367 - Allow addIndex $column to be an array

CRM-20367 - Allow addIndex $column to be an array

Merge pull request #10123 from colemanw/CRM-16633

CRM-16633 - Allow user to change case subject

CRM-20400, Updated completed Text, Added upgrade code

----------------------------------------
* CRM-20400: Update the Payment Receipt to use for Contribution pyament
  https://issues.civicrm.org/jira/browse/CRM-20400

CRM-20400, fixed payment receipt

----------------------------------------
* CRM-20400: Update the Payment Receipt to use for Contribution pyament
  https://issues.civicrm.org/jira/browse/CRM-20400

CRM-20385 : IIDA-72 oversensitive deferred revenue account validation

Merge pull request #9773 from monishdeb/CRM-19663

CRM-19663: A scheduled reminder set to use an absolute date does not allow repeats

CRM-16633 - Allow user to change case subject

CRM-16633 - Return data with crmEditable formSuccess event

Fix script loading race condition

Merge pull request #10113 from JMAConsulting/IIDA-116

IIDA-116 Refactored Deferred Revenue report to follow more standardized approach.

IIDA-116 Fixed batch labels

IIDA-116 Added extra columns and made existing columns selectable

Conflicts:
CRM/Report/Form/Contribute/DeferredRevenue.php

Conflicts:
CRM/Report/Form/Contribute/DeferredRevenue.php

IIDA-116 Indentation for better readability

IIDA-116 Refactored deferred revenue report

Conflicts:
CRM/Report/Form/Contribute/DeferredRevenue.php

Conflicts:
CRM/Report/Form/Contribute/DeferredRevenue.php

IIDA-119 Used API rather than query

Conflicts:
CRM/Financial/BAO/FinancialAccount.php

IIDA-116 Added Revenue Recognition date toggle filter

IIDA-116 Added hotlinks

IIDA-116 Fixed jenkins errors

Merge pull request #10122 from seamuslee001/CRM-20308

CRM-20308 CRM-19657 Toward CRM-20308 & support CRM-19657 Extract using domain from and lo…

Toward CRM-20308 & support CRM-19657 Extract using domain from and logged in user from email details into separate function for re-use

Merge pull request #10028 from jitendrapurohit/is_public_group_fix

CRM-20318: Add is_public setting on Custom groups

Merge pull request #10120 from seamuslee001/CRM-19741

CRM-19741 Add back in fix and add in unit tests to lock in fix

Fix additional issues

CRM-19741 Add back in fix and add in unit tests to lock in fix

Merge pull request #10084 from jitendrapurohit/CRM-19243

CRM-19243: Free membership results in two email receipts

Merge pull request #10116 from colemanw/CRM-20390

CRM-20390 - Fix select2 data type mismatch

Merge pull request #10119 from eileenmcnaughton/deprecated

Add deprecation comments, CRM-19987

Merge pull request #10021 from jitendrapurohit/CRM-20246

CRM-20246: maxfilesize on import should respect 0 as meaning 'no limit'

Merge pull request #10093 from jitendrapurohit/CRM-20343

CRM-20343: skip status calculation when form dates are not modified.

Merge pull request #9276 from seamuslee001/CRM-19517

CRM-19517 allow price fields that use disabled financial types to sho…

Add deprecation comments, CRM-19987

Merge pull request #10095 from mattwire/membership_force_auto_renew

CRM-20375: Show message instead of checkbox when renew required. Hide checkbox w…

Merge pull request #10118 from civicrm/4.7.18-rc

4.7.18 rc

Merge pull request #10096 from jmcclelland/CRM-20376

Crm 20376 - Fix field names on PDF/Print vesion of survey report

Merge pull request #9916 from JMAConsulting/CRM-20207

CRM-20207 Added invocation for selectWhereClause hook in activity SQL

Merge pull request #10117 from colemanw/CRM-20179

CRM-20179 - Fix race condition on manage group page

CRM-20179 - Fix race condition on manage group page

CRM-20390 - Fix select2 data type mismatch

CRM-20388 - ensure connection errors are properly reported to user.

CRM-20376 - fix typo in variable name.

Merge pull request #10110 from mattwire/contributor

Add my details

Merge pull request #9880 from JMAConsulting/CRM-20158-4

CRM-20158, added function to update credit card details like card typ…

minor

Add is_public setting on Custom groups

Additional test fixes and adding CRM unit test

refactor activity listing using API instead of SQL

CRM-20207 Added invocation for selectWhereClause hook in activity SQL

----------------------------------------
* CRM-20207: Introduce selectWhereClause hook for activity results on activity tab contact summary page
  https://issues.civicrm.org/jira/browse/CRM-20207

CRM-20158, added comments

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

 CRM-20158, added function to update credit card details like card type and pan truncation for contribution

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

CRM-20158, added test

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

CRM-20158, changed code to use api for update to financial trxn table and to retrieve data from financial trxn table

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

CRM-20158, placed function contribution BAO to update card details

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

CRM-20158, changed function defination and params and updated function calling

----------------------------------------
* CRM-20158: Store card type and last 4 digits of credit card
  https://issues.civicrm.org/jira/browse/CRM-20158

Add my details

Merge pull request #10106 from eileenmcnaughton/index

CRM-20312 Add indices to DAO classes

 CRM-20312 regenerated DAOS with indexes

CRM-20312 Add indices to DAO classes

Merge pull request #10108 from eileenmcnaughton/localizable

CRM-20312 add field localisation data to DAO.  …

Merge pull request #10098 from agh1/4.7.18-release-notes

Release notes 4.7.18

4.7.18 release notes: update contributors and key

CRM-20312 regenerated DAO with localisation

CRM-20312 add field localisation data to DAO.

Towards index reconciliation

Merge pull request #10107 from eileenmcnaughton/index_tidy

Minor spelling & casing  tidy ups

Minor spelling & casing  tidy ups

4.7.18 release notes: finished issues, still need to fix credits

Merge pull request #9616 from ErichBSchulz/feature/drupal_boot_no_exit

Feature/drupal boot no exit

CRM-20376 - ensure questions are labelled Q1, Q2, etc in pdf report.

Also, replace deprecated method.

Merge pull request #10036 from jitendrapurohit/CRM-20329

CRM-20329: Backend Membership with Payment set to Failed results in M…

Merge pull request #10101 from eileenmcnaughton/contribution

CRM-20379 move payment details above less important info

Merge pull request #10100 from seamuslee001/CRM-19961-schema

CRM-19961 Schema changes to make sms provider multisite aware

CRM-20379 move payment details above less important info