Merge pull request #17867 from MegaphoneJon/mailing-70

mailing#70 Don't create users for test mail if user doesn't have permission

Merge pull request #17879 from colemanw/relCacheApi

Add APIv4 and pseudoconstants for RelationshipCache

Merge pull request #17580 from samuelsov/dev/core#1670

dev/core#1670 copy custom fields from master to shared address

Merge pull request #17560 from eileenmcnaughton/import

[Ref] Unit test attempt to create reported bugs , minor cleanup

Merge pull request #17897 from seamuslee001/setver_new

Update version in the test_data_second_domain file and also update th…

Merge pull request #17898 from demeritcowboy/smartgroupmarker

dev/core#1888 - Fix one line in PR 17888

Update version in the test_data_second_domain file and also update the setVersion script to update the file version as necessary

Set setting indicating system has been installed after loading in the 2nd domain

somehow messed up a line in revert ba2ad46

[Ref] Unit test attempt to create reported bugs , minor refactor

This lays the ground work to test a couple of reported bugs. Minor cleanup of code

Merge pull request #17896 from eileenmcnaughton/pdfletter

Be a little less supportive to cvs

Merge pull request #17888 from demeritcowboy/groups-warnings

dev/core#1888 and dev/core#1885 - Fatal error on advanced search and warnings and missing group display on contact form

Be a little less supportive to cvs

Merge pull request #17808 from colemanw/bridge

APIv4 - Specify BridgeEntities to assist with joins

Merge pull request #16736 from ixiam/dev_report_issue#27

Contribution Summary Report: Taking the currency filtered in the "gen…

Merge pull request #17891 from mattwire/eventcartmenu

Event Cart ext: Move menu entries to extension

Merge pull request #17895 from civicrm/5.28

5.28

Merge pull request #17892 from demeritcowboy/tests-outputreport

[Test Framework] - Tests for report downloads

Merge pull request #17890 from christianwach/lab-core-1889-3

Prevent session from starting during WordPress pseudo-cron procedures

Merge pull request #17866 from colemanw/customFix

dev/core#1578 - Fix APIv4 chaining with custom fields

tests for report downloads

Move menu definitions to eventcart extension

Merge pull request #17882 from eileenmcnaughton/jaap

Unit test for #17361

Merge pull request #17885 from mattwire/eventcartext_forms

EventCart ext: Cleanup and move form components to ext

Merge pull request #17884 from mattwire/eventcartext_meta

EventCart ext: Fix autogenerated code, remove unused hooks, update readme

Prevent session from starting during WordPress pseudo-cron procedures

Merge pull request #17828 from eileenmcnaughton/matt

Load contribution page if live payment processor is disabled but test is available

Merge pull request #17887 from colemanw/searchDebug

Search debug

revert PR 13958

Search Ext: fix parseExpr field prefix

Search Ext: Add api params to debug section

Cleanup and move eventcart form components to ext

Fix autogenerated code, remove unused hooks, update readme

Merge pull request #17361 from jaapjansma/dev-1767

dev/core#1767 Fix phone key parsing in CRM_Dedupe_Finder

Unit test for #17361

Merge pull request #17880 from colemanw/headerComments

[NFC] Fix nonstandard header comments

Merge pull request #17480 from tunbola/email-template-perms

dev/core#1751: [Create Email] Only Show Update/Save Template when User has Permission to Edit Templates

Fix nonstandard header comments

Add APIv4 and pseudoconstants for RelationshipCache

Merge pull request #17874 from colemanw/checkPermShort

Use new checkPermissions shorthand in api calls

Merge pull request #17878 from civicrm/5.28

5.28

Merge pull request #17846 from mattwire/optimiseclearcaches

Simplify flushing group contact cache query to reduce table locking and improve performance

Merge pull request #17806 from demeritcowboy/mail-report-csv-bom

dev/core#1869 - Include BOM in attachment when sending CSV CiviReport via mail_report job

Merge pull request #17875 from colemanw/customFieldVersionCheck

Ensure serialize field exists before adding to query

