dev/core#4158 Fix incorrect option_group update in paypal recurring

This line was pointed out as being problematic in the issue
https://lab.civicrm.org/dev/core/-/issues/4158 and in a test
I was able to verify it causes issues if the value for Completed
in the contribution status option group is not the same as the
contribution recur option group. However, it appears to
be more of a theoretical / test creatable regression than a real
world one & is not a credible fix for the bug.

I think an rc merge without a stable back port is OK

Merge pull request #25704 from totten/5.59-token-mismatch

(dev/core#4147) Add test and fix for various flaky tokens

Api4 - Rename Worldregion to WorldRegion

Worldregion - For API-entity-naming, prefer "WorldRegion"

Set version to 5.60.beta1

Style

Merge pull request #25706 from eileenmcnaughton/strict

Stricter typing in Apiv4

Sigh, and they have colons:

Fix key mismatch

Fix preferred_communication method

Also fix over-aggressive test assertion

Fix region token

Stricter typing in Apiv4

Add Worldregion api for join

Fix for unresolved state_province

Merge pull request #25705 from eileenmcnaughton/dep

Remove deprecated function calls to Event::del, Membership functions del, OptionValue

Rename variable - I figured out what it meant

Adhere to comment block code standard

Standard is short one liner first:

Superficial test cleanup, protect variable

Add phpdoc to prevent restoreTime being removed

These unused variables cause quite a few bugs - keeping them 'matters' but
they look like they can go and people doing that happens moderately often,
causing bugs (this has been the cause of more than one 'locks don't work bug'

Remove some calls to OptionValue deleteRecord

Deprecations in Event, Membership

Per https://github.com/civicrm/civicrm-core/pull/25677

Stop calling deprectated del function

(dev/core#4147) Add scenario testTokensIndividually

(REF) MessageTemplateTest - Extract method useFrozenTime()

Merge pull request #25702 from civicrm/5.59

5.59

Merge pull request #25698 from colemanw/removeIds

Remove deprecated use of $ids

Merge pull request #25701 from alifrumin/5.59rn

[NFC] 5.59 release notes sorting and annotating

[NFC] 5.59 release notes sorting and annotating

Merge pull request #25699 from civicrm/5.59

5.59 to master

Merge pull request #25697 from agh1/5.59.0-releasenotes-initial

5.59.0 releasenotes initial run

Merge pull request #25639 from eileenmcnaughton/import_validate

dev/core#4132 Fix import for multi-custom boxes

Remove deprecated use of $ids

5.59.0 release notes: added boilerplate

5.59.0 release notes: raw from script

Merge pull request #25696 from colemanw/delDep

Delete long-deprecated functions

Merge pull request #25695 from eileenmcnaughton/CRM/Block

Pull over a few more deprecations + blockDelete deprecation

Delete long-deprecated functions

Pull over a few more deprecations + blockDelete deprecation

Merge pull request #25693 from eileenmcnaughton/CRM/Block

Address portions of deprecated code replacement

Address portions of deprecated code replacement

Merge pull request #25691 from eileenmcnaughton/dep_mailing

Deprecations for Mailing::delete functions

Merge pull request #25690 from eileenmcnaughton/dep_test

Deprecations for delete on ACL entities

Deprecations for Mailing::delete functions

Deprecations for delete on ACL entities

Merge pull request #25688 from eileenmcnaughton/dep_test

Remove deprecated functions from tests

Remove deprecated functions from tests

Merge pull request #25686 from eileenmcnaughton/dep_2020_address

Remove 2020-deprecated handling of `legacyAddressCreate`

Merge pull request #25685 from eileenmcnaughton/dep_2020

Remove function noisily deprecated when the world was young (2020)

Merge pull request #25684 from eileenmcnaughton/address_delete

Remove deprecated function call

Merge pull request #25646 from colemanw/fixSearchDisplayFilter

Afform - Filter autocomplete fields belonging to a search display

Remove 2020-deprecated handline of legacyAddressCreate

Remove function noisily deprecated when the world was young (2020)

Merge pull request #25669 from eileenmcnaughton/smarty-syntax

dev/core#4146 Remove (old) Smarty-forward incompatible syntax from Address.tpl

Merge pull request #25683 from eileenmcnaughton/logging_dep

Add gentle deprecation to `debug_log_message`

Merge pull request #25682 from eileenmcnaughton/log_smarty

Log Smarty debug to it's own channel

Merge pull request #25680 from eileenmcnaughton/smarty-bank

Smarty notice fix on bank_account_number

Remove deprecated function call

Merge pull request #25678 from eileenmcnaughton/logging-ref

Remove unnecessary pass-by-reference

Add gentle deprecation to debug_log_message

Merge pull request #25679 from eileenmcnaughton/deprecated_mem_type

Remove code deprecated in 2018

Log Smarty debug to it's own channel

I just wrote the docs on the right way to log using this example
https://docs.civicrm.org/dev/en/latest/framework/logging/

- so we should make it the same. I tested locally
& this results in the smarty debugging going to a separate file (good).

I personally think we should send all the IPN files off to their
own channel but am loath to be pro-active on changing that

Smarty notice fix on bank_account_number

Also remove CRM_Utils_Array call

Remove code deprecated in 2018

Make function private

Remove unnecessary pass-by-reference

Merge pull request #25675 from eileenmcnaughton/notice_location

Remove empty on location - it should always be set now

Remove empty on location - it should always be set now

See https://github.com/civicrm/civicrm-core/pull/25668

Merge pull request #25618 from mattwire/caserelationship

Afform: Allow selecting case for relationship

Merge pull request #25668 from eileenmcnaughton/notice_location

Fix e-notice pattern around location not defined in formButtons.tpl

Merge pull request #25309 from MegaphoneJon/flexmailer-return-path

closes core#4070 - respect default return path

Merge pull request #25585 from herbdool/core-4127-4

dev/core#4127 move cms user parameters to userSystem

Merge pull request #25671 from colemanw/viewSql

SearchKit - Clarify how to view SQL output

SearchKit - Clarify how to view SQL output

Merge pull request #25670 from eileenmcnaughton/cust_it

Remove unused variables

Remove unused variables

Merge pull request #25466 from eileenmcnaughton/sep

Remove unused property - not used since import code refactor `_lineCount`

Merge pull request #25237 from eileenmcnaughton/mem_receipt

Remove conditional assignment around `is_deductible`

Merge pull request #25621 from eileenmcnaughton/event_test_cleanup

Event test cleanup, fix test to use submitted form values, rather than require a lot of contorting

Merge pull request #25270 from eileenmcnaughton/separate

Extend testing for ContributionConfirm & consolidate `isSeparateMembershipPayment`

Merge pull request #25667 from eileenmcnaughton/mixin

Fix mixin to use `addTemplateDir`

Remove (old) Smarty-forward incompatible syntax from Address.tpl

Fix e-notice pattern around location not defined in formButtons.tpl

Fix mixin

Merge pull request #25662 from eileenmcnaughton/token_dep

Add noisy deprecation to deprecated function, after universe search

dev/core#4127 move cms user parameters to userSystem

Merge pull request #25665 from totten/master-nfc-url

(NFC) crmURL - Add examples and whitespace. Crosslink docs.

Merge pull request #25664 from eileenmcnaughton/tpl_fix

Remove forward incompatible syntax from template

crmURL - Add examples and whitespace. Crosslink docs.

Merge pull request #25661 from eileenmcnaughton/smarty-url

Add `crmUrl function` to smarty in the standard way

Merge pull request #25663 from eileenmcnaughton/civix

run `civix update` on `recaptcha`, `legacyCustomSearches`, `financialacls`

Remove forward incompatible syntax from template

Civix upgrade on financialAcls

Civix update on legacyCustomSearches

Civix update on Recaptcha

Merge pull request #25659 from totten/master-ext-up

ext/* - General update to civix v23.02.0

Add crmUrl function in the standard way

Even during Smarty2 this method of registering functions was being phased out
for putting the file in the folder

Add noisy deprecation to deprecated function, after universe search

Merge pull request #25653 from totten/master-htxt

Smarty - {htxt} blocks should not be evaluated unless needed

Merge pull request #25655 from MegaphoneJon/sub-history-date

HTML definition for subscription history date