Fix extensions tabs following deprecation of old tab code

Merge pull request #19061 from christianwach/lab-core-2217

Prevent "add_action" from being called multiple times

Merge pull request #19083 from eileenmcnaughton/contsoft

Add ContributionSoft v4 api

Merge pull request #19103 from eileenmcnaughton/import_extract

[REF] Extract duplicate handling code

Merge pull request #19066 from colemanw/tabCleanup

Fix dev/core#2215 & remove the tab selection inline script from TabHeader.tpl

Merge pull request #19072 from eileenmcnaughton/url

dev/financial#157 [REF] Extract functions

[REF] Extract duplicate handling code

This winds up being a very strange process - however the goal seems to be
to unravel error handling to appropriate exception throwing & catching
& a bit of back & forth helps us isolate the code places

Merge pull request #19108 from eileenmcnaughton/import

Remove obsolete IF

Remove obsolete IF

Per previous PR there is an early return making this if meaningless but
due to the whitespace change it was not in that PR

Merge pull request #19098 from eileenmcnaughton/import

[REF] Duplicate possibly-used parts of createProfileContact onto Parser_Contact

Minor cleanup in test class

Add ContributionSoft v4 api

Merge pull request #19102 from eileenmcnaughton/err

[REF] use early return for errors rather than confusing assignment

[REF] Duplicate possibly-used parts of createProfileContact onto Parser_Contact

The parser_Contact class should call the api. Getting to that point is
complicated because
1) it's such a mess and
2) it does a lot of work to prepare to call createProfileContact
and it's hard to unravel that.

By copying most of createProfileContact onto the class we can start to
unravel it & pare down to calling the api (note we went through
this process in the dedupe class, at least in part).

I'm pretty sure the hooks reflect 'the way the code grew up' rather than
any logical use/ requirement for the profile hook

[REF] use early return for errors rather than confusing assignment

This just makes the code slightly clearer. The next if can go, but
needs to wait on another PR to be merged to avoid conflict

Merge pull request #19095 from nishant-bhorodia/Issue#537-owner-notification-email-sending-with-every-save

Issue#537: Fix syntax to check pcp_id is NULL in api params

Merge pull request #18939 from colemanw/serializeFix

Fix updating custom field schema when toggling search or multiple

Merge pull request #19067 from eileenmcnaughton/weight

Quick fix on summary actions conflict

Issue#537: Fix syntax to check pcp_id is NULL in api params

Set version to 5.34.alpha1

Set version to 5.33.beta1

Merge pull request #19093 from civicrm/5.32

5.32

Merge pull request #19094 from totten/5.32-bump0

(NFC) release-notes/5.32.0 - Add RC-stage updates

release-notes/5.32.0 - Add RC-stage updates

Merge pull request #19092 from seamuslee001/dev_core_2228

dev/core#2228 Fix issue where by changing location type tries to pass…

Merge pull request #19091 from eileenmcnaughton/form

Fix fatal error on payment edit form

dev/core#2228 Fix issue where by changing location type tries to pass in number:... in the API

Fix fatal error on payment edit form

Merge pull request #19090 from civicrm/5.32

5.32

Merge pull request #19014 from eileenmcnaughton/cancel

dev/core#927 Add test demonstrating that an extraneous activity is being created & fix

Merge pull request #19088 from seamuslee001/dev_core_2227

Ensure that the is_deleted filter is added correctly if we are not se…

Ensure that the is_deleted filter is added correctly if we are not searching for deleted contacts but have access to view deleted contacts

Simplify IF down as per feed back from ACD and also comment from Eileen in chat and extend unit test as well

Merge pull request #19053 from totten/master-afform-tweak

Afform - Update fields and HTML mode in web-based editors

afGuiEditor - Add optional preview of HTML output

afGuiEditor - Switch canvas panel to use tabs

(1) This makes the canvas and palette line-up more nicely.

(2) This will allow more tabs.

Afform.convert - Add stateless helper API to convert between layouts

crmMonaco - Accept pass-thru arguments to Monaco

Convert af-monaco to crm-monaco

_af_fill_field_metadata() - Provide metadata about selected fields (regardless of DB permissions)

afHtmlEditor - Add descriptions. Add more missing fields.

afHtmlEditor - Use more BS3

afGuiEditor - Allow editing `is_public`

afGuiEditor - Add help blocks. Reorder in anticipation of more fields.

Merge pull request #19087 from civicrm/5.32

5.32

Merge pull request #19082 from compucorp/dev-core-1790-contact-card-delay

dev/core#1790: Add short delay before closing tooltip elements

Merge pull request #19084 from eileenmcnaughton/prod

dev/drupal#148 Fix product form such that it is possible to unseletc …

dev/drupal#148 Fix product form such that it is possible to unseletc duration unit, frequency unit

Merge pull request #19078 from seamuslee001/emoiji_test_api3

[NFC] Add in an Emoji test for APIv3 as per Eileen's PR

Merge pull request #19075 from colemanw/searchable

Improve schema metadata for Search Kit

dev/core#1790: Add short delay before closing tooltip elements

Merge pull request #19081 from civicrm/5.32

5.32

Merge pull request #19077 from seamuslee001/dev_core_2213

dev/core#2213 Protect against 404s when wpBasePage is mixed case

