Merge pull request #17763 from eileenmcnaughton/dps

Remove PaymentExpress ipn class

Merge pull request #17776 from totten/master-org-check

Status Checks - Use more specific label regarding "Domain"/"Organization" check

Status Checks - Use more specific label regarding "Domain"/"Organization" check

Before
------

Suppose you install CiviCRM anew.  The installer shows a bulleted list of
things to do.  One of those takes you to another, much longer list of things
to do.  And *then* there's a popup on the sidebar which says you need to
"Complete Setup".

You say to yourself, "I thought that's what I was doing!" But you click it anyway.

That takes you to a page with another list of things to do, one of which is
to "Complete Setup".  The text and link clarify that "Complete Setup"
actually means *set the name and address of the organization*.

After
-----

There are still several different pages telling you what to do after setup.

But at least the label is more precise. :)

Merge pull request #17770 from civicrm/5.28

5.28 to master

Merge pull request #17768 from seamuslee001/flexmailer_distmaker

[NFC] Include flexmailer in distmaker

[NFC] Include flexmailer in distmaker

Merge pull request #17754 from demeritcowboy/resourceurl-check

dev/drupal#114 and dev/core#1647 - Remove resource url status check

Merge pull request #17750 from colemanw/upgradeBump

Bump minimum upgradable version to 4.4.7

Bump minimum upgradable version to 4.4.7

Merge pull request #17767 from demeritcowboy/loss-of-face

[REF] Unused interface CRM_Report_Interface

Merge pull request #17256 from demeritcowboy/test-for-16559

[TEST] Failing test for PR 16559

unused file

Merge pull request #17764 from jaapjansma/caseActivity

update to pr 16559

update to pr 16559

Merge pull request #17761 from eileenmcnaughton/acl

Remove unused, deprecated functions

Merge pull request #17745 from artfulrobot/artfulrobot-findfiles

Improve efficiency of findFiles

Payment express - remove all other traces, bar the main class (pending further discussion)

Remove PaymentExpress ipn class

I was able to establish that this was accidentally broken five years ago with this commit
https://github.com/civicrm/civicrm-core/commit/204c86d59f0cfc4c4d917cc245fb41633d36916e#diff-b00e65c9829c27da8b34e35f2e64d9b6L114

Resulting in the ipn url of
http://dmaster.local/sites/all/modules/civicrm/extern/pxIPN.php?result=00003100032465271fc8c82d94ad434d&userid=Fuzion_Dev

dying with the fatal error

 Uncaught Error: Call to undefined method CRM_Core_Payment_PaymentExpressIPN::singleton() in /Users/eileenmcnaughton/buildkit/build/dmaster/sites/all/modules/civicrm/CRM/Core/Payment/PaymentExpressIPN.php on line 391

This means we can be sure the ipn code does not work & hence can't be in use and can remove it.

Can we remove the main class too? My thinking is that there could be old instances of the payment processor in
people's installs and that if we remove or de-activate the payment processor instance it could cause
other breakage.

However, I think we could alter the processor type in the upgrade script to have
CRM_Core_Manual as the class_name in place of 'Payment_PaymentExpress'

Remove unused, deprecated functions

These functions were deemed unused in January and deprecation warnings were added at that point.
Time to go

Merge pull request #17746 from colemanw/apiExpHaving

APIv4 Explorer: Improve selection of fields for HAVING

Merge pull request #17756 from demeritcowboy/unused-gencode-2

More unused functions in GenCode

Merge pull request #17758 from mattwire/wpaclpermsphpnotice

Fix PHP notice on wordpress permissions form

Merge pull request #17645 from MegaphoneJon/core-1826

core#1826: Ignore location_type_id when deduping postal address

Fix PHP notice on wordpress ACL permissions form

Merge pull request #17755 from demeritcowboy/test-activity-tag-search

[Unit Test] dev/core#1173 and dev/core#1827 - Test for activity tag search - PR 17655

Merge pull request #17757 from colemanw/karmaTime

Convert CRM.utils.formatDate tests to karma

