Merge pull request #22758 from civicrm/dependabot/npm_and_yarn/karma-6.3.14

Bump karma from 6.3.4 to 6.3.14

Bump karma from 6.3.4 to 6.3.14

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.14.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.14)

---
updated-dependencies:
- dependency-name: karma
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #22761 from civicrm/dependabot/npm_and_yarn/ajv-6.12.6

Bump ajv from 6.6.1 to 6.12.6

Merge pull request #22713 from colemanw/billingBlockCleanup

[REF] Cleanup billingBlock.js

Bump ajv from 6.6.1 to 6.12.6

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.6.1 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.6.1...v6.12.6)

---
updated-dependencies:
- dependency-name: ajv
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

[REF] Fix issue where spaces in payment_processor_type.name field causes issues with class output on form

[REF] Simplify Location::getValues

Merge pull request #22728 from eileenmcnaughton/local

Format money in custom fields once, on the tpl layer

Remove unused parameter, pass-by-ref

Merge pull request #22750 from colemanw/fixProcessorPseudoconstant

Fix ContributionRecur.payment_processor_id pseudoconstant

BAO_Mailing::create - stop passing by reference

We have been removing this pass-by-refs on an ongoing basis. CRUD calls should use
the api but I checked the core calls and none are looking for anything altered in

Fix ContributionRecur.payment_processor_id pseudoconstant

Before: Pseudoconstant mistakenly uses 'name' instead of title
After: Name and title are treated correctly

dev/core#3063 APIv3 - Fix numeric option matching

Before: Option matching was skipped for all FK fields if a numeric value was given
After: Only skipped for `campaign_id` field, if positive integer given

The optimization was overly broad and had unintended side-effects

Fix apiv4 Contribution delete & all line items

APIv4 - remove unnecessary field from System::check

Following on #22730, this removes the 'severity' field from the System::check api
because it's redundant with 'severity_id:name', and does not appear to be used anywhere.

Merge pull request #22743 from civicrm/5.47

5.47

Fix for #22685 financial acl aggression

Just realised this is required to make #22685 non-changey

Fix to not show payment for templates

Fix missing variables

Note this updates the code such that the buttons are consistent on this form with the payment info block

Fix contribution view to use checkAccess

Generate button links in PHP instead of smarty for ContributionView form

Tidy up getContributionPaymentLinks

