Civi\API\ExternalBatch - Improve test for variables_order/$_ENV

The recent revision #9595 updated `ExternalBatch` to produce a more
informative error message.  However, in the test environment used by
`flexmailer`, there was exactly 1 value in `$_ENV` (even if
`variables_order` was misconfigured).

This should make the test a bit harder to trip-up by affirmatively checking
for the most common environment variable.

Review note: In the known universe, the only direct references to
`ExternalBatch` are in `CiviUnitTestCase`, so this shouldn't impact any
runtime logic.

Merge pull request #9836 from jmcclelland/CRM-20020

CRM-20020 - fix search with parent groups + other criteria

Merge pull request #9936 from eileenmcnaughton/sig

CRM-20220 fix signature fields to be longtext

Update PaymentProcessor.php

Merge pull request #9963 from kainukempowerment/CRM-20239

CRM-20239 Extra tests for the Individual::format function

CRM-20239 Extra tests for the CRM_Contact_BAO_Individual::format function

Merge pull request #9815 from WeMoveEU/CRM-19962

CRM-19962 - Wider name field for A/B mailings

Merge pull request #9662 from JMAConsulting/CRM-19792

CRM-19792, fixed payment processor params to include email

Merge pull request #9948 from KarinG/CRM-19966

Fixes for CRM-19966 Unfair Double Taxation and net_amounts.

Added Test to Create Contribution -> then Edit it via BackOffice form - using default price_set.

CRM-20220 fix signature fields to be longtext

Merge pull request #9961 from colemanw/Tweak

Tweak icon when expanding popup to fullscreen

Tweak icon when expanding popup to fullscreen

Merge pull request #9860 from JMAConsulting/CRM-20058

CRM-20058, added submit credit card payment link

Merge pull request #9952 from JMAConsulting/CRM-20025

CRM-20025 [ready for review] Prevented query from fetching contacts without contributions

Merge pull request #9958 from seamuslee001/CRM-20023

CRM-20023 Add test showing passing in just state_province_id and coun…

Use assertApproxEquals as per Tim's suggestion

Fix tear down function

CRM-20023 Add test showing passing in just state_province_id and country only geoloates based on country not state as well

Merge pull request #9959 from seamuslee001/logged_in_user_api_call

(NFC) Update API examples and add example of using user_contact_id as id fo…

Update API examples and add example of using user_contact_id as id for current logged in user

Merge pull request #9839 from fuzionnz/CRM-20023-geocode_state_province_id

CRM-20023: Do not require state_province value to use state_province_id value.

Merge pull request #9957 from civicrm/4.7.17-rc

4.7.17 rc

Merge pull request #9956 from seamuslee001/master

Update to 4.7.18

Merge pull request #3 from eileenmcnaughton/karing

