Merge pull request #27491 from JKingsnorth/patch-21

Exclude inactive groups on recipient lists for mailings

Merge pull request #27489 from eileenmcnaughton/payment_block

Fix last notices on Online event registration by removing variables

Merge pull request #27461 from colemanw/colType

SearchKit - Add crm-search-col-type-* css classes

Merge pull request #27477 from eileenmcnaughton/confirm

Deprecate Form_Event::testSubmit, migrate test to full flow

Exclude inactive groups on recipient lists

Fix last notices on Online event registration by removing variables

I hunted through to see why these variables are assigned
and they pre-date the switch to load the payment block from the

CRM_Financial_Form_Payment

There is no evidence of that form looking them up in
any way - so I think they can go....
https://github.com/civicrm/civicrm-core/commit/aaff4c697d6f94f01f5dacf5a3d876a95e50f5cb#diff-660258b71b4d1db49bdef38c7b3fcb4b698004ab39b57dcbd49083e256444272

Deprecate Form_Event::testSubmit, migrate test to full clow

In the procss I looked at what the test is actually testing & the goal of the test is ensure the
doPayment contract is met, specifically regarding contactID
https://docs.civicrm.org/dev/en/latest/extensions/payment-processors/create/#getpaymentformfields

I updated the test to use the alterPaymentProcessorParams hook to actually check for that.

Merge pull request #27469 from eileenmcnaughton/no_event_cart

Stop enabling event_cart on new installs

Merge pull request #27456 from eileenmcnaughton/user_email

PHP8.2 Remove undefined property userName

Stop enabling event_cart on new installs

Although we have not fully extracted this, it seems that disabling does not
cause any errors, so it makes sense to stop enabling. Once we have been running
like this on our dev & demo sites for a while we can disable it
for any existing sites that do not have
event cart enabled

Merge pull request #27473 from eileenmcnaughton/style

Rename locally assigned variable for clarity

Rename locally assigned variable for clarity

Remove undefined property userName

This is causing issues on a bunch of classes - but in fact
is only used on the ContributionForm

Merge pull request #27468 from eileenmcnaughton/contribution_json

Hardened smarty - fix mishandling of json

Merge pull request #27467 from eileenmcnaughton/contribution_trivia

Superficial tidy up in backoffice contribution form

Merge pull request #27466 from eileenmcnaughton/part_

Fix test template to use the same tokens as the templates

Hardened smarty - fix mishandling of json

Superficial tidy up in backoffice contribution form

Fix template to use the same tokens as the templates

Merge pull request #27465 from civicrm/5.66

5.66

Merge pull request #27464 from demeritcowboy/oh-snap

Upgrade - don't crash on earlier versions of mariadb

don't crash on earlier versions

Merge pull request #27462 from demeritcowboy/dao

DAO out of date

Merge pull request #27463 from demeritcowboy/regexp

[NFC] phpunit 9 compatibility

regexp

dao out of date

Merge pull request #27460 from braders/scheduled-job-dynamic-properties

[REF][PHP8.2] Tidy up properties in scheduled jobs

Merge pull request #27455 from eileenmcnaughton/conf

php 8.x Remove conference sessions from core workflow messages

Merge pull request #27458 from eileenmcnaughton/online_more

php 8 / notices Fix online event receipt to use the same location tokens as offline

SearchKit - Add crm-search-col-type-* css classes

Use token for event ID

Fix online event receipt to use the same location tokens as offline

Remove conference sessions from core workflow messages

These cause notices is some circumstance and it seems to make more
sense to remove than support them.

They are only available to be configured when the Event Cart is in play so
it's not clear it makes sense to fix core code for them

The Event cart is kinda on it's way out & I don't think it makes sense
to maintain conference sessions in the core templates to support this
extension which despite many attempts has never had much use & has always
been somewhat broken AFAIK.

Note that this change would affect sites with non-customised templates
but in time I think we would either move the assign to the cart or stop
assigning it

Also not the goal is to provide event-cart type functionality via form
builder

Merge pull request #27457 from eileenmcnaughton/online

Whitespace clean up on event online receipt

Merge pull request #27232 from artfulrobot/artfulrobot-add-smart-group-profiler

Add simple smart group profiler

[REF][PHP8.2] Tidy up properties in scheduled jobs

Whitespace clean up on event online receipt

Update xml/schema/Contact/Group.xml

Co-authored-by: colemanw <coleman@civicrm.org>

Move from 5.66 to 5.67

Move cache_fill_took to float/double; only update when there's a good value

Change xml added and upgrader from 5.65 to 5.66

Move to using CRM_Utils_Constant for CIVICRM_SLOW_SMART_GROUP_SECONDS

Add xml, DAO and Upgrader task for smart group profiler

Remove smart group profiling during populateTemporaryTableWithContactsInGroups

Expose 5s 'slow' query to optional constant

Minor nfc tweak to smart group profile code

Add simple smart group profiler

Merge pull request #27430 from colemanw/bunnyDance

Resurrect Bunny Dance

