Merge pull request #27427 from eileenmcnaughton/notice_it

Fix another EventInfo notice

Fix EmailMode

Fix customData Include notice

Add getEventID() - this is part of an effort to ensure consistency

All event & participant forms should have public api functions for
getParticipantID() and getEventID() (participant ID
not so relevant here

Merge pull request #27426 from eileenmcnaughton/pcp

Fix php notices in php dashboard code

Remove unused variable

Fix another EventInfo notice

Merge pull request #27423 from eileenmcnaughton/notice_it

Fix notice on phone_ext

Fix php notices in php dashboard code

Fix notice on phone_ext

Merge pull request #27294 from colemanw/afformDataType

Afform - Fix getting and setting boolean/numeric types

Merge pull request #27366 from eileenmcnaughton/CRM

Remove always true if for required role_id field, add test for value

Merge pull request #27421 from demeritcowboy/desc

Revert just merged 27412

Revert "Fix notice on Membership Component Settings page"

This reverts commit e3f43b1ab5af0d3cf1063ee45805b3c6b66b6c4c.

Merge pull request #27412 from eileenmcnaughton/admin_notice

Fix notice on Membership Component Settings page

Merge pull request #27419 from eileenmcnaughton/no_push

Remove legacy error handling

Merge pull request #27420 from larssandergreen/follow-up-to-add-comment-on-custom-field-move

(NFC) Add comment for IS NOT NULL in moving custom fields

Merge pull request #27418 from demeritcowboy/reg

regen

Add comment for IS NOT NULL in moving custom fields

Remove legacy error handling

Our practice now is to throw exceptions - this results in a hard
fatal within apiv4 usage

Merge pull request #27413 from larssandergreen/Don't-copy-nulls-when-moving-custom-fields

dev/core#4568 Don't copy nulls when moving custom fields

regen

Merge pull request #27400 from eileenmcnaughton/legacy

dev/core#4112 stop installing legacycustomsearches on new installs

Merge pull request #27416 from larssandergreen/Weights-for-custom-field-sets-page

Weights for custom field groups page

Weights for custom field groups page

Merge pull request #27415 from PrzemyslawTabor/india-provinces

India-provinces update outdated provinces abbreviations

Merge pull request #27411 from colemanw/civiGrantCleanup

CiviGrant - Cleanup and simplify grant form, remove unused tab code

India-provinces update outdated provinces abbreviations

Merge pull request #27414 from pradpnayak/entitybatch

Add Entity batch to Search kit

Add Entity batch to Search kit

Don't copy nulls when moving custom fields

Fix notice on Membership Component Settings page

CiviGrant - Cleanup and simplify grant form, remove unused tab code

Deletes unused code for the contact summary tab (now handled by SearchKit).
Simplifies overcomplicated form context and detangles form embedded in page.

CiviGrant - Add missing note_used_for metadata

Merge pull request #27406 from larssandergreen/Fix-MoveField-PHP-error

Fix MoveField PHP notice

Merge pull request #27396 from colemanw/addNoteNote

Metadata - Add "civicrm_note" to "note_used_for" option group

Fix MoveField PHP error

DAO - Prevent recursion adding acl clause to entity that references itself

Metadata - Add "civicrm_note" to "note_used_for" option group

Merge pull request #27389 from colemanw/coreManaged

Scan core `civicrm/managed` directory for entity declarations

Merge pull request #27402 from eileenmcnaughton/pledge_pay

Smarty notices - Fix a couple more templates to look for elements not a variable that says the element is present

Merge pull request #27403 from eileenmcnaughton/php8

dev/core#4421 Fix some attempts to count NULL (php8.x)

Scan core `civicrm/managed` directory for entity declarations

Switch to a more complete module option list using CRM_Core_Module::getAll()

Merge pull request #27392 from totten/master-module-label

CRM_Core_Module - Generate optional `$label` property

dev/core#4421 Fix some attempts to count NULL (php8.x)

Generate `$label`s for modules in `Drupal8` (D8/9/10)

Generate `$label`s for modules in `DrupalBase` (D7/Backdrop)

Fix a couple more templates to look for elements not a variable that says the element is present

Merge pull request #27401 from civicrm/5.66

5.66 to master

Merge pull request #27399 from colemanw/fixContactJoin

APIv4: Implicit joins for email_primary, etc cause error

Merge pull request #27398 from eileenmcnaughton/noticey

Add test cover for handling of returnPath

dev/core#4112 stop installing legacycustomsearches on new installs

Generate $label for module-extensions

CRM_Core_Module - Add optional $label property

Merge pull request #27379 from colemanw/fileApi

Api4 - Enable file attachments to be displayed in SearchKit

Add test cover for handling of returnPath

Fix dev/core#4562 - APIv4: Implicit joins for email_primary, etc cause error

Merge pull request #27241 from eileenmcnaughton/type_2

php 8.x Use internal property rather than undefined _type

Merge pull request #27397 from civicrm/5.66

5.66 to master

Merge pull request #27395 from eileenmcnaughton/weight

Fix weight notices on ActivityType (option values) pages

Merge pull request #27394 from eileenmcnaughton/zet

Revert "[REF] Update Zetacomponents/mail to be 1.9.5 to fix email validation handling"

APIv4 - Giving up on fine-grained File ACLs for now

APIv4 - Fix rendering GROUP_CONTACT(UNIQUE) with field suffix

SearchKit - Support aggregated styles in links

SearchKit - Support aggregated icon fields

APIv4 - Support output_formatters with aggregated columns

APIv4 - Enforce ACLs for File entity, dedupe ACL clauses for bridge entities

APIv4 - Improve performance with explicit join ACL query deduping

Api4 - Enable file attachments to be displayed in SearchKit

File - Update completely wrong docblock.

Before: Function does the opposite of what the docblock says it does with .unknown suffixes.
After: Docblock accurately says what the function does.

Fix weight notices on ActivityType (option values) pages

Merge pull request #27393 from colemanw/conformanceTestFix

APIv4 - Fix conformance test for entities with multiple primary keys

Merge pull request #27356 from eileenmcnaughton/body

dev/core#2800 Fix bounce processing to handle verp emails

Revert "[REF] Update Zetacomponents/mail to be 1.9.5 to fix email validation handling"

This reverts commit 020a1178ac869e64691c4e5fd49ee09f01d8f194.

APIv4 - Fix conformance test for entities with multiple primary keys

Merge pull request #27385 from mlutfy/versionCheckSid

VersionCheck: replace hash by sid, add the environment

Afform - Fix js error when setting default value for the first time

Afform - Fix getting and setting boolean/numeric types

Fixes dev/core#4465
Replaces https://github.com/civicrm/civicrm-core/pull/27006

Add detail about bounce to activity

Note adding fetch_activities to the test exposed a mysql error so I had
to add handling there

Use bounce activity type, if exists

We can look at adding in upgrade script in later version after bedding in

Merge pull request #27240 from eileenmcnaughton/type

Use internal variable rather than undeclared property

Move handling of empty from obscure function to main email handler class

I wondered if the zeta components update would make this fix obsolete but adding
a test  says not...

So this locks in the fix on
https://issues.civicrm.org/jira/browse/CRM-19215

Move from handling to processor, test

dev/core#2800 Fix bounce processing to handle verp emails

This fixes 2 issues when combining bounce processing with verp emails

1) the emails being processed are being matched to email they come from -
ie the verp email. Hence the created activities are not linked to the
actual contact and instead create endless variants of the sender.
This happens with both the job.process_activities job and with
job.fetch_bounces with is_create_activities = TRUE.
This fix adapts the handling to identify verp emails (using existing regex)
and look up the contact ID from the mailing_event_queue. This is
used as the source (I didn't add a target or assignee at this stage cos
I wasn't sure if that was just data-cruft or useful stuff - but it would
be retrieved from the mailing)