Ensure serialize field exists before adding to query

Merge pull request #16838 from mlutfy/core1280

dev/core#1280 Fix ContributionPage soft_credit translation

Use new checkPermissions shorthand in api calls

Merge pull request #17872 from eileenmcnaughton/ids

[NFC] Comment block cleanup

dev/core#1280 Test for ContributionPage soft_credit translation

[NFC] Comment block cleanup

Merge pull request #17524 from mattwire/memberbaocreate

REF Extract addToRecentItems from membership create

Merge pull request #17526 from mattwire/frontendrequiredpaymentfrequency

Disable frequency/interval fields if not required. Mark required if they are so they are validated before submit

Merge pull request #16298 from mattwire/membership_disabled

dev/membership#18 Enhance parameters for Job.process_membership

Merge pull request #17703 from mattwire/contributionpage_currency

Fix currency symbol for Total Amount on contribution page

Merge pull request #17781 from totten/master-vortex-max

RelationshipCache - Add a high-level index to facilitate relationship queries (more fields)

Merge pull request #17871 from seamuslee001/deprecated_jquery

[REF] Fix a couple of jQuery errors that have cropped up

Merge pull request #17831 from totten/master-no-preboot

Hooks/Dispatcher - Close loopholes that occur around "preboot" hooks

Merge pull request #17834 from colemanw/api4perm

APIv4 - Add shorthand for setCheckPermissions()

Merge pull request #17870 from eileenmcnaughton/line

Use PrematureExit exception instead of weird hack in tests

Merge pull request #16429 from ixiam/dev/core#1113

dev/core#1113 - Decimal Separator - Invalid value "total_amount" (NaN,N) creating or editing a membership

Merge pull request #17869 from totten/master-regen-zipcode

(REF) regen.sh - Remove unusual handling of `zipcodes.mysql`

Merge pull request #17863 from demeritcowboy/be-greener

dev/report#43 - Icon after saving a civireport instance is misleading

[REF] Fix a couple of jQuery errors that have cropped up

Merge pull request #17823 from colemanw/upgradeSafe

Remove unnecessary try/catch per #17729

Use PrematureExit exception instead of weird hack in tests

Passing is_unit_test to a function to avoid the civiExit routine has been superceded by catching
PrematureExitExceptions

Merge pull request #17458 from eileenmcnaughton/export

dev/core#1725 Only export primary address fields

Merge pull request #17860 from eileenmcnaughton/test

api_v3_TaxContributionPageTest fix - remove hard coded processor id

Regenerate sql/civicrm_generated.mysql

RelationshipCache - Remove case_id

Rename RelationshipVortex to RelationshipCache

RelationshipVortex - Add to list of example fields in MergerTest::testGetCidRefs

RelationshipVortex - Create and populate table during upgrade (5.29)

RelationshipVortex - Define MySQL trigger to keep data sync'd. Add test. Use syncInto().

RelationshipVortex - Add schema files (incl start_date, end_date, etal)

This is the more maximal variant of the schema which includes mirrors for most fields.

GenCode - Fix pluralization of words ending in x

Merge pull request #16626 from pradpnayak/lineItemFixes

Fixed DB Error: syntax error if line item refers to civicrm_case

(REF) regen.sh - Remove unusual handling of `zipcodes.mysql`

Overview
--------

