Merge pull request #15031 from mfb/upgrade-fix

Fix upgrade version for #13487

Fix upgrade version for https://github.com/civicrm/civicrm-core/pull/13487

Merge pull request #15027 from demeritcowboy/is-nullish

[Test] Demonstrate the truthiness of CRM_Utils_System::isNull

Merge pull request #15012 from seamuslee001/dev_core_752

dev/core#752 Add cid parameter when loading custom data via backend f…

Merge pull request #14988 from eileenmcnaughton/sc

Unit test for custom date parsing

Merge pull request #14999 from demeritcowboy/activityTName-1116

dev/core#1116 - just rename local var activityTName

Merge pull request #15026 from civicrm/5.17

5.17 to master

Merge pull request #15025 from seamuslee001/5_16_1_release_notes

5.16.1 release notes

Merge pull request #15016 from eileenmcnaughton/act_speed

[NFC] [Test] dev/core#1098 Add unit test & code comments relating to the slow activity search

Update 5.16.1.md

Add in release notes for 5.16.1

Merge pull request #15024 from civicrm/5.17

5.17

Merge pull request #15021 from seamuslee001/5.17

core#1182 - revert crm.menubar.js changes

demonstrate isNull

core#1182 - revert crm.menubar.js changes

Merge pull request #15005 from colemanw/empty

Swap CRM_Utils_Array::value for empty() in conditionals

Merge pull request #15020 from MegaphoneJon/core-1182-517

core#1182 - revert crm.menubar.js changes

Merge pull request #15019 from MegaphoneJon/core-1182

core#1182 - revert crm.menubar.js changes

core#1182 - revert crm.menubar.js changes

core#1182 - revert crm.menubar.js changes

Swap CRM_Utils_Array::value for empty() or isset() in conditionals

Merge pull request #15013 from eileenmcnaughton/member_import

[Import] Minor cleanup on membership import

dev/core#1098 Add unit test & code comments relating to the slow activity search

The issue is the joining in of component filtering - we should fix this but do it correctly....
We probably need to extend the test to reflect contact ACLs first

[Import] Minor cleanup on membership import

We have a nasty construct (perhaps due to php4 limitations) where the code iterates the _mapperKeys
array & sets a class property specifying the key it occupies. We later use this to check it occupies a key.

This could be much more readable by just checking the array at the right time.

Also I removed an unused private property & a require_once to api.php which is always included

dev/core#752 Add cid paramater when loading custom data via backend forms so that hooks can modifiy based on contact_id being viewed

Add in handling for cid not being set

Fix assign of cid and switch to using javascript if rather than smarty

Merge pull request #15010 from eileenmcnaughton/event_inv

Event confirm - (very) minor cleanup + test

Update test to reflect test setup change

Event confirm - minor cleanup + test

I've just started delving into this & have
1) added a test
2) removed an assign that is  ALSO done in the sendMail function - ie participantID - this is tested
3) removed a pass-by-reference that I checked was not required

Merge pull request #15008 from civicrm/5.17

5.17

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

core#1175 - fix custom searches

core#1175 - fix custom searches

just rename local var activityTName

Merge pull request #14995 from mfb/context-fix

Error log improvements: Provide priority level

Merge pull request #14715 from eileenmcnaughton/cache_sub

Rewrite subTypeInfo to use caching mechanism

Merge pull request #14998 from seamuslee001/manual_geo_code_export

Allow Manual Geo Code flag to be exported

Allow Manual Geo Code flag to be exported

FIx Export tests by altering to include the manual_geo_code field

Update to title case

Merge pull request #14955 from eileenmcnaughton/rel_alok_ref

[REF] Extract chunk of code relating to whether to disabled an inherited relationship

Rewrite subTypeInfo to use caching mechanism

This seems to be getting a lot of cache misses on prod - presumably due to a lack of sub types
so fixing to use 'modern caching';

CRM_Core_Error::debug_var() should have a priority arg.

Merge pull request #14980 from eileenmcnaughton/mem_renew

[clean up][ export] Remove membership_auto_renew from defaultReturn properties

Merge pull request #14997 from civicrm/5.17

5.17 to master

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.

Merge pull request #14990 from eileenmcnaughton/free

Remove ?last? instances of ->free() dev/core#562

Merge pull request #14927 from seamuslee001/remove_template_field_mosaico

Remove template field if using Mosaico in non shoreditch mode

Set version to 5.18.alpha1

Set version to 5.17.beta1

Merge pull request #14991 from eileenmcnaughton/enotice_regress

Further e-notice regression fix

Remove template field if using Mosaico in non shoreditch mode

Use double equals instead of single equals

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

Unit test for custom date parsing

Unit test for https://github.com/civicrm/civicrm-core/pull/14986
along with some code comments in the test about issues I hit doing what I thought was the right fix
- copied here for visibility

    // @todo I feel like we should work towards this actually parsing $params here -
    // & dropping formatting but
    // per https://github.com/civicrm/civicrm-core/pull/14986 for now $formatted is parsing
    // The issue I hit was that when I tried to extend to checking they were correctly imported
    // I was not actually sure what correct behaviour was for what dates were accepted since
    // it seems to ignore the latter in favour of the former - which seems wrong.

Remove ?last? instances of ->free() dev/core#562

These are almost the last places where ->free is to be found. The remaining ones are in the dao
and perhaps out of scope / need more careful investigation

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

Remove membership_auto_renew from defaultReturn properties

This field is in the defaultReturnProperties for the BAO_Query object but it's a pseudofield and as demonstrated in this PR
https://github.com/civicrm/civicrm-core/pull/14956

the ability of the [search selector](https://github.com/civicrm/civicrm-core/pull/14956/files#diff-ce8713d2ed34e1061d2b3304ec3ff6bbR69)
to determine this value is unrelated to it's presence in the Query object (which always returns an empty value as we can
see in the [export test](https://github.com/civicrm/civicrm-core/pull/14956/files#diff-4686601201fee8a8402a894c1b7df226R291)
despite contribution_recur_id being calculated (as we see a few lines earlier)

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

[REF] Extract chunk of code relating to whether to disabled an inherited relationship

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