Merge pull request #27303 from eileenmcnaughton/mailing_const

Remove unused/ unnecessary table name variables

Merge pull request #27417 from colemanw/searchKitSpreasheetPerformance

SearchKit - Improve performance of downloading spreadsheet

Merge pull request #27453 from colemanw/extComponentCheck

Add system check for missing component extensions

Add system check for missing component extensions

CRM_Case_ManagedEntities - Fix bugs caused by double-caching

The list of case types was being cached twice, once by CRM_Core_Pseudoconstant
and yet again in CRM_Case_XMLRepository::$allCaseTypes.
This made getting a fresh list of case types unnecessarily difficult.

Merge pull request #27454 from civicrm/5.66

5.66

Merge pull request #27452 from eileenmcnaughton/weight

Add weights on profile fields page (notices

Merge pull request #27451 from eileenmcnaughton/email_comments

Add code comments re the cleanup I didn't do

Merge pull request #27450 from demeritcowboy/weight566

backport of 27448

Merge pull request #27391 from eileenmcnaughton/user_email

Fix undefined properties on back-office pledge form

Merge pull request #27435 from eileenmcnaughton/uf_profile_prefix

8.x Fix all instances of call Profile template to specify the prefix

Add weights on profile fields page

ie https://dmaster.localhost:32353/civicrm/admin/uf/group/field?reset=1&action=browse&gid=1

Add code comments re the cleanup I didn't do

I want the code to be stable in 5.66 for a few releases, absent any bugs so
commenting the further cleanup for 'one day'

fix order

Fix all instances of call Profile template to specify the prefix

Merge pull request #27449 from braders/php8-CustomValueTest

[REF][PHP8.2] Remove dynamic property use in api_v3_CustomValueTest

[REF][PHP8.2] Remove dynamic property use in api_v3_CustomValueTest

Merge pull request #27447 from demeritcowboy/block-local

Fix e_warning in smarty `{localize}`

Merge pull request #27425 from eileenmcnaughton/transfer

Remove some unused undeclared properties on SelfSvcTransfer form

Merge pull request #27424 from eileenmcnaughton/properties

Suppress rather than declare exception

Merge pull request #27448 from demeritcowboy/contributionpageweight

fix ordering of contribution page links

CiviCase - Optimize managed reconcile

CiviCase - Prevent recursion when reconciling managed activity/relationship types & fix test

The createManagedRelationshipTypes and createManagedActivityTypes functions always use 'civicrm'
as the module name, so this reconciliation can target just that module to prevent recursion.

The ManagedCaseTypeTest was failing due to caching/timing issues.

Try this...

Merge pull request #27434 from eileenmcnaughton/uf_array

8.x Fix a couple of notices when rendering profiles

Merge pull request #27431 from eileenmcnaughton/delete_participant

Fix notices when deleting participants

Merge pull request #27444 from eileenmcnaughton/super

Superficial cleanup on Event Confirm - null object

fix order

Merge pull request #27442 from eileenmcnaughton/all

Remove unused variable from function signature

e_warning

Merge pull request #27445 from civicrm/5.66

5.66

Merge pull request #27433 from eileenmcnaughton/help_pre

8.x Profiles Remove e-notices by removing never assigned help_pre, help_post

Merge pull request #27443 from eileenmcnaughton/part_

Fix variable leakage in online event receipt

Merge pull request #27407 from colemanw/contactNoteTab

SearchKit - Improve metadata, joins and links

Merge pull request #27428 from eileenmcnaughton/custom_weight

Fix missing weights on Custom data screen (notices)

Update BunnyDance.mgd.php

Superficial cleanup on Event Confirm

str_starts_with is in our polyfill as @colemanw pointed out

Fix variable leakage in online event receipt

Merge pull request #27438 from eileenmcnaughton/assign

Fix some variables to always assign

Merge pull request #27437 from eileenmcnaughton/no_1028

Remove references to invalid action

Merge pull request #27439 from eileenmcnaughton/captca

Remove broken recaptcha from profile preview

Merge pull request #27441 from eileenmcnaughton/test_set

Remove isset from template within unit tests

Move scenario set up to trait

Remove unused variable from function signature

Remove isset from template within unit tests

Merge pull request #27440 from civicrm/5.66

5.66 to master

Remove broken recaptcha from profile preview

The reCaptcha variable is never assigned to profile preview - resulting
in it never rendering (but rendering notices). This code has been broken
for a while without someone noticing - remove

Merge pull request #27436 from eileenmcnaughton/dep

Add deprecation notice to code that seems to be superceded

Fix some variables to always assign

Fix notices when deleting participants

Remove outdated code comment

Remove references to invalid action

All these references refer to comparing action to 1028. The comments make is clear that
they are looking for 'preview' mode - but preview mode is 1024. The checks are all
does-not-equal. So, they would have always been true for a very long time.

Copy & paste is dead long live copy & paste

Merge pull request #27432 from eileenmcnaughton/profile_weight

Add weights on profile links (remove notices)