[NFC] Minor margin tidy up

Just fixes all the things that get marked as 'not best practice' in the IDE

Merge pull request #19011 from demeritcowboy/event-enotice

E_NOTICE when deleting participant

E_NOTICE when deleting participant

Merge pull request #19004 from civicrm/5.32

5.32

Merge pull request #18987 from colemanw/moduleBundles

Allow Angular modules to require Resource bundles

Merge pull request #19001 from eileenmcnaughton/532

Include oauth-client in tarball

Merge pull request #19003 from civicrm/5.32

5.32 to master

Merge pull request #18999 from colemanw/searchStuff

Search kit bug fixes & List display

Include oauth-client in tarball

Merge pull request #18994 from totten/master-ang-json

crm-ui-debug - If in debug mode, then load pretty-printer for JSON data

Merge pull request #19000 from civicrm/5.32

5.32

Merge pull request #18996 from kcristiano/5.32.1931

dev/core#1931 Prevent PayPal from double-encoding the IPN Notify URL 5.32 backport

Search kit: Fix issues with variable pollution and clean up shared display code

Merge pull request #18772 from eileenmcnaughton/ids

dev/core#2066 Extract getSelectedIDs

Merge pull request #18983 from eileenmcnaughton/trans

[REF] Remove always FALSE variable from transitionComponents

crm-ui-debug - If in debug mode, then load pretty-printer for JSON data

Prevent PayPal from double-encoding the IPN Notify URL

Merge pull request #18986 from demeritcowboy/drupal-permissions-page

dev/drupal#146 - Wrong link for Drupal 8 permissions page / Make CMS permissions url lookup more OO-ey

Merge pull request #18980 from christianwach/lab-core-1931

dev/core#1931 Prevent PayPal from double-encoding the IPN Notify URL

Search ext: Fix duplicate group names on search admin page

Search ext: Add List display

Breaks search displays into their own modules, one per display type.
This gives finer-grained dependency management per display type.

Search ext: fix loss of configuration while changing search criteria

oo-ify permissions page links

Merge pull request #18993 from giant-rabbit/153-fix-paypal-redirect

dev/financial#153 Fix redirect to PayPal

Merge pull request #18988 from seamuslee001/dev_core_2197

dev/core#2197 Deploy monaco-editor using composer.json / composer dow…

Fix redirect to PayPal

If you put in double slashes in the path part of the PayPal URL and then
PayPal asks you to do their captcha, they have some sort of bug where
they do the wrong redirect after the captcha to https://cgi-bin/webscr.

Issue #153

dev/core#2197 Deploy monaco-editor using composer.json / composer downloads method

Remove setup.sh as no longer needed and npm related package files

Merge pull request #18991 from demeritcowboy/serialize-notice

dev/core#2196 - serialize E_NOTICE when saving custom field

Merge pull request #18989 from totten/master-afform-label

afform - Get default field `<label>` from `label` instead of `title`

Merge pull request #18981 from mattwire/extensionsuitweak

Extensionsuitweak

afform - Get default field `<label>` from `label` instead of `title`

gitignore - Whitelist ext/afform

fix serialize E_NOTICE

Merge pull request #18990 from totten/master-afform-gui-url

afform/gui - Fix rendering of the "Form Settings"

afform/gui - Fix rendering of the "Form Settings"

The `<textarea/>` tag is misinterpreted, causing the subsequent fields to disappear.

Allow Angular modules to require Resource bundles

Merge pull request #18985 from demeritcowboy/create-table-logging

dev/core#2190 - Remove unused lines, replace CRM_Utils_Array::value

don't assume tablespec exists

[REF] Remove always FALSE variable from transitionComponents

This is not ever passed in now we have removed the call from membership form

I've kept this PR restricted to changes in transitionComponents
since that means the main reviewer task is to confirm the
paramter is never passed in & hence is false, but
quite a lot of other stuff can be ripped out as follow ups

Merge pull request #18979 from eileenmcnaughton/trans

