Merge pull request #17949 from eileenmcnaughton/export_im

[REF] Remove unnecessary complexity on im export

[REF] Remove unnecessary complexity on im export

This just removes a couple of lines that were already marked as being likely unhelpful. Because the impact of
these lines is tested the test needs tweaking too - but it only affects the name of the fields in the temp
table and in internal php keys

Merge pull request #17945 from colemanw/commentCleanup

NFC - Docblock cleanup

Merge pull request #17944 from colemanw/caseCheck

CRM_Utils_Check_Component_Case - Guard against post-upgrade crash

NFC - Expand CRM-* comments with full @see annotations

NFC - Fix @return annotation in CRM_Utils_Check_Component_*

CRM_Utils_Check_Component_Case - Guard against post-upgrade crash

Merge pull request #17940 from eileenmcnaughton/abort

Remove error checking by-pass in tests

Merge pull request #17936 from eileenmcnaughton/cache

Remove extraneous opportunistic cache flush.

Remove error checking by-pass in tests

Remove extraneous opportunistic cache flush.

The opportunistic cache flush is already called by contact create and then again by each of the places
where the contact might be added to a group so this is just extraneous

Merge pull request #17930 from twomice/1902_contrib_source_profile

dev/core#1902: "Contribution Source" profile field has no effect

Merge pull request #17928 from colemanw/groupContactCleanup

[REF] GroupContact BAO - Minor code cleanup

Merge pull request #17931 from civicrm/5.28

5.28 to master

Merge pull request #17929 from demeritcowboy/required-not-5.28

dev/core#1903 - Avoid E_WARNING and remove code

dev/core#1902: "Contribution Source" profile field has no effect on new contribution.

avoid E_WARNING and remove code

GroupContact BAO - Minor code cleanup

Merge pull request #17922 from eileenmcnaughton/group_contact

[REF] Minor function signuture cleanup

Merge pull request #17921 from eileenmcnaughton/format

[REF] Do not pass variable by reference

Merge pull request #17926 from civicrm/5.28

5.28

[REF] Minor function signuturre cleanup

 is never altered so it is unnecessary to pass by reference. visibility is just
used to set ignorePermissions

Merge pull request #17925 from seamuslee001/5.28

Add release-notes/5.27.3

Add release-notes/5.27.3

Merge pull request #17918 from mlutfy/docUpdates

[NFC] Update a few doc/wiki links in code comments

[REF] Do not pass variable by reference

This variable is not altered in the function so does not need to be passed by reference

Merge pull request #17916 from colemanw/domainCache

Improve caching of current domain

Merge pull request #17778 from totten/master-setup-validate

Setup UI - Validate that at least one "Component" is enabled

Merge pull request #17911 from agh1/memberdetailreportautorenew

Member detail report: nest "in" options in parentheses

Update a few doc/wiki links in code comments

APIv3 - If requesting current domain, read from cache

Use sensible caching for current domain

Merge pull request #17917 from christianwach/lab-core-1901

Fix sticky table header on "Find Activities" page

Fix sticky table header on "Find Activities" page

Member detail report: nest "in" options in parentheses

Merge pull request #17908 from civicrm/5.28

5.28

Merge branch 'master' into 5.28

Merge pull request #17906 from seamuslee001/lab_core_1846

dev/core#1846 Make DAO upgrade safe

Fix failing settingTest and mark setting.fill as deprecated

The settingTest class was being too aggressive about creating and deleting domains,
this teaches it to not delete pre-existing domains.
Also marks an old unused api function deprecated.

Add upgrade-safe DAO::getSupportedFields method

Switches api v3 and v4 to use that method so they are upgrade-safe by default.

Merge pull request #17903 from eileenmcnaughton/format

[REF] remove unnecessary variable variables

Merge pull request #17901 from demeritcowboy/outputhandler2

dev/core#1855 - Allow different output formats for CiviReport results and untangle code

Merge pull request #17650 from colemanw/getMultilingual

[REF] - Add helper function for the repetitive task of fetching multilingual

Merge pull request #17905 from MegaphoneJon/fix-dummy-processor

[unreleased regression] Dummy processor now is ID 7 on buildkit sites

[REF] - Add helper function for the repetitive task of fetching multilingual

Adds the function CRM_Core_I18n::getMultilingual() and replaces a bunch of
places where the work was being done manually.

This also fixes a suspected bug where multiple domains may not have been compatible
with multiple languages, as the current domain id was previously ignored,
but is respected in the helper function.

Merge pull request #17904 from christianwach/lab-core-1890

Remove unused "ufUniqID" session variable

Dummy processor now is ID 7 on buildkit sites

Merge pull request #17900 from aydun/wiki_changes

Replace a load of references to the wiki with docs links

Remove unused "ufUniqID" session variable

Replace a load of references to the wiki with docs links
Change docs links to 'latest' not 'stable'

Merge pull request #17902 from demeritcowboy/test-activity-qill

dev/core#1894 - Make CRM_Activity_Form_SearchTest::testQill less time-sensitive

[REF] remove unnecessary variable variables

For a brief but terrifying spell this arvo I delved into this function to try to understand
the purpose of 'preserveDBName'. Obviously I ran screaming from the room but I thought I could disarm these
few lines to slighly file down the fangs of the beast.

Code is heavily covered by tests

make test less time-sensitive

Allow extensions to provide their own formats and clean up tangled code

Merge pull request #17827 from eileenmcnaughton/utf_import

dev/core#183 Use TempTable builder to generate table for import

Merge pull request #17771 from mattwire/ufmatch_validemail

Remove check for valid email in synchronizeUFMatch

Merge pull request #17881 from eileenmcnaughton/activity

Call apiv4 from Contribution create rather than fugly addActivity function

Merge pull request #17899 from colemanw/basicEntity

APIv4 - Add BasicEntity helper class

Call apiv4 from Contribution create rather than fugly addActivity function

I took a look at https://github.com/civicrm/civicrm-core/pull/17274 which has blocking test failures - but felt that
the shared function was adding nothing and simply using the api to create the activity made more sense. The
shared function does a lot of silly wrangling for very little shared functionality and is
hard to read. In this call only 2 params are passed in - so most of the wranglingg
doesn't apply anyway. I ensured the 2 JIRA issues referenced in the removed code have test cover (one already
had a test written by Monish & I added in the campaign check

dev/core#183 Fix import csv to use temp table builder

Use TempTable builder to generate table for import

Merge pull request #17867 from MegaphoneJon/mailing-70

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

APIv4 - Add BasicEntity helper class

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