Merge pull request #22355 from colemanw/searchKitAddJoins

SearchKit - Enable joins for custom fields and option groups

Merge pull request #22496 from civicrm/5.46

5.46

Merge pull request #22443 from eileenmcnaughton/download

Remove support for deprecated key

Merge pull request #22485 from eileenmcnaughton/ab

Fully remove contactAB function

Merge pull request #22498 from eileenmcnaughton/mem_act

Don't default membership type create to inactive

Merge pull request #22464 from colemanw/extRequirementCheck

Extensions - Add dependency status check

Don't default membership type create to inactive

We don't do this for any other entity...

Merge pull request #22494 from eileenmcnaughton/fin

Remove handling for impossible array

Merge pull request #22490 from colemanw/refreshButton

SearchKit - Facilitate popup forms

Merge pull request #22403 from colemanw/api4WriteRecords

[REF] Improve APIv4 save functions

Merge pull request #22493 from eileenmcnaughton/546

Cleanup test class

Merge pull request #22467 from colemanw/civigrantSearchKitTab

CiviGrant - Use SearchKit for contact summary tab

Merge pull request #22492 from braders/system-utils-docblock

Improve method phpdoc for CRM_Utils_System

Remove handling for impossible array

CRM_Financial_BAO_FinancialType::buildPermissionedClause is only called from one place
outside of tests. That place passes in this->_whereClause which is the result of calling
this->whereClause() a few lines earlier. As that function always returns a string
we know that it cannot be an array & can remove the handling

Cleanup test class

crmDialog - Add convenience binding to automatically launch a dialog when clicking an element

SearchKit - Auto-refresh search when popup forms submit

This expands the listener to the entire surrounding <form> element,
in case there are related popups on the Afform.

Improve method phpdoc for CRM_Utils_System

CiviGrant - Ensure dependencies are installed before running tests

Merge pull request #22486 from eileenmcnaughton/rel_check

Remove always-true-if

Remove always-true-if

We just ensured organization is numeric so .. it is

Fully remove contactAB function

The function was a really long winded way of doing what the 3 lines do.

Universe search did not reveal other usages and it is not the
sort of function we would expect to be used by extensions so
I'm straight removing it

Merge pull request #22468 from eileenmcnaughton/ab

Stop calling confusing function that doesn't do much .....

Merge pull request #22484 from civicrm/5.46

5.46 to master

Merge pull request #22479 from demeritcowboy/fulltext-quickfix-546

dev/core#3026 - 5.46 version of 22478

quickfix for fulltext search drupal block

Merge pull request #22469 from eileenmcnaughton/help

Remove seemingly unreachable help text

Merge pull request #22472 from eileenmcnaughton/import

Move function to the class that 'owns' it

Merge pull request #22451 from MegaphoneJon/member-count-fix

Crash when adding a relationship when CiviMember is disabled

Merge pull request #22474 from braders/incoming-mail-broken-fn-reference

Fix broken function reference

fix accessing member/contribution count when not enabled

Move function to the class that 'owns' it

Following https://github.com/civicrm/civicrm-core/commit/f54e87d9cf1e216036adf92f744cdbac3c1fe79e#diff-5864d6a1d03dffb7699b930837c1c222b41fba16dad7d0a5f73b4923ac53dd42R198
neither of these functions are called from outside their class (I did a universe search)
so making them private (& in one case moving off the parent) means a universe seach won't be needed
next time

Merge pull request #22473 from eileenmcnaughton/script

Remove broken script

Fix broken function reference

Merge pull request #22459 from mlutfy/core983acl

dev/core#983 Fix Access CiviCampaign permissions (instead of admin)

Remove broken script

Remove seemingly unreachable help text

This text is e-notice causing (with default modifiers on). I could not find the tpl file
or the potentially corresponding php file & concluded they were likely long-gone

(If I'm wrong some possibly-outdated help text disappears but I couldnt find it)

Stop calling confusing function that doesn't do much .....

Merge pull request #22448 from mattwire/recurid

Simplify getting contact_id from recurring contribution when generating subscription URLs

Merge pull request #22456 from eileenmcnaughton/legt

Remove & deprecate silly function

CiviGrant - Use SearchKit for contact summary tab

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

Remove & deprecate silly function

This function just calls another function & it's slightly MORE complicted to call that then the function
it calls - deprecate it....

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

Extensions - Add dependency status check

Adds a system status check to ensure extension dependencies are met

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

APIv4 - Refactor writeObjects to choose BAO function based on annotations

Instead of using a hardcoded list of BAO functions to call, with extra
hardcoded params to e.g. the Contact & Address BAOs, this switches
to picking the BAO's create/add only if it's not `@deprecated`,
and falling back to WriteRecords.

Also refactors the large `writeObjects` function into two smaller
functions so individual action classes can more easily override them.

Merge pull request #22454 from braders/phpdoc-types

A whole bunch of phpdoc fixes

dev/core#983 Fix Access CiviCampaign permissions (instead of admin)

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 getting contact_id from recurring contribution

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

Remove support for deprecated key

downloadUrl causes notices when default modifier is on.

It turns out is was deprecated approximately 2 forevers ago

https://docs.civicrm.org/dev/en/latest/extensions/info-xml/#downloadUrl

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