true

Merge pull request #27513 from eileenmcnaughton/mailing

dev/core#4567 Add mailing_id, is_test to civicrm_mailing_event_queue

Merge pull request #27542 from civicrm/5.66

5.66

Merge pull request #27483 from eileenmcnaughton/8_notice

Php 8.x clean up notices on Profiles

Merge pull request #27533 from eileenmcnaughton/hash_less

CiviMail - Generate email auth-code as random string

php8.x Fix notices on displaying FieldSet

Stop using mode as magic for showing the fieldset

Loosen regex

Merge pull request #27536 from totten/master-getclass

PHP 7.x - Use get_class() instead of $object::class

Merge pull request #27537 from eileenmcnaughton/recur

Fix mistake from last week on string wrangling

Fix mistake from last week on string wrangling

This came from
https://github.com/civicrm/civicrm-core/pull/27419

PHP 7.x - Use get_class() instead of $object::class

CiviMail - Fixup edge-case chars. Use consistent function.

1. Base64 can output some funny chars that are awkward in VERP and URL contexts.
   Use strict lowercase alphanumeric.
2. The new code was being used for the test mailing - but not for real mailings. They should use same formula.

Merge pull request #27409 from seamuslee001/5.66

[REF] Fix bug caused by recent ACL changes where combination of edit …

Use random string rather than sha string

As discussed on chat
https://chat.civicrm.org/civicrm/pl/r7h6cc7xo78zjgsd8af6ff9tee

this function just generates a hard-to-guess string that is stored
in the civicrm_mailing_event_bounce table. This unique value is
used in conjunction with the unique id value from the table to
determine if it is a match but it's not ever reverse calculated
so we don't need to use a reversable function

Merge pull request #27472 from colemanw/noteForm

API - Improve ACL processing, deprecate `hook_civicrm_notePrivacy`

Merge pull request #27534 from artfulrobot/artfulrobot-fix-group-removal-count

Stop saying N contacts removed when they were already removed.

Stop saying N contacts removed when they were already removed.

Add userId and conditions to hook_civicrm_selectWhereClause signature

Note - Deprecate hook_civicrm_notePrivacy in favor of hook_civicrm_selectWhereClause

This enables note privacy to be enforced by SearchKit and the API,
and gives an upgrade path for implementations of hook_civicrm_notePrivacy to be refactored out.

DAO/APIv4 - Optimize ACL clauses

Before: no values passed to BAO::addSelectWhereClause

After: $entityName and $conditions passed.
$entityName is rarely needed, but useful for dynamic entities like ECK.
$conditions contains field/value pairs gleaned from the WHERE clause or ON clause
  (depending on how the entity was added to the query)

API/DAO - Improve permissioning sql clauses: support field names and OR

Before: selectWhereClause was limited to returning arrays of clauses which would be
joined with AND. They could not reference any other fields on the entity.

After: selectWhereClause can return sub-arrays which are joined with OR,
and can reference any field on the entity using {curly_brace} syntax.

Stricter type checking emits noisy deprecations if arrays are expected and strings are given.

DAO - addSelectWhereClause should always return an array

Activity - More accurate calculation of permittedActivityTypes

The strict array key comparison seemed wrong, array_diff_key works better.
I was hoping this would be more efficient by skipping the where clause for
privledge users, but it doesn't. Oh well at least this is more correct.

Make Note.privacy required

Merge pull request #27531 from eileenmcnaughton/verify

CiviMail - VERP should no longer require job_id for verification

Fix verify verp function to cope with job_id having been deleted

