Merge pull request #23283 from eileenmcnaughton/import_saved_map

[REF] Remove handling for non-existent 'savedMapping' field

Merge pull request #23675 from demeritcowboy/test-sequential

[NFC] Failing test for api chain using sequential

Merge pull request #23676 from eileenmcnaughton/white

Reformat whitespace

Merge pull request #22762 from totten/master-queue-api4-alt

(dev/core#1304) Queues - Allow background worker to drive tasks via APIv4

Merge pull request #23520 from eileenmcnaughton/old_code

Deprecate stuff getting super deep into the BAO unformatted

failing test for api chain with sequential

Merge pull request #23666 from eileenmcnaughton/import_combo

Contact Import cleanup - includes fixes to contact matchine

Reformat whitespace

Merge pull request #23672 from seamuslee001/fix_sequential_enforcement

APIv3 - Fix regression in handling chained calls with `sequential`

Merge pull request #23671 from seamuslee001/master

Fix issue #116 api3 chain check_permissions

[REF] Fix issue where forceing sequential broke webform API chained usage

Fix issue #116 api3 chain check_permissions

Civi\Api4\Queue - Add APIs for claiming and running enqueued tasks

CRM_Queue_TaskRunner - Add background-friendly handler for `CRM_Queue_Task`s

SqlParallel - Implement BatchQueueInterface

Before
------

Each of the `CRM_Queue_Queue_*` drivers supports a set of methods for
claiming/releasing one queue-item at a time (eg `claimItem()`,
`releaseItem()`, `deleteItem()`).

After
-----

All drivers still support the same queue-item methods.  Additionally, the
`SqlParallel` driver supports batch-oriented equivalents (`claimItems()`,
`deleteItems()`, etc).

Comments
--------

I initially looked at updating all of the drivers to support queues.  There
were a few obstacles.  Firstly, batched-claims seem semantically
questionable for queues that run 1-by-1 (`Sql`, `Memory`).  Secondly, there
are a few extensions in contrib that extend these classes and override
methods (consequently, they're looking for stable signatures).

The approach here seemed to resolve those two concerns in an OOP-safe way:
define an optional interface (`BatchQueueInterface`) which can be used to
mark enhanced functionality on queues where it makes sense (eg
`SqlParallel`).

CRM_Queue_Queue_* - Respect `$queueSpec['lease_time']` (if given)

Before: The lease-time is one of the following:

1. A value requested at runtime
2. The constant 3600

After: The lease-time is either supplied as

1. A value requested at runtime
2. A value configured for the queue
3. The constant 3600

CRM_Queue_Queue_* - Respect `$queueSpec['retry_interval']` (if given)

Background:

* A queue runner should call `releaseItem()` if it tries and aborts some task.
* The `retry_interval` is defined as the extra time to wait before trying again.

Before: The `releaseItem()` always releases for immediate execution.

After: The `releaseItem()` checks `retry_interval`.  If it's set, then it
will add an extra delay before retrying.

CRM_Queue_Queue_* - Track and report `run_count` for each item

Before: Whenever you `claimItem()` from the queue, it marks the item `release_time`.

After: Whenever you `claimItem()` from the queue, it marks _both_ the
`release_time` and the `run_count`.

Comments:

* This is the basis for enforcing a `retry_limit` policy.

* This doesn't require any extra queries or joins - it fits into the
  existing update query.

CRM_Queue_Queue_* - Add fetchItem($id)

Note: The return types are a bit wonky here.  Unfortunately, as I recall,
the original specification for all the methods here allowed backend-specific
return types -- anti-standardized return-types.

CRM_Queue_Queue_* - Add isActive() and setStatus()

Merge pull request #23646 from seamuslee001/improve_get_coordinates

[REF] Improve Get Coordinates by supporting other geocoding providers…

Add in unit test of getCoordinates Address action and update TestProvider to be like the Google provider change

Fix Tear down of geoProvider setting

Merge pull request #23668 from colemanw/civiGrantPerm

CiviGrant - Add prefix to permission label

CiviGrant - Add prefix to permission label

Merge pull request #23667 from totten/master-queue-status-error

(dev/core#1304) Queues - Define 'status' and 'error' columns

Do not rely in init to set whether address parsing is on

Remove now-unused getActiveFields

Remove a couple of unused params

Stop setting now-unused updateWithId

Remove transitional getMetadataParams

Remove now-unused function

Move function to the only class that still calls it

Move isErrorInCustomData to shared class

Fold getParams into getMapped row

No point having a separate  line function

Remove function that no longer does anything

This is now only checking booleans in a way that duplicates getTransformedValue

Cleanup unparsed address message handling

Contact import - reduce over-formatting

Extract create relationship

Remove always true if

Only call getParams once

Remove code that handled now-changed relationship key

Note this code is no longer called by contact import, and the others do not pass
in relationship key - so we can remove rather than fix

Fix Import related contact stealing

Handle error when creating related contact

This addresses the e-notice identified here
https://github.com/civicrm/civicrm-core/pull/23643#discussion_r887406505

I believe the bug report in that thread was a variant of the e-notice. Ie the import()
function was failing to catch an exception & it was bubbling up to run()

While there is no support currently for calling import()
outside of run() - the intent is to replace run() with something
more sane - which WILL be callable from the outside (likely
via api).

The right behaviour for import() is to catch all errors

Remove unused getting & setting (copy & paste)

Import flow - separate out 'validate' function from 'run'

(NFC) Update stale comments

Merge pull request #23616 from pradpnayak/dev2540-1

Print report or export report if user have access report permission

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

[REF] Improve Get Coordinates by supporting other geocoding providers other than google

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

dev/core#3482 mailing approval workflow fix

Deprecate stuff getting super deep into the BAO unformatted

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