Merge pull request #17780 from totten/master-sync-into

CRM_Utils_SQL - Add "onDuplicate()" and "syncInto()" helpers

Merge pull request #17800 from colemanw/cronCheck

CheckEnv - Give new installs a grace period before 'Cron Not Running' msg

Merge pull request #17816 from JKingsnorth/patch-16

Fix 'Undefined variable: jsSet in CRM_Core_BAO_Mapping::loadSavedMapping()'

Merge pull request #17812 from eileenmcnaughton/uff

dev/core#1861 fix failure to unset location_type_id when saving uffield

Merge pull request #17807 from colemanw/nonProdCronCheck

dev/core#1863 Downgrade checkEnvironment level and skip non-prod checks

CheckEnv - Give new installs a grace period before 'Cron Not Running' error

[NFC] Ensure jsSet returned variable exists

dev/core#1861 fix failure to unset location_type_id when saving uffield

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

If this is a regression it probably from 2017
https://github.com/civicrm/civicrm-core/commit/2e74ff5574027b2ce84177b82884ebd7c9b87fd1#diff-1911d0c1a896701ba9aa044181f5972bR556

Merge pull request #15435 from MegaphoneJon/reporting-21

reporting#21 - don't multiple contribution details when a 1-to-many r…

Downgrade checkEnvironment level and skip non-prod checks

Merge pull request #17789 from eileenmcnaughton/search

Add hidden tag to search extension

Merge pull request #17788 from eileenmcnaughton/update

[REF] Follow up cleanup

Merge pull request #17723 from seamuslee001/feature_contribution_detail_no_acl

[REF] Remove ACL join on temp table creation in Member ContributionDe…

Merge pull request #17769 from seamuslee001/lab_core_1858

dev/core#1858 Prevent Duplicate contact records being created and har…

Merge pull request #17801 from colemanw/statusPrefCleanup

REF - Cleanup StatusPreference BAO to be more standard

Merge pull request #17794 from eileenmcnaughton/act_perms

Sort permittedActivityTypes

dev/core#1858 Ensure that no duplicate contact is created by if we have the contactID in the post variable from the contact task form that we map that to contact_id for the dedupe params

Add in a code comment describing the issue

REF - Cleanup StatusPreference BAO to be more standard

Merge pull request #17795 from demeritcowboy/comment-resultset

[REF] Do or do not - there is no try

unnecessary try block

Merge pull request #17683 from agh1/memberdetailreportautorenew

Add auto-renew status to membership detail report

Merge pull request #17791 from colemanw/dev/core#1866

APIv4 - Fix saving custom fields with same name

Sort permittedActivityTypes

When determining whether to add a clause filtering activity types the permitted types
are compared to all types. If all are permitted no clause is needed.

However, without this sort it is incorrectly seeing them as not matching and adding extraneous joins

Merge pull request #17698 from agh1/checkwpbasepage

Add system check to ensure WP base page exists

Fix saving custom fields with same name

Merge pull request #17786 from demeritcowboy/profile-description

dev/core#1868 - Regression - Description field is always blank on profiles admin page and slew of E_NOTICES

Add hidden tag to search extension

This was a suggestion from @mattwire that we hide the extension from the UI until the styling dependency is resovled.

It seems reasonable to me as this is primarily in the civicrm-core repo for code management purposes and
making it too easy to enable before the styling is resolved could lead to confusion.

I would quite like to enable on buildkit builds just for dev visibility - but that is something to think
about

Merge pull request #17787 from pradpnayak/REF-1

Fixed notice error on Relationships report

[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

Fixed notice error on Relationships report

put description back

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

WP base page system check: suggest both the modified default and the default default

CRM_Utils_SQL - Add "syncInto()" helper

This is slightly more sugary variant of "INSERT INTO...SELECT...ON DUPLICATE UPDATE..." which requires
less boilerplate/duplication.

Before
------

```php
CRM_Utils_SQL_Select::from('foo_table')
  ->select(['foo_name', 'foo_value1', 'foo_value2'])
  ->insertInto('bar_table', ['bar_name', 'bar_output1', 'bar_output2'])
  ->onDuplicate(['bar_output1 = foo_value1', 'bar_output2 = foo_value2'])
```

After
------

```php
CRM_Utils_SQL_Select::from('foo_table')
  ->syncInto('bar_table', 'bar_name', [
    'bar_name' => 'foo_name',
    'bar_output1' => 'foo_value1',
    'bar_output2' => 'foo_value1',
  ])
```

CRM_Utils_SQL - Add support for "INSERT INTO...SELECT...ON DUPLICATE UPDATE..."

This is a query structure in which you want to build a SELECT query - and,
using the results, perform a mix of INSERTs and UPDATEs.

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

Member detail report: move autorenew filter to `whereClause()` override

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