Merge pull request #23638 from jensschuppe/fix/iCalHtmlEntities

Decode HTML entities in iCalendar text

Merge pull request #23837 from civicrm/5.51

5.51 to master

Merge pull request #23823 from eileenmcnaughton/contact_test

dev/core#3663 fix output regression on csv output for import

Merge pull request #23825 from colemanw/profileExport

APIv4 - Enable exporting profiles with custom fields

Merge pull request #23818 from colemanw/afformDep

Afform - fix dependency injection for autogenerated forms

Always throw exception in tests for pdf out

headers_sent() was always true before and hence the tests were written to expect the
exception would be called. Fixing that caused the tests to fail - but fixing it is good

Merge pull request #23832 from demeritcowboy/casedetails-static

dev/core#2983 - Make static function static - AJAX::caseDetails()

make static function static

APIv4 - Enable exporting profiles with custom fields

Before: Impossible to export a profile with custom fields as a managed entity because of the dependence on custom ids as the fieldname.

After: Uses APIv4-style names when exporting custom fields for profiles, which makes them portable.

Merge pull request #23816 from eileenmcnaughton/inv_org

Fix double organization name in invoice, use token

Merge pull request #23827 from eileenmcnaughton/empty_date

Handle empty date token gracefully

Handle empty date token gracefully

Output to stdErr rather than echo

Merge pull request #23824 from totten/mixin-refresh

