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] simple extraction of function to check required fields are present

Merge pull request #14921 from eileenmcnaughton/url_search2

Core/dev#692 support for some additional url params

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

Merge pull request #14914 from eileenmcnaughton/export_test

[Export] Convert testGender Export to new test format.

Merge pull request #14913 from eileenmcnaughton/export

[REF] [Export] Stop passing header rows around

Merge pull request #14925 from civicrm/5.16

5.16

Merge pull request #14924 from colemanw/menubar5.16

Backport menubar fixes to 5.16

Merge pull request #14926 from eileenmcnaughton/conf

dev/core#1084 Contribution confirmation page: Translation won't work

Update Confirm.tpl

Merge pull request #14920 from eileenmcnaughton/url_search

Fix accidental move of sort_name field on advanced search

Merge pull request #14904 from eileenmcnaughton/activ_516

dev/core#1042 fix  duplicate activities on custom data block

Menubar - get rid of hardcoded pixel values in crm.menubar.js

Menubar - Use class instead of id for flexibility

Fix accidental move of sort_name field on advanced search

The code that made this field url-enterable ALSO re-ordered them by accident - this fixes

Merge pull request #14922 from civicrm/5.16

5.16 to master

Merge pull request #14918 from totten/5.16-prevnext

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Overview
-------------------

This reverts commit a5ca2ad8e16b145edf3a5604fab923d1e2386b37
and fixes a bug in management of the prev/next cache. The
bug is visible when following this procedure:

1. In browser, use "Quick search" to search for text "am"
2. In browser, select checkboxes next to two items
3. In MySQL CLI, run `select * from civicrm_prevnext_cache where is_selected =1;`
4. In browser, continue to next page. Observe that there are no more selection.
5. In MySQL CLI, re-run SQL query. Observe that the selections have disappeared.

Before
-------------------

Proceeding to another page of search results causes loss of selections

After
-------------------

Selections are preserved

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

Merge pull request #14911 from seamuslee001/ref_prev_next_cleanup_service

[REF] Add in cleanup function to prevnext service and utilise in clea…

Merge pull request #14915 from eileenmcnaughton/ex_test

[REF] [Export] [Test] Update 2 more tests to use new helper

Test conversion

Convert testGender Export to new test format.

This includes a tangental cleanup & fix whereby the test covers a label longer than 16 char & the code for
prefix_id & suffix_id is consolidated by handling the special casing in the metadata declaration
rather than in multiple places.

[REF] Add in cleanup function to prevnext service and utilise in cleanup cache situation

Set TTL on prevNextSQL Cache to be 2 days so cleanup code works as previously

Switch to using a class constant for the number of cache days

Stop passing header rows around

Merge pull request #14912 from eileenmcnaughton/export

[REF] [Export] Remove now redundant param

Remove now redundant param

Merge pull request #14784 from kirk-circle/1050-delete-repeating-activities

dev/core#1050 - Delete repeat activities that are selected for deletion

Merge pull request #14907 from eileenmcnaughton/force_pay

[REF] remove never-set, mispelt parameter

Merge pull request #14887 from mattwire/membership_deprecateids

Further deprecate use of $ids array in membership functions

Merge pull request #14747 from mattwire/contributionapi_getsoftperformance

Improve performance on getSoftContribution details - only run one query instead of one per contribution

Merge pull request #14909 from eileenmcnaughton/net_amount

Add unit test for net_amount when fee_amount is set

Merge pull request #14898 from eileenmcnaughton/export_rel

[REF] [Export] Move fetch Relationship details to processor

Merge pull request #14910 from JKingsnorth/core-1153

Remove unused standalone-mode code

Merge pull request #14906 from eileenmcnaughton/event

[NFC] comment fixes, function mis-casing fix

Test the activity form's deletion of single and repeating activities

Delete repeat activities that are selected for deletion

Add unit test for net_amount when fee_amount is set

This just improves testing & ensures fee_amount is used to set net amount.

I got some odd results on financial_trxn & financial_item but just adding checks for
what I found atm

Remove more use of ids array on CRM_Member_BAO_Membership::create

dev/core#1153 Remove unused standalone-mode code

Merge pull request #14889 from demeritcowboy/logging-report-clarification

dev/core#1149 - Make it clearer which record the logging report is displaying

Merge pull request #14902 from eileenmcnaughton/mem

Fix membership end date on confirming a pending contribution

Merge pull request #14908 from seamuslee001/master

5.16

Merge in 5.16