Merge pull request #23155 from eileenmcnaughton/ret2

Remove-single-value-loop

Merge pull request #23162 from agileware/additional-not-additonal

✍🏼 Fix spelling of "additional". Numerous uses of "additonal"

Fix spelling of additional

Merge pull request #22159 from colemanw/isCurrentDomain

APIv4 - Add `is_active` extra field to Domain entity

Merge pull request #23144 from colemanw/noMoreSaveButtonInApiExplorer

API4 Explorer - Remove confusing "Save" button

Remove-single-value-loop

The communicationType array only has one value. This removes the foreach, swapping
the values for the only value they could be. This resulted in 2 always-true
ifs being replaced too

Merge pull request #23149 from colemanw/searchIcons

Add icons for SavedSearch and SearchDisplay entities

Merge pull request #23150 from braders/deleteurl-ensure-variable

Ensure deleteURL variable for ContactImage.tpl

Merge pull request #23153 from braders/nodefaults-default-form

Add smarty:nodefaults to default.tpl form.

Merge pull request #23154 from civicrm/5.49

5.49

Merge pull request #23152 from braders/row-weight-nodefaults

Don't overescape weights in default-escape mode

Merge pull request #23151 from braders/financialaccount-notices

Avoid PHP notices on FinancialAccount table

Merge pull request #23148 from totten/5.48-cache-ver-suffix

