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

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

Merge pull request #14963 from seamuslee001/ref_move_self_serivice

[REF] Move Self service handlng to shared function to allow for use i…

Merge pull request #14961 from eileenmcnaughton/import

[REF] simple extraction of function to check required fields are present

[REF] Move Self service handlng to shared function to allow for use in other contribution recur functions

[REF] minor code cleanup on import mapping

[REF] simple extraction of function to check required fields are present

Minor cleanup around invoicing

Another minor cleanup on invoicing vars

Add test cover for Member_BAO_Query auto_renew field

This adds test coverage to demonstrate that the auto_renew field in the Member_BAO_Query::defaultReturnProperties
achieves nothing.

auto_renew is NOT returned for the export - regardless of it's presence. It IS returned for getRows
but tha is calculated there now in the query object.

Once this is merged I will put up the commit to remove auto_renew from defaultReturnProperties

Merge pull request #14921 from eileenmcnaughton/url_search2

Core/dev#692 support for some additional url params

[NFC] minor cleanup

Remove duplication, unused param

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

Overview
----------------------------------------
Steps to replicate:

1. Create a current relationship (set empty end-date) on individual A as  organization A and mark as current employer
2. Create a past relationship on individual A as  organization B without marking as current employer
3. Run Disable expired relationships cron job

Before
----------------------------------------
Result: Clear the employer field of Contact A

After
----------------------------------------
Result: Does not clear the employer field of Contact A

Technical Details
----------------------------------------
This PR replaces https://github.com/civicrm/civicrm-core/pull/13334

Merge pull request #14954 from eileenmcnaughton/rel_alok

Add serialization metadata for MembershipType api

Add serialization metadata for MembershipType api

Merge pull request #14945 from JKingsnorth/is-public-fixes

Add missing is_public flags to public paths

Merge pull request #14953 from eileenmcnaughton/export_test

[TEST] [REF] [Export] Convert a couple more tests

Add support for email & contact_tags in url

Test conversion

Convert test

Merge pull request #14950 from eileenmcnaughton/rel_up

Move the code to add employer from relationship backoffice form to BAO

Merge pull request #14937 from eileenmcnaughton/export_cust_test

dev/core#439 [Export] [Ref] Convert custom data export test to use newer function.

Merge pull request #14933 from eileenmcnaughton/export_test

[REF] [Test] [Export] Convert some more tests to use the newer function

Merge pull request #14840 from MegaphoneJon/core-1130

core#1130 - allow search views to show contact subtype

Merge pull request #14952 from demeritcowboy/document-activityTypeName

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

Merge pull request #14946 from eileenmcnaughton/ev_test

Add test for failed payment

seek and document activityTypeName

Move the code to add or clear employer from relationship backoffice form to BAO

This is a reduced version of https://github.com/civicrm/civicrm-core/pull/13331 which stalled on the QA over handling of
removed employers - that code is simply excluded. I tested adding & removing & editing relationships with the
current_employer checkbox ticked or unticked & this seems to work fine

Merge pull request #14949 from eileenmcnaughton/rel_test

[NFC] Formatting in BAO_Relationship class

Add test for failed payment

https://github.com/civicrm/civicrm-core/pull/14930

Merge pull request #14947 from seamuslee001/lab_core_1162

dev/core#1162 [TEST] [Mailing] add in unit test checking that no bulk email flag is re…

[NFC] Formatting in BAO_Relationship class

Merge pull request #14948 from civicrm/5.16

5.16 to master

Merge pull request #14944 from colemanw/color

Better support for hookable menubar colors

dev/core#1162 add in unit test checking that no bulk email flag is respected when building recipients

Merge pull request #14252 from jitendrapurohit/core-961

dev/core#961 - Contribution page including 2 email fields does not re…

Merge pull request #14930 from seamuslee001/backoffice_participant_payment_processor_error

Catch Payment Processor Exception if thrown when registering via bac…

Merge pull request #14940 from eileenmcnaughton/mem_import

Fix mishandling of renamed membership status labels on membership import

Add missing is_public flags to public paths

Improve color utils

Better support for hookable menubar colors

NFC Style fixes

Fix removeNullContactTokens compatibility with custom tokens

Merge pull request #14941 from seamuslee001/allow_non_primary_join_reports

[REPORT] Allow extensions to join address, email and phone tables without limi…

Allow extensions to join address, email and phone tables without limiting to primary fields

Merge pull request #14942 from seamuslee001/fix_location_type_handling

[Report] Fix handling of location type in Reports

Fix handling of location type in Reports

Merge pull request #14734 from yashodha/dev-1104

(dev/core#1104) make admin panels hookable

Merge pull request #14320 from sushantpaste/reporthook

dev/core#987 Reporthook

Fix mishandling of renamed membership status labels on membership import

Catch Payment Processor Exception if thrown when registering via back office participant registration

Make changes as per Matt's and Eileen's comments

Merge pull request #14939 from eileenmcnaughton/cont

Extend contribute search url parsing to advanced search

Extend contribute search url parsing to advanced search

Convert custom data export test to use newer function.

Fix handling for custom country fields to be where it is actually hit now

Convert some more tests to use the newer function

In testing this I checked and all the exports export an integer in the campaign_id field. Contribution
ALSO exports campaign_title field - which is unaffected by the removed lines (& for contribution both are tested)

I see the reference to https://issues.civicrm.org/jira/browse/CRM-14398 is for contribution which is covered by the
unit test

Merge pull request #14934 from civicrm/5.16

5.16

Merge pull request #14932 from eileenmcnaughton/export_fn_move

[REF] [Export] Move writeToTable fn to exportProcessor

Merge pull request #14931 from eileenmcnaughton/mysql516

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

Merge pull request #14917 from eileenmcnaughton/zero_ex

[REF][Event]  Extract calculation of 'zero-ness' in form rule

Merge pull request #14881 from eileenmcnaughton/cont_payment_method_import

dev/financial#36 [IMPORT] fix & test mishandling on payment_instrument labels

Move write to table to exportProcessor

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

Merge pull request #14877 from eileenmcnaughton/hist_rep

dev/core#1141 remove unused deprecated sql_calc_rows