Merge pull request #15950 from eileenmcnaughton/export_ref

[REF] Further cleanup on address handling in merge code.

Merge pull request #15928 from seamuslee001/case_advanced_search

Enable Case search metadata on Advanced Search Form

Merge pull request #15941 from seamuslee001/dev_core_560_sms

dev/core#560 Replace instances of CRM_Core_Error::fatal with Exceptio…

Enable Case search metadata on Advanced Search Form

[REF] Further cleanup on address handling in merge code.

This is a follow on from https://github.com/civicrm/civicrm-core/pull/15949

This doesn't make any change - it just improves legibility

dev/core#560 Replace instances of CRM_Core_Error::fatal with Exceptions or Status bounces in SMS processing and appropriately handle error after

Merge pull request #15949 from eileenmcnaughton/export_ref

[REF] Minor code cleanup on the setting of contact greetings.

Merge pull request #15944 from magnolia61/Sort_CMS_tables_alphabetically

Sort CMS Database Table list

Merge pull request #15943 from seamuslee001/sort_arrow_activity_search

[UI] Ensure that when sorting on columns in the find activity search …

[REF] Minor code cleanup on the setting of contact greetings.

I'm trying to review https://github.com/civicrm/civicrm-core/pull/15725 but through no fault of that PR it hits
on one of the parts of the Export code that I still find unreadable :-( This is a further readability fix & it
starts to point me to how to get past the underlying WTF about this code.

Note that I removed the IF around sharedAddress. From Monish's PR I found out that we have a
fairly long-standing regression where the sharedAddress code doesn't actually work :-( so
I'm comfortable this won't break anything :-). We also have good test cover on this chunk from
probably around when the shared address part got broken....

Looking in the UI there is no implication that the greeting for a shared address would display differently -
which is the only explanation I can think of for different handling here. Of course until we
remove the later IF the shared address would do things differently - erm if it worked.

ALl of which is a long way of saying the removal of the IF won't change anything

Sort CMS Database Table list

[UI] Ensure that when sorting on columns in the find activity search the arrow shows which field is being sorted and direction

Merge pull request #15930 from seamuslee001/standardise_pledge_search

[REF] Standardise form handling in the Find Pledges search form

Merge pull request #15936 from seamuslee001/advanced_search_metadata

Include Metadata on advanced search form for membership, grant, pledg…

Merge pull request #15938 from civicrm/5.20

5.20

Merge pull request #15935 from seamuslee001/case_id_search_field

Reinstate Case ID search field

Reinstate Case ID search field

Merge pull request #15934 from eileenmcnaughton/part_sane

[REF] return determination of whether to show expired fields to the calling function

Include Metadata on advanced search form for membership, grant, pledge, fields

Merge pull request #15933 from civicrm/5.20

5.20

[REF] return determination of whether to show expired fields to the calling function

As part of my efforts to ensure we are consistency creating valid transactions I'm trying to sort out the way we calculate
amounts to be re-usable from tests but I feel stymied at every turn by spaghetti code.

This unravels a small piece

Merge pull request #15929 from christianwach/lab-1412

Reinstate traversal as "method of last resort" to find WordPress

[REF] Standardise form handling in the Find Pledges search form

Reinstate traversal as "method of last resort" to find WordPress

Merge pull request #15847 from artfulrobot/opaque-menu

Remove background opacity in menus; improving accessibility/readability

dev/core#1384 Joomla: fix JPATH_BASE when using a virtual directory on Windows (#15915)

Merge pull request #15907 from eileenmcnaughton/ex

[NFC] [Test] Minor test cleanup on CiviMailUtils

Merge pull request #15924 from seamuslee001/dev_core_1414

dev/core#1414 Fix E-notices on the lack of an html key in the psudo m…

Merge pull request #15921 from civicrm/5.20

5.20

Merge pull request #15816 from MiyaNoctem/dev-core-1383-fix-reinstallation-of-extensions-with-logging

dev/core#1383: Fix Re-Installation of Extensions With Logging Enabled

Merge pull request #15920 from seamuslee001/dev_core_1410