Merge pull request #19080 from seamuslee001/master

5.32

Merge in 5.32

Merge pull request #19076 from seamuslee001/update_cache_integration_tests

[NFC] Update Cache/integration-tests version as new version recently …

[NFC] Add in an Emoji test for APIv3 as per Eileen's PR

Merge pull request #18918 from eileenmcnaughton/emoji

dev/core#2165 Test for Handle emojis less fatally where not supported

Annotate DAO files with COMPONENT, exclude disabled components' entities from APIv4 Entity::get

Merge pull request #19044 from seamuslee001/dev_drupal_149

dev/drupal#149 Override sessionStart function for Drupal8 using appro…

dev/core#2213 Protect against 404s when wpBasePage is mixed case

[NFC] Update Cache/integration-tests version as new version recently released that contains the patches already applied

dev/core#2165 Test & fix for Handle emojis less fatally where not supported

Fix emoji handling by returning always-false when unwelcomely present

Add @searchable annotation to denote which API entities should be visible in Search Kit UI

Merge pull request #19063 from christianwach/lab-core-2213

Protect against 404s when wpBasePage is mixed case

Remove the tab selection inline script from TabHeader.tpl

This is part of ongoing cleanup to get the logic for selecting default tab
out of the tpl, which was redundant with the logic already in tabHeader.js.

dev/financial#157 [REF] Extract functions

This is just preliminary cleanup

Merge pull request #19070 from mattwire/extensionupgrademsg

Extract function that generates upgrade link for extensions

Extract function that generates upgrade link for extensions

Merge pull request #18612 from mattwire/tokenprocessorsimplify

REF Simplify tokenProcessor code

Quick fix on summary actions conflict

Per https://github.com/eileenmcnaughton/org.wikimedia.contacteditor/issues/8 &
https://github.com/mattwire/uk.co.mjwconsult.checksum/issues/1

menu items are indexed by weight, resulting in a conflict between 2 items with the same weight -
this is just a quick & dirty fix to ensure we don't overwrite one menu item with another

Merge pull request #19025 from sunilpawar/favourite_country

Feature to provide mostly used countries in top section of Country select list

Simplify tokenProcessor code for activities

Merge pull request #19065 from colemanw/campaignTabs

Minor tabs cleanup toward fixing dev/core#2215

Minor tabs cleanup

1. Avoid overwriting CRM.tabSettings.active by cloning the variable rather than copying by reference
2. Initialize tabs only once on the Campaign dashboard

Merge pull request #19059 from eileenmcnaughton/2199_ufgroup_pre_post_hook

Remove use of nullArray in delete hooks

Protect against 404s when wpBasePage is mixed case

Merge pull request #19060 from eileenmcnaughton/ufadd

Deprecate UFGroup::add parameter ids

Prevent "add_action" from being called multiple times

added test case, ts for help text in setting file, changed static variable logic

Deprecate UFGroup::add parameter ids

This is only passed in from one place. This PR removes it & adds noisy deprecation

Remove use of nullArray in delete hooks

Looking at #18995 I was struck by the delete hook being non-standard. In general we don't load from the DB just to pass
to the hook as the hook can do that itself. However, when I looked at other hooks
I found that they were passing out nullArray - this is a legacy method that precedes
being on a php version that supported default params when passing by reference.

It's further known to introduce intermittent hard to debug issues. This adds the new hook
and also adds standardisation to the other hooks for pre+delete.

I've left the create one for now but GroupContact is a good example of something
close to what we want to standardise on there

dev/drupal#149 Override sessionStart function for Drupal8 using appropriate session functions

Use Drupal Session service and don't worry about calling migrate

Merge pull request #19050 from eileenmcnaughton/pp

[REF] Remove excess handling around contact_id

Merge pull request #19051 from eileenmcnaughton/pps

Remove extra handling around contact id

Merge pull request #18916 from colemanw/crudLink

Remove unused hook_civicrm_crudLink and switch to using metadata for crudLinks

Merge pull request #19052 from colemanw/angModuleCleanup

[REF] Cleanup Ang modules in core to follow conventions

Merge pull request #19058 from mattwire/pcpshortcodeaction

wordpress#63 Add action parameter to PCP shortcode

Merge pull request #19046 from dwoods-encircle/Resource-URL-helptext-update

Update Resource URL Helptext

small change

changed lable and field name from favourite to Pinned

wordpress#63 Add action parameter to PCP shortcode

Merge pull request #19056 from civicrm/5.32

5.32

Merge pull request #19055 from colemanw/fixSearchActionButton

Search ext: Fix search action button

Search ext: Fix search action button

Remove extra handling around contact id

It's retrieved a few rows earlier from the contribution - this is from formerly
shared code that does not apply here

[REF] Remove excess handling around contact_id

The removed code was formerly in a shared function. On looking the
getIDS actually just gets the contact id from the contributionRecur
object & every thing else is extraneous

Merge pull request #19049 from colemanw/crmExampleGone

Delete outdated/unused crmExample Angular module

Merge pull request #19047 from colemanw/crmD3

[REF] Decouple crmD3 angular module from CiviMail

Merge pull request #19045 from seamuslee001/remove_xss_function

[REF] Remove xssString as it is providing a false sense of security

Fix crmStatusPage file name to match module name per convention