Merge pull request #26253 from civicrm/5.62

5.62

Merge pull request #26252 from demeritcowboy/missing-help

Import help text gone missing

help gone missing

Merge pull request #26249 from larssandergreen/Remove-old-form-rule-on-/-in-page-titles-for-paypal

Remove form rule for Contribution Page and Event titles preventing /

Merge pull request #26234 from eileenmcnaughton/deceased_token

Add deceased date token

Remove form rule for titles preventing /

Merge pull request #26244 from eileenmcnaughton/weight_linke

Add weight to a couple more links

Add weight to a couple more links

Merge pull request #26241 from eileenmcnaughton/weight

Add some weights to menu link items

Fix test to expect delete last

Also has some fixes to get away from undeclared property issues on
touched test & helpers

Adjust weights so link order is inchanged

Add some weights

Add deceased date token

Per https://github.com/eileenmcnaughton/nz.co.fuzion.civitoken/issues/63

Merge pull request #26242 from civicrm/5.62

5.62

Merge pull request #26193 from eileenmcnaughton/member

Fix code handling to pre-5.61 for membership block

Merge pull request #26240 from civicrm/5.62

5.62

Merge pull request #26239 from totten/5.62-uninstall

setup - Fix uninstall on MySQL 8.0

Merge pull request #26233 from eileenmcnaughton/weight

Add weight to links spec

setup - Fix uninstall on MySQL 8.0

Overview
--------

Fix a bug with uninstalling or reinstalling via cv (`cv core:uninstall` or `cv core:install -f`) on MySQL 8.0.

Before
------

On MySQL 8.0, it fails - because all table-names appear as null.

It's looking for a result column named `table_name` but actually receives `TABLE_NAME`.

After
-----

Doesn't matter if the column is called `table_name` or `TABLE_NAME`.

Merge pull request #26238 from eileenmcnaughton/part_public

Notice fix on edit Household

Notice fix on edit Household

Add weight to links spec

This allows a weight to be set & for it to alter the order.

