Merge pull request #22582 from civicrm/5.46

5.46

Merge pull request #21865 from colemanw/SearchKitTags

SearchKit - add/remove tags action for all taggable entities

Merge pull request #22537 from jensschuppe/fix/typeErrorCustomFieldTokens

[dev/core#3029] Avoid risking a TypeError when evaluating tokens for non-existent custom fields

Merge pull request #22577 from braders/remove-method-getWordReplacement

Remove method getWordReplacement.

Merge pull request #22576 from seamuslee001/update_jquery_ui

[REF] Upgrade JQuery UI to 1.13.0

Merge pull request #22574 from braders/send_unsub_response-phpdoc

(NFC) Minor phpdoc fixes to CRM_Mailing_Event_BAO_Unsubscribe::send_unsub_response

Merge pull request #22480 from eileenmcnaughton/create_date

Add created_date & modified_date to civicrm_relationship

Remove method getWordReplacement.

It appears to have never been used since its introduction 8 years ago,
and furthermore calls functions which don't exist.

Therefore, it would be unlikely to work even if it were used.

Merge pull request #22575 from braders/getGroupByFromSelectColumns-tidy

(NFC) Minor amends to getGroupByFromSelectColumns

[REF] Upgrade JQuery UI to 1.13.0

(NFC) Minor amends to getGroupByFromSelectColumns

(NFC) Minor phpdoc fixes to CRM_Mailing_Event_BAO_Unsubscribe::send_unsub_response

Merge pull request #22562 from braders/hook-preevent-docblock

(NFC) Correctly document CRM_Utils_Hook::pre to allow null ID

Merge pull request #22573 from colemanw/componentFn

[REF] Components - Use extracted function

Components - Use extracted function

Uses function extracted in #22567

Merge pull request #22458 from eileenmcnaughton/test

dev/core#2757 test to demonstrate contact delete actions were not called for v4 api

Merge pull request #22565 from braders/report-form-docblock

(NFC) CiviReport, etal - More phpdoc fixes

Merge pull request #22507 from colemanw/searchImportCleanup

[REF] SearchKit - Simplify import/export popup code

Merge pull request #22568 from demeritcowboy/details-wrong-name

Activity Details field on batch update via profile has wrong fieldname

Add created_date & modified_date to civicrm_relationship

As with other tables we have done this to, knowing when we added a record to the
DB is useful (and differs from the relationship start_date).

I'm not sure yet if I will try to expose in the UI or whether search-kit
is all we need

Merge pull request #22569 from demeritcowboy/duplicate-file

Remove duplicate file

remove duplicate file

Merge pull request #22563 from braders/alterDeferredRevenueItems-phpdoc

(NFC) Update PHPDoc for alterDeferredRevenueItems

Merge pull request #22567 from eileenmcnaughton/enabled

(REF) Extract function `CRM_Core_Component::isEnabled()`

fix wrong field name

Add function Core_Component::isEnabled

It always feels like there should be an easier to discover way to check if
a component is enabled ....

More phpdoc fixes, this time related to report form methods

Merge pull request #22555 from eileenmcnaughton/stricter

(NFC) Comment fixes Stricter phpcs

Merge pull request #22553 from eileenmcnaughton/css

(NFC) Remaining css phpcs fixes

Update PHPDoc for alterDeferredRevenueItems

Correctly document CRM_Utils_Hook::pre to allow null ID

[dev/core#3029] Throw and catch exception instead of risking a TypeError when evaluating tokens for non-existent custom fields

Merge pull request #22556 from eileenmcnaughton/from_where

Follow up fn removal

Merge pull request #22549 from eileenmcnaughton/tool_clean

Remove obsolete scripts

Follow up fn removal

Merge pull request #22542 from colemanw/deprecateGetName

[REF] SavedSearch - Deprecate/remove redundant/unused functions

Remainin css phpcs fixes

Stricter phpcs

Merge pull request #22466 from eileenmcnaughton/searchtype

Re-fix enotice on searchType to work with default modifiers

Merge pull request #22550 from eileenmcnaughton/tool_clean1

Remove obsolete postIPN script

Merge pull request #22529 from eileenmcnaughton/phpcs4

PHPCS standards on css....

Merge pull request #22544 from colemanw/deconstructor

(REF) Remove constructors that do nothing

Merge pull request #22545 from eileenmcnaughton/build

Remove unused variable

Remove obsolete postIPN script

Per https://github.com/civicrm/civicrm-core/pull/22534#issuecomment-1014934985

Remove obsolete scripts

Per https://github.com/civicrm/civicrm-core/pull/22534#issuecomment-1014932778

Merge pull request #22534 from eileenmcnaughton/tools

Remove some old scripts

Re-fix enotice on searchType to work with default modifiers

Use a value that we ensure we assign

Remove unused variable

This is over-written a couple of lines later in the template so
we don't need it & ergo don't need to assign it either

Note that the paramter is assigned in postProcess for the
receipt so this is just this usage

Remove unused pass-by-reference

This pass-by-ref was causing `civilint` to crash and appears to be unnecessary and possibly harmful.

Remove constructors that do nothing

Merge pull request #22541 from braders/buildpremiumpreviewblock-refactor2

Refactor buildPremiumPreviewBlock function to make intended usage clearer

SavedSearch - Deprecate/remove redundant/unused functions

Refactor buildPremiumPreviewBlock function to make intended usage clearer.

Merge pull request #22540 from civicrm/5.46

5.46 to master

Merge pull request #22539 from colemanw/searchKitTokenFix

SearchKit - Fix token selector to use expression alias

Merge pull request #22533 from eileenmcnaughton/phpcs

More comments & style fixes

SearchKit - Fix token selector to use expression alias

Merge pull request #22470 from eileenmcnaughton/isAdmin

Smarty notice fix on Email help

Merge pull request #22536 from eileenmcnaughton/regen

Re-run regen

Merge pull request #22535 from eileenmcnaughton/style

Fixes to spaces from phpcs

Re-run regen

Fixes to spaces from phpcs

Remove some old scripts

Merge pull request #22531 from seamuslee001/example_regenerate

[NFC] Update test example template and regenerate examples

More comments & style fixes

This is mostly generated by code sniffer but I also added the deprecation to the isEnvironment function
as I wound up having to do cleanup on something never called....

Merge pull request #22530 from eileenmcnaughton/category

Remove category from case activity template

[NFC] Update test example template and regenerate examples

Merge pull request #22527 from eileenmcnaughton/phpcs2

Run stricter phpcs on Civi dir

Remove category from case activity template

Merge pull request #22528 from eileenmcnaughton/phpcs3

Run 'full drupal strictness' standard over api/v3/examples

Merge pull request #22444 from eileenmcnaughton/isform

Remove empty check

Merge pull request #22517 from eileenmcnaughton/skip

Move deprecated function back to the only class that calls it

Merge pull request #22526 from eileenmcnaughton/phpcs

Fix php comments

PHPCS standards on css....

I ran phpcbf over our code base & it made these changes to the css - I guess the
standard must prefer them....

Run 'full drupal strictness' standard over api/v3/examples

I just removed all the rules we weren't enforcing cos too-hard from
our drupal code standard & ran phpcbf over the directory. I figure
we aren't really updating these so getting them to what strict phpcs will
tolerate could save time when we next tighten the noose
and if someone ever regenerates them
they can deal with re-running it or improving the tpl

Merge pull request #22482 from eileenmcnaughton/token

Add unit test on updateGreeting & remove deprecated fn call

Run stricter phpcs on Civi dir

Merge pull request #22520 from colemanw/contactTypeCleanup

[REF] - Use CRM_Contact_BAO_ContactType::basicTypes()

Remove empty check

We had to ensure this was always assigned for default modifier so remove the
empty to make it clear it is 'sorted'

Fix php comments

As per https://github.com/civicrm/civicrm-core/pull/22515

Merge pull request #22524 from braders/contacttype-types

[NFC] Fixes to the type definitions for contactType methods

Merge pull request #22525 from braders/remove-charts-colours

[Ref] Remove unused _colours property

Merge pull request #22522 from braders/phpdoc-fixes-2

Another set of PHPDoc fixes

Merge pull request #22521 from braders/phpdoc-links

Fix more phpdoc comments in member links functions

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

Merge pull request #22518 from colemanw/cleanupBadCode

Remove unreachable, nonsensical code

Merge pull request #22515 from eileenmcnaughton/phpcs

Fix php comments

Merge pull request #22442 from demeritcowboy/tests-22429

Unit tests for 22429 and apply same treatment to other money format functions

[Ref] Remove unused _colours property

[NFC] Fixes to the type definitions for contactType methods

Another set of PHPDoc fixes

Fix more phpdoc comments in member links functions

Fix php comments

This commit contains 14 files worth of changes from the following steps

1) install https://github.com/civicrm/coder.git
2) comment out <rule ref=Drupal.Commenting.FunctionComment.IncorrectParamVarName><severity>0</severity></rule>
3) run phpcbf --standard=Drupal