Resolve immediate issues with test & add checks for mail content (sin…

Merge pull request #9655 from eileenmcnaughton/activities

CRM-19836 Enable activity create from replies to CiviMail.

CRM-20025 Added check to restrict test contributions from appearing

----------------------------------------
* CRM-20025: DB Error on email task for advanced search for contributions
  https://issues.civicrm.org/jira/browse/CRM-20025

CRM-20058 Added check for backoffice credit card payment enabled

----------------------------------------
* CRM-20058: Add Credit Card Payment link for partially paid contribution
  https://issues.civicrm.org/jira/browse/CRM-20058

Update to 4.7.18

Resolve immediate issues with test & add checks for mail content (since that function is edited

Merge pull request #9955 from eileenmcnaughton/joe

Towards CRM-19715, Remove Close Accounting Period code

First pass at PHP Unit test for reSubmit (without priceset).

CRM-19836 Enable activity create from replies to CiviMail.

This relies on a parameter being added to the api call:

civicrm_api3('Job', 'fetch_bounces', array('is_create_activities' => TRUE));

PHP Unit test fix: net_amount must equal total_amount minus fee_amount.

Merge pull request #9954 from colemanw/CRM-20235

CRM-20235 - Suppress annoying extension download error messages

Towards CRM-19715, Remove Close Accounting Period code

Remove the Close Accounting Period form. This is a part of #9498 which is stand alone & reduces the of that PR.

There are some things in the PR that need more work, but this part can be done in isolation

Merge pull request #9769 from scardinius/crm-19958

CRM-19958 Fix invalid name of variable subtype

CRM-20235 - Don't set status messages from the BAO

This function is already returning error codes. Calling functions ought to take responsibility for reporting the errors.

Merge pull request #8807 from JMAConsulting/CRM-16189-4

[ready-for-review]CRM-16189 (IIDA-16) - Changes to Deferred Revenue report

CRM-20020 - ensure search works when both smart and regular groups are used.

Merge pull request #9947 from davidjosephhayes/master

CRM-20232 Fix Relationship (Type) filter on Relationship report template. Any r…

Merge pull request #9946 from lcdservices/CRM-18747

CRM-18747

Remove php unit test for function that no longer exists.

Merge pull request #9772 from jitendrapurohit/CRM-4287fix

CRM:4287 - add searchPrimaryDetailsOnly setting

Merge pull request #9912 from fuzionnz/CRM-20200-editorconfig

[WIP] CRM-20200. Add .editorconfig

Merge pull request #9951 from JKingsnorth/CRM-19993-4.7.18

CRM-19993 Fixes for ISO compliance with countries and counties

Merge pull request #9953 from agh1/release-notes-4.7.17-lastminute

Last-minute release note fixes

4.7.17 release notes: improve description per @xurizaemon's confusion

4.7.17 release notes: Aidan works for Squiffle

Merge pull request #9670 from johanv/CRM-19869-pdf_table_header_issue

Closes CRM-19869 - table layout problem multiple page PDF.

CRM-19993 Fixes for ISO compliance with countries and counties

Merge pull request #9865 from lcdservices/CRM-20146

CRM-20146 dont hardcode check_number toggle in .tpl

Merge pull request #9907 from eileenmcnaughton/dedupe_code

CRM-20157, CRM-20195 Dedupe code cleanup

Merge pull request #9937 from eileenmcnaughton/kenya

CRM-20062 update Kenyan provinces

Merge pull request #9850 from aydun/CRM-20039

CRM-20039 Add Supplemental Address lines to Advanced Search

Merge pull request #9824 from sunilpawar/CRM-20001

CRM-20001 Tokens in Subject Not replaced

Merge pull request #9072 from jeromelebleu/patch-19386

CRM-19386: Retrieve Activities custom fields in import parser

Merge pull request #9920 from agh1/release-notes-4.7.17

[ready to merge] 4.7.17 release notes

4.7.17 release notes: finished descriptions

CRM-20062 upgrade component of Kenyan provinces

CRM-20062 update Kenyan provinces for new installs

Fixes for CRM-19966 and net_amounts.

Fix Relationship (Type) filter on Relationship report template. Any relationship types that were the same bidirectional (eg Sibling of) were completely fitlered out of the list of possibilities on the select box on the filter.

4.7.17 release notes: updated raw run

4.7.17 release notes: update of remaining contributors

4.7.17 release notes: update of some contributors

NFC: spacing on contributor key

4.7.17 release notes: raw first run

4.7.16 release notes: added to contributor key

Merge pull request #9930 from JMAConsulting/CRM-20193

[ready-for-review]CRM-20193, Draw currency from better table on Batch Transaction form

Merge pull request #9935 from eileenmcnaughton/activity_cust_date

CRM-20219 Update activity fields in profiles to use date picker

Merge pull request #9631 from eileenmcnaughton/contribution_search

CRM-20199 Contribution search performance

LCD-3559 revert original change and handle in tpl file

CRM-20146 dont hardcode check payment instrument

Merge pull request #9943 from totten/master-20224

CRM-20224 - Cleanup internal event names

CRM-20224 - Civi\ActionSchedule\Events - Rename internal events (`actionSchedule.fooBar` vs `civi.actionSchedule.fooBar`)

If you look at the list of events emitted through the EventDispatcher, you'll
find several different formats for the names:

```
   actionSchedule.getMappings
   actionSchedule.prepareMailingQuery
   api.authorize
   api.exception
   api.prepare
   api.resolve
   api.respond
   civi.core.install
   civi.token.eval
   civi.token.list
   civi.token.render
   DAO::post-delete
   DAO::post-insert
   DAO::post-update
   hook_civicrm_caseChange
   hook_civicrm_post::Activity
   hook_civicrm_post::Case
   hook_civicrm_unhandled_exception
```

This patch renames events using the `actionSchedule.fooBar` convention.  Based on
grepping universe, I don't believe any extensions rely on the `DAO::*`
events.

CRM-20224 - Civi\API\Events - Rename internal events (`api.foo` vs `civi.api.foo`)

If you look at the list of events emitted through the EventDispatcher, you'll
find several different formats for the names:

```
   actionSchedule.getMappings
   actionSchedule.prepareMailingQuery
   api.authorize
   api.exception
   api.prepare
   api.resolve
   api.respond
   civi.core.install
   civi.token.eval
   civi.token.list
   civi.token.render
   DAO::post-delete
   DAO::post-insert
   DAO::post-update
   hook_civicrm_caseChange
   hook_civicrm_post::Activity
   hook_civicrm_post::Case
   hook_civicrm_unhandled_exception
```

This patch renames events using the `api.foo` convention.  Based on
grepping universe, I don't believe any extensions rely on the `DAO::*`
events.

CRM-20224 - CRM_Core_DAO - Rename internal events (`DAO::foo-bar` vs `civi.dao.fooBar`)

If you look at the list of events emitted through the EventDispatcher, you'll
find several different formats for the names:

```
   actionSchedule.getMappings
   actionSchedule.prepareMailingQuery
   api.authorize
   api.exception
   api.prepare
   api.resolve
   api.respond
   civi.core.install
   civi.token.eval
   civi.token.list
   civi.token.render
   DAO::post-delete
   DAO::post-insert
   DAO::post-update
   hook_civicrm_caseChange
   hook_civicrm_post::Activity
   hook_civicrm_post::Case
   hook_civicrm_unhandled_exception
```

This patch renames events using the `DAO::foo-bar` convention.  Based on
grepping universe, I don't believe any extensions rely on the `DAO::*`
events.

Merge pull request #9941 from eileenmcnaughton/comment

Add docblock comment to help with CRM-20058

Add docblock comment to help with CRM-20058

Merge pull request #9938 from JKingsnorth/CRM-20206

[NFC] CRM-20206 - Clarify comments on html param

[NFC] Clarify comments

Merge pull request #9931 from ErichBSchulz/commentfixes

comment changes only

Merge pull request #9934 from civicrm/4.7.17-rc

4.7.17 rc

CRM-20219 Update activity fields in profiles to use date picker

Merge pull request #9922 from eileenmcnaughton/cust_date_part

CRM-20213 Change participant_register_date to use datepicker widget

Merge pull request #9933 from spalmstr/new

CRM-20004 Fix regression Event payment receipt sent twice

CRM-20004 Fix regression Event payment receipt sent twice

comment changes only

The magic __call method provides the ability to snoop on all hooks

CRM-20193, moved set of code into if condition for better readability

----------------------------------------
* CRM-20193: Draw currency from better table on Batch Transaction form
  https://issues.civicrm.org/jira/browse/CRM-20193

CRM-20193, optimized sql to use INNER JOIN with contribution table rathe LEFT JOIN

----------------------------------------
* CRM-20193: Draw currency from better table on Batch Transaction form
  https://issues.civicrm.org/jira/browse/CRM-20193

CRM-20193, changed the table alias for Currency field to use civicrm_financial_trxn table and have name field table alias

----------------------------------------
* CRM-20193: Draw currency from better table on Batch Transaction form
  https://issues.civicrm.org/jira/browse/CRM-20193

Merge pull request #9899 from colemanw/CRM-20179

CRM-20179 - jstree upgrade and other js performance improvements

Merge pull request #9917 from JKingsnorth/CRM-20206

CRM-20206 Fix encoding in plain text checksum links

Merge pull request #9923 from eileenmcnaughton/acl

CRM-19773 unit test to verify acls apply

tags and group file is modified for the group description

CRM-19773 unit test to verify acls apply

Merge pull request #9570 from colemanw/CRM-19773-report

CRM-19773 - Check permisions for all entities in reports

CRM-20213 Change participant_register_date to use datepicker widget

Merge pull request #9921 from civicrm/4.7.17-rc

4.7.17 rc