2) this second one is a doozy & probably only showed up in tests because the
tests mostly cover obscure email formats. The create activities code does
a call to get the body and attachments using a function that looks like it
came from a tutorial on how to best use ezcmail. It seems to parse
all the email variants and attachments and, importantly, moves those attachments
to the civicrm files as part of creating the activity.

The code used to getBody() for bounces looks like it was ... written by us.
It does a subset of the processing in earlier call to get body and in
many of our test cases falls back on a blunt generateBody(). This call parses the
email, including the attachments, which are no longer there cos they got moved....
so it fatals.

I dug into the 2 functions and I feel pretty sure the bounce one doesn't add any
additional value but DOES have less capability than the other. So I removed that code,

Unfortunately the person received weird email from Exchange 2003 did not log
the headers https://issues.civicrm.org/jira/browse/CRM-9361 - but even if the
preferred version of the function doesn't parse that there is default text so
it won't fail & it seems like the risk of having a bit less info about an (untested)
obscure email bounce body is not that probematic

Merge pull request #27332 from larssandergreen/Show-payment-block-if-payment_instrument_id

dev/core #4556 Show payment block if payment_instrument_id already set

Add test for participant role, check mail

Remove always true if

This removes an if that is always true

role_id is a required field on the participant form and the register from
search that extends it - so we don't need to check if it is filled out.

In addition we know that it is always an array because that's just the type of field it is.

Removing handling for pre-select 2 makes the code clearer...

Merge pull request #27388 from colemanw/disableGetForCheckAccess

Disable broken unit test

Document that civi.api4.authorizeRecord does not work for get actions

Merge pull request #27386 from civicrm/5.66

5.66

Merge pull request #27384 from eileenmcnaughton/566test

Add email helper to new form test helper

VersionCheck: replace hash by sid

Merge pull request #27383 from larssandergreen/Fix-SearchDisplay-totals

dev/core#4559 Don't include test, template, deleted entities in SearchDisplay totals

Merge pull request #27381 from sebalis/dev_core_4542_sebalis

dev/core#4542 Fix priority handling in ACLs when dealing with objects…

Add email helper to new form test helper

Don't include test, template, deleted entities in searchdisplay totals

Merge pull request #27382 from mlutfy/getStartedStatus

GettingStarted: add token for System Status

GettingStarted: add token for System Status

add change to ACLPermissionTest.php from original pull request

Co-authored-by: Seamus Lee <seamuslee001@gmail.com>

dev/core#4542 Fix priority handling in ACLs when dealing with objects other than contacts, and remove code duplication

Co-authored-by: Seamus Lee <seamuslee001@gmail.com>