Merge pull request #15213 from agh1/5.17.0-releasenotes-final

5.17.0 release notes: late changes and tidy-up

5.17.0 release notes: late changes and tidy-up

Merge pull request #15207 from seamuslee001/5_16_4_5_17_release_notes

[NFC] Bring up 5.16.4 release notes into 5.17

Update 5.16.4.md

release-notes - Add 5.16.4

Merge pull request #15197 from eileenmcnaughton/5.17

[NFC] comment fixes

Merge pull request #15194 from eileenmcnaughton/517_dates

Fix failure to save relative dates on legacy fields

[NFC] comment fixes

Fix failure to save relative dates on legacy fields

Merge pull request #15169 from pradpnayak/relDateFixes

1226, fixed Change log relative date search for creating smart group

1226, fixed Change log relative date search for creating smart group

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

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

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

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

composer.lock - Run: "composer update php"

(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 #15087 from agh1/5.17.0-releasenotes

5.17.0 release notes - initial run

5.17.0 release notes: added boilerplate

5.17.0 release notes: raw from script

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 #15048 from eileenmcnaughton/5.17

Fix unreleased regression on import

Merge pull request #15058 from seamuslee001/dev_core_1186

dev/core#1186 add in unit test to lock in fix from dmeritcowboy in #1…

Merge pull request #15055 from demeritcowboy/file-no-description

dev/core#1186 - ignore exception if no description field

dev/core#1186 add in unit test to lock in fix from dmeritcowboy in #15055

Merge pull request #15047 from eileenmcnaughton/pptx

dev/core#1190 Add pptx to safe file types

catch exception when no description

dev/core#1190 Add pptx to safe file types

I feel like it would be better to store these as an array somewhere - maybe as the first step to
a metadata construct for Option groups...

Fix unreleased regression on import

https://github.com/civicrm/civicrm-core/commit/2adc819d0e59d71ffcf14c6799a379f108b07d60
made the code a little more readable but changed the indexing from 0-based to column based.

See https://pear.php.net/manual/en/package.database.db.db-common.setfetchmode.php

THis can be replicated by trying to update an existing mapping

Merge pull request #15041 from seamuslee001/5.17

Add release-notes/5.16.2.md

Add release-notes/5.16.2.md

Merge pull request #15037 from demeritcowboy/fin-account-limbo

dev/financial#65 - Revert recent changes that cause financial account edit form to be blank

dev/financial#65

Merge pull request #15025 from seamuslee001/5_16_1_release_notes

5.16.1 release notes

Update 5.16.1.md

Add in release notes for 5.16.1

Merge pull request #15021 from seamuslee001/5.17

core#1182 - revert crm.menubar.js changes

core#1182 - revert crm.menubar.js changes

Merge pull request #15007 from MegaphoneJon/core-1175-517

core#1175 - fix custom searches

core#1175 - fix custom searches

Merge pull request #14996 from mfb/revert-14222

Revert #14222 on RC branch

Revert "CRM_Core_Error::debug_var() should have a level arg; CRM_Core_Error methods could use Civi::log() internally; prefix param support for Civi::log()."

Due to regression https://lab.civicrm.org/dev/core/issues/1174

This reverts commit 3ac984a463d71d796096d5c78c4abd2de0d0a27d.

Set version to 5.17.beta1

Merge pull request #14991 from eileenmcnaughton/enotice_regress

Further e-notice regression fix

Merge pull request #14222 from mfb/debug-var

Error log improvements: Provide severity level and use Civi::log()

Merge pull request #14985 from herbdool/issue-1170

Issue 1170

Merge pull request #14983 from greenpeace-cee/fix-contact-reference-merge

dev/core#1109 - Fix merge not updating ContactReference fields

And this was the pre-existing enotice

Further e-notice regression fix

Merge pull request #14978 from eileenmcnaughton/map_field

 Fix (unreleased regression) e-notices on import form

Merge pull request #14987 from civicrm/5.16

5.16 to master

Merge pull request #14986 from eileenmcnaughton/luciano

[REF][Import] add formatted parameter to formatInput

[REF][Import] add formatted parameter to formatInput

Merge pull request #14960 from eileenmcnaughton/event_test

dev/drupal#72 Ensure front end profile title is used in event confirmation emails

Issue 1170: Fix URL in CRM_Utils_System_Backdrop::getLoginURL

dev/core#1109 - Fix merge not updating ContactReference fields

This fixes an issue that causes ContactReference custom fields pointing
to a duplicated contact that's being merged to not be updated to the
main (surviving) contact.

Switch to helper function

Merge pull request #14979 from eileenmcnaughton/map_field_zip

[REF] [Import] extract function that sets field metadata

dev/core#224 Ensure front end profile title is used in confirmation emails

Extract loadMessageParams for events

dev/drupal#72 fix display of profiles on events to use front end titles

Preliminary cleanup - use CRM_Core_Exception, fix comments

Merge pull request #14976 from eileenmcnaughton/import

[REF][Import]   very minor cleanup - 3 lines of code to one

Merge pull request #14956 from eileenmcnaughton/mem_auto

[Test] Add test cover for Member_BAO_Query auto_renew field

[REF] [Import] extract function that sets field metadata

My thinking is that we want to migrate over to a new class (similar to export processor) that does a lot a metadata functions
- perhaps it might be importMetadata. A big part of the reason it's all so crazy appears to be that the functions to
wrangle the metadata are on the first class in the sequence (DataSource) and because they are not shared & do not have a shared
parent they are passed around instead via the form set &  get methods. In fact the metadata & array wrangling functions just
need to be available - hmm sounding like a trait - ok next commit....

[REF][Import]   very minor cleanup - 3 lines of code to one

Fix (unreleased regression) e-notices on import form

When doing this extraction
https://github.com/civicrm/civicrm-core/commit/0e3fc0224be2e0db48691ca96cc10dec4ab801db

The function pass-by-reference calls hid the fact that the  &  needed to be passed in

causing e-notices.

In fact there are still some enotices after this which I'm still investigating but this partially fixes
& is mergeable in itself - the other can be follow up

Merge pull request #14975 from eileenmcnaughton/mapping_field

Add pseudoconstant metadata to mapping_field.location_type_id

Reduce php4 support

Merge pull request #14977 from eileenmcnaughton/map_field

[NFC] code formatting only

[NFC] code formatting only

Merge pull request #14969 from colemanw/abbr

Use metadata for getOptions abbreviation & include currency symbol

Merge pull request #14965 from seamuslee001/ref_use_shared_function

[REF] Update selfService in updateBilling to use shared function

Add pseudoconstant metadata to mapping_field.location_type_id

Merge pull request #14964 from seamuslee001/dev_core_1061

dev/core#1061 Ensure that custom data is not loaded on update subscription form when in self service mode and re-use shared function

dev/core#1061 Ensure that custom data is not loaded on update subscription form when in self service mode and re-use shared function

Update to call function rather than variable as function does early return if variable aready set as per EIleen's comment

[REF] Update selfService in updateBilling to use shared function

Minor spelling fix

Update as per Eileen's comment

Merge pull request #14971 from mlutfy/ts-fixes-20190806

Translation 'ts' usage fixes.

Merge pull request #14974 from civicrm/5.16

5.16.0 to master

5.16.0 release notes - late changes (#14973)

* Add release-notes/5.15.1.md

* Add release-notes/5.15.2.md

* Update 5.15.2.md

* 5.15.2 release notes: added missing bug fix

* 5.16.0 release notes: late changes

* 5.16.0 release notes: contributors for late changes

Translation 'ts' usage fixes.

Merge pull request #14959 from eileenmcnaughton/event_inv

Minor cleanup around invoicing on event code

Use metadata for getOptions abbreviation & include currency symbol

Merge pull request #14957 from eileenmcnaughton/event_nfc

[NFC] minor cleanup

Merge pull request #14970 from demeritcowboy/document-activityTypeName-3

[NFC] dev/core#1116 - seek and document where activityTypeName is name and where it's label

seek and document activityTypeName

Merge pull request #14962 from eileenmcnaughton/import_map

[REF] minor code cleanup on import mapping

Merge pull request #14943 from JKingsnorth/patch-12

Fix removeNullContactTokens compatibility with custom tokens

Merge pull request #14844 from chamilwijesooriya/issue-1135

dev/core#1135 Participants having multiple roles affects maximum event registration count

Add test for custom token compatibility

Merge pull request #14951 from eileenmcnaughton/rel_dis

dev/core#470: Current employer disapears when disabling expired relationships