It is a bit hard to know whether to add weights to existing ones as
it would be a lot to do all of them - but I did hear a specific request
(can't recall who from) to have Delete last to reduce mis-clicks
and with the View & Edit declared in the same place
I felt I should add them with low values for some sort of consistency

This would mess with people who have implemented weight in a hook - but
probably any change is low impact & an easy-ish fix

Note that I will update docs once agreed

Merge pull request #26235 from ufundo/getsearchtasks-qfkey-for-page-callback

dev/core#4295 SearchKit: check controller class for qfKey when loading legacy search actions for contributions

check route callback for generating qfkey

Merge pull request #26214 from larssandergreen/Improve-slow-download-from-SearchKit-results

Speed up Download Spreadsheet from SearchKit results

Improve-slow-download-from-SearchKit-results

Merge pull request #26230 from colemanw/contributionUpgrade

Upgrader - Use generic (non-api4) code in upgrader

Upgrader - Use generic (non-api4) code in upgrader

This was causing issues with https://github.com/civicrm/civicrm-core/pull/26208
because the Contribution API is no longer available without enabling the
civi_contribute extension, which doesn't happen until a later upgrade step.

Merge pull request #26068 from colemanw/arrayOperators

dev/core#1321 - SearchKit - Limit operator list for array fields

Merge pull request #26223 from eileenmcnaughton/participant_sent

Clarify sent & not sent variables

Merge pull request #26225 from olayiwola-compucorp/addNameToContributor

Added Olayiwola Odunsi to contributor-key.yml

Added Olayiwola Odunsi to contributor-key.yml

Merge pull request #26196 from seamuslee001/skip_vendor_old_files_non_civi_vendor

[REF] dev/core#1777 - Skip checking vendor provided orphaned files if the vendor directory is not within the CiviCRM root

Merge pull request #26145 from olayiwola-compucorp/changeContributionDateLabel

dev/core#1403 - CRM/Contribute -  Change "Date Received" to "Contribution Date"

Clarify send & not sent variables

Its silly to pass an array around just to have it finally counted...

Merge pull request #26221 from eileenmcnaughton/part_public

Make some internal functions protected

Merge pull request #26220 from eileenmcnaughton/crazy_test_4

Fix Participant tests to use full form flow

dev/core#1403 - CRM/Contribute -  Change "Date Received" to "Contribution Date"

Merge pull request #26222 from eileenmcnaughton/extract

Remove unreachable validation

Add getParticipantID to avoid non-php8-friendly path check

This particular fail is due to the path being empy in the test environment - But
it's wrong to check it as it is a crufty check for whether
the search task is being used

Remove unreachable validation

Some time ago we fixed it such that only Pending and Completed statuses are in
the drop box for contribution_status_id. This fragment of code handles an edge case
that is no longer reachable - ie when the value is Failed and some other confusing stuff
is true

Fix Participant tests to use full form flow

This doesn't get all of them but fixes a bunch

Improve fromEmail address handling

We don't need to validate the from Email address in the postProcess as the
form submit validation expects it to be a supplied value.

Fixing this makes testing easier

Improve test consistency on capitalisation

Merge pull request #26217 from eileenmcnaughton/crazy_test

Minor cleanup on function

Make some internal functions protected

I did a universe search to be sure but these 'belong' to the form

Minor cleanup on function

The primary goal here was to check that it ALWAYS returns an array. It does,
so I added a type hint.

I also confirmed that
- it is not called from universe
- it is not called with no primaryParticipantID
- it is not called with excludeCancel set for false
so I did some deprecating.

Lastly there was code to handle there being no cancelStatus -
assigning a variable cancelID of 0 as a fallback. But, it
was not being used - so I fixed it to cast-to-int.

Merge pull request #26213 from dontub/AbstractAction-drop-throws-phpdoc

AbstractAction: Drop `@throws` from phpdoc of `__construct()` because it's not thrown

AbstractAction: Drop `@throws` from phpdoc of `__construct()` because it's not thrown

Merge pull request #26210 from civicrm/5.62

5.62

Merge pull request #26205 from eileenmcnaughton/event_backoffice

Further separate register task from participant form

Merge pull request #26209 from larssandergreen/Don't-show-send-confirmation-description-if-registering-multiple

Don't show send broken confirmation description if registering multiple contacts

Don't show send confirmation description if registering multiple

Merge pull request #26206 from eileenmcnaughton/event_mail

Extract sendEmail function

Merge pull request #26172 from eileenmcnaughton/562_import

dev/core#4227 Fix import to not blank birth_date etc on update

Merge pull request #26204 from eileenmcnaughton/deprecated

Add missing deprecated tag

Merge pull request #26036 from colemanw/componentExtensions

dev/core#3961 Create stub extensions for all components

Extract sendEmail function

This is a straight forward extraction
There is test cover

Further separate register task from participant form

Component Extensions - Fix upgrade and test errors

Component/Extensions - Remove unnecessary cache clearing and prevent recursion

Ensure the cached settings are updated _before_ post_change hooks are called for settings.

All the cache clears being performed by on_change hooks were redundant with what happens when
extensions are enabled/disabled so I removed it.

Also moved the extension install hook to use postInstall which should be more stable, and
added checks in onToggleComponents to prevent recursion.

Add test for component-extension lifecycle

Sync enable/disable status of component extensions with core components

Remove missing deprecated tag

Merge pull request #26203 from eileenmcnaughton/reg_vars

Remove unused constants

Remove unused constant

Merge pull request #26202 from pradpnayak/extcleanup

flexmailer remove cruft code

Merge pull request #26201 from civicrm/5.62

5.62

flexmailer remove cruft code

Merge pull request #26199 from pradpnayak/fatalerror

Regression: Fatal error when importing membership

Fixed fatal error when importing membership

Add stub extensions for all components

Merge pull request #26195 from totten/master-esm

dev/core#4279 - Add support for ECMAScript Modules (ESM, part 1)

Fix capitalization in comments

add more esms

(NFC) Emphasize abbreviation "ESM"

Merge pull request #26192 from MegaphoneJon/entity-ref-joins

proper EntityRef joins

[REF] Skip checking vendor provided orphened files if the vendor directory is not within the CiviCRM root

Merge pull request #26194 from larssandergreen/Ttile-case-all-of-Support-menu

Title case Register Your Site in support menu

Title case Register Your Site in support menu

Fix code handling to pre-5.61 for membership block

The work for #26170 concluded that membershipPriceset is always TRUE
at this point. Since we were pushing out a regression fix I left the
variable where it didn't seem to be causing regressions.

However, I think clarifying the variable and
removing the redundant if-else in the rc will set us up well as
1) if there is any further regression we haven't found in
5.61 then people will be able to check on the rc / master
2) if we need to do any further fixes / back-ports the
code will be more synced if we do this in the rc

Note there is some further clean up I feel should be done
folloiwng what we learnt dealing with #26170 - but
I want to let the dust settle so that I'm not creating a
hard-to-backport situation

Resources - Define helper methods `addModule()`, `addModuleFile()`, `addModuleUrl()`

CRM_Core_Region::add() - Allow JS resources to be flaged as ES5 modules (`esm=>TRUE`)

proper EntityRef joins

Merge pull request #26191 from larssandergreen/Uncheck-is_email_receipt-when-hidden-on-New-Contribution

Uncheck is_email_receipt when hidden on New Contribution

Uncheck is_email_receipt when hidden

Merge pull request #26184 from reflexive-communications/crmMailingAB-error

crmMailingAB/services: catch exception, report error and rethrow

Merge pull request #26178 from stesi561/drupalconfig

Fix config is null on hooks

Merge pull request #26189 from larssandergreen/Prevent-self-service-cancellation-when-prohibited-by-using-back-office-url

Prevent prohibited self service cancellation by using is_backoffice=1 in URL

Merge pull request #26188 from larssandergreen/Fix-authorization-failed-on-self-service-participant-transfer

Fix authorization failed on self-service participant transfer

Merge pull request #26187 from larssandergreen/Preserve-participant-status-when-transfering

Preserve participant status when transferring

Prevent unauthorized self-service participant update via URL

Fix authorization failed on self-service participant transfer

Preserve participant status on transfer

Merge pull request #26181 from mattwire/confirmeventmsg

Show/hide billing name/address on event confirm based on if they're set instead of a magical collection of booleans

crmMailingAB/services: catch exception, report error and rethrow

Merge pull request #26123 from colemanw/afformValidate

Afform - Ensure checkboxes and radios are validated client-side

Show/hide billing name/address on event confirm based on if they're set instead of a magical collection of booleans

Merge pull request #26167 from larssandergreen/imrpove-footer-and-support-menu

dev/core#3920 Improve footer and support menu

Fix config is null on hooks