Improve error handling on IPN

https://civicrm.stackexchange.com/questions/37277/paypal-standard-payments-are-being-accepted-but-marked-as-incomplete-transaction/37279#37279

shows how unhelpful this error is - getting data from the exception should help.

Targetting 5.28 in case the gitlab relates to a regression & we need to solicit more debug info

Merge pull request #18053 from seamuslee001/test_ports

[NFC] Port some recent test fixes from master to 5.28

(REF) WebsiteTest - Mitigate flaky failures

Overview
--------

In recent days, api_v3_WebsiteTest has emitted sporadic failures like this:

```
api_v3_WebsiteTest::testDeleteWebsite with data set #0 (3)
Failed asserting that 3 matches expected 0.

/home/jenkins/bknix-max/build/build-2/web/sites/all/modules/civicrm/tests/phpunit/api/v3/WebsiteTest.php:75
/home/jenkins/bknix-max/build/build-2/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:209
/home/jenkins/bknix-max/extern/phpunit7/phpunit7.phar:615
```

and

```
api_v3_WebsiteTest::testDeleteWebsiteInvalid with data set #0 (3)
Failed asserting that 4 matches expected 1.

/home/jenkins/bknix-max/build/build-2/web/sites/all/modules/civicrm/tests/phpunit/api/v3/WebsiteTest.php:88
/home/jenkins/bknix-max/build/build-2/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:209
/home/jenkins/bknix-max/extern/phpunit7/phpunit7.phar:615
```

These failures do not reproduce for me in isolation.

Before
------

Both the failing assertions make an implicit assumption that the baseline content of `civicrm_website` is empty.

After
-----

The failing assertions use an explicit baseline (`$beforeCount`).

Comments
--------

The test failures are sporadic and only seem to seem occur when run in the full suite.

My theory is that something else is leaking `civicrm_website` records;
however, it's hard to track that down amidst a full suite (when the full
suite takes so long to execute).  Therefore, I cannot be certain that this
is actually fixes the problem.  However, this really just tightens up the
assumptions of the test - as long as it passes the PR tests, it should be
safe to merge and then watch in the `CiviCRM-Core-Matrix`.

Fix for failing test

BY ensuring join_date is in the past we get away from situations where there is no valid status

re-re-fix test

Re-fix test

The strtotime calculation adds 4 months before setting the day of month. However

July 31 + 4 months is 1 Dec - ie the month is 12 not 11 due to there being only 30 days. So to
get 27 Nov we need to get the July month (7) and add 4 and voila 11, not 12

update failing test

make test less time-sensitive

api_v3_TaxContributionPageTest fix - remove hard coded processor id

Merge pull request #18017 from seamuslee001/dev_mail_72

dev/mail#72 Remove call to custom fatal error handler from CRM_Core_E…

dev/mail#72 Remove call to custom fatal error handler from CRM_Core_Error::debug_log_message

Merge pull request #18011 from totten/5.28-tbl

FiveTwentyEight - Provide concrete details about civicrm.files

FiveTwentyEight - Provide concrete details about civicrm.files

This hopefully makes it easier to decide what to do without needing a
scavenger hunt.

Merge pull request #17983 from kcristiano/5.28-wp-notices

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Merge pull request #17868 from seamuslee001/remove_civicrm_files_override

[REF] Remove civicrm.files override for WordPress to fix issues with…

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Signed-off-by: Kevin Cristiano <kcristiano@kcristiano.com>

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

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

default not being set for price fields

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

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

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

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

dev/core#1903 - Avoid E_WARNING and remove code

avoid E_WARNING and remove code

Merge pull request #17925 from seamuslee001/5.28

Add release-notes/5.27.3

Add release-notes/5.27.3

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 #17890 from christianwach/lab-core-1889-3

Prevent session from starting during WordPress pseudo-cron procedures

Prevent session from starting during WordPress pseudo-cron procedures

Merge pull request #17875 from colemanw/customFieldVersionCheck

Ensure serialize field exists before adding to query

Ensure serialize field exists before adding to query

[REF] Remmove civicrm.files override for WordPress to fix issues with users struggling to find extensions

Merge pull request #17836 from seamuslee001/dev_core_1874

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

Merge pull request #17851 from alifrumin/5.28rn

[NFC] 5.28 Release Notes First Pass

Merge pull request #17857 from demeritcowboy/membershiptest-5.28

[Test framework] - Backport of membership test fix

Merge pull request #17854 from demeritcowboy/missing-codeversion-5.28

dev/core#1882 - Missing version number in status check message about db version vs code version

backport of membership test fix

Merge pull request #17853 from seamuslee001/dev_wordpress_62

dev/wordpress#62 Update adrienrn/php-mimetyper gitignore file to ensu…

missing version number in message

5.28 release notes: first pass

dev/wordpress#62 Update adrienrn/php-mimetyper gitignore file to ensure that sites that manage their systems using git can access the db.json file

Merge pull request #17840 from seamuslee001/5272_rns

5.27.1 and 5.27.2 Release Notes

Add release-notes/5.27.1.md

Update 5.27.2.md

Remove dev/core#1847 - the initial fix caused other regressions, and the full fix merits a longer incubation

5.27.2 - Add release notes

dev/core#1874 Fix fatal error by passing through the string formatType rather than the date format when adding datepicker fields from metadata

Merge pull request #17820 from seamuslee001/1867_qill