(dev/core#3166) "Metadata" cache - Strictly separate by version

Merge pull request #22967 from eileenmcnaughton/ret2

[REF] Code cleanup on location entities for the Contact Summary screen

Add smarty:nodefaults to default.tpl form.

This allows core forms (e.g. search forms) to submit correctly,
even with CIVICRM_SMARTY_DEFAULT_ESCAPE mode.

Without this change, quote marks were turned into HTML attributes,
meaning the form essentially had no method or action.

Don't overescape weights in default-escape mode

Avoid PHP notices on FinancialAccount table

Ensure deleteURL variable for ContactImage.tpl

Add icons for SavedSearch and SearchDisplay entities

Set version to 5.50.alpha1

Set version to 5.49.beta1

(dev/core#3166) "Metadata" cache - Strictly separate by version

This addresses some symptoms identified in the discussion of 3166.

Before
------

The "metadata" cache is always stored with the name `metadata`.

If you load a newer codebase, it still reads `metadata` from the older code base (until something explicitly clears the cache).

After
-----

The "metadata" cache is always stored with the name `metadata_{version}`, eg `metadata_5_48_0`.

If you load a newer codebase (eg `5.49.0`), it will ignore the older `metadata_5_48_0` and create a new `metadata_5_49_0`.

Comments
--------

The upgrade system also clears cache, but many users rely on navigating through the site to get to the upgrader.

The change could make sense with other caches, but I only applied the change to the `metadata` cache because there is _some_ risk;
eg if some caller bypasses the cache objects. (I was also slightly concerned about the possibility the some caller relies
on `$cache->getName()` or `$cache->getPrefix()` or similar... but I haven't been able to find any methods like that...). It seems
safer to put that sort of change into a monthly update rather than minor patch.

Merge pull request #23146 from colemanw/api4ExplorerIndex

APIv4 Explorer - Adjust 'Index' field to fit better on narrow screens

api4Explorer.css style fix

Merge pull request #23145 from braders/crm-button-icon-fixes

Fix incorrect use of icon argument for crmButton function

Fix incorrect use of icon argument for crmButton function

APIv4 Explorer - Adjust 'Index' field to fit better on narrow screens

API4 Explorer - Remove confusing "Save" button

This button was added as a prototype for saving a smart group from the Explorer.
But since that time SearchKit has evolved with the same functionality.
Since the presence of the button in the Explorer is confusing and redundant with SearchKit,
I think it's best to remove it.

Merge pull request #23132 from eileenmcnaughton/imp

Remove unnecessary pass-by-ref

APIv4 - Add `is_active` extra field to Domain entity

Merge pull request #23138 from eileenmcnaughton/map

Consolidate MapTable.tpl

Merge pull request #23136 from eileenmcnaughton/notice_mem_search

Enotice fix - membership search

Merge pull request #23137 from eileenmcnaughton/unused

Remove unused property

Consolidate MapTable.tpl

Remove unused property

Enotice fix - membership search

Remove unnecessary pass-by-ref

Merge pull request #23134 from totten/master-buggable

(REF) Schema - Fix boolean fields in various tables - Split commits

Merge pull request #23133 from eileenmcnaughton/mem

Remove 'in-between' class

Merge pull request #23129 from colemanw/api4_log_entity

Add in APIv4 Logging Entity

Schema - Boolean blob

Schema - Fix boolean fields in 'SMS'

Schema - Fix boolean fields in 'Report'

Schema - Fix boolean fields in 'Pledge'

Schema - Fix boolean fields in 'Price'

Schema - Fix boolean fields in 'PCP'

Schema - Fix boolean fields in 'Member'

Schema - Fix boolean fields in 'Mailing'

Schema - Fix boolean fields in 'Friend'

Schema - Fix boolean fields in 'Financial'

Schema - Fix boolean fields in 'Event'

Schema - Fix boolean fields in 'Dedupe'

Schema - Fix boolean fields in 'Cxn'

Schema - Fix boolean fields in 'Core'

Schema - Fix boolean fields in 'Contribute'

Schema - Fix boolean fields in 'Contact'

Schema - Fix boolean fields in 'Case'

Schema - Fix boolean fields in 'Campaign'

Schema - Fix boolean fields in 'Activity'

Schema - Fix boolean fields in 'ACL'

Schema - Add base function for FiveFortyNine boolean search

Revert "Schema - Fix boolean fields in various tables"

This reverts commit c1e814c7ddd3406dbada69de369e07b1eb85c689.

Add in APIv4 Logging Entity

Remove 'in-between' class

Merge pull request #23130 from colemanw/managedOptionValue

Fix OptionValue BAO to call pre/post hooks to prevent force-reset of managed option values

Fix OptionValue BAO to call pre/post hooks to prevent force-reset of managed entities

Managed entities rely on pre/post hooks being called in order to track whether an entity
has been modified. Without those hooks OptionValues were being force-reverted by the
managed system even when the 'update' mode was set to 'unmodified', which is supposed
to defer to user modifications.

Merge pull request #22850 from ixiam/dev_Issue#3080

Fixes issue with duplicate is_billing on inline address forms.

Fixes style warnings

Merge pull request #22941 from sunilpawar/batch_copy_radio_clear_value

Copy Clear value in batch update process for radio button field

Merge pull request #20437 from JMAConsulting/core_2633

core#2633 : Cannot remove Group Organisation selection in multisite setup

Merge pull request #22954 from JMAConsulting/booleanFields

[WIP] Schema - Fix boolean fields in various tables

Schema - Fix boolean fields in various tables

Merge pull request #23126 from eileenmcnaughton/imp

Remove good intentions

Merge pull request #23127 from civicrm/5.48

5.48

Merge pull request #23125 from agh1/5.48.0-releasenotes-final

5.48.0 release notes: added late changes

Remove good intentions

No - this isn't a todo we are about to do all of a sudden....

5.48.0 release notes: added late changes

Merge pull request #23124 from eileenmcnaughton/html

Tpl cleanup - whitespace, wrong closing tag

Merge pull request #23121 from eileenmcnaughton/imp

Simplify class inheritance

Merge branch '5.48' to master

Merge pull request #22977 from eileenmcnaughton/issetit

Ensure pay_later_text is always assigned

Merge pull request #23123 from eileenmcnaughton/private

Remove unused private variables

Merge pull request #23122 from eileenmcnaughton/rr

Fix function calls to be non-static

Merge pull request #23016 from pradpnayak/optionValue

Respect zero value

Merge pull request #23120 from colemanw/contentBlock2

Afform - Ensure upgrade fully converts content blocks

Tpl cleanup - whitespace, wrong closing tag

Merge pull request #23119 from eileenmcnaughton/silly6

Notices on import - consolidate mapping variables

Merge pull request #23116 from colemanw/removeContactViewOptions

CiviGrant - Remove upgrade that adds removed option value

Simplify class inheritance

CRM_Contribute_Import_Parser_Contribution is the only class in our universe to
extend CRM_Contribute_Import_Parser and this class only adds confusion to the
mix as functions are 'distributed' between them. This removes the inheritance,
leaving only the constant that is still used in the now-deprecated class

Merge pull request #23117 from colemanw/shorcuts

Emit deprecated warning for typo version of hook_civicrm_links

Merge pull request #23099 from colemanw/RecentItemAPI

Add APIv4 for RecentItems

Remove unused private variables

Merge pull request #23118 from colemanw/grantTab

CiviGrant - Fix missing column and translate headers in contact summary tab

Fix function calls to be non-static

Merge pull request #23115 from eileenmcnaughton/548

Fix broken civgrant shortcut link

Afform - Ensure upgrade fully converts content blocks

Ensures `{block: '*'}` doesn't get converted to `{entity_type: '*'}`
because that's not a real entity type.

Merge pull request #23104 from colemanw/contentBlock

Afform - Fix regression editing a content block with no entity type

Notices on import - consolidate mapping variables

This consolidates 2 variables mappingName and loadedMapping into
one (savedMappingName) - loadedMapping is only ever used as
a boolean so presence or otherwise of savedMappingName is enough.

This addresses enotices and simplifies the code

Merge pull request #23114 from eileenmcnaughton/silly6

Fix enotice, simplify text on Saved field mapping

Adds PHPUnit testing for multiple addresses with is_billing on