(dev/core#3660) Ensure that rebuildMenuAndCaches has current mixins+classloaders

dev/core#3663 fix output regression on csv output

This fixes a very-recent rc regression whereby the file error output was
not correctly referring to the fields to be exported.

Note that in order to make it testable I had to
- remove header-setting that duplicates work done by the csv package
- remove extraneous echos in the GenCode

(dev/core#3660) Ensure that rebuildMenuAndCaches has current mixins+classloaders

Overview
--------

This fixes an issue in transitioning from `hook_managed` to `mixin/mgd-php@1`, wherein
managed-entities are briefly deleted (but later recreated).

Steps to Reproduce
------------------

1. (Web) Install an extension with a revision that uses `hook_managed`
2. (CLI) Switch the extension to a revision that uses `mixin/mgd-php@1`
3. (CLI) View contents of `civicrm_managed`
4. (Web) Run `civicrm/menu/rebuild`
5. (CLI) View contents of `civicrm_managed`
6. (Web) Run `civicrm/menu/rebuild`
7. (CLI) View contents of `civicrm_managed`

Before
------

While processing step 4 (`civicrm/menu/rebuild`), it fails to run the hooks for `mgd-php`.

Consequently, the list of managed-entities is lost and will disappear at step 5.

After
-----

While processing step 4 (`civicrm/menu/rebuild`), it activates the hooks for `mgd-php`.

Technical Details
------------------

When processing `civicrm/menu/rebuild`, there are a couple big substeps:

* `Civi\Core\Container::boot()` - During this process, it loads extensions. As usual,
  this reads cached metadata, sets up classloaders, sets up mixins/hooks, etc.
* `CRM_Core_Invoke::rebuildMenuAndCaches()` - During this process, it clears
  out caches and rebuilds several things (menus, managed-entities, etc).

The problem is this:

* The cache used during `boot()` has stale metadata (specifically,
  `civicrm_cache` has old values of `mixinScan` and `mixinBoot`).
  So it doesn't setup any new mixins/hooks.
* Then `rebuildMenuAndCaches()` depends on the mixins/hooks that are already setup.
  While the function does clear persistent caches, it assumes that the
  PHP runtime environment is up-to-spec. But it's not -- becuase our hooks
  were based on the caches.

The patch uses the same `refresh()` mechanism as the extension-administration subsystem (which
has to reset the classloaders+mixins after enabling or disabling an extension).

Merge pull request #23807 from colemanw/fixAfformTitle

SearchKit - Only add filter values to Afform title if passed internally

SearchKit - Only add filter values to Afform title if passed internally

This narrows the scope of contextual titles from #22319 to
only those filters passed internally, as used in drilldown forms.

It makes sense to exclude values from exposed filters because that
can lead to a confusing UI.

Afform - fix dependency injection for autogenerated forms

Merge pull request #23800 from seamuslee001/ref_date_filter

[REF] [PHP8.1] Replace usage of smarty's date_filter to ensure we don't call s…

Merge pull request #23419 from chrisgaraffa/contactheader-regions

Add crmRegions to contact headers and associated inline forms

Merge pull request #23817 from civicrm/5.51

5.51 to master

Merge pull request #23815 from seamuslee001/date_second_customformat

[REF] Fix second output in customFormat function and extend unit test

Fix double organization name in invoice, use token

[REF] Fix second output in customFormat function and extend unit test

[REF] Replace usage of smarty's date_filter to ensure we don't call strftime which is deprecated in php8.1

Ensure start date format is the same

Merge pull request #23806 from eileenmcnaughton/invoice_domain

Invoice Workflow template - add domain tokens

(NFC) Domain Tokens - Use consistent capitalization (title-case)

The current labels have mixed capitalization. Notwitstanding
one notable exception (`{action.*}`), most tokens use title-case.
So this patch applies title-case.

Merge pull request #23697 from bsilvern/CRM-3495

dev/core#3495 - Fix fields with wildcards in Advanced Search

Merge pull request #23813 from totten/master-tokproc-exc

(REF) TokenProcessor - Tweak error message

Merge pull request #23811 from totten/master-wfmsg-dataprovider

(REF) WorkflowMessageTest - Use dataProvider

Merge pull request #23810 from totten/5.51-msgadm-htmlraw

message_admin - Enable another preview mode ("HTML (Raw)")

Merge pull request #23814 from civicrm/5.51

5.51 to master

TokenProcessor - Tweak error message

(REF) WorkflowMessageTest - Use dataProvider

message_admin - Enable another preview mode ("HTML (Raw)")

This is similar to the "HTML" preview, except it shows the raw HTML (with syntax highlighting).

Merge pull request #23801 from eileenmcnaughton/cont_work

Fix breakage in message preview on workflow contributions

Merge pull request #23768 from mlutfy/cloud17

dev/core#3542 Store extdir cache in sqlcache rather than the filesystem

Merge pull request #23723 from mattwire/ufmatchsearchkit

Allow UFMatch to be used in SearchKit - this allows you to filter by eg. 'has a CMS account'

Merge pull request #23790 from mattwire/editcontributionlinkshook

Add hook to modify 'contribution payment links'

Metadata - Add title for UfMatch entity

Invoice Workflow template - add domain tokens

Add hook to modify 'contribution payment links'

Merge pull request #23805 from mattwire/devfinancial197alt

dev/financial#197 Revert freezing on total_amount field on recurring form (alt to #23796)

dev/cloud-native#17 Store extdir cache in sqlcache rather than the filesystem

Merge pull request #23799 from turrisxyz/Pinned-Dependencies-GitHub

chore: Set permissions for GitHub actions

Revert freezing on total_amount field on recurring form

Merge pull request #23798 from seamuslee001/fix_auto_renew_membership

Fix Automatic renewal membership from front end contribution forms

Fix breakage in message preview on workflow contributions

Merge pull request #23797 from eileenmcnaughton/remove_field_types

[REF] Remove param that has a set value

chore: Set permissions for GitHub actions

 Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much.

- Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs

[Keeping your GitHub Actions and workflows secure Part 1: Preventing pwn requests](https://securitylab.github.com/research/github-actions-preventing-pwn-requests/)

Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>

[REF] Fix Automatic renewal membership from front end contribution forms

Properly fix regression and update unit tests to match what the form does

Merge pull request #23774 from demeritcowboy/recur-curious

dev/financial#199 - Add failing test to demonstrate the problem

[REF] Remove param that has a set value

Merge pull request #23785 from totten/5.51-hook-log

dev/core#3502 - CiviEventDispatcher - Softer errors for not-ready. More comments.

Merge pull request #23789 from civicrm/5.51

5.51

Merge pull request #23753 from eileenmcnaughton/import_validate_int

Add integer validation to import

Merge pull request #23763 from eileenmcnaughton/act_test

Fix target contact label back to include target

Merge pull request #23764 from eileenmcnaughton/activity_update_mutli

dev/core#2706 dev/core#2308 fix activity import to update multi-select fields

Merge pull request #23749 from colemanw/uiUtils

AngularJS - Remove UI-Utils library

Merge pull request #23788 from ufundo/5.51

only check batch currency match when adding a financial_trxn

Merge pull request #23791 from mattwire/phpnotice1

NFC: Fix PHP notice on ContributionRecurSelector.tpl

Merge pull request #23792 from mattwire/recordrefund

Make it clear what "record refund" form does

Make it clear what record refund form does

Fix PHP notice

Merge pull request #23786 from mattwire/devfinancial199

dev/financial#199 Fix additional details on edit template contribution

Fix additional details on edit template contribution

only check batch currency match when adding a financial_trxn

dev/core#3502 - CiviEventDispatcher - Tune down the warning

dev/core#3502 - CiviEventDispatcher - Tweak error message. Add explanation.

Merge pull request #23783 from totten/5.51-snapshots

FiveFiftyOne - Enable snapshots for data modified data

FiveFiftyOne - Enable snapshots for data modified data

Before
------

The 5.51.alpha upgrade modifies a some data-structures (`civicrm_mapping_*`, `civicrm_queue`),
but it doesn't take any snapshots.

After
-----

It does snapshots.

Comments
--------

Both tables should generally be quite small.

Strictly, the `civicrm_mapping_*` updates are more in-place modifications,
so that's more important.  The `civicrm_queue` changes are only adding new
columns.  But the table is so small - it doesn't hurt.

I suppose one could argue that these particular fields aren't likely to have
problems - but it's important to get in the habit of enabling snapshots so
that we have them when they do matter.

Merge pull request #23781 from civicrm/5.51

5.51 to master

Merge pull request #23771 from eileenmcnaughton/cont_update

dev/core#3651 dev/core#1337 Fix failure to show last column data in output, spaces in names

add failing test

Merge pull request #23780 from totten/5.51-hush

(dev/core#3653) CRM_Queue_Runner - Fix upgrade warnings. Extract disableBackgroundExecution().

Merge pull request #23776 from eileenmcnaughton/electro

(dev/core#2198) Switch to more maintained phpQuery

Merge pull request #23766 from eileenmcnaughton/cont_up

dev/core#3513 Remove 'done' from sumary/import monitoring page

CRM_Queue_Runner - Fix upgrade warnings. Extract disableBackgroundExecution().

This is a revision to #23775.  As before, it prevents a hard error when
upgrading to 5.51.  However, the try-catch produces confusing error messages
in the `CiviCRM.log.*` (ie they indicate a SQL error, but the situation is
actually a normal/expected during upgrade).  This avoids the error messages.

Tested (`r-run`) with `civicrm/upgrade`, `drush civicrm-upgrade-db`, and `cv
upgrade:db` -- for both 5.45=>5.51 and 5.45=>5.49=>5.51.

Merge pull request #23741 from ufundo/entitybatchcurrency

only check batch currency match when adding a financial_trxn

Merge pull request #23777 from darrick/fix_import_disableUSPS

fix disableUSPS since import running in separate context of Queue.

fix disableUSPS since import running in separate context of Queue.

Merge pull request #23778 from civicrm/5.51

5.51

Merge pull request #23775 from eileenmcnaughton/5.51

dev/core#3653 Fix on queue runner not working in upgrade

dev/core#2198 switch to more mainted phpquery

dev/core#3653 Fix on queue runner not working in upgrade

Merge pull request #23773 from civicrm/5.51

5.51 to master

Merge pull request #23772 from eileenmcnaughton/cont_contact_fields

Fix import regression - contribution import borks on do_not_import

Merge pull request #23757 from colemanw/dupeFixAgain

Deduepe - Fix form buttons (again)

dev/core#3651 dev/core#1337 Fix failure to show last column data in output, spaces in names

Fix import regression - contribution import borks on do_not_import

Merge pull request #23769 from civicrm/5.51

5.51

dev/core#3513 Remove 'done' from sumary/import monitoring page

See dev/core#3513 for why it's confusing

Merge pull request #23759 from colemanw/fixTotalCount

[REGRESSION] SearchKit - Fix angular console error when totalCount is not given

Merge pull request #23754 from eileenmcnaughton/act_date

dev/core#2325 import second-handling fix

Merge pull request #21809 from demeritcowboy/site-path

dev/drupal#167 - Deprecated service to be removed in Drupal 10

Merge pull request #23765 from eileenmcnaughton/guzzle_it

Catch & convert Guzzle exception

Merge pull request #23758 from colemanw/noCiviGrant

Setup - Remove CiviGrant from available components

Merge pull request #23760 from eileenmcnaughton/validate_fields

dev/core#3511 Improve handling of invalid saved mapping fields

Merge pull request #23761 from eileenmcnaughton/invalid_state_with_country

dev/core#3512 fix rc fatal error when state is invalid