- removes an unused variable.
- fixes to not show 'record Refund' for pending contributions (is displayed
on the expand payments section of a contribution

Tidy up getContributionPaymentLinks

- removes an unused variable.
- fixes to not show 'record Refund' for pending contributions (is displayed
on the expand payments section of a contribution

Merge pull request #22744 from eileenmcnaughton/grump

Bypass grumpy mode for contribution search totals & criteria

Merge pull request #22684 from eileenmcnaughton/contacl

Move acl check for contributionView to the extension

Merge pull request #22742 from eileenmcnaughton/cust

Fix deprecated warnings, smary grumpy-mode issues in legacy searches

Some more notice fixes in smarty

Merge pull request #22741 from MegaphoneJon/fix-duplicate-contact-typing

enforce required fields on Contact.duplicatecheck

Bypass grumpy mode for contribution totals

These are assigned to the template from php & have nbsp in them (for better or worse)

Fix deprecated warnings, smary grumpy-mode issues in legacy searches

This gets rid of some deprecated warnings and also makes the pager work in smarty grumpy mode

Merge pull request #22714 from demeritcowboy/grant-searchbuilder

dev/core#3056 - Crash with search builder if civigrant not enabled and have admin rights

enforce required fields on Contact.duplicatecheck

quickfix for crash if civigrant not enabled and have admin rights

OptionValues - Specify id+name+label for option callbacks that use it

Previously some API calls relied on a bug which conflated name with label,
that bug has been fixed, causing some test failures. The solution is to
update the option lists with a full multidimensional array with translated labels
and untranslated names.

Merge pull request #22738 from civicrm/5.47

5.47

Merge pull request #22727 from eileenmcnaughton/mod_money

dev/core#3059 Regression fix - be tolerant with smarty money

Merge pull request #22735 from civicrm/5.47

5.47

Merge pull request #22733 from totten/5.47-dedupe-check

(dev/core#3055) Fix dedupe check to not require elevated permissions

PseudoConstant - Allow multi-dimensionaly arrays to be returned from option callbacks

Before: Only flat arrays could be returned by a pseudoconstant callback fn.
After: Callbacks can return arrays with id/name/label/abbr.

Fix dedupe check to not require elevated permissions

Merge pull request #22628 from colemanw/civicrm_admin_ui

Add civicrm_admin_ui extension

Pseudoconstants - Use id instead of label for name

Each item in a field option list contains keys like id, name, label, color, icon, description, abbr.
But some option lists only consist of simple key/value pairs. To convert those simple associative arrays
into a full option list, the name should be derived from the id, not the label, because machine names
are expected to be stable, and labels can be translated.

Before: `name` derived from `label` when converting simple associative to an option list
After: `name` derived from `id`.

Merge pull request #22731 from colemanw/customvaluefailingtest

APIv4 - Fix setting/getting a multi-record customfield with date+time

APIv4 - Fix setting/getting a multi-record custom field with date+time

Merge pull request #22677 from eileenmcnaughton/aclmem

Move financial acls for membership to extension

Merge pull request #22722 from colemanw/restExplorer

APIv4 Explorer - Add REST syntax

Merge pull request #22380 from braders/core-483-show-customised-preferences-on-validation

Show customised fields on load, if customised option selected.

APIv4 Explorer - Add REST syntax

Format money in custom fields once, on the tpl layer

This ensures the raw value is available to the template layer and uses it
rather than the formatted value for money display.

It rather feels it would be better to separate out this view use case in case
there are others hidden in there - but I think this tpl is only
used with this assign

Regression fix - be tolerant with smarty money

I've seen issues in extended reports and in core where already-formatted money from
custom fields is formatted again in the smarty layer and, with recent changes
, throwing an exception when the money is greater than 1000
(because the presence of a comma makes it invalid). This adds tolerance
that seems consistent with prior code

Merge pull request #22715 from braders/batchentry-non-numberic-warning

[NFC] Batch entry - Avoid warnings when performing math on empty string.

Merge pull request #22723 from civicrm/5.47

5.47

Avoid null column showing on groups search page

Cast value to int before performing math

CustomField - Fix preview link

Add civicrm_admin_ui extension

So far this extension replaces 2 screns with SearchDisplay Afforms:
- Custom field groups
- Custom fields

Merge pull request #22716 from colemanw/sortableEntityFix

APIv4 SortableEntity - Fix sorting custom fields with option groups

Merge pull request #22721 from demeritcowboy/grantinfo

dev/core#3057 - follow-up to fix civigrant info.xml version number in master

fix version number

Fix regression with flexible merging of contact information

Merge pull request #22718 from civicrm/5.47

5.47

APIv4 SortableEntity - Fix sorting custom fields with option groups

Before: APIv4 would guess which fields to use for grouping when sorting by weight.
this caused a bug when sorting custom fields which also had an option_group_id, which
was incorrectly guessed to be used for grouping.

After: New `@groupWeightsBy` annotation removes the guesswork.

Merge pull request #22717 from demeritcowboy/grant-stats

dev/core#3057 - Fix missing civigrant

Merge pull request #22699 from colemanw/searchKitImportCheck

SearchKit - Improve import UI to handle update & preview the import

Merge pull request #21097 from mattwire/extensionguzzle

Switch extension downloader and extensions feed to use Guzzle HTTP library

fix missing civigrant

api_v3_ExtensionsTest - Switch to Guzzle-style mocking

Merge pull request #22688 from colemanw/getComponentName

REF - Use function to get component name from permission

REF - Use function to get componenent name from permission

Merge pull request #22711 from braders/br-selfclosing-syntax

Use valid syntax for self-closing br elements

Use valid syntax for self-closing br elements

Merge pull request #22636 from eileenmcnaughton/exampley

Add basic contribution example base for contribution workflows

Merge pull request #22706 from eileenmcnaughton/notice3

Notice fixes

Merge pull request #22707 from eileenmcnaughton/notice5

Enotice fix

Set version to 5.48.alpha1

Set version to 5.47.beta1

Merge pull request #22058 from kainuk/lecacy-auth

authx - Add configuration screen

Merge pull request #22687 from colemanw/cleanupComponentCheck

REF - Switch to using new `CRM_Core_Component::isEnabled()`

authx.setting.php - Add/tweak help text so it appears on the settings page.

Merge pull request #22710 from braders/fulltext-template-accessibility

Improve field accessibility in legacycustomsearches

authx.setting.php - Change sizes and weights to make settings more readable

Civilint

This code seems obsolote

Add settings screen

Cleanup billingBlock.js

Merge pull request #22709 from braders/dedupe-limit-smarty-notice

Don't throw PHP notice when rendering dedupefind template.

Merge pull request #19727 from totten/master-authx-rest

(dev/core#2077) Make 'civicrm/ajax/rest' interoperable with 'extern/rest.php' parameters

Merge pull request #22705 from eileenmcnaughton/notice2

Enotice fixes with datepicker

Improve field accessibility in legacycustomsearches

Merge pull request #22708 from braders/feature/priceset-notice-fix

Enotice fix with priceset template

Don't throw PHP notice when rendering dedupefind template.

Enotice fix with priceset template

Enotice fix

Notice fixes

These primarily affect 'grumpy-mode'

Enotice fixes with datepicker

Datepicker requires some fields to be set to avoid enotices (at least with grumpy mode disabled).

Merge pull request #22704 from braders/remove-invalidprofiles-template-variable

Remove unused invalidProfiles variable, and simplify template.

authx - Extend support for legacy-auth to APIv4

Convert existing `X-Requested-With:` guards to  `isWebServiceRequest()`

_Overview_:  `civicrm/ajax/api4` and `returnJsonResponse()` inspect the
web-request to see if it comes via AJAX/REST. If so, the call is
allowed and formatted as JSON. The patch refines the test.

_Before_: `X-Requested-With:` signals that a call is AJAX/REST.

_After_: `X-Requested-With:` still signals that a call is AJAX/REST.
Additionally, if `authx` is enabled, then some requests will be treated as
AJAX/REST based on how they are authenticated (ie `xheader`/`X-Civi-Auth:`
and `param`/`?_authx=` are AJAX/REST).

(REF) CRM_Utils_REST::isWebServiceRequest - Mark public. Expand comments.

Remove unused invalidProfiles variable, and simplify template.
As well as simplifying things, this avoids PHP notices.