This allows the record to be validated based on queue_id & hash
(which is still pretty insane to brute force & allows
us to purge job records

Add mailing_id, is_test to civicrm_mailing_event_queue

Merge pull request #27529 from civicrm/5.66

5.66 to master

Merge pull request #27528 from demeritcowboy/weight-again

Fix weights again on contribution page settings dropdown

fix weights again

Merge pull request #27527 from colemanw/searchToolbar

SearchKit Toolbar - Fix conditionals, add tests

SearchKit Toolbar - Fix conditionals, add tests

Merge pull request #27525 from larssandergreen/Remove-spaces-around-curly-braces-to-prepare-for-Smarty-3+

Remove spaces around curly braces to prepare for Smarty 3+

Remove spaces around curly braces to prepare for Smarty 3+

Merge pull request #27521 from civicrm/5.66

5.66

Merge pull request #27520 from colemanw/dfk

APIv4 - Add dfk metadata to getfields

Merge pull request #27476 from eileenmcnaughton/part_amounts

Add test cover, use token for `event.is_share`

APIv4 - Hack around bad dfk metadata in unit tests

APIv4 - Add dfk metadata to getfields

(mostly) fixes some bad guesswork in the APIv4 conformance test,
and now the dfk_entities metadata is available in getFields

Merge pull request #27487 from eileenmcnaughton/pay_later

Fix notices in text versions of event receipts

Merge pull request #27518 from demeritcowboy/bread

dev/core#4605 - Add yet another guard against failed upgrades

Merge pull request #27517 from eileenmcnaughton/custom_param

Move nasty chunk of code to where it is used

guard during upgrades

Merge pull request #27516 from eileenmcnaughton/participant82

Php8.x Fix notices when creating custom field

Move nasty chunk of code to where it is used

This code to get the customFields array is tough going. I haven't unravelled it, just moved it from being
figured out in one function & used in another such that it is figured out in the
chunk of code it is used in.

This will probably fix some php 8.2 tests because rather than all tests
hitting it only those that actually have custom fields + receipts do.

It's a bit of a fake win but it will reduce fails & make further cleanup easier

Php8.x Fix notices when creating custom field

Merge pull request #27514 from eileenmcnaughton/regen

Run regen

Run regen

Merge pull request #27510 from totten/master-event-dao

Regenerate CRM_Event_DAO_Event

Fix notices in text versions of event receipts

Pretty minor change but gets past the notices

Regenerate CRM_Event_DAO_Event

Recently changed by #27478

Merge pull request #27478 from eileenmcnaughton/carty

Move Conference slots functionality to event cart

Merge pull request #27482 from eileenmcnaughton/show_cms

Consolidate Profile Preview with other profile code

Merge pull request #27488 from eileenmcnaughton/event_notice_custom_post

php 8.x Ensure profiles are assigned to template, even when empty

Merge pull request #27497 from eileenmcnaughton/sessions_offline

Remove event cart sessions handling from offline event receipt text as gone from html

Merge pull request #27502 from eileenmcnaughton/legacy

Fix legacy custom searches to not use hacky method to access protected property on core class

Merge pull request #27509 from eileenmcnaughton/renew

php8.2 stop using yet-another undeclared property for _membershipId.

Merge pull request #27503 from eileenmcnaughton/getvar

Add deprecation annotations for getVar & setVar

Merge pull request #27508 from civicrm/5.66

5.66

Merge pull request #27507 from eileenmcnaughton/survey

Remove getVar call, access protected property in same class directly

php8.2 stop using yet-another undeclared property for _membershipId.

Instead follow our new practice of having consistent (externally supported)
functions for getting the main id variables

Merge pull request #27475 from totten/5.66-comp-status

Status Check - Ensure that components are enabled

Remove getVar call, access protected property in same class directly

Merge pull request #27494 from agh1/5.66.0-releasenotes-initial

5.66.0 release notes initial run

Merge pull request #27499 from mlutfy/cleanupAndSend

[NFC] Mail Logger: cleanup CIVICRM_MAIL_LOG_AND SEND

Fix legacy custom searches to not use hacky method to access protected property on core class

This avoids passing the form object to a function simply to extract the
links value & instead only calls the form object is it actually implements
the links function - which they basically don't...

This still works - ie I checked the actions are still available in custom searches
but respects the principle that protected variables are not public

Merge pull request #27504 from civicrm/5.66

5.66 to master

Merge pull request #27501 from eileenmcnaughton/weight

Add a couple more weights - legacy searches

Add deprecation annotations for getVar & setVar

Add a couple more weights - legacy searches

Merge pull request #27500 from civicrm/5.66

5.66 to master

Merge pull request #27492 from pradpnayak/entitybatch

make grouptree protected so that it can be accessed via hook

Mail Logger: cleanup CIVICRM_MAIL_LOG_AND SEND

Merge pull request #27481 from totten/5.66-upg-comp-timing

(dev/core#4521) Upgrader - Enable component-extensions before system-flush

Merge pull request #27485 from eileenmcnaughton/payment_fields

[php 8.2] Declare _paymentFields with public visibility

make grouptree protected so that it can be accessed via hook

Remove event cart sessions handling from offline event receipt text as gone from html

Merge pull request #27490 from eileenmcnaughton/activity_report

php 8.2, ensure properties are declared on activity report

Merge pull request #27480 from eileenmcnaughton/templates

Add some missing weights

Exclude inactive groups on recipient lists

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

5.66.0 release notes: added boilerplate

5.66.0 release notes: raw from script

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

php 8.2, ensure properties are declared on activity report

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

Ensure profiles are assigned to template, even when empty

Declare _paymentFields with public visibility

Consolicate Profile Preview with other profile code

Add some missing weights

Jenkins rejected the deprecated function in the style check

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.

Upgrader - 5.63 upgrade should use simple extension installation

Upgrader - Multiple fixes for new enableSimpleExtension()

Upgrader - Implement addSimpleExtensionTask(). Update docblocks.

Move Conference slots functionality to event cart