financial#160 - set correct from_financial_account_id on an edited fee amount FinancialTrxn

Merge pull request #19102 from eileenmcnaughton/err

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

[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

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

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

Use settingsFactory intead of settings for angular modules.

We are moving toward the use of settingsFactory and away from settings for angular modules.
This would allow the list of modules to be cached as it would only contain static metadata.

Merge pull request #19048 from colemanw/angComment

[NFC] Remove boilerplate comment from .ang.php files

Merge pull request #19054 from eileenmcnaughton/pp1

[REF] Simplify sendNotification determination

Remove boilerplate comment from .ang.php files

Link to wiki was outdated and comment was not very helpful

[REF] Simplify sendNotification determination

Delete outdated/unused crmExample Angular module

Decouple crmD3 angular module from CiviMail

The crmD3 module was being conditionally loaded based on whether CiviMail was enabled.
This changes it to always exist but only loaded as required.

Update Resource URL Helptext

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

Merge pull request #19043 from eileenmcnaughton/pp

dev/financial#148 fully deprecate validateData function

Merge pull request #19042 from eileenmcnaughton/pps

Remove legacy check

Merge pull request #19005 from totten/master-afform-dashlet

Afform - Generate dashlets based on `Afform.is_dashlet` property.

dev/financial#148 fully deprecate validateData function

Remove legacy check

This check dates right back to when we started extracting IPN code from code blocks in the processor. I don't believe it's meaningful

Merge pull request #19040 from eileenmcnaughton/pp2

[REF] remove obscure use of objects from A.net