Merge pull request #17729 from colemanw/upgradeSafeApi

Make api get upgrade-safe

Merge pull request #17799 from civicrm/5.28

5.28

Merge pull request #17820 from seamuslee001/1867_qill

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

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

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

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

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

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

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

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 #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 #17762 from eileenmcnaughton/date

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

Merge pull request #17789 from eileenmcnaughton/search

Add hidden tag to search extension

Merge pull request #17788 from eileenmcnaughton/update

[REF] Follow up cleanup

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 #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 #17798 from seamuslee001/dev_core_1864

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

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

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

5.28.0 release notes initial run

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

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

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

5.28.0 release notes: added boilerplate

5.28.0 release notes: raw from script

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.