Merge pull request #22064 from colemanw/civigrant

Migrate CiviGrant component to an extension

Merge pull request #22264 from sunilpawar/dev_recurring_error

Fix Fatal Error on contribution tab and user dashboard when recurring payment generated using extension get disabled

Merge pull request #22462 from eileenmcnaughton/inv2

Remove unused variable

Merge pull request #22455 from eileenmcnaughton/legg

Remove IF that only loops through one value

Merge pull request #22421 from eileenmcnaughton/tax

Replace all isset checks on line.tax_rate

Merge pull request #22453 from braders/autorespond-failure

Fix syntax error in CRM_Mailing_Event_BAO_Reply

Merge pull request #22313 from mlutfy/eventDefaultUrlValues

Event Registration: allow default Price Set values from the URL

Merge pull request #22457 from eileenmcnaughton/label

Mark function as private

Remove unused variable

Merge pull request #22460 from civicrm/5.46

5.46

Merge pull request #22445 from eileenmcnaughton/546

Use field value function to get html signature

Merge pull request #22454 from braders/phpdoc-types

A whole bunch of phpdoc fixes

Ensure tax is set when it comes form Order BAO

Replace all isset checks on line.tax_rate

It is actually always set here
https://github.com/civicrm/civicrm-core/blob/64f4eebe75dbaeb429abcaa4dfca90094ce8be20/CRM/Price/BAO/LineItem.php#L246-L253

Remove IF that only loops through one value

Mark function as private

I did a universe search & this is not called outside this class (the
one on the LabelCommon class is called from one place outside that class

A whole bunch of phpdoc fixes

Fix syntax error in CRM_Mailing_Event_BAO_Reply

CiviGrant - Regenerate SQL

CiviGrant - Install extension + managed entities during upgrade

Disable & delete component, install extension as either "enabled" or "disabled" depending on the status of the component.
Migrate option values to managed entities
Migrate navigation menu to extension managed entities

CiviGrant - Migrate legacy search query code

This fixes up the `CRM_Contact_BAO_Query_Interface` class which had incorrectly
omitted `static` from some static functions.

Adds defaultReurnProperties to search hook interface.

CiviGrant - Remove or mitigate hardcoded references to civicrm_grant table in core

Removes CiviGrant from hardcoded lists when possible, or at least puts up
guards to check that it's available before attempting to use it.

CiviGrant - Move unit tests

CiviGrant - Move report files

CiviGrant - Migrate schema files

Insert and then delete CiviGrant to avoid messing up legacy code with expectations of component IDs.

CiviGrant - Migrate component functions to hooks

CiviGrant - Move CRM & API code from core to extension

Generate CiviGrant extension

Merge pull request #22439 from eileenmcnaughton/legt

Simplify variables pass in & out of function

Merge pull request #22447 from eileenmcnaughton/legg

Remove if & foreach that no longer do anything

Merge pull request #22446 from eileenmcnaughton/php4

Remove call to php 4 function

Simplify variables pass in & out of function

Rather than compiling an array, pass in the 3 values. Also
its now obvious that there is only a max of 1 contact_check in the function
so just return rather than 'continuing' (allowing us to de-loop in a more
whitespacey follow up)

Remove if & foreach that no longer do anything

Remove call to php 4 function

This CRM-2122 seems to be php4 thing - it won't exist anymore

https://php.uz/manual/en/function.fribidi-log2vis.php

Use field value function to get html signature

The  function is used for all other tokens on the contact object and will better access
the right value and return an empty string rather NULL if not set.

I hit this as an enotice writing a test - but it is possibly the cause of this issue
https://github.com/eileenmcnaughton/nz.co.fuzion.civitoken/issues/58#issuecomment-966611004

hence targetting the rc

The tokens() function does weird things if you pass it NULL

Merge pull request #22440 from eileenmcnaughton/legg

Clean up input and outputs

Merge pull request #22441 from civicrm/5.46

5.46

Merge pull request #22435 from colemanw/subscriptionHistory

APIv4 - Add SubscriptionHistory entity

Merge pull request #22436 from eileenmcnaughton/noticeme

Enotice fix

Merge pull request #22437 from MegaphoneJon/broken-search-urls

fixes report#93: SearchKit shows broken external URLs

Merge pull request #22429 from colemanw/fixMoneyType2

Fix input type for smarty number formatting (more forgiving)

Clean up input and outputs

By grepping universe we can see that params['contact_check'] only ever has one value when called
from the supported core place and the unsupported jma grant application extension place.

By clarifying that here we can do further cleanup without having to re-search universe...

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

fixes report#93: SearchKit shows broken external URLs

Enotice fix

APIv4 - Add SubscriptionHistory entity

Fix input type for smarty number formatting

Some smarty templates pass NULL to crmMoney, so it needs to handle that possibilty

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