Merge pull request #22404 from eileenmcnaughton/hooky

Stop calling hook twice

Stop calling hook twice

Per the code comments this hook is called here but also in add. This is all about the crazy
history of this function - which used to be tied to the relationship form.

I can't think of any valid use for calling the hook twice - but I can think of reasons it
would be bad...

Merge pull request #22399 from eileenmcnaughton/data

Remove always-true IF

Merge pull request #22398 from eileenmcnaughton/rel_add

[REF] Remove unused variable

Merge pull request #22397 from eileenmcnaughton/silly

Remove unnecessary pass-by-ref

Merge pull request #22320 from colemanw/activityDateMeta

Add metadata so Activity Created Date is available in Form Builder

Merge pull request #22379 from eileenmcnaughton/not1

Ensure assigned smarty array has empty keys (not no keys)

Merge pull request #22402 from eileenmcnaughton/no_rel

Clarify contents of ids for further fixes

Set version to 5.47.alpha1

Set version to 5.46.beta1

Merge pull request #22393 from eileenmcnaughton/no_ref

Remove unnecessary pass-by-ref

Merge pull request #22392 from eileenmcnaughton/protect

Privatise function

Clarify contents of ids for further fixes

Remove always-true IF

Merge pull request #22391 from eileenmcnaughton/legacy

[REF] Remove handling for relationshipID

Merge pull request #22394 from eileenmcnaughton/return

Ensure getDuplicateContacts always returns an array

[REF] Remove unused variable

Remove unnecessary pass-by-ref

Ensure getDuplicateContacts always returns an array

Calling code already handles an empty array as that is what it would receive - unless the hook set it to NULL

Remove unnecessary pass-by-ref

Privatise function

Merge pull request #22389 from colemanw/contactTypes

REF - Use `CRM_Contact_BAO_ContactType::basicTypes()` instead of hardcoded lists

[REF] Remove handling for relationshipID

We can see that the only call to this function does not set params['id'] & hence it is NULL

Merge pull request #22383 from colemanw/dedupeRuleScreenFixes

DedupeRules - Translate contact type labels, respect enabled contact types

Merge pull request #22390 from eileenmcnaughton/legacy2

[REF] Further cleanup on employer create

Merge pull request #22319 from colemanw/afformSearchFilters

Afform - UI and contextual titles for search displays

Merge pull request #22323 from eileenmcnaughton/taxterm

Smarty modifier - stop using isset to check taxTerm

Merge pull request #21178 from eileenmcnaughton/perm

dev/core#2752 Allow financial_trxns to be viewed

[REF] Simplify handing of ids

Rather than compile contactID into an array to pass it in pass as itself. It is then
clear that ids['relationship'] will always be null - so remove the check

REF - Use `CRM_Contact_BAO_ContactType::basicTypes()` instead of hardcoded lists

Makes code more flexible/forgiving if new contact types are added in the future,
or if existing contact types are disabled.

Merge pull request #22388 from colemanw/unusedFunctionDel

Remove unused function

Remove unused function

Duplicate legacyCreateMultiple to contact_utils