This is a small refactoring in service of the broader goal of simplifying
the sample-data generation and incorporating it into the installer (where it
can be localized/configured and where we don't have to commit any blobs).

The current process revolves around `regen.sh` and `civicrm_generated.mysql`.

Before
------

`regen.sh` builds an example DB and dumps the content into `civicrm_generated.mysql`.

As part of this, it loads `zipcodes.mysql` into the example DB,
drops/creates a table `zipcodes`, then runs some setup work,
and then re-drops `zipcodes`.

This is only safe on an isolated database. If you're trying to run it on
a pre-existing database created by `$cms`, then you don't know if there's going
to be a conflict on `zipcodes`.

After
-----

It doesn't drop/create/drop the `zipcodes` table. It just reads from the file.

Comment
-------

The main change is fairly small.  It looks big because `zipcodes.mysql` as
to be reformatted as `zipcodes.json`.

Merge pull request #17865 from colemanw/summary

Fix potential js error on summary screen when reloading blocks

api_v3_TaxContributionPageTest fix - remove hard coded processor id

Merge pull request #17864 from colemanw/searchFix

Search Ext: fix loading options and parsing custom field names

mailing#70 Don't create users for test mail if user doesn't have permission

dev/core#1578 - Fix APIv4 chaining with custom fields

Fix potential js error on summary screen when reloading blocks

Merge pull request #17861 from totten/master-dao-event

EventCart - Resolve BAO identity and uncommitted DAO changes

Search Ext: fix loading options and parsing custom field names

Merge pull request #17862 from colemanw/jobCleanup

[REF] ScheduledJob cleanup, remove unused var

ScheduledJob cleanup, remove unused var

tests for processReportMode

icon in status message is misleading

Merge pull request #17855 from seamuslee001/dev_core_1090

dev/core#1090 Update extendedSerializeData to use the Backbone namesp…

Merge pull request #17835 from demeritcowboy/test-contact-form

dev/core#1874 - Failing test for new Individual form

EventCart - Resolve BAO identity and uncommitted DAO changes

Overview
--------

This fixes a recent issue in which `setup.sh` is producing local-only changes to the `Cart`
and `EventInCart` DAO files.

To reduce ambiguity, I'm rephrasing the normal "Before/After" and giving a 3-step evolution/history.

Evolution
---------

(1) Traditionally, the `Cart`/`EventInCart` entities have both BAO+DAO in main source-tree.

(2) With #17743 (685bf3d218a9dad7e90d24aec165c63df12c1f67), the BAO moved away. But then
we get some detritus in the code-tree whenever you run `setup.sh`.

(3) With this patch, we don't get the detritus anymore.

Technical Details
-----------------

When I ran `setup.sh -g`, it produced local modifications like this:

```
diff --git a/CRM/Event/Cart/DAO/Cart.php b/CRM/Event/Cart/DAO/Cart.php
index 4683add13d..3a7f721dc1 100644
--- a/CRM/Event/Cart/DAO/Cart.php
+++ b/CRM/Event/Cart/DAO/Cart.php
@@ -94,7 +94,7 @@ class CRM_Event_Cart_DAO_Cart extends CRM_Core_DAO {
           'where' => 'civicrm_event_carts.id',
           'table_name' => 'civicrm_event_carts',
           'entity' => 'Cart',
-          'bao' => 'CRM_Event_Cart_BAO_Cart',
+          'bao' => 'CRM_Event_Cart_DAO_Cart',
           'localizable' => 0,
           'add' => '4.1',
         ],
```

(Let's ignore the cyclic dependency in this design - that's a messier/pre-existing
problem.)

That metadata is inaccurate.  The `bao` should point to the BAO, sinec the BAO does exist.

It's inaccurate because the BAO exists in a different location.

This patch allows you override the file-existence check -- saying, "this entity will
use BAO's even though they're not in the file-location you might normally expect."

Merge pull request #17858 from civicrm/dependabot/npm_and_yarn/lodash-4.17.19

Bump lodash from 4.17.15 to 4.17.19

Load contribution page if live payment processor is disabled but test is available

Unit test for https://github.com/civicrm/civicrm-core/pull/16860

Merge pull request #17859 from civicrm/5.28

5.28

Merge pull request #17836 from seamuslee001/dev_core_1874

dev/core#1874 Fix fatal error by passing through the string formatTyp…

Merge pull request #17841 from seamuslee001/eventcart_setting

[REF] Migrate Event Cart Setting into the Extension

Bump lodash from 4.17.15 to 4.17.19

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.15...4.17.19)

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #17856 from civicrm/5.28

5.28

Merge pull request #17851 from alifrumin/5.28rn

[NFC] 5.28 Release Notes First Pass