Merge pull request #15958 from civicrm/5.20

5.20

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

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

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)