Merge pull request #15127 from mattwire/deprecatedefaultcountry

Deprecate CRM_Core_Config::defaultContactCountry and defaultContactCountryName

Merge pull request #15125 from eileenmcnaughton/mem_param

Remove unused skipRedirect param

Merge pull request #15116 from eileenmcnaughton/import_website

[Ref] [Import] Use processor to determine website type, location type, test

Merge pull request #15120 from mattwire/apimembershiptype_defaultdomain

Add default domain to membershiptype API

Deprecate CRM_Core_Config::defaultContactCountry and defaultContactCountryName

Merge pull request #14367 from MegaphoneJon/financial-58

financial#58: Don't crash the View Batches page after a failed batch …

Remove unused skipRedirect param

No longer used for anything - it can go

Merge pull request #15124 from mattwire/REF_processMembershipPending

Minor refactor of 'Pending' parameter for processMembership

Merge pull request #15112 from demeritcowboy/whats-my-name

[Test] Fix Last_name in CiviUnitTestCase::activityCreate

Merge pull request #15121 from demeritcowboy/test-bao-relationship

test for PR 15103

Minor refactor of 'Pending' parameter for processMembership

Merge pull request #15103 from mattwire/relationshipaddbao

Refactor CRM_Contact_BAO_Relationship::add to autoload missing params from existing record

test for PR 15103

Add default domain to membershiptype API

Refactor CRM_Contact_BAO_Relationship::add to autoload missing params from existing record

[Ref] [Import] Use processor to determine website type, location type, test

Add test to ensure cleanup does not change js

Merge pull request #15097 from eileenmcnaughton/import_cleanup

[Import] [ref] - minor cleanup to reduce code complexity

fix last name

Use Processor to  determine the phone_type_id & im_provider_id

What we are doing here is actually very simple - despite the code being kinda insane - we are just getting
the  fields from the database for  each column in the mapping.

The fields are all loaded into the processor already & those functions are tested. This switches to  using them
and to passing around less variables...