Having a lot of trouble fixing a bug via this method so gonn break up & clean up the function in this context
(leaves on the import class & a JMA extension calling it.

Merge pull request #22369 from braders/dedupeRulesSystemCheck

Add system status check for missing dedupe rules

Merge pull request #22378 from eileenmcnaughton/notice

Ensure dashboardLinks assigned to prevent e-notice

Merge pull request #22382 from colemanw/api4ExplorerFormatArrayParams

APIv4 Explorer - Format array params using add methods instead of set

DedupeRules - Translate contact type labels, respect enabled contact types

Add system status check for missing dedupe rules

APIv4 Explorer - Format array params using add methods instead of set when available

Improves OOP generated code by using sugar methods `addGroupBy()` and `addRecords()`

Merge pull request #22267 from eileenmcnaughton/cont_test2

[Ref] Add getter for priceSetID and use full form flow

Ensure assigned smarty array has empty keys (not no keys)

Ensure dashboardLinks assigned to prevent e-notice

Rather than no assign assign an empty array. It is processed with a foreach so all good

Merge pull request #22366 from colemanw/fixSearchKitEditableDates

SearchKit - Fix editable dates on standalone form

Merge pull request #22367 from colemanw/searchKitViewButton

SearchKit - Contextual "View" button

dev/core#2752 Allow financial_trxns to be viewed

Per https://lab.civicrm.org/dev/core/-/issues/2752 this allows non admins to
see payment details in search for contributions they can see.
I only added for 'get' because I didn't want to loosen 'delete'
(in particular) and this was the minimum extra to achieve
search access

Merge pull request #22373 from demeritcowboy/memwarn

Fix E_WARNING access array offset on null when creating new contribution page

Merge pull request #22375 from demeritcowboy/searchkit-chicken-egg

dev/report#90 - Don't crash search_kit on upgrade from 5.35

Merge pull request #22377 from eileenmcnaughton/format

dev/core#2773 Add an ACL to demo data

Merge pull request #22372 from eileenmcnaughton/acl4

enotice fix (with default modifiers)

Add an ACL to demo data

This creates an acl user (jenny@example.com) who can view members of the Volunteer group.
A change will also be needed in buildkit to add her user account

Merge pull request #22365 from colemanw/importMultiValuedCustom

Fix "Import Multi-Value Custom Data"

Merge pull request #22371 from eileenmcnaughton/acl2

Do not escape showHideBlocks by default

Merge pull request #22374 from eileenmcnaughton/format

Formatting cleanup

don't crash on upgrade from 5.35

Merge pull request #21483 from jmcclelland/leaky-honoree-variable

Ensure we don't get previously set honoree values in receipts

Formatting cleanup

Merge pull request #22370 from eileenmcnaughton/acl

Fix notices on acl page

enotice fix (with default modifiers)

fix array element access on null

Merge pull request #22356 from colemanw/searchKitLinks

SearchKit - Allow aggregate columns to be links

Do not escape showHideBlocks by default

Affects civicrm/acl?action=add&reset=1

Fix notices on acl page

civicrm/acl?reset=1

Merge pull request #21517 from mattwire/propertybagrecurinstallments

Fix PropertyBag setRecurInstallments to accept 0

Regenerate sql for new "Import Custom Data" menu item

Merge pull request #22309 from eileenmcnaughton/brickit

Use new money formatting util for smarty formatting

Merge pull request #21916 from mattwire/recurprocessorid

dev/financial#57 Hide recur trxn_id if it matches recur processor_id

Merge pull request #22361 from braders/phone-email-label-accessibility

Improve accessibility - associate label and fields

SearchKit - Contextual "View" button

This should make it less confusing. When you click on the View button it will take you to the display you are currently editing.
When not on a display, the View button looks different so it's easier to tell where it's taking you.
Fixes dev/core#3016

SearchKit - Fix editable dates on standalone form

For better or worse, the crmDatePicker widget requires a parent <form> element.
Fixes dev/core#3017

Import - Fix loading and saving multiValued custom data field mapping

Import - Fix multi-record custom data import PHP errors

Fixes fatal PHP error `Declaration of CRM_Custom_Import_Parser::run(...) should be compatible with CRM_Contact_Import_Parser::run(...)`
by switching to inherit the base class instead of the contact import class. This simplifies the code in general.

Merge pull request #22364 from eileenmcnaughton/defaults

Fix campaign dashboard to work with default escaping

Merge pull request #22363 from eileenmcnaughton/noticey2

Fix over-escaping when default escaping is on

Merge pull request #22362 from eileenmcnaughton/noticey

Fix escape-by-default

Fix campaign dashboard to work with default escaping

Fix over-escaping when default escaping is on

Fix escape-by-default

Merge pull request #22357 from yashodha/dev-3012

(dev/core#3012) Expose email on hold as filter for report

Merge pull request #22359 from colemanw/api4ActivityCreate

APIv4 - Set 'activity_type_id' to required

Cleanup multi-record custom import tpl and warn user if no multi-custom groups exist.

Add menu item for "Import MultiValued Custom Data"

Merge pull request #22360 from braders/import-field-docblock-improvements

[NFC] docblock improvements to Import_Field classes

Improve accessibility - associate label and fields

docblock improvements to Import_Field classes

APIv4 - Set 'activity_type_id' to required

In APIv4 GetFields, sets activity field to required because the
database default is problematic.

Merge pull request #22349 from braders/chart-label-translatable

Allow chart dropdown labels to be translatable.

Allow chart dropdown labels to be translatable.

Prior to this change, the strings 'Tabular', 'Bar Chart' and 'Pie Chart' were hardcoded.

Merge pull request #22318 from jitendrapurohit/state_name

Add state name to export fields

(dev/core#3012) expose email on hold as filter for report

SearchKit - Allow aggregate columns to be links

Previously aggregate columns were excluded from being shown as links.
That's overly strict. They can't have autogenerated links, but they can
have custom links.

Always assign taxTerm to templates

I do wonder about making this a domain token - I go back & forth a bit but as
a domain token it would be available in non-smarty contexts too & could be exported

Merge pull request #22350 from demeritcowboy/managed-entities-drupal

dev/drupal#172 - module status used incorrectly in drupal 8 to determine which modules to care about for managed entities

Merge pull request #22347 from braders/price-set-unnecessary-ids

Remove unnecessary id attribute.

Merge pull request #22354 from civicrm/5.45

5.45

Merge pull request #22353 from seamuslee001/5.45

[REF] Fix tests that fail after 2021

fix expired credit card

fix tests that will fail after 2021

Merge pull request #22352 from demeritcowboy/stuck-in-2020

Fix tests that fail after 2021

fix expired credit card

fix tests that will fail after 2021