Convert CRM.utils.formatDate tests to karma

Merge pull request #17655 from samuelsov/lab1827

dev/core#1827 activity search - fixing search by tags

unused functions

test for activity tag search

Merge pull request #17753 from demeritcowboy/unused-gencode

Unused functions in GenCode

Merge pull request #17751 from civicrm/5.28

5.28

Merge pull request #17684 from colemanw/time

Teach CRM.utils.formatDate to also show time

Teach CRM.utils.formatDate to also show time

remove resource url check

unused functions

Merge pull request #17748 from demeritcowboy/double-vision

Pre-upgrade message for php-intl shows twice

Improve efficiency of findFiles

only show message once

Merge pull request #17747 from mlutfy/fixReportGroupbyTpl

Fix typo in templates/CRM/Report/Form/Tabs/GroupBy.tpl

test for ignoring location_type_id on import

Ignore location_type_id for dedupe on import also

Fix typo in templates/CRM/Report/Form/Tabs/GroupBy.tpl

APIv4 Explorer: Improve selection of fields for HAVING

Merge pull request #17741 from eileenmcnaughton/eventcart

Add eventcart shell

Set version to 5.29.alpha1

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

Add eventcart shell

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

Fix logical errors and fix ENV file as well

Merge pull request #17669 from seamuslee001/flexmailer_core

[REF] Ship Flexmailer extension with Core

Merge pull request #17730 from agh1/moneyformatcheckgrammar

dev/core#1494 System check for moneyvalueformat: grammar & clarity

Merge pull request #17727 from MegaphoneJon/fix-lcd

[NFC] Fix contributor key for Lighthouse Consulting and Design

Merge pull request #17728 from agh1/php-intl-upgrade-message

dev/translation#48: fix up system check grammar and add pre-upgrade message.

Ensure pricefield value label column is not null on upgrade

On testing locally I found I did have a null value in a field & this fatalled if I enabled
multilingual

5.27.0 release notes: further edits

dev/core#1494 System check for moneyvalueformat: grammar & clarity

Merge pull request #17726 from colemanw/noWiki

Remove obsolete function formatWikiURL

PHP-Intl: fix up system check grammar and add pre-upgrade message.

Fix contributor key for Lighthouse Consulting and Design

Fix version check for custom field serialize col

Mark unused functions as deprecated and update doc link

Remove obsolete function formatWikiURL

[REF] Ship Flexmailer extension with Core

Hacks to avoid failure on upgrade due to new field not yet being in the DB

On attempting to upgrade I found I couldn't get to the upgrade screen add the new serialize field as
I got fatal errors in these 2 places (maybe more to come).

The new field is civicrm_custom_field.serialize

Merge pull request #17701 from demeritcowboy/name-label-accountrelationship

dev/wordpress#61 - Financial Account by Relationship lookup is incorrectly using label not name

Merge pull request #17602 from mattwire/supportsrecurring

REF Access the method not the property directly to check if a payment processor supports recurring

Merge pull request #17719 from civicrm/5.27

5.27

Merge pull request #17720 from alifrumin/5.27.0-releasenotes

[NFC] 5.27 release notes - first pass

5.27 release notes first pass

Merge pull request #17718 from agh1/5.27.0-releasenotes

5.27.0 release notes - initial run

Merge pull request #17716 from mattwire/setnamesutf8mb4

Connect to database using SET NAMES utf8mb4

Merge pull request #17717 from colemanw/jsComments

Fix CRM_Utils_JS::dedupeClosures to ignore comments

dev/core#1827 activity search - fixing search by tags - simple and tag set

5.27.0 release notes: added boilerplate

5.27.0 release notes: raw from script

Fix CRM_Utils_JS::dedupeClosures to ignore comments

This runs stripComments BEFORE dedupeClosures so that comments don't interfere with the deduping.
Also fixes stripComments to remove the line break as well as the comment line.

SET NAMES utf8mb4

Merge pull request #17714 from colemanw/jsDocblocks

NFC cleanup docblocks in CRM_Utils_JS