[REF] [export] remove chunk of non-functional code

I worked through this & groups is only available as a return property from contact export so can only be hit from there.

However, the search string has a very specific format - involving parentheses - that is NOT present in the from
clause as currently calculated

      case 'civicrm_group_contact':
        return " $side JOIN civicrm_group_contact ON contact_a.id = civicrm_group_contact.contact_id ";

I don't kmow if there is some bug that if this worked it would fix or not but the original bug
it addressed referred to an old query format we don't use https://issues.civicrm.org/jira/browse/CRM-3618 now.

If there is a bug this won't change it & if we identify it we can add tests & fix properly

Merge pull request #14768 from eileenmcnaughton/export_test

[REF] [export] . Move greeting params retrieval to the place in the code where it is used

Move greeting params retrieval to the place in the code where it is used

Merge pull request #14765 from eileenmcnaughton/export_test

[REF] Update export test to new function

Merge pull request #14745 from eileenmcnaughton/fgb

dev/core#578 follow up fix on activity summary report

Merge pull request #14615 from mattwire/contributionpage_title

Set title using standard form method and use for success message on contributionpage

Merge pull request #14762 from eileenmcnaughton/mapping_whomp

[REF] Extract mapping converter function, kinda brutally

Extract mapping converter function, kinda brutally

Update export test to new function

Merge pull request #14706 from ixiam/dev-SmartyCompiled

expose smarty's compile_check to be overridden in civicrm.settings.php

Merge pull request #14758 from eileenmcnaughton/export_test_improve

[REF] export code simplification

Add unit test for saveFieldMapping

Merge pull request #14757 from eileenmcnaughton/mapping_fields

[REF] Extract saveMapping Field

Merge pull request #14729 from eileenmcnaughton/enotice

e-notice fix & unit test

Remove unecessary pass-by-reference

[REF] export code simplification

On review I find the value ['exportOption'] is a parameter used at the form later to determine whether to
export 'Primary fields only' (ahem) or the user should select an array of fields. By the time the form is submitted we
either have an array of selected fields or NULL denoting that there is no selection (& hence the defaults should be used)
and the selection at the form layer is redundant. However it is still being referenced in one place (and hence the tests
are passing it in to avoid an enotice). This fixes that one place & removes it from the tests. It also extracts part of
a test to move towards making it easier to alter the signature of exportComponents without messing with so many places in
the code

Merge pull request #14756 from eileenmcnaughton/wrangle

Extract field wrangling to determineReturnProperties

Extract saveMapping Field

Basic extraction. Tangental to export mapping cleanup

Merge pull request #14754 from eileenmcnaughton/cust_field_fix

Fix obscure bug on updating custom fields (not necessarily hittable via UI)

Merge pull request #14755 from eileenmcnaughton/mapping_sane

[REF] Remove call to getMappingFields in favour of api call.

Extract field wrangling to determineReturnProperties

This encases the wrangling of the field format to it's own function

Merge pull request #14742 from eileenmcnaughton/payment_out

[REF] un-extract createProportionalFinancialEntities

Fix obscure bug on updating custom fields (not necessarily hittable via UI)

This fixes a bug which I think only applies to programatically created custom fields -basically if the field has an empty option_group_id
and then you update it to have one it does a check to see if the existing option_group_id is in use and does not handle the
fact the field is empty - leading to a mysql error.

Remove call to getMappingFields in favour of api call.

The getMappingFields function is confusing - it turns out we just need an array of names

Merge pull request #14752 from eileenmcnaughton/export

[REF] Move function onto the processor class

NFC - reformat test class

Move function onto the processor class

I went to extract the function that determines fields off to be a funciton on the processor class
but moving this first simplifies it

Merge pull request #14514 from eileenmcnaughton/export_sc_jus_test

dev/core#1015 Unit test for fix regression on exporting soft credits

Merge pull request #14751 from colemanw/extractCompFields

[REF] Extract CRM_Core_BAO_Mapping::addComponentFields

Extract CRM_Core_BAO_Mapping::addComponentFields

Merge pull request #14744 from lcdservices/dev-report-17

dev/report#17 fix postal_code_suffix col

e-notice fix & unit test

This addresses a fairly obscure enotice. The notice occurs when a table has been excluded from
triggers by a hook (as makes sense if the table includes only calculated fields) and then a new
custom field is added to the table (or presumably edited or removed). In this case we should
not go through the whole trigger handling loop.

Note I see it as a feature rather than a bug that this would also apply to triggers that
update the contact.modified_date as it's not helpful to update this when updating a
calculation - if someone decides to intervene by hook we should trust them

Merge pull request #14616 from mattwire/contributionpage_tabset_cleanup

REF Simple cleanup of tabset code for contributionpages

Merge pull request #14720 from pradpnayak/REF-1

Removed hardcoded activity status and used Activity create

Merge pull request #14749 from civicrm/5.16

5.16 to master

Merge pull request #14746 from eileenmcnaughton/order_report

dev/core#1081 fix for error on contribution detail when using custom data order by without select

Removed hardcoded activity status and used Activity create

Added unit test

Use unique subject for activity in test

dev/core#1081 fix for error on contribution detail when using custom data order by without select

dev/core#578 handle full group by on activity summary.

