Merge pull request #25233 from braders/remove-isHtml

Remove broken method CRM_Utils_File::isHtml()

Remove broken method CRM_Utils_File::isHtml()

Merge pull request #25227 from eileenmcnaughton/dedupe_rule

Support DedupeRules as Managed entities

Merge pull request #25228 from civicrm/5.57

5.57 to master

Merge pull request #25225 from agh1/5.57.0-releasenotes-initial

5.57.0 release notes initial run

Support DedupeRules as Managed entities

5.57.0 release notes: added boilerplate

5.57.0 release notes: raw from script

Merge pull request #25223 from eileenmcnaughton/dep_opt

Remove deprecated ids from OptionValue::add

Remove deprecated ids from OptionValue::add

Merge pull request #25217 from braders/profile-image-accept-types

Only allow valid image types to be uploaded to image_URL field on profiles

Merge pull request #25216 from civicrm/5.57

5.57

Only allow valid image types to be uploaded to image_URL field on profiles

Merge pull request #25213 from demeritcowboy/fract-qty

dev/core#4039 - Restore ability to have fractional quantity for text price fields

restore ability to have fractional quantity

Merge pull request #25212 from seamuslee001/ref_composer_patches

[REF][PHP8.2] Update Composer Patches to latest version to fix notice…

[REF][PHP8.2] Update Composer Patches to latest version to fix notice issue in php8.2

Merge pull request #25201 from demeritcowboy/empty-checkbox

[NFC] php8 - Unit test for empty checkbox array #23305

Merge pull request #25208 from civicrm/5.57

5.57

Merge pull request #25204 from kurund/contribution-import-fix

dev/core#4038 import contribution fixes in update mode

Merge pull request #23305 from ginkgomzd/custom-empty-array

correct logic for handling empty-array values for checkboxes;

Merge pull request #25205 from mattwire/activityimporttypo

NFC: Fix variable name in activity import

correctly handle empty arrays in custom-fields;

Fix variable name in activity import

import contribution fixes in update mode

Merge pull request #25202 from seamuslee001/php82_deprecation_classvariables

[REF][PHP8.2] Further resolve some deprecation issues around dynamic …

failing test for empty checkbox array

Merge pull request #25203 from seamuslee001/update_compile_lib

[REF] Update Composer Compile Lib with latest version

[REF][PHP8.2] Further resolve some deprecation issues around dynamic class properties

[REF] Update Composer Compile Lib with latest version

Merge pull request #25186 from eileenmcnaughton/data_array

Remove reference to 'dataArray' from text version of contribution_online_receipt

Merge pull request #25200 from MegaphoneJon/gotv-static

PHP8: callback function on GOTV must be static

Merge pull request #25178 from totten/master-civiunitestcase

(REF) CiviUnitTestCase - Cleanup and simplify the DB-reset mechanism

callback function on GOTV must be static

Merge pull request #25164 from alifrumin/mailingLabels

