[NFC] Minor cleanup on test class

Merge pull request #19857 from eileenmcnaughton/task1

[REF] Extract getQueryParams

Merge pull request #19859 from civicrm/5.36

5.36

Merge pull request #19728 from totten/master-authx-guards

(dev/core#2077) AuthX - Allow optional "guards"

Merge pull request #19812 from eileenmcnaughton/recur

[REF] Clarify & simplify is_recur param

Merge pull request #19825 from colemanw/api4BridgeJoinSubquery

APIv4 - Use subquery to LEFT JOIN via a bridge entity

Extract getQueryParams

SearchKit - Add "EXCLUDE" join type, to search for entities that do not have a relationship to another entity

APIv4 - Use subquery to LEFT JOIN via a bridge entity

Bridge entities are supposed to be transparent in APIv4, but when LEFT joining,
an artifact of the double-join would give extraneous results.

APIv4 - Simplify long function by extracting part of it

Merge pull request #19828 from eileenmcnaughton/act_contact_hook

dev/core#2390 Add hook support for Activity Contact

Merge pull request #19842 from eileenmcnaughton/task

[unreleased regression] Fix export contribution to share tasktrait

Merge pull request #19853 from colemanw/api4ExplorerCssFix

Api4 Explorer - Prevent long results from breaking layout

Fix ts to E::ts

authx - If 'authx_guards' is set, then enforce them

authx - Declare setting "authx_guards"

Merge pull request #19826 from demeritcowboy/nomulticase

dev/core#2463 - Remove loop that since at least 5.23 does nothing for single cases and crashes for multiple, and for multiple recipients only uses the last activity id for 5.36+

Api4 Explorer - Prevent long results from breaking layout

Merge pull request #19852 from eileenmcnaughton/nfc_test

[NFC] Minor test cleanup

Merge pull request #19851 from demeritcowboy/ids-php74

dev/core#2366 - Failing test for php 7.4 and IDS for non-admins

[NFC] Minor test cleanup

Merge pull request #19850 from seamuslee001/mysql8_logging_test_fix

[NFC] Fix logging schema test to work on MySQL8 as it no longer outpu…

failing test for php 7.4 and IDS

[NFC] Fix logging schema test to work on MySQL8 as it no longer outputs a length for int columns

Merge pull request #19849 from seamuslee001/member_event_form_test_fixes

[NFC] Fix test failures on PHP 7.4 caused by either NULL being set fo…

Merge pull request #19847 from seamuslee001/fix_test_failures_mysql8

[NFC] Fix Test failures on MySQL 8 caused by change in output of eith…

Merge pull request #19846 from seamuslee001/fix_contribution_page_tests

[NFC] Fix PHP 7.4 errors on ContributionPage tests

Merge pull request #19845 from seamuslee001/fix_membership_type_test

[NFC] Fix Membership Type Test error on php7.4

Merge pull request #19844 from seamuslee001/php74_membership_api_fix

[REF] Fix APIv3 MembershipJob test failure on php7.4 by only doing ch…

[NFC] Fix test failures on PHP 7.4 caused by either NULL being set for the membership_type_id or Payment Processor not being set on the form

[NFC] Fix Test failures on MySQL 8 caused by change in output of either create table or Column names in queries

[NFC] Fix PHP 7.4 errors on ContributionPage tests

[NFC] Fix Membership Type Test error on php7.4

[REF] Fix APIv3 MembershipJob test failure on php7.4 by only doing checking  if it is not empty

Fix export contribution to share tasktrait

Merge pull request #19787 from demeritcowboy/cancel-button

dev/financial#170 - Cancel button on financial batch export is no longer an input so code needs updating

Merge pull request #19839 from tschuettler/core_2466

(dev/core#2466) - Drop HTML markup in exports for link custom fields

Merge pull request #19834 from eileenmcnaughton/seq

Code cleanup in bootstrap.php

Merge pull request #19835 from eileenmcnaughton/task

[REF] Minor function extraction

Drop HTML markup in exports for link custom fields

Merge pull request #19838 from eileenmcnaughton/nfc

[NFC] Minor coding standards cleanup

Merge pull request #19836 from eileenmcnaughton/legacy

Remove legacy phpunit param

[NFC] Minor coding standards cleanup

[REF] Minor function extraction

Merge pull request #19833 from eileenmcnaughton/test

Remove safe+mode setting

Remove legacy phpunit param

Merge pull request #19832 from seamuslee001/core_ext_void

[NFC] Add in Void statements on varios Core Ext Tests to future support PHP…

Merge pull request #19624 from agileware/CIVICRM-1670

Issue #2384 - Remove text about email receipts for each recurring contribution

Code cleanup in bootstrap.php

Remove safe+mode setting

This is php5.4 compatibility = not needed

Add in Void statements on varios Core Ext Tests to future support PHPUnit8

Merge pull request #19831 from colemanw/angularUpgrade

Upgrade angularJs from 1.8.0 to 1.8.2

Merge pull request #19753 from eileenmcnaughton/format

dev/core#1494 & dev/translation#65 call brickmoney number formatting, where supported

Merge pull request #19830 from kewljuice/patch-6

Fix deprecated Redis::delete() (issue #2468)

Upgrade angularJs from 1.8.0 to 1.8.2

Fix deprecated Redis::delete() (issue #2468)

Merge pull request #19827 from civicrm/5.36

5.36

dev/core#2390 Add hook support for Activity Contact

Alternative to https://github.com/civicrm/civicrm-core/pull/19623
per comments on that PR.

Add release-notes/5.35.1

(security/core#105) Joomla::authenticate() - Fix escaping

(security/core#105) Joomla::checkUserNameEmailExists - Fix mismatched escaping

This uses the escaping rule from CRM_Core_DAO to construct a query for JDatabaseDriver.
However, they use different connections and (therefore) could require different
escaping rules.

(security/core#104) CRM_Utils_System::authenticateKey - Use secure equality test

(security/core#97) PHP CLI guard is the opposite of correct

Escape api params in APIv4 Explorer

Purify PCP introductory text field

Escape information supplied by extensions to prevent XSS

security/core#97 Ensure that php scripts where applicable in sql and tools that should only be run in CLI can be run in CLI

Use more portable check for cli and add in 404 header as per Rich's comments

security/core#100 Escape uploaded data to prevent Reflected Cross site scripting from uploaded CSVs

remove loop that since at least 5.23 does nothing for single cases and crashes for multiple, and for multiple recipients only uses the last activity id for 5.36+

Merge pull request #19824 from eileenmcnaughton/url_form

Remove now-empty function

Merge pull request #19765 from colemanw/inPlaceEdit

SearchKit - Add "in-place edit" feature

Remove now-empty function

Merge pull request #19823 from tunbola/fix-isfrontpage-wrong-value

dev/core#2464: Fix Drupal Base 'isFrontEndPage' Returns Wrong Value After Saving A Settings Page

dev/core#2464: Rebuild Menu after clearing db cache

Merge pull request #19766 from WeMoveEU/faster-select2-groups

dev/core#2451 Faster processing of optgroups in select2 elements

Merge pull request #19816 from eileenmcnaughton/dupe_state

[NFC] Minor IDE cleanup in test class

Merge pull request #19821 from eileenmcnaughton/import

Remove an unnecessary ampersand

Remove an unnecessary ampersand

Merge pull request #19794 from MegaphoneJon/core-2459

dev/core#2459 Fix custom fields changed from multiple-choice data type to Text

Merge pull request #19798 from colemanw/api4cleanup

APIv4 - Code cleanup & improve links to @see annotations in Explorer

SearchKit - add draggable arrows when configuring display columns

SearchKit - Additional link settings for opening in new tab or popup

SearchKit - Add "in-place edit" feature

Merge pull request #19813 from eileenmcnaughton/dupe

[NFC] IDE cleanup on test class

Merge pull request #19815 from civicrm/5.36

5.36 to master

[NFC] IDE cleanup on test class

[NFC] Minor IDE cleanup

Merge pull request #19811 from jmcclelland/reserve-respondents-5.36

avoid error when reserving respondents in a survey.

[REF] Clarify & simplify is_recur param

The only time is_recur leaves the function is when it is passed to
the payment processor so this simplifies so it is only set in that
array & otherwise consistently calculated

avoid error when reserving respondents in a survey.

The full error is:

Error: Call to a member function getSelectedIDs() on null in CRM_Campaign_Form_Task->preProcess() (line 38 of /var/www/powerbase/sites/all/modules/civicrm/CRM/Campaign/Form/Task.php).

I'm not sure if we should somehow be passing &$form to this function
instead?

Merge pull request #19779 from artfulrobot/artfulrobot-improve-quicksearch

Improve quicksearch result presentation to enable browser link functions

Merge pull request #19803 from eileenmcnaughton/little_bit_super

[REF] Minor extraction to help cleanup permission code

Merge pull request #19810 from jaapjansma/dev_2447

Fix for dev/core#2447: wrong event fee stored

Merge pull request #19805 from MegaphoneJon/escape-boolean-booleans

core#2461 - allow booleans to be of type boolean

Merge pull request #19807 from jaapjansma/dev_2449

Fixed issue dev/core#2449: remove an unnecessary call to cleanMoney

Fix for dev/core#2447: wrong event fee stored

Fixed issue dev/core#2449

Improve docs in comments

Fixed issue dev/core#2449

Merge pull request #19799 from eileenmcnaughton/mem_recur

[REF] Minor parameter cleanup