Merge pull request #14408 from eileenmcnaughton/payment

Switch recordAdditionalPayment fully over to api

Merge pull request #15148 from totten/master-debower

distmaker - Finish migrating bower.json => composer.json

Merge pull request #15149 from totten/master-d8json

(dev/release#9) distmaker - Report on head for commit civicrm-drupal-8

(dev/release#9) distmaker - Report on head for commit civicrm-drupal-8

Overview
--------

The general goal is to produce tags for `civicrm-drupal-8` during the release
process.

The workflow for producing release tags is generally as follows:

* Run `distmaker` to prepare a release candidate (a collection of files - e.g.
  `civicrm-5.18.0-drupal.tar.gz`, `civicrm-5.18.0-joomla.zip`, `civicrm-5.18.0.json`, etc).
  Note that the JSON file describes the build (i.e. the list of inputs/git-commits and
  outputs/tarballs).
* Use the candidate for `civihydra` testing (i.e. use the tarballs for normal GUI-based installation)
* If they work, then run `releaser` - which takes the release candidate files and:
    * Pushes the tarballs out to CDNs
    * Pushes the git tags to Github

Before
------

* The JSON report describes inputs from the following repos:
  `civicrm-{backdrop,core,drupal,joomla,packages,wordpress}.git`
* To create a complete set of build artifacts, one should have file tree:
    ```
    src/ (civicrm-core.git)
      WordPress/ (civicrm-wordpress.git)
      backdrop/ (civicrm-backdrop.git)
      drupal/ (civicrm-drupal.git)
      joomla/ (civicrm-joomla.git)
      packages/ (civicrm-packages.git)
    ```

After
-----

* The JSON report *additionally* reports on `civicrm-drupal-8.git`
* To create a complete set of build artifacts, one should also have `drupal-8` in the tree:
    ```
    src/ (civicrm-core.git)
      WordPress/ (civicrm-wordpress.git)
      backdrop/ (civicrm-backdrop.git)
      drupal-8/ (civicrm-drupal-8.git)
      drupal/ (civicrm-drupal.git)
      joomla/ (civicrm-joomla.git)
      packages/ (civicrm-packages.git)
    ```

Technical Details
-----------------

* To produce a report with the D8 commits, one needs have a copy of
  `drupal-8/` (`civicrm-drupal-8.git`).
* However, `distmaker` can still be used without `drupal-8`.  (As in the
  past, so this is still: *if you're only building the Joomla zip file, then
  you don't need the code for drupal-8 or WordPress*).
* There will be a separate patch to:
    * Checkout `drupal-8` when CI does an automated build
    * Use the JSON to create tags

distmaker - Finish migrating bower.json => composer.json

This is a follow-up to #15044 - since we don't have/need `bower.json`, it
doesn't make sense for `distmaker` to call `bower install`.

Merge pull request #15136 from eileenmcnaughton/batch_test

Add minimal test for testExportFinancialBatch

Merge pull request #15141 from colemanw/permission

Code cleanup in CRM_Core_Permission::check

Merge pull request #15140 from jaapjansma/issue_event_14

dev/event#14 Fix excessive cache clearing on creating an event

Code cleanup in CRM_Core_Permission::check

Fixed issue #14

Merge pull request #15004 from davejenx/dev-core-860-discount-line-items-fix2

dev/core/issues/860: discount not applied to line item: call buildAmount hook in CRM_Member_Form_Membership::submit().

Add minimal test for testExportFinancialBatch

This adds the minimal test suggested as needed for https://github.com/civicrm/civicrm-core/pull/14621

In the process I needed to add some api defaults

Merge pull request #15134 from mattwire/927_extractparticipant_membership

Extract cancelParticipant and cancelMembership functions in baseIPN

Merge pull request #15113 from eileenmcnaughton/weird_stuff

[Ref] Extract addInheritedMembership

Extract cancelParticipant and cancelMembership functions in baseIPN

Merge pull request #15030 from mattwire/changecasestatus_relationshipapi

Switch to relationship BAO when modifying relationships in change case status so we don't bypass hooks

Merge pull request #15133 from mattwire/activityemail_extract

Extract function to create email activity when sending an email to contact

Switch to relationship API when modifying relationships in change case status so we don't bypass hooks

Extract function to create email activity when sending an email to contact

Merge pull request #15129 from mattwire/refactor_baseipn_1

Initial refactor of BaseIPN

Merge pull request #15123 from mattwire/fixhardcodedheadofhouseholdrelationship

Fix hardcoded ID for Head of Household relationship

Merge pull request #15044 from totten/master-composer-all-the-things

Migrate bower.json into composer.json via composer-downloads-plugin

Merge pull request #15128 from mattwire/minorcleanup

(dev/core#927) (NFC) Add comments to relate to gitlab ticket

Merge pull request #15126 from mattwire/deprecateaddresssequence

Deprecate CRM_Core_Config::addressSequence()

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

Add comments to relate to gitlab ticket

Initial refactor of BaseIPN

Deprecate CRM_Core_Config::defaultContactCountry and defaultContactCountryName

Deprecate CRM_Core_Config::addressSequence()

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

Fix hardcoded ID for Head of Household relationship

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

[Ref] Extract addInheritedMembership

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

composer.lock - Update to include civicrm/composer-downloads-plugin

composer.json - Add all bower assets via composer-downloads-plugin

Before
------

* JS and CSS libraries and downloaded via `bower`.
* `bower` is deprecated.
* `bower` is slow.
* `bower` requires installing NodeJS
* `bower` requries installing Bower

After
-----

* `composer install` downloads the `bower_components` folder
* `composer` has a better cache
* `composer` is fast
* You can setup a dev build with fewer dependencies.

Comments
--------

There are many, many composer plugins which can be referenced when managing
assets. This particular one has an important distinction:

* It does not require root-level package configuration.
* It produces a file-structure that closely matches the current one.
* It works just as well as whether 'civicrm-core' is used as a root-package
  or as an dependency.

Remove bower.json

This will be replaced in an adjacent commit with updates to composer.json

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