Use bounce activity type, if exists

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

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 #27384 from eileenmcnaughton/566test

Add email helper to new form test helper

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

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 #27372 from larssandergreen/Expose_fee_label_for_event_tokens

Expose fee_label for event tokens

Expose fee_lable for event tokens

Merge pull request #27367 from eileenmcnaughton/566

Fix notice on unpaid event

Merge pull request #27368 from seamuslee001/5.66

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

Fix notice on unpaid event

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

Merge pull request #27128 from christianwach/lang

Support retention of locale on links

Merge pull request #27364 from seamuslee001/5.66

FiveSixtyFour: disable i18n rewrite on cancel_URL

Merge pull request #27363 from eileenmcnaughton/event_id

Ensure event ID is an integer

FiveSixtyFour: disable i18n rewrite on cancel_URL

Ensure event ID is an integer

Merge pull request #27358 from colemanw/pseudoconstantGuard

PseudoConstant - Prevent fatal when entity not available

PseudoConstant - Prevent fatal when entity not available

This can happen e.g when Views tries to lookup an option value for a disabled extension.
See https://github.com/civicrm/civicrm-drupal/pull/656#issuecomment-1094362139

Merge pull request #27350 from eileenmcnaughton/incoming_util

Remove unused variable

Remove unused parameter

Set version to 5.66.beta1

Merge pull request #27330 from alifrumin/4555

dev/core#4555 When using a Contribution Page with a Membership Price Set, the contribution amount information never shows on the thank you page

Merge pull request #27345 from eileenmcnaughton/eileen

Fold deprecated function back into the caller

Merge pull request #27344 from seamuslee001/update_zetacomponents

[REF] Update Zetacomponents/mail to be 1.9.5 to fix email validation …

Fix variable name clash

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

Merge pull request #27348 from seamuslee001/master

5.65

Merge branch '5.65'

Update release-notes/5.65.0.md

Purify in the social network section as well

security/core#114 Fix Stored XSS in event title field

Add code comment as per Coleman

Minor fix

Also fix deleting from PCPAccount form class

security/core#126 Ensure that QFKey is validated when deleting a contact image

Fix security/core#126 csrf bug to delete contact image

Fix security/core#125 Stored xss in survey title

CIVI-SA-2023-12 - jQuery Validation

CIVI-SA-2023-11 - Potential XSS

CIVI-SA-2023-10 - Potential SQLI

CIVI-SA-2023-09 - SQLI via API getFields

Merge pull request #27346 from seamuslee001/5.65

Correct Credit card month description in release notes

Correct Credit card month description in release notes

Merge pull request #27343 from seamuslee001/master

5.65

Merge pull request #26861 from totten/master-url

(dev/core#4433) - Implement Civi::url() with prefixes and OOP enhancements

Fold deprecated function back into the caller

We have a situation where the first function called (parseMailingObject) formats stuff weirdly just to be undone by the second.

Moving the handling to a single place (the calling function) will allow us to unravel this

Merge branch '5.65'

Merge pull request #27333 from eileenmcnaughton/tok

dev/core#4551 Fix escaping on token event.title

Merge pull request #27337 from eileenmcnaughton/email_start

Move regex processing in EmailProcessor to handling class

Fix schema for pay_later_receipt input type

dev/core#4551 Fix escaping on token event.title

Merge pull request #27342 from seamuslee001/master

5.65

Merge branch '5.65'

Merge pull request #27340 from seamuslee001/565_regen

[REF] Regen after civicrm/civicrm-core#27335

Merge pull request #27331 from colemanw/searchKitDefaults

SearchKit - Fix default search columns

Merge pull request #27341 from civicrm/5.65

5.65

Merge pull request #27321 from eileenmcnaughton/email_api

Improve error handling by using civicrm_api3 not civicrm_api

Merge pull request #27339 from agh1/5.65.0-releasenotes-final

5.65.0 release notes: added late changes

[REF] Regen after civicrm/civicrm-core#27335

5.65.0 release notes: added late changes

Merge pull request #27338 from colemanw/crmStatusRevert

Revert "Update CRM.status for compatability with native js promises"

Move regex processing in EmailProcessor to handling class

This is a partial of https://github.com/civicrm/civicrm-core/pull/27319
and includes moving a big chunk of code but no logic changes

There is good test cover in CRM_Utils_Mail_EmailProcessorTest

I have split this out as it gets some of the 'bulk' out of that PR which
has a lot of more complex changes to grok

SearchKit - Use angular helper crmStatus for in-place-edit

Followup to #27105 this uses native angular service instead of jQuery-based function

Revert "Update CRM.status for compatability with native js promises"

This reverts commit 5e6740703f417d2d4d0c3d625bbe0cdfd00cd549.
It was causing loading errors at ext/message_admin/ang/crmMsgadm/User.js#L18

Merge pull request #27336 from civicrm/5.65

5.65 to master

Merge pull request #27334 from colemanw/getCalcFields

SearchKit - Don't crash afforms with non-dao entities

Merge pull request #27335 from larssandergreen/fix-missing-event-pay-later-receipt-text

Fix missing event pay later receipt text

Fix missing event pay later receipt text

SearchKit - Don't crash afforms with non-dao entities

update confirm too

SearchKit - Fix default search columns

Merge pull request #27329 from mlutfy/eventDashboardEndDate

Event Dashboard: show events that have not ended yet

dev/core#4555 When using a Contribution Page with a Membership Price Set, the contribution amount information never shows on the thank you page

Event Dashboard: show events that have not ended yet

Merge pull request #27305 from eileenmcnaughton/preview

Remove unreachable page

Merge pull request #27327 from demeritcowboy/weight

Set link weight on manage case

weight

Merge pull request #27320 from eileenmcnaughton/processor

Remove unused parameter, pass by reference in private function with one caller

Merge pull request #27323 from eileenmcnaughton/weight

Define weight for links on extensions page, financial types, financial accounts

Merge pull request #27326 from civicrm/5.65

5.65 to master

Merge pull request #27324 from eileenmcnaughton/565

Fix enotice (possible regression)

Add weights to contribution page links

Fix enotice (possible regression)

It's clear in the function that lineItem is only defined for add - I hit the enotice editing
the participant to be paid. Not sure if this is a regression but there was a related one
fixed in 5.65 or 5.64 so targetting the rc

Add weight to contribution tasks

Add weights to Manage Event links

Add weights on assign financial account page

Add weights to financial type screen

Add weights on financial accounts page

Define weight for links on extensions page

Merge pull request #27317 from eileenmcnaughton/process

Remove long-broken function

Improve error handling by using civicrm_api3 not civicrm_api

Remove unused parameter, pass by reference in private function with one caller

Merge pull request #27318 from eileenmcnaughton/legacy

Finish CiviCRM 2.1 deprecation - old style verp

Merge pull request #27259 from eileenmcnaughton/lookup

Use new lookup trait to eliminate use of undefined properties `userDisplayName`