[dev/core#3470] Search Kit: Mailing labels don't work

Merge pull request #25193 from demeritcowboy/act-targ

E_WARNING on bulk email activity view

Merge pull request #25197 from colemanw/fixOptions

SearchKit - Fix option loading & rendering for in-place edit

Merge pull request #25147 from colemanw/bumpPhpMin

Bump min PHP version to 7.3.0

Merge pull request #25120 from briennekordis/event_fees_tab

Configure Event Fees tab

Merge pull request #25196 from seamuslee001/php82_errors_2

[REF] Fix a couple of errors in PHP8.2

Merge pull request #25173 from eileenmcnaughton/no_const

Standardise method of getting entity in imports

Merge pull request #25194 from eileenmcnaughton/build_divide

Post code-divide tidy-up

Merge pull request #25154 from totten/master-month-names

CRM_Utils_Date - Get month names from IntlDateFormatter. Ignore timezones.

SearchKit - Fix option loading & rendering for in-place edit

Fixes dev/core#4037

Standardise method of getting entity

[REF] Fix a couple of errors in PHP8.2

Post code-divide tidy-up

Follow up to https://github.com/civicrm/civicrm-core/pull/25184

E_NOTICE

Merge pull request #25192 from mlutfy/addCroatian

List of languages: Enable Croatian language by default

Merge pull request #25190 from civicrm/5.57

5.57

List of languages: Enable Croatian language by default

Merge pull request #25189 from MegaphoneJon/batch-form-entity

Fix 'cannot determine default entity' error on New Batch page

Merge pull request #25184 from eileenmcnaughton/build_divide

Divide & conquer, `initSet` no longer shared as much code is not

Fix 'cannot determine default entity' error on New Batch page

Merge pull request #25179 from jaapjansma/civix_issue_257

 totten/civix#257 Fix Civix not correctly loading Mixinx

Remove reference to 'dataArray' from text version of contribution_online_receipt

CiviUnitTestCase - Simplify console message. Remove unused method 'getDBName()'.

CiviUnitTestCase - Inline method '_populateDB()'

At this point, it's just a bunch of verbosely written conditions that
appear to be trueisms;

* When called via `setUpBeforeClass()`, the `$perClass` flag is TRUE,
  so it does the reset.
* When called via `setUp()`, the `$object` flag is non-null,
  so it does the reset.

At this point, the actual reset bit is 1-line. Easier to just call that.

CiviUnitTestCase - Extract method 'buildEnvironment'. Future target for method-overrides.

CiviUnitTestCase - Move UF check

In practice, this check is the first thing done by `setUpBeforeClass()`.
Might as well move it there.

Technically, this check is also called by way of `setUp()`, but it's
redundant at that point. (Constants are constants...)

CiviUnitTestCase - Inline misleading function 'getConnection'

This function returns a connection that nobody uses.  It has only one
caller.  It's mostly interesting because it can reset the database.
Hiding it under the name `getConnection` is misleading.

CiviUnitTestCase - Remove unused property $_dbconn

CiviUnitTestCase - Remove unused property $populateOnce

The property is initialized to FALSE. It seems that the idea was to let some classes
override this, but it is never used.

CiviUnitTestCase - Remove unused method requireDBReset

CiviUnitTestCase - Remove ancient/unused property '$_dbName'

CiviUnitTestCase - Remove ancient/unused method 'getDataSet()'

Merge pull request #25185 from civicrm/5.57

5.57

Merge pull request #25174 from seamuslee001/ref_create_alter_column_function

[REF] Create Alter Column function in upgrade base class

Merge pull request #25177 from totten/5.57-testing

Fix miscoordination between CiviUnitTestCase/CiviEnvBuilder. Fix flaky ConformanceTest.

Merge pull request #25183 from eileenmcnaughton/build_memy

Remove conditionals that rely on NULL not being NULL

Merge pull request #25161 from mattwire/genericsqlimport

Add datasource selector to generic import (so it can be used eg. with contribution import)

Divide & conquer, initSet no longer shared as much code is not

AutocompleteTest - Use SK as part of the baseline environment

Merge pull request #25182 from eileenmcnaughton/build_mem

Simplify params on now-private function

Remove conditionals that rely on NULL not being NULL

Simplify params on now-private function

Fix for CiviX issue 257

Action Type as variable

Merge pull request #25144 from mattwire/afformmembership

Add Membership entity to formbuilder

ConformanceTest - Remove override, which causes failures in `ConformanceTest`

CiviUnitTestCase - Fix miscoordination with CiviEnvBuilder

Background
----------

Recall that `CiviUnitTestCase` and `CiviEnvBuilder` are two different
implementations of a similar concept.

* `Concept`: If two tests use the same baseline DB/environment, and if they
  preserve the baseline, then you don't need to reset everything in between
  tests.  But if they change, then you need to reset.

* `CiviUnitTestCase`: The older rendition.  Tightly coupled.  Only allows
  one specific baseline.  Mingled with a lot of unrelated functionality in
  `CiviUnitTestCase`.

* `CiviEnvBuilder`: The newer rendition.  Loosely coupled.  Allows different
  baselines.  Can be mixed into any plain-old `TestCase`.  Used by
  `Civi\Test::headless()` and `Civi\Test::e2e()`.

Problem Scenario
----------------

Suppose you have a mix of different tests running with the same DB -- e.g.

1. Run some tests based on `CiviEnvBuilder`
2. Run some tests based on `CiviUnitTestCase`
3. Run some tests based on `CiviEnvBuilder`

This wasn't originally anticipated, but it can happen -- either because
the test-suite is large+mixed, or because a developer is manually
running specific tests (which happen to be written differently).

The problem goes like this:

1. Run some tests based on `CiviEnvBuilder`
    * This resets the DB and also stores a DB signature. ("Here is how
      we setup the DB...")
2. Run some tests based on `CiviUnitTestCase`
    * This resets the DB, but leaves the old DB signature in place.
3. Run some tests based on `CiviEnvBuilder`
    * This sees the old DB signature and wrongly concludes that we
      still have the DB from step (1).

Solution
--------

Whenever one resets the DB, it should update the DB signature.

(REF) CiviEnvBuilder - Simplify `$name` calculation

Tweaking the `$name` is theoretically useful if you have entirely
independent data-sets that happen to live in the same DB.  In practice, we
only use CiviEnvBuilder to manage Civi-related data-sets.

[REF] Create Alter Column function in upgrade base class

Minor fixes as per coleman

Merge pull request #25172 from colemanw/noLocalize

Remove incorrect usages of {localize} in civicrm_data.tpl

Merge pull request #25123 from mattwire/searchKitTasks

Add search/display details to searchKitTasks hook - allows for eg. filtering actions by search display

Remove incorrect usages of {localize} in civicrm_data.tpl

The {localize} block was being used in this template in ways that could not plausibly work

- Using the tag in some rows but not others within the same INSERT statement
- Adding it to the rows but not the columns of an INSERT statement

I think the only reason this didn't cause a crash is because this file is only used during installation,
and it's not possible to install CiviCRM with multiple languages - you have to switch to multilingual
post-installation. So these {localize} blocks were never actually used.

Merge pull request #25143 from aydun/option_group_descriptions

Add Option Group descriptions from templates/CRM/Admin/Page/Options.tpl

Merge pull request #25146 from colemanw/defineVars

SearchKit - Ensure id & name are defined when loading saved searches

Add search/display details to searchKitTasks hook - allows for eg. filtering actions by search display

use id instead of label

OptionGroup - Update some descriptions and fix multilingual upgrader

Merge pull request #25169 from demeritcowboy/act-targ-value

E_NOTICES on activity view

E_NOTICES

Merge pull request #25168 from totten/master-ornery

(NFC) PrevNextTest - Fix warning about "@group"

Merge pull request #25167 from civicrm/5.57

5.57

(NFC) PrevNextTest - Fix warning about "@group"

When running this test, there are warnings like so:

```
WARNING: Class E2E\Core\PrevNextTest implements EndToEndInterface. It should declare "@group e2e".
```

In PHPUnit docs (https://phpunit.readthedocs.io/en/9.5/annotations.html#group), they
encode multiple groups using multiple lines. Following this convention fixes the warning.

Merge pull request #25160 from totten/5.57-file-test

CRM_Utils_FileTest - Fix compatibility with private data folders

Merge pull request #25145 from MegaphoneJon/core-4027-rc

fixes core#4027 - don't crash when a contribution has no line items

Merge pull request #25165 from eileenmcnaughton/smarty_case

Fix smarty notices on Open Case

Fix smarty notices on Open Case