Deprecated alterFieldSQL

This function is only used by one place and feels like the wrong separation of concerns - ie
we have a single create action & a multiple create action. They both need to share the generation
of the sql to run but can handle the batching of that & rebuilding of triggers etc
separately

Merge pull request #14757 from eileenmcnaughton/mapping_fields

[REF] Extract saveMapping Field

Merge pull request #14729 from eileenmcnaughton/enotice

e-notice fix & unit test

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

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

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

(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

Merge pull request #14690 from eileenmcnaughton/profile_copy

dev/core#1088 Fix fatal error when copying profile with civirules enabled

Merge pull request #14713 from pradpnayak/REF-1

Grab contribution status id from database

Merge pull request #14608 from eileenmcnaughton/csv_test

 [test] convert export test to handle exception rather than early return

Grab contribution status id from database

Merge pull request #14707 from demeritcowboy/why-is-php-case-insensitive-for-classnames-2

dev/joomla#13 - followup 2 - Some class names have wrong upper/lower case spelling

Merge pull request #14708 from demeritcowboy/why-is-php-case-insensitive-for-classnames-3

dev/joomla#13 - followup 3 - Some class names have wrong upper/lower case spelling

Merge pull request #14709 from demeritcowboy/why-is-php-case-insensitive-for-classnames-4

dev/joomla#13 - followup 4 - Some class names have wrong upper/lower case spelling

Merge pull request #14710 from demeritcowboy/why-is-php-case-insensitive-for-classnames-5

dev/joomla#13 - followup 5 - Some class names have wrong upper/lower case spelling

Merge pull request #14704 from pradpnayak/REF-1

Grab contribution status id from database

Merge pull request #14685 from eileenmcnaughton/mark_conflict

Consolidate handling of conflicts between the batch job and get_conflicts api

fix wrong spelling of classname

fix wrong spelling of classname

fix wrong spelling of classname

fix wrong spelling of classname

Merge pull request #14693 from eileenmcnaughton/cust_field_sql

[REF] remove instances of pass-by-reference where no change takes place

Merge pull request #14503 from pradpnayak/ReportFix

$this->_selectedTables is not populated incase of boleen filters