Merge pull request #22784 from mattwire/recurformstests

RecurForms test fixes

Merge pull request #22753 from eileenmcnaughton/mail

BAO_Mailing::create - stop passing by reference

Merge pull request #22766 from eileenmcnaughton/mailit

Move check for mailing workflow permisisons to BAO layer

Merge pull request #22782 from civicrm/5.47

5.47

RecurForms test fixes

Merge pull request #22781 from iswilson/patch-2

Contributor key: Add Ian Wilson

Merge pull request #22778 from colemanw/noMoreSearchBuilder

Remove "Search Builder" from menubar on new installs.

Contributor key: Add Ian Wilson

Merge pull request #22724 from braders/feature/group-search-null-columns

Avoid null column showing on groups search page

Merge pull request #22667 from braders/avoid-notices-uf-settings-page

Avoid PHP notices on the UF settings page.

Merge pull request #22768 from colemanw/searchKitDedupe

SearchKit - Add 'merge contacts' task

Merge pull request #22780 from civicrm/5.47

5.47 to master

Merge pull request #22779 from eileenmcnaughton/pledgey

dev/core#3066 - Check CiviPledge is enabled before disconnecting pledge payments

Merge pull request #22754 from colemanw/extensionApi

APiv4 - Add Extension.get

Merge pull request #22740 from colemanw/dev/core#3063

dev/core#3063 APIv3 - Fix numeric option matching

Merge pull request #22543 from colemanw/deprecateRetrieve

[REF] - Deprecate & delegate BAO::retrieve functions

dev/core#3066 - Add check for CiviPledge component

Check that component is enabled before attempting to disconnect pledge payments from Failed / Cancelled contributions.

Remove "Search Builder" from menubar on new installs.

Search Builder is deprecated in favor of SearchKit, so the menubar should
stop showing both.

Merge pull request #22277 from demeritcowboy/isdir2

dev/core#2927 - Avoid flooding logs with open_basedir in effect

Merge pull request #22633 from eileenmcnaughton/msg_template

dev/core#2866 Ignore preferred mail format when sending message

Merge pull request #22775 from eileenmcnaughton/pref

Follow up fix on removing preferred_mail_format

Merge pull request #22776 from colemanw/fixApi4EnttiyGet

APIv4 - Fix typo in Entity.getFields

Merge pull request #22749 from eileenmcnaughton/liney

Fix apiv4 Contribution delete & all line items

APIv4 - Fix typo in Entity.getFields

dev/core#2866 Ignore preferred mail format when sending message

Follow up fix on removing preferred_mail_format

APiv4 - Add Extension.get

Merge pull request #22635 from eileenmcnaughton/trait

dev/core#2866 Remove preferred_mail_format from email trait

Merge pull request #22730 from colemanw/fixOptionValueNames

Pseudoconstants - Fix and improve handling of option callbacks

Merge pull request #22773 from seamuslee001/npm_audit_fix

[REF] Results of running npm audit fix

Merge pull request #22770 from colemanw/searchKitEmptyText

SearchKit - Customizable "No Results" text

[REF] Results of running npm audit fix

SearchKit - Customizable "No Results" text

DedupeMerge - Cleanup form code to fix redirects and undefined variables

Fixes undefined template variables, and does the final redirect in an ajax-friendly way.

Merge pull request #21539 from mattwire/deprecaterecurtrxnid

Deprecate civicrm_contribution_recur.trxn_id

SearchKit - Add 'merge contacts' task

Merge pull request #22679 from mattwire/dummyreturn

Clean return values for dummy payment processor

Add upgrader to update comments on civicrm_contribution_recur table

Deprecate civicrm_contribution_recur.trxn_id

Merge pull request #22765 from eileenmcnaughton/block_2

[REF] Simplification in Contact::getValues

Merge pull request #22764 from colemanw/searchKitFixNonDAO

SearchKit - Fix support for non-DAO entities

Move check for mailing workflow permisisons to BAO layer

Merge pull request #22745 from eileenmcnaughton/noticey

Some more notice fixes in smarty

[REF] Simplification in Contact::getValues

The call to location::getValues takes more lines & is
less readable than just doing the thing.

Note that
1) I could not find any evidence of 'noWebsite' being passed in so I switched to
always add
2) the signature on website:getValues is slightly different (of course).
3) I found that the others that were receiving params as a reference were
not altering it so removed those pass-by-refs

Merge pull request #22763 from demeritcowboy/classclassclass

Adjust css class for payment processor to use `-`

SearchKit - Fix support for non-DAO entities

Before: SearchKit would crash if running a search on a non-DAO entity (Extension, Entity, Afform, etc)
After: Works, test added

Merge pull request #22422 from eileenmcnaughton/taxtermprice

Cleanup around taxTerm assignment

class

Merge pull request #22757 from eileenmcnaughton/block_2

[REF] Simplify Location::getValues

Merge pull request #22756 from eileenmcnaughton/block

Remove unused parameter, pass-by-ref

Merge pull request #22760 from seamuslee001/fix_space_in_paymentprocess_class_name

[REF] Fix issue where spaces in payment_processor_type.name field cau…

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

Merge pull request #22743 from civicrm/5.47

5.47

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