Merge pull request #15130 from eileenmcnaughton/import_rel

[ref] [ import] Extend tests & simpler retrieval to relationship_type_id

Merge pull request #15163 from colemanw/eventClone

[REF] Move clone template functionality from api to bao

Merge pull request #15167 from mattwire/testRenewMembership_note

Add note about testRenewMembership failing on certain days

Merge pull request #15119 from seamuslee001/fix_dashboard_domain_fk

Ensure that civicrm_dashboard has a foreign key to civicrm_domain

Merge pull request #15099 from christianwach/lab-1200

Ensure Dashboard respects multiple Domains

Merge pull request #15155 from eileenmcnaughton/merge_perm

Use merge duplicate contacts as api permission for merging

Merge pull request #15157 from eileenmcnaughton/dedupe6

Adjust permissions on dedupe exception api to 'merge duplicate contacts'

Add note about testRenewMembership failing on certain days

Merge pull request #15164 from mlutfy/ppIpnTranslation

Fix PayPalImpl Contribution Status PseudoConstant for non-English

Fix PayPalImpl Contribution Status PseudoConstant for non-English (use name, not label)

Merge pull request #15161 from vakeesan26/dev/core#1218

dev/core#1218 Fix Word Replacements on manage case Roles

[REF] Move clone template functionality from api to bao

fix for dev/core#1218

Merge pull request #15114 from eileenmcnaughton/weird2

Make sure inherited memberships are not themselves inherited

Use merge duplicate contacts as api permission for merging

Adjust permissions on dedupe exception api to 'merge duplicate contacts'

Merge pull request #15152 from eileenmcnaughton/dedupe

Open up permissions on Dedupe.getduplicates

Open up permissions on Dedupe.getduplicates

Currently the action Dedupe.getduplicates is defaulting to 'administer CiviCRM'.

The function manages permissions internally (ie. you can't retrieve contacts you don't have permission  to see)
so the function itself can have fairly open  permissions - hence I went for  'access CiviCRM' rather than
'merge duplicate contacts' - it might even be argued this should be open & rely solely on contact ACLs but
I have not gone that far

This function is part of moving towards api based rather than form based logic. It is not in use in core yet
outside tests.

Merge pull request #15145 from colemanw/js

Better error handling in CRM_Utils_JS::decode

Merge pull request #15150 from demeritcowboy/teststatusbycasetype

[Test] Bring to life dormant test

Merge pull request #14892 from eileenmcnaughton/lines

 dev/financial#40 add missing financial item when altering a radio amount

Merge pull request #14408 from eileenmcnaughton/payment

Switch recordAdditionalPayment fully over to api

Add check to ensure only 3 rows

dev/financial#40 add missing financial item when altering a radio amount

[test] dev/financial#40: Missing Financial type and Credit Account Code in Bookkeeping Transaction Report

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

resurrect old test

(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`.

Better error handling in CRM_Utils_JS::decode

Make sure inherited memberships are not themselves inherited

In trying to make sense of the code / fix / test for https://github.com/civicrm/civicrm-core/pull/15062
I discovered the tests wouldn't pass due to a weird edge case where an individual inherited a membership
and that membership was inherited in turn via a relationship the individual had (with the same organization)

I don't believe that inheriting inherited memberships is an intentional feature & it DOEs cause issues
- this removes that possibility

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

Extend tests & simpler retrieval to relationship_type_id

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

Ensure that civicrm_dashboard has a foreign key to civicrm_domain

Fix upgrade on multilingual installs

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

Ensure Dashboard respects multiple Domains

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