dev/core#1867 Ensure that the qill matches the filter when no time is…

dev/core#1867 Ensure that the qill matches the filter when no time is supplied

Merge pull request #17811 from eileenmcnaughton/fromto

dev/core#1867 CiviReport filters: Fix incorrect defaulting to the beginning rather than end the to-range day

Merge pull request #17814 from eileenmcnaughton/dedupe

Fix dedupe on same-activity-connection

Fix dedupe on same-activity-connection

Merge pull request #17803 from eileenmcnaughton/dedupe

dev/core#1862 Skip cache tables during merge, rely on cache management processes

Fix incorrect defaulting to the beginning rather than end the to-range day

dev/core#1862 Skip cache tables during merge, rely on cache management processes

We later call Contact.create on both contacts so that should manage the caches for this update 'as well as for any other update'
and solve the bug + reduce locking queries

Merge pull request #17797 from seamuslee001/dev_core_1865

dev/core#1865 Remove civiwp and option query strings from redirected …

Merge pull request #17762 from eileenmcnaughton/date

dev/core#1847 Fix datepicker to respect the searchDate offsets

dev/core#1847 Fix datepicker to respect the searchDate offsets

I have some misgivings about this since I suspect the rationale behind the search offsets was primarily about the clunky UI.

Having said that I think the datepicker UI is a bit clunky on this front too as it's not obvious you can choose earlier
dates. This does at least restore established behaviour.

https://lab.civicrm.org/dev/core/-/issues/1847

Merge pull request #17798 from seamuslee001/dev_core_1864

dev/core#1864 Ensure that when search in trash is not checked that on…

Merge pull request #17792 from agh1/5.28.0-releasenotes-initial

5.28.0 release notes initial run

dev/core#1865 Remove civiwp and option query strings from redirected url as they only relate to CiviCRM internal uses

dev/core#1864 Ensure that when search in trash is not checked that only contacts not deleted are returned in the advanced search form

5.28.0 release notes: added boilerplate

5.28.0 release notes: raw from script

Merge pull request #17779 from seamuslee001/nfc_php74_flexmailer

[NFC] Update flexmailer to be PHP7.4 compatible

[NFC] Update flexmailer to be PHP7.4 compatible

Merge pull request #17772 from seamuslee001/dev_core_1860

dev/core#1860 Fix installing on PHP7.3 Ubuntu 20.04 and MariaDB 10.3

Merge pull request #17760 from seamuslee001/joomla_track_url

Ensure that when normal routing trackable urls and opens in Joomla go…

dev/core#1860 Fix installing on PHP7.3 Ubuntu 20.04 and MariaDB 10.3

Merge pull request #17768 from seamuslee001/flexmailer_distmaker

[NFC] Include flexmailer in distmaker

[NFC] Include flexmailer in distmaker

Only encode url if it is the front end in unit tests

Fix function declaration to ensure that passing in frontend variable doesn't cause htmlizing by accident in unit tests

Ensure that when normal routing trackable urls and opens in Joomla go to the frontend not administrator site

Merge pull request #17748 from demeritcowboy/double-vision

Pre-upgrade message for php-intl shows twice

only show message once

Set version to 5.28.beta1

Merge pull request #17733 from eileenmcnaughton/pvv

Set locales for all domains when enabling multilingual

Merge pull request #17721 from colemanw/isDeletedPerm

Add permission metadata to contact is_deleted field

Merge pull request #17738 from colemanw/upgradeLocales

dev/core#1852 Ensure all domains have the same value for locales

Ensure that we are testing on multiple domains

Co-authored-by: colemanw <coleman@civicrm.org>

Merge pull request #17739 from colemanw/byeByeWiki

[NFC] Fix some outdated wiki links

Fix some outdated wiki links

Add permission metadata to contact is_deleted field

Merge pull request #17737 from christianwach/lab-core-1851

Fix saving of State/Province Multi-select values

Ensure all domains have the same value for locales

Fix saving of State/Province Multi-select values

Merge pull request #17736 from civicrm/5.27

5.27

Merge pull request #17731 from agh1/5.27.0-releasenotes-final

5.27.0 release notes: final edits

Merge pull request #17735 from totten/master-field-ver

API/DAO Metadata - Retain versioning metadata for possible usage in runtime+tooling

Update 5.27.0.md

Update 5.27.0.md

Set locales for all domains when enabling multilingual

In the process of trying to test a possible wordpress regression I found that enabling
mutlilingual on a domain other than domain 1 breaks the site. The reason is that without
only domain 1 is being updated with the new language and locales. Since I'm on domain
2 when I do this I can no longer load any page as the locales value has not been set
and the queries look for option_value.label not option_value.label_us.

Since 'no locale' is no longer valid the locale needs to be set on all sites. Note the
code change is primarily an extraction inn order to only run the table update part once

Merge pull request #17732 from civicrm/5.27

5.27

Merge pull request #17722 from eileenmcnaughton/up

dev/core#1846 Regression (sorta) Hacks to avoid failure on upgrade due to new field not yet being in the DB

API/DAO Metadata - Retain versioning metadata for possible usage in runtime+tooling

[NFC] Update Domain version number in tests to be at test v5.27.alpha1

Merge pull request #17725 from eileenmcnaughton/pvv

Ensure pricefield value label column is not null on upgrade

Ensure that the unexpected version check doesn't show if the version matches