[REF] Follow up cleanup

This removes an IF block that relies on the always-true parameter

I'm pretty sure it just wasn't done last round as all the white space created a lot of noise

Search extension - improve results display

Merge pull request #17783 from pradpnayak/REF-1

getLoggedInContactID() is a static function

Merge pull request #17782 from civicrm/5.28

5.28

Merge pull request #17744 from eileenmcnaughton/repeat

dev/core#1679: Ensure Paypal IPN always updates the next scheduled payment date

Merge pull request #17749 from totten/master-setup

Support for installing CiviCRM-Backdrop via "setup" UI

getLoggedInContactID() is a static function

Merge pull request #17779 from seamuslee001/nfc_php74_flexmailer

[NFC] Update flexmailer to be PHP7.4 compatible

Merge pull request #17774 from civicrm/5.28

5.28

Search extension - blur auto toggle on click

Rename search base page class

Merge pull request #17765 from colemanw/dev/core#1853

dev/core#1853 - Fix validation errors when removing contact subtype

Rename searchBuild to less ambiguous Create Search

Merge pull request #17775 from colemanw/search

Add search extension

[NFC] Update flexmailer to be PHP7.4 compatible

Add search extension

Merge pull request #17636 from MikeyMJCO/patch-5

[dev/core#750] Don't check server variables if we're running in CLI

Merge pull request #17763 from eileenmcnaughton/dps

Remove PaymentExpress ipn class

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 #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 #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 #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

Setup UI - Allow callers to wrap UI in their own page chrome. Clarify response format.

The general idiom for calling SetupController is like this:

```php
$ctrl = \Civi\Setup::instance()->createController()->getCtrl();
$ctrl->setUrls(...);
\Civi\Setup\BasicRunner::run($ctrl);
```

This changes the data exchanged between `BasicRunner` and `SetupController`
to allow more flexible runners. The `BasicRunner` is the only one currently
used by existing installers, so it's drop-in compatible.

Before
------

The `BasicRunner` calls `$ctrl->run(...)` and gets back an array with the response data
(`[0 => $headers, 1 => $body]`).

After
-----

The `BasicRunner` calls `$ctrl->run(...)` and gets back a `SetupResponse`. This
object provides the `[0]` and `[1]` keys (for backward compat), but it's better
to access the object properties (`$headers`, `$body`, etc).

It provides several additional properties - eg `$title`, `$assets`, `$code`.

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

This is a drop-in update for anything that uses `BasicRunner`.

Strictly speaking, if one wrote a different runner, it may or may not be
drop-in compatible.  But since I don't think there are any others, it's not
really worth spending much energy on verifying.

The bump in `Setup::PROTOCOL` (`1.0` => `1.1`) indicates that old installers
should generally be forward compatible with this protocol.  However, if one
writes a new installer and specifically uses the `SetupResponse` type,
then they should assert protocol `1.1`.

setup/ - Fix CSS loading on "finished" page. Split out "page.php".

The various `finished.*.php` files were outtputting the incorrect URL for
`template.css`. The code for outputting `template.css` is duplicated in a few files.

This commit fixes by extracting a helper (`renderPage()` / `page.php`) to encapsulate
that bit and reuse it for each page-view in the Setup UI.

Setup UI - Fix translation of post-install messages

Here's what happens:

- The "Setup" subsystem does its internal/partial bootstrap; it sets `$tsLocale` and uses `CRM_Core_I18n`. Hooray.
- The "Setup" initializes the database/settings and then does a full bootstrap. The full bootstrap step calls `applyLocale()`, which replaces the active `$tsLocale`... with the *unconfigured default*. Ooops.
- This step runs next - and it updates the DB setting. But we've already trampled `$tsLocale`. Calling `applyLocale()` a second time means we use the preferred `$tsLocale` again. Hooray.

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

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

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

dev/core#1853 - Fix validation errors when removing contact subtype

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

Drupal.civi-setup.php - Don't assume that `$language` is available

finished.Drupal.php - Let it work!

FlushDrupal.civi-setup.php - Update to work with setup UI

FlushBackdrop.civi-setup.php - Update to work with setup UI

finished.Backdrop.php - Fix the FIXMEs. Align closer to finished.Drupal.php

setup/res/template.css - Fix "Sample Data" alignment

This rule appears to be targetted at "Components" section, but it actually affects the entire document
Consequently, the "Localization" and "Sample Data" segments bleed together weirdly.

setup/res/template.css - Twiddle column width

The aim is to have 3 columns (each 1/3 of the screen width) and allow for
some padding in between.  Unfortunately, the current formulation is futzy.
In my browser, when viewing on Backdrop, the columns are just slightly too
wide (causing them to wrap prematurely).

There must be a better way to do this, but it's not my main issue right now.

Twiddling is better than status quo.

SetupController - Need better bootstrap to support localization

While trying the setup UI on Backdrop, I found that changing the language
led to a crash.  The problem: it needed to load the `l10n` files, which led
to more path resolution, which led to failures - because `civicrm.settings.php`
hasn't been loaded. (It doesn't exist yet!)

Fortunately, most of the needful information is available.  The
`Setup::init()` should have found the info and put it into `$model`.  We
just need to load the configuration into the right globals/constants.

SetupController - If you install and then reload the install screen, re-render the finish message.

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'

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

Remove unused, deprecated functions

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

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

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

dev/core#1679: Ensure Paypal IPN always updates the next scheduled payment date

This is from https://github.com/civicrm/civicrm-core/pull/16952 and https://github.com/civicrm/civicrm-core/pull/17028

is to not change the scheduled date if it seems to have already been intentionally changed. However, per #16952
time stamps can lead to 'now' being a day in the future - so this loosens the check from equals today to
is less then now + 1 day

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