dev/core#1410 Fix E-notice when doin a force case search with a prede…

dev/core#1414 Fix E-notices on the lack of an html key in the psudo metadata for these psudofields

dev/core#1410 Fix E-notice when doin a force case search with a predefined case subject field

Merge pull request #15919 from kcristiano/5.20-1412

dev/core/1412  CiviMail sent via wp-cli and cron mangles mailing urls…

Merge pull request #15922 from totten/5.20-e2e-cli

(dev/core#1412) E2E - Add test case for path+URL construction in CLI

(dev/core#1412) E2E - Add test case for path+URL construction in CLI

Overview
--------

In dev/core#1412, the regression involved CLI processes producing incorrect URLs.
This adds some test-coverage.

Before
------

No test coverage for URLs in CLI environment.

After
-----

There is some test-coverage. It's able to reveal the existence of the bug from 1412.

Comments
--------

A lot of path/URL construction is dependent on the environment/site-build.
The test does not, on its own, guarantee that all environments work.
Rather, it adds a new slice of cheese to the stack.

This slice is moderately generic - in that I've successfully tested it
localy on `drupal-demo` and `wp-demo` builds.  In theory, it should work on
other build types (as long as they set variables `CIVI_CORE`, `CMS_ROOT`,
`CMS_URL`).

Merge pull request #15918 from MegaphoneJon/core-1411-rc

core#1411: Advanced Search crashes when some form values start with 1

Merge pull request #15905 from eileenmcnaughton/test_good

Fix CRM_Contribute_BAO_ContributionTest to no longer use unreliable legacy set up method

dev/core/1412  CiviMail sent via wp-cli and cron mangles mailing urls on WP

core#1411: Advanced Search crashes when some form values start with 1

Merge pull request #15916 from civicrm/5.20

5.20 to master

Merge pull request #15899 from greenpeace-cee/fix-ambiguous-order-column

Fix ambiguous column in search with ORDER BY

Merge pull request #15906 from eileenmcnaughton/odd

[Test] Remove hard coded it reference

dev/core#1383: Fix Re-Installation of Extensions With Logging Enabled

When uninstalling an extension, logging tables associated to custom groups and
fields will not be deleted. On re-installation, addition of custom fields will
cause DB errors to be thrown, as columns existing on logging tables are tried
to be created again (they already exist on logging tables).

Fixed by checking if the column exists on log table before trying to create
it, treating it as a modification of the schema if it exists.

Fix ambiguous column in search with ORDER BY

This fixes an issue where columns whose names are not unique in a
search query cause a DB error when they're used as a sort column.
The issue can be observed in the contribution search when sorting
by contribution status.

The issue is resolved by using the where field of the column spec,
which holds the fully-qualified name of the column.

Minor test cleanup on CiviMailUtils

[Test] Remove hard coded it reference

Honestly - why isn't this failing already? We only need the contributionID so this works

Merge pull request #15904 from eileenmcnaughton/greeting

Move greeting handling up as it is not paymentValidation dependent

Fix CRM_Contribute_BAO_ContributionTest to no longer use unreliable legacy set up method

The whole partial_amount_to_pay params thing works badly & was actually the cause of other fixes stalling for 6 months.

This is part of an effort to deprecate & eliminate it

Move greeting handling up as it is not paymentValidation dependent

Merge pull request #15819 from seamuslee001/dev_core_183_price_set

dev/core#183 Event Price Set Custom Search to using CRM_Utils_SQL_Tem…

Merge pull request #15793 from seamuslee001/depreacete_dao_temp_table_name

dev/core#183 Remove references to and noisly deprecated CRM_Core_DAO::createTempTableName

Merge pull request #15792 from seamuslee001/convert_temp_table_creation_campaign_upgrade

dev/core#183 Use Standard CRM_Utils_SQL_TempTable builder to create temporary tabl…

Merge pull request #15876 from seamuslee001/dev_core_183_random_segement

dev/core#183 Convert the generating of temporary tables within the ra…

Add in unit test of Campaign BAO query voter clause

Add in unit test of the priceset custom search

dev/core#183 Event Price Set Custom Search to using CRM_Utils_SQL_TempTable interface

Remove references to and noisly deprecated CRM_Core_DAO::createTempTableName

Update unit test to match new temp table format

Use Standard CRM_Utils_SQL_TempTable builder to create temporary table in Campaign and upgrade

Merge pull request #15785 from eileenmcnaughton/contribution_url_params

Contribution url params - add contribution_page_id as a supported url parameter

Merge pull request #15902 from eileenmcnaughton/transaction_sillyness

[REF] remove another unnecessary pass-by-reference

Merge pull request #15903 from eileenmcnaughton/except

Convert a few fatals to exceptions.

[REF] remove another unnecessary pass-by-reference

the recur function is only called once. Immediately after it is called there is a return &
the calling function has not passed in any variables by reference - ergo we gain nothing but confusion by using pass-by-ref
here

Merge in 5.20

Merge pull request #15841 from mattwire/participant_cleanup_removeparticipantfrominput

Remove unused variables in completeOrder()

Merge branch '5.20-security_release_release_notes' into '5.20-security'

Release notes for 5.19.2 Release

See merge request security/core!90

Update 5.19.2.md

Update 5.19.2.md

Update release notes for Public MRs being merged into 5.19

Update 5.19.2.md

Update 5.19.2.md

Update 5.19.2.md

Minor update to release notes

Update release notes as per Tim's copy editing of the SA

Release notes for 5.19.2 Release

Update to latest versions of polyfill-ctype and polyfill-iconv

securiry/core#70 Upgrade Symfony version to 2.8.50

ContactApiKeyTest - Add additional test-case

This case was failing before but now passes with the update

Ensure that APIv4 Properly filters out permissable fields when no permission to view field exists

Update unit test to match the output of the CRM_Utils_String function

SavedSearch API - Only accept safe inputs

CRM_Utils_String::unserialize() - Add wrapper for parsing safe exprs (int/string/array/etc)

[NFC] Update Doc block as per Patrick

Revert changes in CRM/Core/BAO{Cache|ConfigSetting}.php CRM/Extension/ClassLoader.php CRM/Queue and CRM/Utils/Cache

Return FALSE instead of throwing Exception if package throws Exception

Implement xKerman/restricted-unserialize package to guard against unsafe unserialize

ContactInterchangeTest - Add coverage for APIv4 joins/chains

(security/core#67) Port APIv3's escaping scheme to APIv4

HTMLInputCoder - Add more variants for encoding arrays

Fix unit test for APIv4 format

Fix api_key handling from joins

security/core#65 - Fix XSS in Dashboard Report Title

This fixes an XSS in report titles displayed on the dashboard page
by escaping the title on output.

security/core#65 Fix Display of Report title that contains XSS in it when shown on dashboard

security/core#59 - Fix SQL injection in civicrm/ajax/dedupefind

This fixes an SQL injection in civicrm/ajax/dedupefind by validating
the order direction parameter.

Fixes #59

Merge pull request #15890 from civicrm/5.20

5.20

Merge pull request #15892 from totten/master-extgettest

(NFC) api_v3_ExtensionTest - Fix when executing on build with 25+ extensions

Merge pull request #15889 from eileenmcnaughton/refund

dev/core#1409 Remove net_amount from Addtional Payment form

dev/core#1409 Remove net_amount from Addtional Payment form

This is causing a validation problem. We used to have an issue on the contribution form which we eventually
resolved by removing net_amount as it's best calculated anyway

In order to make this changed I had to ensure Payment.create adds the net_amount & had
to do a couple of tweaks for the test to pass

(NFC) api_v3_ExtensionTest - Fix when executing on build with 25+ extensions

Overview
--------

This fixes a small issue with running the test locally.

Before
------

If the system has code for 25+ extensions (whether enabled or not), `api_v3_ExtensionTest` fails.

After
-----

The number of extensions doesn't matter.

Merge pull request #15326 from totten/master-headfoot-2

CiviMail - Only display "Headers and Footers" if there are headers and footers