This follows on from efforts from @braders - there are not many
instances (although I kept the comment small) and we could fairly easily get rid of them
all and enable that rule in CI - which would lift the bar a little

Merge pull request #22497 from eileenmcnaughton/mem_ids

Test cleanup - remove direct calls to BAO_Membership::add

Move deprecated function back to the only class that calls it

This function is only called from the importer (in core) - copy back there & noisily
deprecate (one known extension usage).

Note that I removed 2 parts of it in the copy back that are unreachable
as only specific params are passed in we know they don't include
case & always include skipRecent

https://github.com/civicrm/civicrm-core/blob/5484728757de4c85d44305a92bfa2aa35351fbce/CRM/Contact/Import/Parser/Contact.php#L880-L887

Merge pull request #22516 from eileenmcnaughton/legacy2

Merge function back into parent function

Remove unused frozen form blocks

These blocks appear to be unused because these forms are never displayed in frozen "view" mode.
If they were in use they would be broken because the urls are incorrectly output raw
instead of being properly passed to `CRM_Utils_System::url()`.

Remove unreachable, nonsensical code

This block of code appears to be unreachable, because the contribution page
wizard is never frozen in "view" mode.
If it were reachable, it would do nothing sensible, as the button would link
to a completely unrelated page (configure custom data) and the link would be
broken because it is incorrectly bypassing `CRM_Utils_System::url()`.

Merge function back into parent function

It isn't a logical separation of responsibility & doesn't reduce code

Merge pull request #22514 from braders/feature/report-statistics-docblock

Add type to phpdoc comment for report statistic function