Dave J did a tonne of work fixing up this report & adding a unit test - this complete by just handling FGB very specifically
on one query

Merge pull request #13540 from davejenx/dev-core-577-activity-summary-report-db-error

dev/core/issues/577: Activity Summary report fix for db column count error with section header (without ONLY_FULL_GROUP_BY)

dev/report#17 fix postal_code_suffix col

Merge pull request #14743 from colemanw/mappingRef

[REF] Further cleanup & extraction in getMappingFieds

Further cleanup & extraction in getMappingFieds

[REF] un-extract createProportionalFinancialEntities

This is one of those places where, unituitively, I think undoing an extraction makes sense.

The code for allocating monies to financial items is shared with the change payment instrument functionality
but fundamentally these are different processes and sharing the code is making it less rather than more readable.

My goal is to pull back all the code that handles processing financials off a payment back into Payment.create
and to clean it up from there

Merge pull request #14741 from colemanw/mappingExt

[REF] Simple function extraction for buildMappingForm

[REF] Simple function extraction for buildMappingForm

Merge pull request #14727 from eileenmcnaughton/cust1

[REF] simple extraction of getFieldAlterSQL

[REF] simple extraction of getFieldAlterSQL

Merge pull request #14589 from eileenmcnaughton/money

 Fix creation of additional zero value line item when changing fee selection in edge case

Merge pull request #14725 from eileenmcnaughton/cust_field4

[REF] extract portion that creates the custom field record

Merge pull request #14738 from civicrm/5.16

5.16 to master

Merge pull request #14732 from eileenmcnaughton/backoffice

dev/report#16 Unreleased regression - fee levels incorrectly show sol…

Merge pull request #14733 from eileenmcnaughton/payment_reg

Fix refund payment not recording from additional payment form

dev/core#1015 Unit test for (merged) regression on exporting soft credits

Merge pull request #14614 from mattwire/manageevent_title

Improve handling of 'Manage Event' title

Merge pull request #14697 from eileenmcnaughton/schema

[REF] Cleanup fixSchemaDifferencesFor()

[REF] extract portion that creates the custom field record

Clarify appending field to column_name

[REF] further simplification of custom field create action

Merge pull request #14724 from eileenmcnaughton/cust_field3

[REF] clarify  variable (very minor change with good test cover)

Merge pull request #14731 from eileenmcnaughton/leaky

Fix some test leakage

Merge pull request #14736 from pradpnayak/contriblist

Updated Circle-Interactive developers info

Merge pull request #14701 from eileenmcnaughton/act_location

Activity search - convert priority_id to a metadata field and add location as a searchable field

Updated Circle-Interactive developers info

Simple cleanup of tabset code for contributionpages

Set title using standard form method and use for success message on contributionpage

Improve handling of 'Manage Event' title

Merge pull request #14726 from eileenmcnaughton/cust_field_5

[REF] simple extraction of prepareCreateParams

Merge pull request #14721 from kirk-circle/master

dev/core#1097 - Ensure consistent count on Groups tab

Fix refund payment not recording from additional payment form

Revert participant portion of https://github.com/civicrm/civicrm-core/pull/14244

Set version to 5.17.alpha1

Set version to 5.16.beta1

Merge remote-tracking branch 'origin/5.15'

dev/report#16 Unreleased regression - fee levels incorrectly show sold out

Merge pull request #14730 from alifrumin/5.15.0-releasenotes

5.15.0 releasenotes

Set version to 5.15.0

Update 5.15.0.md

cleanup

finished annotating

annotating

finishing changes with git lab issues

linking and annotating

annotating

annotating

contributors

organizing by git lab issue and condensing as well as pulling out obvious misc

5.15.0 release notes: added boilerplate

5.15.0 release notes: raw from script

Fix some test leakage

[REF] simple extraction of prepareCreateParams

[REF] clarify  variable

The variable is clearer being defined directly as 'add' or 'modify' in the main function to reduce wrangling

Merge pull request #14723 from seamuslee001/ref_date_to_local_extract

Extract code converting a date object to local timezone object to own…

Extract code converting a date object to local timezone object to own function

dev/core#1097 - Ensure consistent count on Groups tab

Merge pull request #14717 from totten/master-rm-ser

(dev/cloud-native#3) SerializeCache - Remove unused, incomplete cache-driver

expose smarty's compile_check to be overridden in civicrm.settings.php

(dev/cloud-native#3) SerializeCache - Remove unused, incomplete cache-driver

The class `CRM_Utils_Cache_SerializeCache` provides a cache-driver for
storing cache records in the filesystem (under `CIVICRM_TEMPLATE_COMPILEDIR`,
using PHP `serialize()` format). Why remove it?

1.  As we work through cleanup in file management (e.g.  #12843,
   dev/cloud-native#3), having an unnecessary reference to
   CIVICRM_TEMPLATE_COMPILEDIR makes it harder to reason about the system.

2. The class is not used. I'm pretty sure it was added speculatively
   (i.e.  with an aim to try using it some day), but that never came to
   pass.  Grepping `universe`, I cannot find any usages or references to
   `SerializeCache`.

3. The implementation is incomplete -- parameters like `get(..., $default)`
   and `set(..., $ttl)` will generate exceptions if used.

Merge pull request #14714 from civicrm/5.15

5.15 to master

Merge branch 'master' into 5.15