5.20.0 release notes: final edits

5.20.0 release notes: add late changes

Merge pull request #16013 from eileenmcnaughton/reg

dev/core#1435 Fix lack of filtering in Contribution tab on Membership or Participant View

dev/core#1435 Fix lack of filtering in Contribution tab on Membership or Participant View

Merge pull request #16007 from seamuslee001/5.20

Add release-notes/5.19.3.md

Add release-notes/5.19.3.md

Merge pull request #16002 from seamuslee001/handle_custom_field_deleted

Fix regression from fixing relative key handling where custom field no longer exists in the database

Catch CiviCRM APIv3 Error if custom field no longer exists in the database

Add in unit test demonstrating the regression where smart group fails to load if a custom field has been deleted which was stored in the form values array

Merge pull request #15923 from eileenmcnaughton/act_order

Fix fatal error when sorting by status in activity search

Merge pull request #15962 from seamuslee001/dev_core_1422

dev/core#1422 Ensure that the form values are correctly passed onto t…

Merge pull request #15966 from seamuslee001/permission_access_metadata

Conditionally add metadata for advanced search only if the user has a…

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

Merge pull request #15976 from eileenmcnaughton/5.20

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

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

Fix fatal error when sorting by status in activity search

Conditionally add metadata for advanced search only if the user has access for searching to that perticular component

dev/core#1422 Ensure that the form values are correctly passed onto the Badge label task and others

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

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

Merge pull request #15952 from eileenmcnaughton/ps_ex

Fix deprecation warning on Price Set report

status check for external case xml files

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

Merge pull request #15935 from seamuslee001/case_id_search_field

Reinstate Case ID search field

Reinstate Case ID search field

Merge pull request #15929 from christianwach/lab-1412

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

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

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 #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

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 #15899 from greenpeace-cee/fix-ambiguous-order-column

Fix ambiguous column in search with ORDER BY

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.

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 #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

Merge pull request #15888 from seamuslee001/lab_core_1406

dev/core#1406 Ensure that we use the CRM_Member_DAO_Membership as dao…

dev/core#1406 Ensure that we use the CRM_Member_DAO_Membership as dao for the civicrm_membership table

Merge pull request #15879 from alifrumin/rn5.20

First pass at sorting and annotating 5.20 release notes

release notes 5.20 - first pass at editing, sorting and annotating

Merge pull request #15846 from greenpeace-cee/fix-cancel-date

Fix contribution cancel_date not being loaded

dev/core#1391 - Fix contribution cancel_date not being loaded

This fixes an issue where the contribution cancel_date is not loaded
in the contribution search, causing cancelled contributions not to be
greyed out and the cancel date not to be shown.

Merge pull request #15872 from eileenmcnaughton/sybunt

Fix Sybunt to select mark rows as selected

Merge pull request #15873 from eileenmcnaughton/cont_ag

dev/core#1377 Fix bug where search action doesn't work.

dev/core#1377 Fix bug where search action doesn't work.

Fixes a regression where the search actions are not working on the contributionAggregate custom search

Fix Sybunt to select mark rows as selected

Merge pull request #15844 from mlutfy/fixMsgTpl

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

Merge pull request #15804 from eileenmcnaughton/fns

dev/core#1381 fix fatal on case search (unreleased regression

dev/core#1381 fix fatal on case search (unreleased regression

Merge pull request #15787 from eileenmcnaughton/recur_ug

Remove unhelpful alert from contribution search

Merge pull request #15789 from seamuslee001/5_19_1_version_notes

5.19.1 Version Release Notes

Update 5.19.1.md

Add in release notes for 5.19.1 drop

Remove unhelpful alert from contribution search

Merge pull request #15781 from eileenmcnaughton/cont_search

dev/core#1374 Fix search formValue handling on contribution search

Merge pull request #15782 from seamuslee001/dev_mailing_56_2

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix i…

dev/core#1374 Fix search formValue handling on contribution search

This fixes 2 issues
1) Search params being lost when editing a related entity per
https://lab.civicrm.org/dev/core/issues/1374
2) force=1&sort_name=p not working in contribution search url

In digging I concluded the problem is we have 3 underlying arrays which we keep jumbling together

1) formValues - the actual submitted values, augmented by any  url passed params
2) the default values - values to load by default on the form
3) our working query params - a copy of formValues that we have prepared for the query

We need to stop mangling them. I added subtle code comments

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix issue where by mailing_name has been namespaced also in 5.20

Merge pull request #15779 from seamuslee001/dev_mailing_56

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namesp…

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namespaced fields when building the row array for mailing reports

Merge pull request #15772 from agh1/5.20.0-releasenotes-initial

5.20.0 release notes initial run

5.20.0 release notes: added boilerplate

5.20.0 release notes: raw from script

Merge pull request #15765 from seamuslee001/5.20

dev/core#1376 Catch Invalid Argument Exception when looking for APIv4…

dev/core#1376 Catch Invalid Argument Exception when looking for APIv4 Services