dev/financial#139 prep: standardize markup, closing spans on bulk payment form

dev/financial#139 tests: contribution receive_date shouldn't change when payments received

Merge pull request #17974 from eileenmcnaughton/receipt

[REF] Minor code clean up

Merge pull request #17976 from civicrm/5.28

5.28 to master

Merge pull request #17973 from seamuslee001/ref_profile_date_fields

[REF] Fix regression where adding any date based field onto a profile…

[REF] Fix regression where adding any date based field onto a profile triggers an error date preferences not configured when previewing the profile

[REF] Minor code clean up

This just consolidates a little of the code used to determin if an email receipt should be sent.
It's clear that there is more code above that could be pulled into this & combined
with the isEmailReceipt function but I wanted to just offer up a small reviewable change unless
there is reviewer enthusiasm for more

Merge pull request #17967 from eileenmcnaughton/grant

[REF] Grant cleanup

Merge pull request #17965 from seamuslee001/widget_test

[NFC] Add in a unit test of calling the contribution page widget endp…

[REF] Grant cleanup

This does a couple of things to cleanup the Grant BAO -
1) moves money handling back to the form layer
2) removes unnecessary date handling - the format that comes from the form layer
is now supported by the BAO.

I was looking to work on another but but then figured I needed a v4 api before
writing the tests and then realised we shouldn't do a v4 api before getting
rid of the legacy money handling.

It does occur to me it might make more sense to start moving to an
extension before doing any of the other steps

[NFC] Add in a unit test of calling the contribution page widget endpoint and remove unneded file

Merge pull request #17963 from civicrm/5.28

5.28

Merge pull request #17962 from seamuslee001/dev_core_1909_profile

dev/core#1909 Fix E-notice when adding a field on a profile

Merge pull request #17964 from seamuslee001/dev_core_1909_paymentprocessor

dev/core#1909 Fix e-notice when adding a payment processor

dev/core#1909 Fix e-notice when adding a payment processor

Merge pull request #17960 from demeritcowboy/contribution-default

dev/core#1911 - Default not being set for fixed contribution amounts or any price field that is not type text

dev/core#1909 Fix E-notice when adding a field on a profile

Merge pull request #17959 from demeritcowboy/file-case-notice

dev/core#1909 - E_NOTICE opening file-on-case

Merge pull request #17941 from eileenmcnaughton/fieldspec

[Ref] Simplify field reference

default not being set for price fields

E_NOTICE opening file-on-case

Merge pull request #17957 from demeritcowboy/combine-createcase

[Test framework] - Combine triplicate createCase functions

Merge pull request #17932 from seamuslee001/ref_radio

[REF] Refactor to use the standard CRM_Core_Form::addRadio function f…

combine duplicate functions

Merge pull request #17948 from seamuslee001/mail_mime_upgrade

Upgrade PEAR/mail_mime package to be compliant with PHP7.4 and deploy it using composer

Merge pull request #17938 from seamuslee001/ref_fix_financial_account

[REF] Fix the default to_financial_account_id for generated transactions

Merge pull request #17955 from colemanw/superstar

APIv4 - Add keyword to select all custom fields

Upgrade PEAR/mail_mime package to be compliant with PHP7.4 and deploy it using composer

Merge pull request #17946 from seamuslee001/ref_dompdf_upgrade

[REF] Upgrade dompdf version to be more compatible with PHP7.4

APIv4 - Add keyword to select all custom fields

Merge pull request #17939 from eileenmcnaughton/ex_meta

[REF] [Tests] Cleanup test declaration to take advantage of mapping improvements

Merge pull request #17914 from demeritcowboy/invisible-messages

dev/drupal#127 - CRM_Core_Session::setStatus() gets ignored sometimes

Merge pull request #17953 from civicrm/5.28

5.28 to master

Merge pull request #17947 from agh1/dev-core-1899-5-28

dev/core#1899 specify display mode for action links with icons

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

dev/core#1899 specify display mode for action links with icons

[REF] Upgrade dompdf version to be more compatible with PHP7.4

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.

[Ref] Simplify field reference

Remove error checking by-pass in tests

[REF] [Tests] Cleanup test declaration to take advantage of mapping improvements

[REF] Fix the default to_financial_account_id for generated transactions

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

[REF] Refactor to use the standard CRM_Core_Form::addRadio function for a number of radio elements

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

avoid messages not being shown

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