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)

Merge pull request #14743 from colemanw/mappingRef

[REF] Further cleanup & extraction in getMappingFieds

Further cleanup & extraction in getMappingFieds

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

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

Grab contribution status id from database

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

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

Merge pull request #14689 from eileenmcnaughton/cust_field_create

[REF] extract prepareCreate from CustomField.create

WIP

[REF] Cleanup fixSchemaDifferencesFor()

This is always called with FALSE as the last param except from the unit test - the test can do it's own work on it
Also passing NULL for param 2 is the same as not passing anything

Merge pull request #14699 from eileenmcnaughton/activity_search

[NFC][test] reformat activity search test

Merge pull request #14700 from eileenmcnaughton/search

[NFC][test] code formatting only

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

The location field on activity in a generally exposed field so it makes sense it should be in the search. Priority_id
conversion is part of ongoing standardisation which includes adding url parameter support for the field

[NFC][test] code formatting only SearchTest, SelectorTest

[NFC][test] reformat activity search test

Merge pull request #14695 from eileenmcnaughton/revert

Remove failing assertion

Merge pull request #14698 from civicrm/5.15

5.15 to master

Merge pull request #14692 from agh1/pseudoconstant-caps-5.15

dev/joomla#13 lower-case C on CRM_Core_PseudoConstant breaks 5.14 upgrade on Joomla w/ CiviCase

Remove failing assertion

This was a 'never worked' as the original call was doing no check. I changed it to do a check but
got confused & didn't realise thee check was actually failing when I merged. This check was never successful
we should remove it

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

I did a grep on these and the passed params are not altered - down with php4

Consolidate handling of conflicts between the batch job and get_conflicts api

This ensures that conflicts are stored during batch_merge to the prev_next cache with the same format as when the api
calls get_conflicts. The code doing this wrangling is moved from the api to the BAO layer.

We add a test to ensure the output is the same & use the previously added test to check the string is the same too.

Test cover here is pretty good

[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results

[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results

Add test ensuring conflicts key existing behaviour is retained

Use correct caps for CRM_Core_PseudoConstant everywhere

Merge pull request #14688 from eileenmcnaughton/revert

 Fix erroneous variable passed to callAPISuccessGetValue

dev/joomla#13 CiviCase: use correct caps for CRM_Core_PseudoConstant