Merge pull request #14779 from eileenmcnaughton/export_test2

[ref] [export] [test] Improve csv test to test final output rather than the csv

Merge pull request #14750 from eileenmcnaughton/pay_notice

Fix enotice on formatting credit card details

Improve csv test to test final output rather than the csv

Merge pull request #14770 from totten/master-config-ids

Finish removing references to Config.IDS.ini

Merge pull request #14696 from totten/master-d8-url

Migrate CivicrmHelper::parseUrl() to CRM_Utils_System_Drupal8::parseUrl().

Merge pull request #14777 from totten/master-clean-key

GenCode, Cache::cleanKey() - Fix deploop during clean initialization

Merge pull request #14778 from totten/master-basefilepath

(dev/cloud-native#3) CRM_Utils_File - Deprecate baseFilePath() et al

CRM_Utils_File::relativeDirectory() - Mark as deprecated

Computation of a relative path requires some base.  This implementation is
problematic because it relies on an implicit base which was constructed
problematically.  Moreover, the function is not used anywhere in the known
`universe`.

CRM_Utils_File::absoluteDirectory() - Make `$basePath` required

Previous versions interpreted `$basePath=NULL` to mean "default to `self::baseFilePath()`".
However, no code in the known `universe` relies on this interpretation, and
the `baseFilePath()` function is problematic/deprecated.

We could almost deprecate the entire function `absoluteDirectory()`;
however, it's fine to keep around: the semantics are reasonable if one
provides `$basePath`, and there is one active use-case in which
`Civi\Core\Paths` calls `absoluteDirectory()` with a suitable `$basePath`.

CRM_Utils_File::baseFilePath() - Mark function as deprecated

The rationale is provided in the docblock.

GenCode, Cache::cleanKey() - Fix deploop during clean initialization

Overview
--------

Suppose you have a clean codebase with no DAO files, and you try to run `xml/GenCode.php`.
This currently crashes.

The problem is not symptomatic day-to-day because we commit DAOs, so it's
very rare to run a full/clean initialization.  However, it does get in the
way of stress-testing the correctness of GenCode.

Before
------

Since 76e697a9d750d568d0d12c10dd8173f656e8bb1e, I believe we've had a dependency-loop
in the clean-gencode scenario, which works as follows:

* We don't have any DAOs, so we run `GenCode`.
* Running `GenCode` does a partial bootstrap (i.e. `CRM_Core_Config::singleton($loadFromDB===FALSE)`)
* The partial bootstrap creates some thread-local caches (`Arraycache`)
* Before creating the cache, it normalizes the name by calling `CRM_Core_BAO_Cache::cleanKey()`
* `CRM_Core_BAO_Cache` extends `CRM_Core_DAO_Cache`
* `CRM_Core_BAO_Cache` doesn't exist - that's why we called `GenCode` at the beginning.

After
-----

This migrates the utility function `cleanKey()` to another class which is always available and
does not rely on DAOs.

Merge pull request #14772 from mlutfy/drupal75

dev/drupal#75 Drupal8: fix call to languageNegotiationURL() when called from cv

dev/drupal#75 Drupal8: fix call to languageNegotiationURL() when called from cv

Merge pull request #14768 from eileenmcnaughton/export_test

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

CRM-20926 - Finish killing Config.IDS.ini

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

Fix entoice on formatting credit card details

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