Merge pull request #23567 from kurund/honor-roll-fixes

show honor roll only if it's enabled for the PCP page

Merge pull request #23640 from aydun/casetype

Make CaseType usable in Afform & SearchKit

Merge pull request #23623 from eileenmcnaughton/pcm

Deprecate crazy BAO handling of preferred_communication_method

Make CaseType usable in Afform

Merge pull request #23056 from yashodha/dev-3141

(dev/core#3141) Membership should be listed chronologically by join d…

Merge pull request #23626 from eileenmcnaughton/import_try

Minor cleanup on import return codes

Merge pull request #23597 from colemanw/searchKitProximity

SearchKit - add address proximity (map radius) search

Merge pull request #23631 from eileenmcnaughton/authx

Add Authx settings to navigation

Update ext/authx/authx.php

Co-authored-by: Tim Otten <totten@civicrm.org>

Add Authx settings to navigation

Minor cleanup on import return codes

Merge pull request #23634 from eileenmcnaughton/import_unreach

Remove legacy calls to `civicrm_error`, unpack return

Another unreachable place - code is always duplicate if it is an array

Remove more checks that can be is_array

Remove unreachable code

Remove legacy calls to civicrm_error, unpack return

The function can only return an array or a BAO object

Merge pull request #23633 from briennekordis/patch-2

Update contributor-key.yml

Update contributor-key.yml

add @briennekordis as a contributor

Merge pull request #23610 from eileenmcnaughton/import_dupe

[Import] [Ref] Trivial - fold function contents back into parent

Merge pull request #23628 from totten/5.50-alsoinc-test

(NFC) Scheduled Reminders - Reinstate and analyze test-case for "Also Include"

Merge pull request #23627 from eileenmcnaughton/import_test

Test cleanup - use names keys in mapper array where possible for clarity

Merge pull request #23625 from eileenmcnaughton/import_suff

Remove obsolete code - this parameter is not possible

Deprecate crazy BAO handling of preferred_communication_method

In the context of import this is tested in
testImportFieldsWithVariousOptions

- for the api it is tested via testPseudoFields - although in
api context it is just a deprecation

(NFC) Scheduled Reminders - Reinstate and analyze test-case for "Also Include"

After reading #23445 and #23101, I wanted to see the test-coverage for "Also
Include".  There's a little...  but not a lot.  There was an interesting-but-inert
test-case.

Before
------

* Test-case is commented-out
* Comments indicate a vague sense of confusion

After
-----

* Test-case is enabled
* Comments indicate a well-developed, robust sense of confusion

Test cleanup - use names keys in mapper array where possible for clarity

Merge pull request #23619 from darrick/pull/23534

Fix swapOptions writing over selected options on mapField page load.

Merge pull request #23624 from mattwire/sqlgroup

Release lock when deleting cache key

[Import] [Ref] Trivial - fold function contents back into parent

The parent is actually longer by virtue of calling the child than without it....

Remove obsolete code - this parameter is not possible

Release lock when deleting cache key

Merge pull request #23474 from eileenmcnaughton/import_dep

[Import] Unpack the weird wild error return

Merge pull request #23609 from briennekordis/leading_zero_fix

core 3459 getlist leading zero fix

Merge pull request #23613 from eileenmcnaughton/member_unused

Remove unused variables

Merge pull request #23614 from eileenmcnaughton/import_unused2

Remove some unused variables

Merge pull request #23621 from eileenmcnaughton/import_1

Whitespace cleanup - remove `if(1)`

Merge pull request #23586 from eileenmcnaughton/import_mem

dev/core#3038 Fix handling for payment_instrument_id, cleanup financial_type_id

Whitespace cleanup - remove if(1)

Fix style errors

Fix swapOptions writing over selected options on mapField page load.

use swapOptions exclusivily to hide fields on mapField form.

Merge pull request #23560 from eileenmcnaughton/import_catcher

[Import] Test + fix for failure to reject invalid ex identifier

[Import] Test + fix for failure to reject invalid ex identifier

Remove some unused variables

Remove unused variables

Merge pull request #23612 from civicrm/5.50

5.50 to master

Merge pull request #23607 from mattwire/23099recentitemspermission

Fix 'Authorization Failed' regression when submitting eg. webform via checksum

Merge pull request #23606 from eileenmcnaughton/import_activity

[Import] [Activity] Fix fatal error regression

core 3459 getlist leading zero fix

Merge pull request #23563 from eileenmcnaughton/import_location_all

Import locations - code cleanup, add tests, fix bugs

[Import] [Activity] Fix fatal error regression

Merge pull request #23599 from aydun/contriberror

Make this error message more informative.

Make this error message more informative.
Update test

Merge pull request #23546 from ixiam/skip-constraint

dev/core#3467 - Drupal 8 - Skip requirement for password when changing your own civi contact's email

Skip requirement for password when changing the current user fields

Fix 'Authorization Failed' regression when submitting eg. webform via checksum

e-notice fix

Remove sneaky files

Fix import to work with phone_ext, test

This fixes a couple of bugs I found writing tests & adds
test cover for website & phone imports + cleanup

Notably - creating a phone with a phone_ext or an email
with signature_text doesn't work prior to this

Country validation

m

Improve comparison for options

Cleanup array explosion

Use metadata to get the relevant entity

Add as-yet-unused functions to reduce PR conflicts

Merge pull request #23601 from colemanw/authxBeta

Authx - Mark extension as beta

Merge pull request #23468 from mattwire/contributionpagedeprecatecompletetransaction

REF Deprecate method completeTransaction on contribution pages

Add as-yet-unused functions to reduce PR conflicts

Update ext/authx/info.xml

Co-authored-by: Tim Otten <totten@civicrm.org>

Merge pull request #23592 from eileenmcnaughton/import_parent

[Import] [ref] Move function to parent for use by siblings

Merge pull request #23565 from eileenmcnaughton/preffered_mail

Remove deprecated preferred_mail_format from more places...

Merge pull request #23605 from eileenmcnaughton/import_indie

Remove some never used fields - copy & paste begone

Remove some never used fields - copy & paste begone

Merge pull request #23562 from eileenmcnaughton/import_cc

Additional test

Simplify parameters for completetransaction for zero value payment

Deprecate method completeTransaction on contribution pages

Authx - Mark extension as beta

Merge pull request #23583 from colemanw/apiDisabledFields

APIv4 - Exclude disabled custom fields

SearchKit - Add proximity search to UI

Fixes dev/core#3163

APIv4 - Add Address::getCoordinates action

APIv4 - Add address proximity search filter

Use 'getParser' rathe than name the parser

[Import] [ref] Move function to parent for use by siblings

Merge pull request #23456 from eileenmcnaughton/import_more

[Import] Consolidate DataSource.tpl - template for first file on activity import

Merge pull request #23590 from eileenmcnaughton/import_member_labels

[Import] Add some not-yet-used `getParser` functions that are prone to conflict

Merge pull request #23596 from civicrm/5.50

5.50 to master

Merge pull request #23594 from totten/5.50-upgrade

Upgrader - Fix error on Snapshot::cleanupTask

Upgrader - Fix error on Snapshot::cleanupTask

This is a follow-up to #23522.

Before
------

Upgrade works on `cv upgrade:db` but fails on `drush civicrm-upgrade-db`

After
-----

Upgrades on both `cv upgrade:db` and `drush civicrm-upgrade-db`

[Import] Add some not-yet-used functions that are prone to conflict

Merge pull request #23589 from demeritcowboy/null

php8.1 - Check null first before sending to strlen

Merge pull request #23559 from eileenmcnaughton/import_yay

Another always-true-if

Merge pull request #23570 from ufundo/mailing-group-token-frontend-title

dev/core#3463 - CiviMail - prefer frontend_title from group for maili…

Merge pull request #23561 from eileenmcnaughton/import_catcher_

Test fix

Merge pull request #23576 from eileenmcnaughton/cust_it

Import - Relax "is_required" check for custom fields

check null first

Merge pull request #23516 from eileenmcnaughton/sms

dev/core#3177 Switch sms to use flexmailer token rendering

Merge pull request #23318 from seamuslee001/php81_preg_replace

[REF] Fix handling of NULL values in count_characters smarty modifier…

dev/core#3038 Fix handling for payment_instrument_id, cleanup financial_type_id

This addresses https://lab.civicrm.org/dev/core/-/issues/3038
by getting rid of the non-standardness around payment_instrument_id
and financial_type_id. The two fields are now
marked as importable and we use the
real names (payment_instrument_id) rather than the
'handy-dandy' names (payment_instrument) - which means
we can simplify in a bunch of places. Also since we are
already cleaning up names in the upgrade we add these in there

Merge pull request #23577 from demeritcowboy/deleted-tests

[NFC] Bring back deleted activity email tests

Merge pull request #23579 from colemanw/searchKitFixCount

SearchKit - Fix pager & tab count after creating a new record in a popup

APIv4 - Exclude disabled custom fields

Merge pull request #23578 from colemanw/managedSearchSegment

SearchKit - Allow SearchSegments to be packaged as .mgd.php

Merge pull request #23587 from civicrm/5.50

5.50 to master