Remove return value on transitionComponents

Tweak information displayed by extensions list so it's aimed more at the end-user rather than the developer

Merge pull request #18971 from colemanw/angularDashboard

Rewrite jquery.dashboard.js as Angular app

Prevent PayPal from double-encoding the IPN Notify URL

Rewrite jquery.dashboard.js as Angular app

New home dashboard written with Angular + APIv4.
Functionality is mostly unchanged.
The motivation for this is to support afforms embedded within dashboard dashlets.

Merge pull request #18950 from MegaphoneJon/event-44

event#44: Don't allow self-service cancelling after an event when 'hours to cancel' is zero

Remove return value on transitionComponents

We no longer use the result from this function so this removes most of the code that calculates the updateResult value.

There are still instances coming back from the cancel & fail functions that I will remove as a follow up

Merge pull request #18964 from eileenmcnaughton/trans

dev/core#2155 Remove obscure broken handling for onlinePendingContribution

Merge pull request #18960 from seamuslee001/utf8mb4_install

dev/core#339 Install CiviCRM tables using utf8mb4

Merge pull request #18977 from civicrm/5.32

5.32 to master

Merge pull request #18976 from colemanw/searchParams

Search ext: use crmRouteBinder to expose params for new searches to url

Search ext: use crmRouteBinder to expose params for new searches as url arguments

Merge pull request #18975 from colemanw/fixAfformDep

Fix undeclared afform requirement of ngRoute

Merge pull request #18974 from civicrm/5.32

5.32

Fix undeclared afform requirement of ngRoute

Merge pull request #18967 from seamuslee001/dev_core_2178

dev/core#2178 Fix regression on adding contacts to a newly created group

Merge pull request #18970 from MegaphoneJon/core-2188

fix default 'is searchable' to prevent breaking advanced search

dev/core#339 Install CiviCRM tables using utf8mb4

Update sample custom value tables as appropriate as well

Fix conflict between test setting ROW_FORMAT being compressed and the ROW_FORMAT of the original table being DYNAMIC

fix default 'is searchable' to prevent breaking advanced search

Merge pull request #18748 from eileenmcnaughton/log

dev/core#2120 Do not attempt to obsolete primary key on log tables

dev/core#2178 Fix regression on adding contacts to a newly created group

Merge pull request #18965 from MegaphoneJon/mailing-80

Don't break clickthrough URLs when there's leading whitespace

Merge pull request #18966 from demeritcowboy/nl-nl-default

[NFC] Follow-on to 18963 to add example setting in civicrm.settings template for setting CMS inheritance for nl_BE

Merge pull request #18962 from colemanw/dashboardPerms

API - Finer-grained permisions for the dashboard & dashlet entities

example setting

Merge pull request #18963 from samuelsov/nli18n

Add a way to choose between nl_NL and nl_BE

API - Finer-grained permisions for the dashboard & dashlet entities

This allows contacts without "administer CiviCRM" permission to access these api entities.
Permissions are restricted to contacts they have access to, and write permissions are
limited to a contact only editing their own dashboard.

Don't break clickthrough URLs when there's leading whitespace

dev/core#2155 Remove obscure broken handling for onlinePendingContribution

Per https://lab.civicrm.org/dev/core/-/issues/2155 there is some obscure functionality for
completing a pending contribution under some cases. My testing suggests it is long-term broken.

See GL for more

Add a way to choose between nl_NL and nl_BE

Merge pull request #18961 from eileenmcnaughton/msg

dev/core#2158 Remove detail about updating related components when editing a contribution status

dev/core#2158 Remove detail about updating related components when editing a contribution status

Note that with this merged it's also possible to remove a lot of code from transitionComponents

Merge pull request #18952 from eileenmcnaughton/aip

Standardise more params in completeOrder

Merge pull request #18912 from yashodha/dev_2153

