Block crud for v4 api

Move validator out of BAO into v3 api

m

m

dev/core#2486 Add v4 Mailing api (+ MailingJob)

ro

Merge pull request #22789 from civicrm/5.47

5.47

Merge pull request #22787 from eileenmcnaughton/t

Update unit test to use exception

Merge pull request #22499 from eileenmcnaughton/fin3

Follow up fix on #22685 financial acl move to extension

Merge pull request #22785 from colemanw/mailingTemplateOptions

Mailing - Add `serialize` and `add` to `template_options` field

Update unit test to use exception

Note that this also updates the money formatting on the contributionSoft list

Mailing - Add `serialize` and `add` to `template_options` field

No need for special handling to json_encode the field, the BAO
takes care of that automatically when `serialize` is declared.

Test coverage in api_v3_MailingTest::testMailerCreateTemplateOptions

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

Fix for #22685 financial acl aggression

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

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.