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

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

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

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>

Merge pull request #27369 from eileenmcnaughton/contribute

Remove unnecessary contributeMode reference - just present trxn_id if defined

Merge pull request #27325 from eileenmcnaughton/fee_label

Standardise fee_label in event workflow messages

Merge pull request #27376 from civicrm/up-merge

5.66 to master

Merge branch 'master' into up-merge

Merge pull request #27372 from larssandergreen/Expose_fee_label_for_event_tokens

Expose fee_label for event tokens

Expose fee_lable for event tokens

Merge pull request #27373 from civicrm/5.66

5.66 to master

Merge pull request #27367 from eileenmcnaughton/566

Fix notice on unpaid event

Merge pull request #27370 from civicrm/5.66

5.66

Merge pull request #27371 from eileenmcnaughton/contribute_mode

Stop checking contributeMode in ContributionPage confirm.tpl before displaying billingName

Merge pull request #27234 from eileenmcnaughton/no_if

Cleanup presentation of total to improve code & reduce notices

Merge pull request #27293 from colemanw/requireAfform

Afform - Make Afform Core extension required

Stop checking contributeMode in ContributionPage confirm.tpl

Merge pull request #27368 from seamuslee001/5.66

[REF] Ensure that any NULL values in the title field are fixed prior …

Remove unnecessary contributeMode reference

If trxn_id is present, then show it.

The variable is always assigned in
https://github.com/civicrm/civicrm-core/blob/2ad38a735f7b26daf99d7b63eb413c6b69f780cb/CRM/Contribute/Form/Contribution/ThankYou.php#L212

So we don't need more (deprecated) conditions

Merge pull request #27149 from totten/master-config-param

hook_civicrm_config: Add extra information to distinguish invocations

Merge pull request #27365 from civicrm/5.66

5.66

Fix notice on unpaid event

[REF] Ensure that any NULL values in the title field are fixed prior to changing the column

Afform - Make Afform Core and Authx extensions required

This enables Afform by default (and its dependency Authx) on all CiviCRM installs;
they cannot be disabled.