(dev/core#2153) BUG: when changing a custom group option value CiviCR…

Standardise more params in completeOrder

This moves the loading of the following params from the first contribution from Authorize.net to
completeOrder
- amount_level
- address_id

In addition campaign_id is moved, however, this is only in it is not an
input param or derived from the recur record

Merge pull request #18956 from civicrm/5.32

5.32

Merge pull request #18958 from alifrumin/5.32rn

[NFC ] 5.32 Release Notes - first pass

5.32 release notes

Merge pull request #18955 from totten/5.32-oauth-return

dev/core#2141 - Tighten up new page `civicrm/oauth-client/return`

dev/core#2141 - Tighten up new page `civicrm/oauth-client/return`

Overview
--------

The route `civicrm/oauth-client/return` is added in 5.32 as the main "Redirect URL".

In normal usage, the page shouldn't be visible to a user (because the
developer should define some alternative UI) -- but one might see it (a)
during development, (b) if there's an error, or (c) if a clever user mucks
about.

Improvements:

* Error handling
    * Present error messages more nicely
    * Record errors in the log
    * Report more info via hook_oauthReturnError
* Other UI
    * Redact token details (dependent upon permission `manage OAuth client secrets`)
    * Set a more sensibile page title
    * Make output blobs conditional and collapsible

Merge pull request #18948 from aydun/core#2172

core#2172 - Fix 'Find Respondents' in CiviCampaigns

Merge pull request #18953 from civicrm/5.32

5.32

Merge pull request #18951 from totten/5.32-ms-imap

dev/mail#79 - Fix MS Exchange/IMAP. Use OpenID Connect.

dev/mail#79 - Use OpenID Connect with MS Exchange

At time of writing, there appears to be a problem in Microsoft's services:
if you request both `User.Read` and `IMAP.AccessAsUser.All`, then the token
does not actually work for IMAP access.

However, it is does work to combine `openid` and `IMAP.AccessAsUser.All`.

This patch revises the MS Exchange definition to get resource-owner details
via OpenID Connect's `id_token`.

dev/mail#79 - Add option to get resource-owner via OpenID Connect

At time of writing, there appears to be a problem in Microsoft's services:
if you request both `User.Read` and `IMAP.AccessAsUser.All`, then the token
does not actually work for IMAP access.

However, it is does work to combine `openid` and `IMAP.AccessAsUser.All`.

This patch to CiviGenericProvider makes it easier to get resource-owner
details via OpenID Connect's `id_token`.

Merge pull request #18949 from colemanw/legacyAutocomplete

APIv3 - Fix legacy handling for html_type Autocomplete-Select

event#44: Don't allow self-service cancelling after an event when 'hours to cancel' is zero

Fix legacy handling for html_type Autocomplete-Select

Legacy handling was returning "Autocomplete-Multi-Select" which was never a real html_type.

Fix SQL

Merge pull request #18946 from eileenmcnaughton/if

Remove always true if

Merge pull request #18947 from eileenmcnaughton/pps

dev/financial#148 dev/financial#148 Copy contents of validateDate onto paypalIPN

Merge pull request #18819 from eileenmcnaughton/import

Minor code cleanup - dedupe isNull checks

dev/financial#148 Copy contents of validateDate onnto paypalIPN

This code isn't really that shared & it can be simplified by NOT sharing it

Remove always true if

By the time the code reaches this point contributionParams['contribution_recur_id']
must be set. We can confirm this by the fact the call to getTemplateContribution
requires id to be passed in and uses it in a lookup. Also the
code just before repeatTransaction ensures it is called

Merge pull request #18934 from eileenmcnaughton/aip

Remove always-true if

Merge pull request #18935 from eileenmcnaughton/pps

[REF] Minor variable cleanup

Merge pull request #18936 from eileenmcnaughton/ppp

[Ref] Use variables directly

Merge pull request #18944 from civicrm/5.32

5.32

Merge pull request #18945 from agh1/5.32.0-releasenotes-fixedinitial

5.32.0 release notes initial run

5.32.0 release notes: added boilerplate

5.32.0 release notes: raw from script