(NFC) Update stale comments

Civi::queue(...$queueSpec) - Accept the 'status' and 'error' options

Note: We only enforce this requirement for queues that use the `runner`
option.  Older queue-users may not specify a `runner` -- in which case
they're responsible for establishing their own runner, and their runner
problem doesn't care what is specified here.

civicrm_queue - Backfill "status" and "error" columns

civicrm_queue - Add 'status' and 'error' columns

Merge pull request #23657 from colemanw/fixGetFieldsWithNoSubType

[UNRELEASED REGRESSION] APIv4 - Fix GetFields to return custom fields with no sub-type

APIv4 - Fix GetFields to return custom fields with no sub-type

Ensures unconditional custom fields are always present.

Merge pull request #23653 from eileenmcnaughton/import_going

Remove no-longer-used function 29 points

Merge pull request #23588 from eileenmcnaughton/import_soft

[Import] Cleanup Contribution flow - esp with regards to soft credits

Clean up preview screen code

Simplify handling of soft credit

Merge pull request #23637 from eileenmcnaughton/import_fn

[Import] [Ref] [Contribution] Additional functions, typo fix

Merge pull request #23635 from eileenmcnaughton/import_unreach2

Simplify return from createContact

Merge pull request #23647 from civicrm/5.50

Merge 5.50 to Master

Merge pull request #23654 from agh1/5.50.0-releasenotes-final

5.50.0 release notes final

5.50.0 release notes: misc edits

Merge pull request #23358 from mattwire/participantwaitlist

Fix missing contactID when registering for paid event from waitlist

5.50.0 release notes: added security release

Remove no-longer-used function

5.50.0 release notes: updated with late changes

Add release-notes/5.49.3.md

Add release-notes/5.49.2.md

Add release-notes/5.49.1.md

Add release-notes/5.48.2.md

Add release-notes/5.48.1.md

Add release-notes/5.47.4.md

Update 5.47.3.md

Add release-notes/5.47.3.md

Add release-notes/5.47.2.md

Add release-notes/5.47.1.md

Add release-notes/5.46.3.md

Add release-notes/5.46.2.md

Add release-notes/5.46.1.md

Add release-notes/5.45.3.md

[Import] [Ref] Additional functions, typo fix

The functions are not yet used -in the interests of managing conflicts"

Further cleanup return checks

Simplify return from createContact

Merge pull request #23642 from eileenmcnaughton/import_c

 Fix State handling to be case insensitive again (unreleased regression)

Merge pull request #23636 from eileenmcnaughton/import_contact_type

Import - ensure userID is set on parser

Merge pull request #23643 from eileenmcnaughton/match

Fixes for contact matching - unreleased regression

Merge pull request #23644 from lcdservices/dev-core-3482c

dev/core#3482 mailing approval workflow fix

Merge pull request #23603 from eileenmcnaughton/rel2

Fix createEmployerRelationship to work with the data it has when doing a lookup, test

Fixes for contact matching

Fix State handling to be case insensitive again

Add geocode import

Fix State handling to be case insensitive again

Add geocode import

Merge pull request #23629 from eileenmcnaughton/import_greetings

Simplify greeting handling, add test

dev/core#3482 mailing approval workflow fix

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

Simplify greeting handling, add test

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

Import - ensure userID is set on parser

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.