[NFC] Cleanup comments, string comparision in grant search

Merge pull request #15931 from seamuslee001/standardise_grant_search

[REF] Standardise the Grant Search form handling

Merge pull request #15980 from eileenmcnaughton/setting_fix

dev/core#1425 Replace deprecated settings fns in test suite

Merge pull request #15979 from eileenmcnaughton/search

Remove CRM_Contact_Form_Search_Custom_Basic from searches added on install.

Merge pull request #15982 from civicrm/5.20

5.20

dev/core#1425 Replace deprecated settings fns in test suite

Merge pull request #15977 from seamuslee001/fix_issue_number_range_smart_group_empty_values

Ensure that Relative key is not added for non Select Date is_search_range custom fields

Only add in relative key when its a custom date field

Ensure that we strip the _to _from _high _low from the element name first

Regenerate sql

Remove CRM_Contact_Form_Search_Custom_Basic from searches added on install.

From https://github.com/civicrm/civicrm-core/pull/15963 this search appears to do what the main basic search does but
we can see that to make it work with sort etc we have to make changes to the BAO_Query object - doing
that to support a custom search scares me & we have a big picture position that we should move these
from core to an extension. However, an easy first step is not to add on new installs. In the case of
this search it seems like an easy call as it doesn't add anything discernable

Merge pull request #15978 from civicrm/5.20

5.20

Merge pull request #15976 from eileenmcnaughton/5.20

Fix 5.20 regression on retrieving template transaction with no logged  in user

Merge pull request #15901 from eileenmcnaughton/matt

[REF] further removal of unnecessary  params

Fix 5.20 regression on retrieving template transaction with no logged in user

Merge pull request #15973 from kcristiano/5.20-msg-tpl

Link to docs.civicrm.org instead of wiki

Link to docs.civicrm.org instead of wiki

Merge pull request #15877 from demeritcowboy/casereport-unittest

[TEST] dev/core#1366 Unit test for case audit

Merge pull request #15909 from eileenmcnaughton/payment

Fix api Payment.create to support overpayments

Merge pull request #15951 from eileenmcnaughton/export_bom

CSV Export: Add deprecation warning

Merge pull request #15967 from seamuslee001/dev_core_1329

dev/core1329 Reduce number of deceased contacts in the demo data from…

dev/core1329 Reduce number of deceased contacts in the demo data from 31 to 12

Limit number of deceased contacts to 4 and add in age checks as per Justin

Merge pull request #15882 from demeritcowboy/audit-timeline

dev/core#1366 - Make case activity audit print report work again

Merge pull request #15965 from colemanw/ufMatchTest

Fix potential test glitch when repeatedly calling createLoggedInUser

Fix potential test glitch when repeatedly calling createLoggedInUser

Merge pull request #15960 from eileenmcnaughton/ex_param

[REF] Remove unused parameter

Merge pull request #15958 from civicrm/5.20

5.20

Remove unused parameter

This parameter is always TRUE so it adds no value, removing

Merge pull request #15939 from demeritcowboy/xml-file-status-check-520

dev/core#1046 Status check for external case xml files

Merge pull request #15953 from eileenmcnaughton/export_bom2

Remove unused parameter

Merge pull request #15952 from eileenmcnaughton/ps_ex

Fix deprecation warning on Price Set report

Merge pull request #15947 from eileenmcnaughton/act

Rename activity search field from status_id to activity_status_id

Merge pull request #15954 from eileenmcnaughton/ex_silly

[REF] CSV Export: Remove impossible checks on  var

status check for external case xml files

Merge pull request #15955 from JMAConsulting/core-1420

core#1420 Quicksearch with phone filter doesn't work with non-numeric character

Merge pull request #15956 from colemanw/removeFn

Remove unused functions

Merge pull request #15861 from JMAConsulting/core-1398

core#1398: Option to open navigation item in new window (if present)

Merge pull request #15734 from seamuslee001/remove_activity_option_join_custom_search

Remove join to civicrm_option_value in favour of using getLabel funct…

Remove unused functions

These functions are no longer needed now that KAM is merged into core.

core#1398: Option to open navigation item in new window (if present)

core#1420 Quicksearch with phone filter doesn't work with non-numeric character

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

[REF] Remove checks on  var

This variable is hard-coded to a double quote so checks to see if it is a single quote will never be true.

I've checked this pretty thoroughly & I think the removed IFs are dead code

Merge pull request #15941 from seamuslee001/dev_core_560_sms

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

Remove unused parameter

There are 3 calls to writeCSVFile - none of them pass in saveFile so it can be removed

Fix deprecation warning on Price Set report

In 5.20 we added a deprecation warning on searches that are borked WRT filling the prev_next
cache & hence doing searches. The price set search falls into this camp & while it has
been broken forever the deprecation notice is new (& the fix is safe) so targettin 5.20

Add deprecation warning.

On digging into this export function I cannot find any way to access these lines.

If no-one else can spot a way I propose adding a deprecation notice & later removing

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

Rename activity search field from status_id to activity_status_id

Currently this is on the form as 'status_id' but as soon as it's submitted the value is renamed - I think
we might not even need to change the smart group for this one.

The other field that is being renamed is 'priority_id'. Unless we have an appetite to
add a uniqueName for this field (which I kind of feel like not doing more of for some
cycles), I think we could stop renaming it & rely on the default where handling for the
where clause. From what I can see the reason for prefixing is not about
uniqueness but rather about adding a prefix so it gets handled by
Activity_BAO_Query - but in fact Contact_BAO_Query should handle it just fine
based on metadata

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

Standardise the Grant Search form handling

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 api Payment.create to support overpayments

We've discussed this before - it's OK to add a payment to a fully paid contribution because ... life.

When this happens there should be no financial items linked to the over payment

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…