Follow ons will do the same for other fields (website_type_id, location_type_id but I will add more tests as I do that

Merge pull request #15109 from civicrm/5.17

5.17

Merge pull request #15111 from seamuslee001/5_16_3_release_notes

Add in release notes for 5.16.3 drop

Merge pull request #15105 from greenpeace-cee/fix-id-additionalinfo

dev/financial#66 - Fix missing contribution ID for AdditionalInfo

Merge pull request #15104 from christianwach/lab-1203

Add `pre` and `post` hooks to Domain create and edit operations

Update release-notes.md

Update 5.16.3.md

Add in release notes for 5.16.3 drop

Merge pull request #15106 from alifrumin/5.17.0-rn

5.17.0 release notes: first pass at sorting and annotating

dev/financial#66 - Fix missing contribution ID for AdditionalInfo

This fixes an issues where the contribution ID is not being passed
when the AdditionalInfo form block is retrieved in the contribution
form. This causes existing values of fields in this block to not be
loaded, leading to data loss when the form is saved.

5.17.0 release notes: first pass at sorting and annotating

Add `pre` and `post` hooks to Domain create and edit operations

Merge pull request #15064 from seamuslee001/fee_amount_id_search_fix

CRM-17182 Fix Search on fee_level_id and ensure that participant.fee_amount isn't updated when label changes

Add in unit test of searching when price field value label has changed

Resolve CRM-17182 by filtering on civicrm_line_item.price_field_value_id instead of th fee_amount column

Pass id in the params array rather than in the ids array

Add in unit test demonstrating that fee label changes when the price field value label changes

Merge pull request #15095 from eileenmcnaughton/export_metadata

[Test] [NFC] upgrade remaining export tests

Merge pull request #15098 from civicrm/5.17

5.17

Merge pull request #15089 from totten/5.17-install-msg

(dev/drupal#79) Fail more gracefully when attempting to install on PHP 5.x

Add test cover for js for an im provider mapping

Merge pull request #15061 from eileenmcnaughton/agile

Fix readability & caching on CRM_Contact_BAO_Relationship::isInheritedMembershipInvalidated

Merge pull request #15068 from eileenmcnaughton/mapping_test

[REF] minor code refactor on import - Pass ProcessorObject into loadSavedMapping & use it to load the formName

Pass ProcessorObject into loadSavedMapping & use it to load the formName

After a few more steps we can eliminate most of this function & use the processor but this is jusut a very small
change....

Merge pull request #15072 from eileenmcnaughton/test_ext2

[NFC] Add ImportProcessor class & unit tests + extend existing cover

[Test] [NFC] upgrade remamining export tests

Fully switch to checking csv output with new helper - this allows us to remove the hack from the core fn

Merge pull request #15093 from mattwire/contributionpage_typo

Fix wrong variable in check for profiles on contributionpage

Merge pull request #15002 from demeritcowboy/activity-type-assignment-1116

dev/core#1116 - refactor/rename activityTypeName

composer.lock - Run: "composer update php"

Merge pull request #14989 from eileenmcnaughton/import_sc

 dev/core#1001 Fix count notice warnings in php 7.2 [Import]

Merge pull request #15092 from christianwach/lab-1196

Fix recursion in crmInArray()

Fix recursion in crmInArray()

Merge pull request #15091 from JKingsnorth/rm-unused-maxlocations

Remove unused method CRM_Contact_BAO_Contact_Utils::maxLocations

Remove unused method CRM_Contact_BAO_Contact_Utils::maxLocations

Merge pull request #15069 from demeritcowboy/1-2-tolerance

[Test] Tolerance parameter in attachmenttest needs to be one higher for integers

(dev/drupal#79) Fail more gracefully when attempting to install on PHP 5.x

Before
------

If an admin stracts the code and navigates to `/sites/all/modules/civicrm/install/index.php`, it displays
a syntax error.

After
-----

If an admin stracts the code and navigates to `/sites/all/modules/civicrm/install/index.php`, it displays
the message:

> __PHP Version Requirement__
> CiviCRM requires PHP 7.0+. The web server is running PHP 5.6.38.

Comments
--------

This is similar to https://github.com/civicrm/civicrm-drupal/pull/583

The canonical representation of the minimum PHP version is in
`$civicrm_root/CRM/Upgrade/Form.php`.  However, setting up the classloader
triggers a syntax error, so we need to read this without having access to
the classloader.

The approach herein has a few effects:

* The minimum PHP can be read from a JSON file.
* That JSON file is also used by `composer`, so you'll also get better errors when downloading that way.
* At some unknown point, the minimum will probably bump up again (7.1 or 7.2 or whatever). When that
  happens, the unit-test will ensure we keep `CRM/Upgrade/Form.php` and `composer.json` in sync.

Note: I was little concerned that the `composer.json` file might not be
available when normal installers run, so I checked the published tarballs
for D7, BD, WP, and J - in all cases, the `composer.json` looks to be
included at the expected location.

Merge pull request #15088 from civicrm/5.17

5.17 to master

Merge pull request #15087 from agh1/5.17.0-releasenotes

5.17.0 release notes - initial run

Merge pull request #15085 from seamuslee001/update_examples

[NFC] Update Examples

5.17.0 release notes: added boilerplate

5.17.0 release notes: raw from script

Merge pull request #14916 from eileenmcnaughton/export_mem

Fix metadata on member export

[NFC] Update Examples

Fix wrong variable in check for profiles

Merge pull request #15078 from mlutfy/remove-old-scripts

Remove a few unused scripts from tools/bin/scripts

Merge pull request #15070 from eileenmcnaughton/deprecation

Remove deprecated unused function.

Merge pull request #15083 from civicrm/5.17

5.17

Merge pull request #15082 from totten/5.17-php-min

(dev/drupal#79) CRM_Upgrade_Form - Raise MINIMUM_PHP_VERSION from 5.6 to 7.0

Merge pull request #15051 from agileware/CIVICRM-1290

Remove "Copy Case custom data" code (circa 2013)

CIVICRM-1290 CIVICRM-967 Add unit test: Migrate custom file attachments to new case on reassignment
Add unit test from PR #14213 Migrate custom file attachments to new case on reassignment. https://github.com/civicrm/civicrm-core/pull/14213

CIVICRM-1290 Remove "Copy Case custom data" code

Remove "Copy Case custom data" code. Fixes https://lab.civicrm.org/dev/core/issues/1183

(dev/drupal#79) CRM_Upgrade_Form - Raise MINIMUM_PHP_VERSION from 5.6 to 7.0

This version requirement officially went up to PHP 7.0 circa Civi 5.14.
However, at that time, the upgrade metadata was kept at PHP 5.6 to allow somewhat
softer landing for stragglers. That's no longer possible in 5.16+, so this
just updates the requirement.

Before
------

When upgrading via drush to Civi 5.16+ on PHP 5.6, the Civi class-loader fails to initialize.

```
Parse error: syntax error, unexpected ':', expecting '{' in /Users/totten/bknix/build/dmaster/web/sites/all/modules/civicrm/vendor/league/csv/src/functions.php on line 33
```

(Approximate call-path: `civicrm.drush.inc` => `civicrm.settings.php` =>
`CRM_Core_ClassLoader` => `vendor/autoload.php` => `vendor/league/csv/src/functions.php`)

After
-----

It still fails, but at least the metadata is accurate.

There will be a separate patch in another repo to make `drush` use this
metadata and present a better error.

Merge pull request #15079 from demeritcowboy/enotice-add-multi-activity

dev/core#1192 - E_NOTICE on using Add Activity action on contact search results

Merge pull request #15052 from 19ATF77/manageGroupFixes

Manage Group search message fix

Add ImportProcessor class & unit tests + extend existing cover

This adds the full ImportProcessor class from the WIP pr #15034 - minus the functions that
replace loadSavedMapping) along with tests. It does NOT start to use the new class as yet.

Note that #15068 will need rebasing if this is merged first & vice versa

The goal is to bring it into use along with extending tests but this is 'safe' in that no funcitonal code is changed

Merge pull request #15076 from sushantpaste/priceset-hook

dev/core#870 priceset hook

Merge pull request #15077 from mlutfy/api-docs-urls

[NFC] Update the APIv3 documentation links.

Merge pull request #15075 from JMAConsulting/core-14172

Fixes for taking dynamic form name in grant search

fix enotice

Remove a few unused scripts from tools/bin/scripts

Update the APIv3 documentation links.

Merge pull request #15074 from sushantpaste/report-hook

dev/core#1077 Fixes making report listing actions links hookable

Changed to standardize message

Merge pull request #14673 from eileenmcnaughton/payment_create

Further work on payment.create consolidation - always handle financials from payment.create

dev/core#870 priceset hook

Fixes for taking dynamic form name in grant search

dev/core#1077 Fixes making report listing actions links hookable

Merge pull request #15071 from totten/master-gat

(NFC) GenericAssertionsTrait - Improve docblocks

(NFC) GenericAssertionsTrait - Improve docblocks

This code was previously extracted from CiviUnitTestCase. It had weak docblocks. Make them better.

Changes:

* Add more types and examples.
* Describe some of the subtleties in the different forms of array-comparison.
* Remove dependency on `ts()`.  Unit-test assertions are not actually translated.

Remove deprecated function.

I noticed this has been deprecated for over a year (& is unused). Removing

Fix readability & caching on CRM_Contact_BAO_Relationship::isInheritedMembershipInvalidated

This removes an sql query & some hard wrangling & uses the api - wrapped in a caching helper.

pull out activitytype block

more tolerance

Merge pull request #15050 from eileenmcnaughton/loadmappingtest

[Import] [Test] Initial test on loadSavedMapping function.

Merge pull request #15066 from seamuslee001/attachment_api_stablize

Stabalize Attachment API Test

Merge pull request #15067 from seamuslee001/fix_test_syntax

[NFC] Fix test syntax

Initial test on loadSavedMapping function.

To facilitate this I
1) made it public - in the end it will go anyway
2) removed a 'pass-through' variable from the return array - it is passed in & returned unchanged
3) made it cope without a notice when a class property is not defined - that will go too soon

Merge pull request #15049 from eileenmcnaughton/loadMapping

[NFC] swap instances where double = is used but it should be triple =

[NFC] Fix test syntax

Stabalize Attachment API Test

Merge pull request #15065 from civicrm/5.17

5.17

Merge pull request #15018 from eileenmcnaughton/metadata

[Ref] [Import] move metadata calculations to a trait