Remove unused function

Merge pull request #23158 from eileenmcnaughton/fail

[REF] Extract getContributionRecurID

Merge pull request #22571 from eileenmcnaughton/trxn

Improve financial trxn spec to require required fields

Merge pull request #23196 from civicrm/5.49

5.49 to master

Merge pull request #23192 from eileenmcnaughton/549

Fix FiveFortyNine is_public to default to 0

Merge pull request #23189 from alifrumin/rn5.49

[NFC] First Pass at Release notes for 5.49

Merge pull request #23191 from twomice/3164_report_show_filter_criteria_zero

dev/core#3164 : Report Filter Statistics don't show filters with vaue of "0"

Improve financial trxn spec to require required fields

It turns out the apiv4 conformance test only passes because it
is bypassing the BAO create method - which requires more parameters

Not this will fail :-( because it messes with the test expectations
- I don't know how to create complex valid test data for it though....

I guess we could create a contribution & pass it as entity_id

[NFC] 5.49 release notes first pass

Fix FiveFortyNine is_public to default to 0

dev/core#3164 : Report Filter Statistics don't show filters with value of "0"

Merge pull request #23139 from kurund/group-subscription-confirmation-email

send group subscription welcome email only if template is enabled

Merge pull request #23188 from colemanw/jsFix

Cleanup crmMailing/Templates.js

Cleanup crmMailing/Templates.js

Merge pull request #23180 from eileenmcnaughton/csv

dev/core#3172 Update phpleague/csv from 9.2 to 9.6 (supports php 8)

Merge pull request #23172 from eileenmcnaughton/import_most

Further simplify variable retrieval

Merge pull request #23181 from civicrm/5.49

5.49 to master

Merge pull request #23179 from totten/5.49-grant-entities

(dev/core#3161) CiviGrant - Cleanup managed entities (5.49-rc)

dev/core#3172 Update phpleague/csv from 9.2 to 9.6 (supports php 8)

Merge pull request #23082 from agileware/CIVICRM-1958

🪲 🪳 Bug Fix: The option "Enable multiple bulk email address for a contact" does not apply when editing a Contact, only works for inline edit of email addresses

CiviGrant - Remove unused managed entities during upgrade

Removes managed entities which may have been added during the 5.47 upgrade,
to prevent the managed system from automatically deleting them.

CiviGrant - Cleanup managed entities - fixes dev/core#3161

- Ensure labels are translated
- Set stricter cleanup policy for entities that should be removed on uninstall
- Make grant_status options unmanaged. They will not be updated by the system ever,
  but they will be deleted on uninstall because the option group is a managed entity.

Merge pull request #23174 from braders/nodefaults-contrib-links

Contribution dashboard links work in escape-on-output mode

Merge pull request #23178 from civicrm/5.49

5.49 to master

Merge pull request #23176 from agh1/5.49.0-releasenotes-initial

5.49.0 release notes initial run

Merge pull request #23175 from pradpnayak/limitcontacts

Limit issue when printing pdf from contribution search

5.49.0 release notes: added boilerplate

5.49.0 release notes: raw from script

Merge pull request #23173 from braders/escape-on-output-event-links

Add smarty:nodefaults to ManageEvent page.

Fix limit issue

Contrib dashboard links work in escape-on-output mode

Add smarty:nodefaults to ManageEvent page.
This ensures that the links render correctly in escape-on-output mode.

Simplify more variables

Simplify value retrieval

Merge pull request #23171 from eileenmcnaughton/saved_map

Remove some copy & paste

Merge pull request #23169 from totten/master-authx-info

(NFC) authx - Update descriptions and links in `info.xml`

Remove some copy & paste

Merge pull request #23135 from eileenmcnaughton/memm

Centralise skipColumnHeader to one function (fixes notices, cleans code)

Merge pull request #23168 from totten/master-ver-compat

(Re-)Set version to 5.50.alpha1

Merge pull request #23166 from pradpnayak/limitcontacts

Set limit to 0 when processing send sms task

Merge pull request #23165 from eileenmcnaughton/ref

Reduce use of reference

(NFC) authx - Update descriptions and links in `info.xml`

Set limit to 0 when processing send sms task

Set version to 5.50.alpha1

Merge pull request #23143 from colemanw/extVersion

Release - Set version compatibility for core extensions when updating core

Centralise skipColumnHeader to one function

Reduce use of reference

Reference does not make sense for objects. assign_by_ref does not make
sense when not used again in the function or returned in a reffy kinda way

Merge pull request #23101 from MegaphoneJon/core-1836-js

core#1836 - restrict scheduled reminder 'Also Include' to non-events

Merge pull request #23163 from mlutfy/fixTsEventSv

getSelfServiceEligibility: fix string translation

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"

getSelfServiceEligibility: fix string translation

Fix spelling of additional

[REF] Extract getContributionRecurID

Merge pull request #23141 from totten/master-mailing-verify

(dev/mail#107) CiviMail - Fix validation error (5.49)

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

Release - Set version compatibility for core extensions when updating core version

APIv4 - Add `is_active` extra field to Domain entity

CiviMail - Fix validation error

Steps to reproduce:

- Disable Flexmailer
- Use web UI
- Create a new mailing
- Fill out fields
- Submit mailing
- Observe inaccurate error - Mailing cannot be sent. There are missing or invalid fields (subject,name,from_name,from_email,body).

See also: https://civicrm.stackexchange.com/questions/41654/after-upgrade-to-civicrm-5-48-0-all-submissions-say-mailing-cannot-be-sent-th

Merge pull request #23138 from eileenmcnaughton/map

Consolidate MapTable.tpl

send group subscription welcome email only if template is enabled

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'