Merge pull request #22428 from civicrm/5.46

5.46

Merge pull request #20884 from MegaphoneJon/financial-89

don't allow multiple currencies in a batch

Merge pull request #22431 from eileenmcnaughton/leg

[REF] Rename variable to be meaningful

Merge pull request #22430 from colemanw/navigationDefaults

Navigation - Make required fields NOT NULL

Merge pull request #22433 from braders/gotv-tpl-formatting

[REF] Tidy up formatting in gotv.tpl, remove extra </div>

Merge pull request #22432 from braders/tidy-report-template-code

Tidy CRM_Report_Page_Options code

Merge pull request #22434 from braders/phpdoc-class-reference-fixes

Fix class references in phpdoc comments

Fix class references in phpdoc comments

Tidy up formatting in gotv.tpl, fix missing </div>

[REF] Rename variable to be meaningful

It turns out key is organizationID - rename ...

I did this in both places the function is called from - as I clean up the copy some
cleanups on the original emerge too

Merge pull request #22401 from eileenmcnaughton/legit

[REF] Stop returning unused variables

Merge pull request #22426 from colemanw/writeRecordsCustom

[REF] Centralize BAO handling of custom data

Navigation - Make required fields NOT NULL

APIv3 Navigation - Add defaults so generic writeRecord can be used instead of BAO::add

Merge pull request #22418 from demeritcowboy/isdir-7-8-546

[NFC] isDir unit test fails on php 7 'min' matrix

Merge pull request #22420 from eileenmcnaughton/case

Remove / deprecate case param handling

Merge pull request #22425 from braders/remove-htmlfile-rule

Remove htmlFile validation rule, as it can't possibly work.

Merge pull request #22417 from eileenmcnaughton/construct

Fight back against copy & paste gone wild

Tidy CRM_Report_Page_Options code

APIv3 - Refactor basic_create to use writeRecords when create/add functions are deprecated

- Stops passing the deprecated `$ids` param to BAO::create
- Uses BAO::writeRecord when BAO::create or BAO::add are `@deprecated`
- Trust BAO::writeRecord to handle custom data

DAO - Update writeRecords function to also save custom data

Refactors a couple BAOs which use writeRecords, as they no longer need to
write the custom data themselves.

Remove htmlFile validation rule,
as it can't possibly work.

Merge pull request #22424 from braders/query-param-phpdoc-fixes

Fix phpdoc types for $_query in CRM_Core_Selector_Base subclasses

Merge pull request #22423 from braders/queryformatter-docblock

[NFC] Tidy up phpdoc in CRM_Utils_QueryFormatter

Fix phpdoc types for $_query in CRM_Core_Selector_Base subclasses

[NFC] Tidy up phpdoc in CRM_Utils_QueryFormatter

Merge pull request #22416 from eileenmcnaughton/depit

Increase GroupContact:create deprecation

Remove / deprecate case code

It's clear it can't be passed in from the separated off version & it also is not reached from
the importer - which leaves only external usages (not supported but deprecation will do for now)

Merge pull request #22410 from eileenmcnaughton/rel

Stop returning unused parameters

[REF] Stop returning unused variables

Only 2 code places call this function (one an unsupported call outside core but in universe & the other in Parser_Contact)
and only the latter uses ANY of the return variables - this fixes the function to stop
returning those that are never used  - this stops them from returning them

Fight back against copy & paste gone wild

Merge pull request #22415 from colemanw/deprecatedDocblocks

APIv4 - Docblock updates, mark BAO::create functions deprecated

Merge pull request #22414 from colemanw/fixTest

APIv4 - Fix test

Merge pull request #22413 from colemanw/fixDecodeRows

APIv4 - Fix incorrect placement of decodeRows in loop

fails in unusual situation on php 7

Increase GroupContact:create deprecation

Note the v3 api does not call this & no-where in core calls it directly.

APIv4 - Docblock updates, mark BAO::create functions deprecated

Fix test

Test can give false failure depending on site's contact display_name settings.

APIv4 - Fix incorrect placement of decodeRows in loop

The funciton is designed to loop through all items, so needs to be placed
outside the loop that creates the items.

Merge pull request #22406 from eileenmcnaughton/rels

Remove unreachable else

Merge pull request #22395 from eileenmcnaughton/legy

[REF] Remove handling for no organization being passed

Merge pull request #22411 from eileenmcnaughton/dep

Remove deprecated code

Merge pull request #22409 from colemanw/upgradeDocblock

NFC - Fix typo in docblock

Remove deprecated code

Simplify determination of organizationID

[REF] Remove handling for no organization being passed

Stop returning unused parameters

NFC - Fix typo in docblock

Merge pull request #22407 from eileenmcnaughton/rel

Remove check for contact_check as it is always an array here

Merge pull request #20561 from WeMoveEU/fix/double-autosave

dev/core#2587 Ignore modified_date when checking for changes during mailing auto-save

Merge pull request #22274 from eileenmcnaughton/mems

Fix testMultipleMembershipsContribution to use order api

Remove check for contact_check as it is always an array here

Remove unreachable else

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

Fix testMultipleMembershipsContribution to use order api

This test was not creating valid financial records prior to this

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